[PATCH v5,07/11] media: mediatek: vcodec: separate struct 'mtk_vcodec_ctx'
AngeloGioacchino Del Regno
angelogioacchino.delregno at collabora.com
Tue Jul 4 05:12:54 PDT 2023
Il 04/07/23 13:13, Yunfei Dong ha scritto:
> Adding different context struct for encoder and decoder and removing
> struct 'mtk_vcodec_ctx'.
>
> Signed-off-by: Yunfei Dong <yunfei.dong at mediatek.com>
> Reviewed-by: Nicolas Dufresne <nicolas.dufresne at collabora.com>
> ---
> .../mediatek/vcodec/mtk_vcodec_com_drv.h | 76 +++++
Only one remark. "com" may mean "communication" or "common", but if you rename
this to `mtk_vcodec_cmn_drv.h`, then "cmn" can only mean "common", which would
increase the chances to immediately understand what the header is about.
Unless, of course, anyone else has strong opinions against. Besides....
Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno at collabora.com>
More information about the linux-arm-kernel
mailing list