[PATCH 4/4] media: mediatek: vcodec: Drop VA check against mtk_vcodec_mem_free()

AngeloGioacchino Del Regno angelogioacchino.delregno at collabora.com
Wed Dec 6 02:19:32 PST 2023


Il 13/11/23 13:26, Fei Shao ha scritto:
> Now mtk_vcodec_mem_free() handles the VA-is-NULL case without generating
> excess error log, so we don't need to check that every time before using
> it in the driver.
> 
> Remove all the unnecessary if branches against mtk_vcodec_mem_free().
> 
> Signed-off-by: Fei Shao <fshao at chromium.org>

For the reasons explained in patch [3/4], I'd prefer to keep those checks.

Cheers,
Angelo



More information about the Linux-mediatek mailing list