lkml.org 
[lkml]   [2020]   [Apr]   [16]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 5.4 033/232] media: imx: imx7-media-csi: Fix video field handling
    Date
    From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

    [ Upstream commit f7b8488bd39ae8feced4dfbb41cf1431277b893f ]

    Commit 4791bd7d6adc ("media: imx: Try colorimetry at both sink and
    source pads") reworked the way that formats are set on the sink pad of
    the CSI subdevice, and accidentally removed video field handling.
    Restore it by defaulting to V4L2_FIELD_NONE if the field value isn't
    supported, with the only two supported value being V4L2_FIELD_NONE and
    V4L2_FIELD_INTERLACED.

    Fixes: 4791bd7d6adc ("media: imx: Try colorimetry at both sink and source pads")
    Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
    Reviewed-by: Rui Miguel Silva <rmfrfs@gmail.com>
    Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
    Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
    Signed-off-by: Sasha Levin <sashal@kernel.org>
    ---
    drivers/staging/media/imx/imx7-media-csi.c | 4 ++++
    1 file changed, 4 insertions(+)

    diff --git a/drivers/staging/media/imx/imx7-media-csi.c b/drivers/staging/media/imx/imx7-media-csi.c
    index bfd6b5fbf4841..d24897d06947f 100644
    --- a/drivers/staging/media/imx/imx7-media-csi.c
    +++ b/drivers/staging/media/imx/imx7-media-csi.c
    @@ -1009,6 +1009,7 @@ static int imx7_csi_try_fmt(struct imx7_csi *csi,
    sdformat->format.width = in_fmt->width;
    sdformat->format.height = in_fmt->height;
    sdformat->format.code = in_fmt->code;
    + sdformat->format.field = in_fmt->field;
    *cc = in_cc;

    sdformat->format.colorspace = in_fmt->colorspace;
    @@ -1023,6 +1024,9 @@ static int imx7_csi_try_fmt(struct imx7_csi *csi,
    false);
    sdformat->format.code = (*cc)->codes[0];
    }
    +
    + if (sdformat->format.field != V4L2_FIELD_INTERLACED)
    + sdformat->format.field = V4L2_FIELD_NONE;
    break;
    default:
    return -EINVAL;
    --
    2.20.1


    \
     
     \ /
      Last update: 2020-04-16 16:05    [W:4.103 / U:0.016 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site