[PATCH 06/21] media: i2c: it6625: use unsigned int loop indices in table lookups
From: Hermes Wu via B4 Relay
Date: Fri Sep 18 2026 - 05:12:41 EST
From: Hermes Wu <Hermes.wu@xxxxxxxxxx>
it6625_csi_format_idx(), it6625_csi_mbus_code_idx(), and
it6625_regdump_print() iterate with a plain-signed loop index against
an unsigned bound. Declare the index as unsigned int directly in the
for() statement instead.
Signed-off-by: Hermes Wu <Hermes.wu@xxxxxxxxxx>
---
drivers/media/i2c/it6625.c | 12 +++---------
1 file changed, 3 insertions(+), 9 deletions(-)
diff --git a/drivers/media/i2c/it6625.c b/drivers/media/i2c/it6625.c
index 97780c3d2d8fd3567ef18b3d2fc46b4e1f0758d7..9077f66187c647f85e41e571ff0d52e5e2ffdfdf 100644
--- a/drivers/media/i2c/it6625.c
+++ b/drivers/media/i2c/it6625.c
@@ -375,9 +375,7 @@ static const struct it6625_format_info {
static inline int it6625_csi_format_idx(u8 csi_format)
{
- int i;
-
- for (i = 0; i < ARRAY_SIZE(it6625_formats); i++) {
+ for (unsigned int i = 0; i < ARRAY_SIZE(it6625_formats); i++) {
if (it6625_formats[i].csi_format == csi_format)
return i;
}
@@ -387,9 +385,7 @@ static inline int it6625_csi_format_idx(u8 csi_format)
static inline int it6625_csi_mbus_code_idx(u32 mbus_fmt_code)
{
- int i;
-
- for (i = 0; i < ARRAY_SIZE(it6625_formats); i++) {
+ for (unsigned int i = 0; i < ARRAY_SIZE(it6625_formats); i++) {
if (it6625_formats[i].mbus_fmt_code == mbus_fmt_code)
return i;
}
@@ -1878,11 +1874,9 @@ static int it6625_v4l2_init_controls(struct v4l2_subdev *sd)
static void it6625_regdump_print(struct seq_file *s, const u8 *reg_buf)
{
- int i;
-
seq_puts(s, " 0x00 0x01 0x02 0x03 0x04 0x05 0x06 0x07 0x08 0x09 0x0A 0x0B 0x0C 0x0D 0x0E 0x0F\n");
- for (i = 0; i < 256; i++) {
+ for (unsigned int i = 0; i < 256; i++) {
if (i % 16 == 0)
seq_printf(s, "[%02X] ", i & 0xF0);
seq_printf(s, "0x%02X ", reg_buf[i]);
--
2.34.1