From: Yunfei Dong <[email protected]>

mtk_q_data::fmt is actually a pointer and must be dereferenced as such.
This went under the radar because mtk_v4l2_debug() evaluates to nothing
unless DEBUG is defined.

Signed-off-by: Yunfei Dong <[email protected]>
[acourbot: split into its own commit]
Signed-off-by: Alexandre Courbot <[email protected]>
---
 drivers/media/platform/mtk-vcodec/mtk_vcodec_dec.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/media/platform/mtk-vcodec/mtk_vcodec_dec.c 
b/drivers/media/platform/mtk-vcodec/mtk_vcodec_dec.c
index 26a55c3e807e..653f111532ab 100644
--- a/drivers/media/platform/mtk-vcodec/mtk_vcodec_dec.c
+++ b/drivers/media/platform/mtk-vcodec/mtk_vcodec_dec.c
@@ -284,7 +284,7 @@ static void mtk_vdec_update_fmt(struct mtk_vcodec_ctx *ctx,
                fmt = &mtk_video_formats[k];
                if (fmt->fourcc == pixelformat) {
                        mtk_v4l2_debug(1, "Update cap fourcc(%d -> %d)",
-                               dst_q_data->fmt.fourcc, pixelformat);
+                               dst_q_data->fmt->fourcc, pixelformat);
                        dst_q_data->fmt = fmt;
                        return;
                }
-- 
2.23.0.187.g17f5b7556c-goog

Reply via email to