media: staging: media: imx: imx7-media-csi: Drop unneeded pixel format validation
authorLaurent Pinchart <laurent.pinchart@ideasonboard.com>
Tue, 10 May 2022 09:47:34 +0000 (10:47 +0100)
committerMauro Carvalho Chehab <mchehab@kernel.org>
Fri, 8 Jul 2022 17:03:08 +0000 (18:03 +0100)
The driver won't accept unsupported pixel formats, there's thus no need
to validate it in imx7_csi_pad_link_validate().

Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Acked-by: Rui Miguel Silva <rmfrfs@gmail.com>
Tested-by: Alexander Stein <alexander.stein@ew.tq-group.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@kernel.org>
drivers/staging/media/imx/imx7-media-csi.c

index ff80187ec1a2c5042976a133dadc177bcc7bf754..5a13aa7244aac4200f879cf0330a7c02438887a1 100644 (file)
@@ -1968,7 +1968,6 @@ static int imx7_csi_pad_link_validate(struct v4l2_subdev *sd,
                                      struct v4l2_subdev_format *sink_fmt)
 {
        struct imx7_csi *csi = v4l2_get_subdevdata(sd);
-       const struct v4l2_pix_format *out_pix = &csi->vdev_fmt;
        struct media_pad *pad = NULL;
        unsigned int i;
        int ret;
@@ -2018,38 +2017,6 @@ static int imx7_csi_pad_link_validate(struct v4l2_subdev *sd,
                break;
        }
 
-       /* Validate the sink link, ensure the pixel format is supported. */
-       switch (out_pix->pixelformat) {
-       case V4L2_PIX_FMT_UYVY:
-       case V4L2_PIX_FMT_YUYV:
-       case V4L2_PIX_FMT_GREY:
-       case V4L2_PIX_FMT_Y10:
-       case V4L2_PIX_FMT_Y12:
-       case V4L2_PIX_FMT_Y14:
-       case V4L2_PIX_FMT_SBGGR8:
-       case V4L2_PIX_FMT_SGBRG8:
-       case V4L2_PIX_FMT_SGRBG8:
-       case V4L2_PIX_FMT_SRGGB8:
-       case V4L2_PIX_FMT_SBGGR10:
-       case V4L2_PIX_FMT_SGBRG10:
-       case V4L2_PIX_FMT_SGRBG10:
-       case V4L2_PIX_FMT_SRGGB10:
-       case V4L2_PIX_FMT_SBGGR12:
-       case V4L2_PIX_FMT_SGBRG12:
-       case V4L2_PIX_FMT_SGRBG12:
-       case V4L2_PIX_FMT_SRGGB12:
-       case V4L2_PIX_FMT_SBGGR14:
-       case V4L2_PIX_FMT_SGBRG14:
-       case V4L2_PIX_FMT_SGRBG14:
-       case V4L2_PIX_FMT_SRGGB14:
-               break;
-
-       default:
-               dev_dbg(csi->dev, "Invalid capture pixel format 0x%08x\n",
-                       out_pix->pixelformat);
-               return -EINVAL;
-       }
-
        return 0;
 }