On Wed Sep 30 14:41:24 2026 +0800, Hermes Wu wrote:
> 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 <[email protected]>
> Signed-off-by: Hans Verkuil <[email protected]>
Patch committed.
Thanks,
Hans Verkuil
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 b09b331b2036..54c23bd44f25 100644
--- a/drivers/media/i2c/it6625.c
+++ b/drivers/media/i2c/it6625.c
@@ -378,9 +378,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;
}
@@ -390,9 +388,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;
}
@@ -1898,11 +1894,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]);
_______________________________________________
linuxtv-commits mailing list -- [email protected]
To unsubscribe send an email to [email protected]
