On Tue Feb 25 14:08:40 2025 +0100, Richard Leitner wrote:
> For better readability use already available register definitions in
> hard-coded common and mode register structs.
> 
> Signed-off-by: Richard Leitner <richard.leit...@linux.dev>
> Reviewed-by: Dave Stevenson <dave.steven...@raspberrypi.com>
> Signed-off-by: Sakari Ailus <sakari.ai...@linux.intel.com>
> Signed-off-by: Hans Verkuil <hverk...@xs4all.nl>

Patch committed.

Thanks,
Hans Verkuil

 drivers/media/i2c/ov9282.c | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

---

diff --git a/drivers/media/i2c/ov9282.c b/drivers/media/i2c/ov9282.c
index 87e5d7ce5a47..c92684225789 100644
--- a/drivers/media/i2c/ov9282.c
+++ b/drivers/media/i2c/ov9282.c
@@ -296,8 +296,8 @@ static const struct ov9282_reg mode_1280x800_regs[] = {
        {0x3813, 0x08},
        {0x3814, 0x11},
        {0x3815, 0x11},
-       {0x3820, 0x40},
-       {0x3821, 0x00},
+       {OV9282_REG_TIMING_FORMAT_1, 0x40},
+       {OV9282_REG_TIMING_FORMAT_2, 0x00},
        {0x4003, 0x40},
        {0x4008, 0x04},
        {0x4009, 0x0b},
@@ -327,8 +327,8 @@ static const struct ov9282_reg mode_1280x720_regs[] = {
        {0x3813, 0x08},
        {0x3814, 0x11},
        {0x3815, 0x11},
-       {0x3820, 0x3c},
-       {0x3821, 0x84},
+       {OV9282_REG_TIMING_FORMAT_1, 0x3c},
+       {OV9282_REG_TIMING_FORMAT_2, 0x84},
        {0x4003, 0x40},
        {0x4008, 0x02},
        {0x4009, 0x05},
@@ -358,8 +358,8 @@ static const struct ov9282_reg mode_640x400_regs[] = {
        {0x3813, 0x04},
        {0x3814, 0x31},
        {0x3815, 0x22},
-       {0x3820, 0x60},
-       {0x3821, 0x01},
+       {OV9282_REG_TIMING_FORMAT_1, 0x60},
+       {OV9282_REG_TIMING_FORMAT_2, 0x01},
        {0x4008, 0x02},
        {0x4009, 0x05},
        {0x400c, 0x00},

Reply via email to