On 8/20/2025 2:37 PM, Dikshita Agarwal wrote:
> 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: fde6161d91bb ("media: iris: Add HEVC and VP9 formats for decoder")
> Signed-off-by: Dikshita Agarwal <quic_dikshita@quicinc.com>
> ---
> drivers/media/platform/qcom/iris/iris_vdec.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/media/platform/qcom/iris/iris_vdec.c b/drivers/media/platform/qcom/iris/iris_vdec.c
> index d670b51c5839d1fad54d34f373cf71d5f3973a96..0f5adaac829f2263fae9ff0fa49bb17bad2edecb 100644
> --- a/drivers/media/platform/qcom/iris/iris_vdec.c
> +++ b/drivers/media/platform/qcom/iris/iris_vdec.c
> @@ -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;
>
Reviewed-by: Vikash Garodia <quic_vgarodia@quicinc.com>