media: imx: Set default sizes through macros in all drivers

All drivers use 640x480 as the default size, but they all hardcode those
values. Create two global macros named IMX_MEDIA_DEF_PIX_WIDTH and
IMX_MEDIA_DEF_PIX_HEIGHT to store the default size, and use them through
the code.

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>

diff --git a/drivers/staging/media/imx/imx-ic-prp.c b/drivers/staging/media/imx/imx-ic-prp.c
index 5b09e11b1a0edd23bba32818d776db669429fe51..f21ed881295f535603a965fcfeb53beb1bfc57de 100644 (file)
--- a/drivers/staging/media/imx/imx-ic-prp.c
+++ b/drivers/staging/media/imx/imx-ic-prp.c
@@ -442,7 +442,9 @@ static int prp_registered(struct v4l2_subdev *sd)
        /* set a default mbus format  */
        imx_media_enum_ipu_formats(&code, 0, PIXFMT_SEL_YUV);
 
-       return imx_media_init_mbus_fmt(&priv->format_mbus, 640, 480, code,
+       return imx_media_init_mbus_fmt(&priv->format_mbus,
+                                      IMX_MEDIA_DEF_PIX_WIDTH,
+                                      IMX_MEDIA_DEF_PIX_HEIGHT, code,
                                       V4L2_FIELD_NONE, NULL);
 }
 

diff --git a/drivers/staging/media/imx/imx-ic-prpencvf.c b/drivers/staging/media/imx/imx-ic-prpencvf.c
index 74f5de466d5dbc1dbae0c5fa6e471951a49a992d..47df1a5a1ae8bf102d95fde7d477f770bcabe40d 100644 (file)
--- a/drivers/staging/media/imx/imx-ic-prpencvf.c
+++ b/drivers/staging/media/imx/imx-ic-prpencvf.c
@@ -1255,8 +1255,9 @@ static int prp_registered(struct v4l2_subdev *sd)
 
        for (i = 0; i < PRPENCVF_NUM_PADS; i++) {
                ret = imx_media_init_mbus_fmt(&priv->format_mbus[i],
-                                             640, 480, code, V4L2_FIELD_NONE,
-                                             &priv->cc[i]);
+                                             IMX_MEDIA_DEF_PIX_WIDTH,
+                                             IMX_MEDIA_DEF_PIX_HEIGHT, code,
+                                             V4L2_FIELD_NONE, &priv->cc[i]);
                if (ret)
                        return ret;
        }

diff --git a/drivers/staging/media/imx/imx-media-csi.c b/drivers/staging/media/imx/imx-media-csi.c
index ef5add079774edf440e525feeb732f0ff7fb8450..b9d3f98a2c8f1d2a507b3b7d26d54ca6bc1a1b82 100644 (file)
--- a/drivers/staging/media/imx/imx-media-csi.c
+++ b/drivers/staging/media/imx/imx-media-csi.c
@@ -1758,8 +1758,9 @@ static int csi_registered(struct v4l2_subdev *sd)
 
                /* set a default mbus format  */
                ret = imx_media_init_mbus_fmt(&priv->format_mbus[i],
-                                             640, 480, code, V4L2_FIELD_NONE,
-                                             &priv->cc[i]);
+                                             IMX_MEDIA_DEF_PIX_WIDTH,
+                                             IMX_MEDIA_DEF_PIX_HEIGHT, code,
+                                             V4L2_FIELD_NONE, &priv->cc[i]);
                if (ret)
                        goto put_csi;
 
@@ -1772,10 +1773,10 @@ static int csi_registered(struct v4l2_subdev *sd)
        priv->skip = &csi_skip[0];
 
        /* init default crop and compose rectangle sizes */
-       priv->crop.width = 640;
-       priv->crop.height = 480;
-       priv->compose.width = 640;
-       priv->compose.height = 480;
+       priv->crop.width = IMX_MEDIA_DEF_PIX_WIDTH;
+       priv->crop.height = IMX_MEDIA_DEF_PIX_HEIGHT;
+       priv->compose.width = IMX_MEDIA_DEF_PIX_WIDTH;
+       priv->compose.height = IMX_MEDIA_DEF_PIX_HEIGHT;
 
        priv->fim = imx_media_fim_init(&priv->sd);
        if (IS_ERR(priv->fim)) {

diff --git a/drivers/staging/media/imx/imx-media-vdic.c b/drivers/staging/media/imx/imx-media-vdic.c
index 879329f81f79ded08cf017c6745508dece9363a8..395b850736facb590c7e2d3b98ec5e63ab28b077 100644 (file)
--- a/drivers/staging/media/imx/imx-media-vdic.c
+++ b/drivers/staging/media/imx/imx-media-vdic.c
@@ -856,8 +856,9 @@ static int vdic_registered(struct v4l2_subdev *sd)
 
                /* set a default mbus format  */
                ret = imx_media_init_mbus_fmt(&priv->format_mbus[i],
-                                             640, 480, code, V4L2_FIELD_NONE,
-                                             &priv->cc[i]);
+                                             IMX_MEDIA_DEF_PIX_WIDTH,
+                                             IMX_MEDIA_DEF_PIX_HEIGHT, code,
+                                             V4L2_FIELD_NONE, &priv->cc[i]);
                if (ret)
                        return ret;
 

diff --git a/drivers/staging/media/imx/imx-media.h b/drivers/staging/media/imx/imx-media.h
index c8b6a43d0d7c386fcb6a7a735dc7e1e5a64884a6..1abb9bb88c12c864962c5a151f47e034408a3eae 100644 (file)
--- a/drivers/staging/media/imx/imx-media.h
+++ b/drivers/staging/media/imx/imx-media.h
@@ -15,6 +15,9 @@
 #include <media/videobuf2-dma-contig.h>
 #include <video/imx-ipu-v3.h>
 
+#define IMX_MEDIA_DEF_PIX_WIDTH                640
+#define IMX_MEDIA_DEF_PIX_HEIGHT       480
+
 /*
  * Enumeration of the IPU internal sub-devices
  */

diff --git a/drivers/staging/media/imx/imx6-mipi-csi2.c b/drivers/staging/media/imx/imx6-mipi-csi2.c
index 4f8fcc91aaae156342e3a46629f5c51d688ece7a..9020541edb89438ef98cfa201ed4baaaf7efc4ed 100644 (file)
--- a/drivers/staging/media/imx/imx6-mipi-csi2.c
+++ b/drivers/staging/media/imx/imx6-mipi-csi2.c
@@ -571,7 +571,9 @@ static int csi2_registered(struct v4l2_subdev *sd)
 
        /* set a default mbus format  */
        return imx_media_init_mbus_fmt(&csi2->format_mbus,
-                                     640, 480, 0, V4L2_FIELD_NONE, NULL);
+                                     IMX_MEDIA_DEF_PIX_WIDTH,
+                                     IMX_MEDIA_DEF_PIX_HEIGHT, 0,
+                                     V4L2_FIELD_NONE, NULL);
 }
 
 static const struct media_entity_operations csi2_entity_ops = {