[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