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

Subject: media: ov5647: Rename SBGGR8 VGA mode
Author:  Jacopo Mondi <[email protected]>
Date:    Thu Nov 19 17:32:35 2020 +0100

Before adding new modes, rename the only existing one to report
the bit depth to distinguish it from future additions.

While at it, briefly describe the mode.

Signed-off-by: Jacopo Mondi <[email protected]>
Signed-off-by: Sakari Ailus <[email protected]>
Signed-off-by: Mauro Carvalho Chehab <[email protected]>

 drivers/media/i2c/ov5647.c | 13 +++++++------
 1 file changed, 7 insertions(+), 6 deletions(-)

---

diff --git a/drivers/media/i2c/ov5647.c b/drivers/media/i2c/ov5647.c
index e5a0e6b7e84e..0234aed43a10 100644
--- a/drivers/media/i2c/ov5647.c
+++ b/drivers/media/i2c/ov5647.c
@@ -115,7 +115,7 @@ static struct regval_list sensor_oe_enable_regs[] = {
        {0x3002, 0xe4},
 };
 
-static const struct regval_list ov5647_640x480[] = {
+static const struct regval_list ov5647_640x480_8bpp[] = {
        {0x0100, 0x00},
        {0x0103, 0x01},
        {0x3034, 0x08},
@@ -205,7 +205,8 @@ static const struct regval_list ov5647_640x480[] = {
        {0x0100, 0x01},
 };
 
-static const struct ov5647_mode ov5647_8bit_modes[] = {
+static const struct ov5647_mode ov5647_8bpp_modes[] = {
+       /* 8-bit VGA mode: Uncentred crop 2x2 binned 1296x972 image. */
        {
                .format = {
                        .code           = MEDIA_BUS_FMT_SBGGR8_1X8,
@@ -220,16 +221,16 @@ static const struct ov5647_mode ov5647_8bit_modes[] = {
                        .width          = 1280,
                        .height         = 960,
                },
-               .reg_list       = ov5647_640x480,
-               .num_regs       = ARRAY_SIZE(ov5647_640x480)
+               .reg_list       = ov5647_640x480_8bpp,
+               .num_regs       = ARRAY_SIZE(ov5647_640x480_8bpp)
        },
 };
 
 static const struct ov5647_format_list ov5647_formats[] = {
        {
                .mbus_code      = MEDIA_BUS_FMT_SBGGR8_1X8,
-               .modes          = ov5647_8bit_modes,
-               .num_modes      = ARRAY_SIZE(ov5647_8bit_modes),
+               .modes          = ov5647_8bpp_modes,
+               .num_modes      = ARRAY_SIZE(ov5647_8bpp_modes),
        },
 };
 

_______________________________________________
linuxtv-commits mailing list
[email protected]
https://www.linuxtv.org/cgi-bin/mailman/listinfo/linuxtv-commits

Reply via email to