[PATCH 11/21] media: i2c: it6625: use unsigned int indices in EDID read/write
From: Hermes Wu via B4 Relay
Date: Fri Sep 18 2026 - 05:03:42 EST
From: Hermes Wu <Hermes.wu@xxxxxxxxxx>
it6625_read_edid() and it6625_write_edid() iterate their block loop
with signed i/bank_ctrl locals holding only non-negative values; make
them unsigned int and keep err signed. Reorder the declarations to put
the pointer before the scalars while these functions are already being
touched.
Signed-off-by: Hermes Wu <Hermes.wu@xxxxxxxxxx>
---
drivers/media/i2c/it6625.c | 6 ++++--
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/drivers/media/i2c/it6625.c b/drivers/media/i2c/it6625.c
index b580256d32f51465a620b639fc0134366cc6fcc1..328bd99f27cf84722caa184302cddeb9c3e667fb 100644
--- a/drivers/media/i2c/it6625.c
+++ b/drivers/media/i2c/it6625.c
@@ -660,8 +660,9 @@ static u64 it6625_get_pclk(struct it6625 *it6625)
static int it6625_read_edid(struct it6625 *it6625, u8 *edid, int start_block,
int num_blocks)
{
- int i, bank_ctrl, err = 0;
struct device *dev = it6625->dev;
+ unsigned int i, bank_ctrl;
+ int err = 0;
if (!edid) {
dev_err(dev, "edid buffer is NULL");
@@ -699,8 +700,9 @@ static int it6625_read_edid(struct it6625 *it6625, u8 *edid, int start_block,
static int it6625_write_edid(struct it6625 *it6625, u8 *edid, int start_block,
int num_blocks)
{
- int i, bank_ctrl, err = 0;
struct device *dev = it6625->dev;
+ unsigned int i, bank_ctrl;
+ int err = 0;
if (start_block < 0 || num_blocks <= 0 ||
start_block > EDID_NUM_BLOCKS_MAX ||
--
2.34.1