Commit 2dbd2645 authored by Dikshita Agarwal's avatar Dikshita Agarwal Committed by Hans Verkuil
Browse files

media: iris: Fix format check for CAPTURE plane in try_fmt 



Previously, the format validation relied on an array of supported
formats, which only listed formats for the OUTPUT plane. This caused
failures when validating formats for the CAPTURE plane.
Update the check to validate against the only supported format on the
CAPTURE plane, which is NV12.

Fixes: fde6161d ("media: iris: Add HEVC and VP9 formats for decoder")
Cc: stable@vger.kernel.org
Reviewed-by: default avatarVikash Garodia <quic_vgarodia@quicinc.com>
Tested-by: Vikash Garodia <quic_vgarodia@quicinc.com> # X1E80100
Tested-by: Neil Armstrong <neil.armstrong@linaro.org> # on SM8550-HDK
Tested-by: Neil Armstrong <neil.armstrong@linaro.org> # on SM8650-HDK
Signed-off-by: default avatarDikshita Agarwal <quic_dikshita@quicinc.com>
Tested-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org> # x1e80100-crd
Signed-off-by: default avatarBryan O'Donoghue <bod@kernel.org>
Signed-off-by: default avatarHans Verkuil <hverkuil+cisco@kernel.org>
parent 8172f577

drivers/media/platform/qcom/iris/iris_vdec.c

+1 −1
Original line number Diff line number Diff line
@@ -158,7 +158,7 @@ int iris_vdec_try_fmt(struct iris_inst *inst, struct v4l2_format *f) 
		}

		break;

	case V4L2_BUF_TYPE_VIDEO_CAPTURE_MPLANE:

		if (!fmt) {

		if (f->fmt.pix_mp.pixelformat != V4L2_PIX_FMT_NV12) {

			f_inst = inst->fmt_dst;

			f->fmt.pix_mp.pixelformat = f_inst->fmt.pix_mp.pixelformat;

			f->fmt.pix_mp.width = f_inst->fmt.pix_mp.width;