WARNING: usleep_range should not use min == max args; see Documentation/timers/timers-howto.txt

Hi Sergio,

FYI, there are new warnings show up in

tree:   git://linuxtv.org/media_tree.git master
head:   c29b8f3149f2916e98fc3b8d6c1df2137d003979
commit: b4a0477c0b87c5b7a20a84df8cf81311d1efb226 [91/499] [media] v4l: omap4iss: Add support for OMAP4 camera interface - CSI receivers

:::::: branch date: 44 minutes ago
:::::: commit date: 9 weeks ago

scripts/checkpatch.pl 0001-media-v4l-omap4iss-Add-support-for-OMAP4-camera-inte.patch

# many are suggestions rather than must-fix

WARNING: usleep_range should not use min == max args; see Documentation/timers/timers-howto.txt #548: drivers/staging/media/omap4iss/iss_csi2.c:516:
+ usleep_range(100, 100);

WARNING: Prefer netdev_err(netdev, ... then dev_err(dev, ... then pr_err(... to printk(KERN_ERR ... #552: drivers/staging/media/omap4iss/iss_csi2.c:520:
+ printk(KERN_ERR "CSI2: Soft reset try count exceeded!\n");

WARNING: usleep_range should not use min == max args; see Documentation/timers/timers-howto.txt #566: drivers/staging/media/omap4iss/iss_csi2.c:534:
+ usleep_range(100, 100);

WARNING: Prefer netdev_err(netdev, ... then dev_err(dev, ... then pr_err(... to printk(KERN_ERR ... #570: drivers/staging/media/omap4iss/iss_csi2.c:538:
+ printk(KERN_ERR

WARNING: quoted string split across lines #789: drivers/staging/media/omap4iss/iss_csi2.c:757:
+ dev_dbg(iss->dev, "CSI2: ComplexIO Error IRQ "
+ "%x\n", cpxio1_irqstatus);

WARNING: quoted string split across lines #799: drivers/staging/media/omap4iss/iss_csi2.c:767:
+ dev_dbg(iss->dev, "CSI2 Err:"
+ " OCP:%d,"

WARNING: quoted string split across lines #800: drivers/staging/media/omap4iss/iss_csi2.c:768:
+ " OCP:%d,"
+ " Short_pack:%d,"

WARNING: quoted string split across lines #801: drivers/staging/media/omap4iss/iss_csi2.c:769:
+ " Short_pack:%d,"
+ " ECC:%d,"

WARNING: quoted string split across lines #802: drivers/staging/media/omap4iss/iss_csi2.c:770:
+ " ECC:%d,"
+ " CPXIO:%d,"

WARNING: quoted string split across lines #803: drivers/staging/media/omap4iss/iss_csi2.c:771:
+ " CPXIO:%d,"
+ " FIFO_OVF:%d,"

WARNING: quoted string split across lines #804: drivers/staging/media/omap4iss/iss_csi2.c:772:
+ " FIFO_OVF:%d,"
+ "\n",

WARNING: Prefer netdev_err(netdev, ... then dev_err(dev, ... then pr_err(... to printk(KERN_ERR ... #1649: drivers/staging/media/omap4iss/iss_csiphy.c:81:
+ printk(KERN_ERR "CSI2 CIO set power failed!\n");

WARNING: line over 80 characters
#1749: drivers/staging/media/omap4iss/iss_csiphy.c:181:
+ if (lanes->data[i].pol > 1 || lanes->data[i].pos > (csi2->phy->max_data_lanes + 1))

WARNING: line over 80 characters
#1759: drivers/staging/media/omap4iss/iss_csiphy.c:191:
+ if (lanes->clk.pol > 1 || lanes->clk.pos > (csi2->phy->max_data_lanes + 1))

---
0-DAY kernel build testing backend Open Source Technology Center
http://lists.01.org/mailman/listinfo/kbuild Intel Corporation