From: Márton Németh <nm127@xxxxxxxxxxx> The loop counter j is declared twice in function error_handler(). Remove the redundant declaration. This will remove the following sparse warning (see "make C=1"): * symbol 'j' shadows an earlier one Signed-off-by: Márton Németh <nm127@xxxxxxxxxxx> --- diff -r 2a50a0a1c951 linux/drivers/media/video/zoran/zoran_device.c --- a/linux/drivers/media/video/zoran/zoran_device.c Sat Jan 23 00:14:32 2010 -0200 +++ b/linux/drivers/media/video/zoran/zoran_device.c Sat Jan 23 10:47:27 2010 +0100 @@ -1229,7 +1230,7 @@ u32 astat, u32 stat) { - int i, j; + int i; /* This is JPEG error handling part */ if (zr->codec_mode != BUZ_MODE_MOTION_COMPRESS && @@ -1280,6 +1281,7 @@ /* Report error */ if (zr36067_debug > 1 && zr->num_errors <= 8) { long frame; + int j; frame = zr->jpg_pend[zr->jpg_dma_tail & BUZ_MASK_FRAME]; printk(KERN_ERR -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html