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 <jacopo@xxxxxxxxxx>
---
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 e5a0e6b7e84e2..0234aed43a10a 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),
},
};
--
2.29.1
