media: staging: media: imx: imx7-media-csi: Drop IC support from imx7_csi_try_colorim...
authorLaurent Pinchart <laurent.pinchart@ideasonboard.com>
Tue, 10 May 2022 02:06:45 +0000 (03:06 +0100)
committerMauro Carvalho Chehab <mchehab@kernel.org>
Fri, 8 Jul 2022 17:02:57 +0000 (18:02 +0100)
The imx7_csi_try_colorimetry() function supports the unrelated image
converter hardware as it originates from shared helpers. Drop that.

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 fd4db66ccb36d62e8b41b4b3702a14020997c640..5f346a2e83c93328ae12a7e4fe5c80c0f3262dc8 100644 (file)
@@ -2031,13 +2031,8 @@ out_unlock:
  * parameters based on the colorspace if they are uninitialized.
  *
  * tryfmt->code must be set on entry.
- *
- * If this format is destined to be routed through the Image Converter,
- * Y`CbCr encoding must be fixed. The IC supports only BT.601 Y`CbCr
- * or Rec.709 Y`CbCr encoding.
  */
-static void imx7_csi_try_colorimetry(struct v4l2_mbus_framefmt *tryfmt,
-                                    bool ic_route)
+static void imx7_csi_try_colorimetry(struct v4l2_mbus_framefmt *tryfmt)
 {
        const struct imx7_csi_pixfmt *cc;
        bool is_rgb = false;
@@ -2069,16 +2064,9 @@ static void imx7_csi_try_colorimetry(struct v4l2_mbus_framefmt *tryfmt,
                tryfmt->xfer_func =
                        V4L2_MAP_XFER_FUNC_DEFAULT(tryfmt->colorspace);
 
-       if (ic_route) {
-               if (tryfmt->ycbcr_enc != V4L2_YCBCR_ENC_601 &&
-                   tryfmt->ycbcr_enc != V4L2_YCBCR_ENC_709)
-                       tryfmt->ycbcr_enc = V4L2_YCBCR_ENC_601;
-       } else {
-               if (tryfmt->ycbcr_enc == V4L2_YCBCR_ENC_DEFAULT) {
-                       tryfmt->ycbcr_enc =
-                               V4L2_MAP_YCBCR_ENC_DEFAULT(tryfmt->colorspace);
-               }
-       }
+       if (tryfmt->ycbcr_enc == V4L2_YCBCR_ENC_DEFAULT)
+               tryfmt->ycbcr_enc =
+                       V4L2_MAP_YCBCR_ENC_DEFAULT(tryfmt->colorspace);
 
        if (tryfmt->quantization == V4L2_QUANTIZATION_DEFAULT)
                tryfmt->quantization =
@@ -2135,7 +2123,7 @@ static int imx7_csi_try_fmt(struct imx7_csi *csi,
                return -EINVAL;
        }
 
-       imx7_csi_try_colorimetry(&sdformat->format, false);
+       imx7_csi_try_colorimetry(&sdformat->format);
 
        return 0;
 }