This is an automatic generated email to let you know that the following patch 
were queued:

Subject: media: i2c: imx219: Drop IMX219_VTS_* macros
Author:  Laurent Pinchart <laurent.pinch...@ideasonboard.com>
Date:    Sun Sep 24 18:33:02 2023 +0300

The IMX219_VTS_* macros define default VTS values for the modes
supported by the driver. They are used in a single place, and hinder
readability compared to using the value directly as a decimal number.
Drop them.

Signed-off-by: Laurent Pinchart <laurent.pinch...@ideasonboard.com>
Reviewed-by: Dave Stevenson <dave.steven...@raspberrypi.com>
Reviewed-by: Jacopo Mondi <jacopo.mo...@ideasonboard.com>
Signed-off-by: Sakari Ailus <sakari.ai...@linux.intel.com>
Signed-off-by: Hans Verkuil <hverkuil-ci...@xs4all.nl>

 drivers/media/i2c/imx219.c | 12 ++++--------
 1 file changed, 4 insertions(+), 8 deletions(-)

---

diff --git a/drivers/media/i2c/imx219.c b/drivers/media/i2c/imx219.c
index 82e5d1dad14a..76551e6db22a 100644
--- a/drivers/media/i2c/imx219.c
+++ b/drivers/media/i2c/imx219.c
@@ -71,10 +71,6 @@
 
 /* V_TIMING internal */
 #define IMX219_REG_VTS                 CCI_REG16(0x0160)
-#define IMX219_VTS_15FPS               0x0dc6
-#define IMX219_VTS_30FPS_1080P         0x06e3
-#define IMX219_VTS_30FPS_BINNED                0x06e3
-#define IMX219_VTS_30FPS_640x480       0x06e3
 #define IMX219_VTS_MAX                 0xffff
 
 #define IMX219_VBLANK_MIN              4
@@ -301,7 +297,7 @@ static const struct imx219_mode supported_modes[] = {
                        .width = 3280,
                        .height = 2464
                },
-               .vts_def = IMX219_VTS_15FPS,
+               .vts_def = 3526,
        },
        {
                /* 1080P 30fps cropped */
@@ -313,7 +309,7 @@ static const struct imx219_mode supported_modes[] = {
                        .width = 1920,
                        .height = 1080
                },
-               .vts_def = IMX219_VTS_30FPS_1080P,
+               .vts_def = 1763,
        },
        {
                /* 2x2 binned 30fps mode */
@@ -325,7 +321,7 @@ static const struct imx219_mode supported_modes[] = {
                        .width = 3280,
                        .height = 2464
                },
-               .vts_def = IMX219_VTS_30FPS_BINNED,
+               .vts_def = 1763,
        },
        {
                /* 640x480 30fps mode */
@@ -337,7 +333,7 @@ static const struct imx219_mode supported_modes[] = {
                        .width = 1280,
                        .height = 960
                },
-               .vts_def = IMX219_VTS_30FPS_640x480,
+               .vts_def = 1763,
        },
 };
 

_______________________________________________
linuxtv-commits mailing list
linuxtv-commits@linuxtv.org
https://www.linuxtv.org/cgi-bin/mailman/listinfo/linuxtv-commits

Reply via email to