media: hantro: Prevent encoders from using post-processing
authorEzequiel Garcia <ezequiel@collabora.com>
Wed, 29 Jan 2020 21:06:08 +0000 (22:06 +0100)
committerMauro Carvalho Chehab <mchehab+huawei@kernel.org>
Mon, 24 Feb 2020 16:21:11 +0000 (17:21 +0100)
The post-processing feature is meant to be used by decoding
only. Prevent encoding jobs from enabling it.

Fixes: 8c2d66b036c77 ("media: hantro: Support color conversion via post-processing")
Signed-off-by: Ezequiel Garcia <ezequiel@collabora.com>
Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
drivers/staging/media/hantro/hantro.h

index b0faa43b3f79c81a39da6859e5d015815414bee8..327ddef45345e2d92cf713f06de86515130e4497 100644 (file)
@@ -423,7 +423,7 @@ hantro_get_dst_buf(struct hantro_ctx *ctx)
 static inline bool
 hantro_needs_postproc(struct hantro_ctx *ctx, const struct hantro_fmt *fmt)
 {
-       return fmt->fourcc != V4L2_PIX_FMT_NV12;
+       return !hantro_is_encoder_ctx(ctx) && fmt->fourcc != V4L2_PIX_FMT_NV12;
 }
 
 static inline dma_addr_t