]> git.openfabrics.org - ~emulex/infiniband.git/commit
usb: gadget: mass_storage: remove >= 0 check for unsigned type
authorSebastian Andrzej Siewior <bigeasy@linutronix.de>
Mon, 3 Dec 2012 14:32:36 +0000 (15:32 +0100)
committerFelipe Balbi <balbi@ti.com>
Thu, 10 Jan 2013 10:36:47 +0000 (12:36 +0200)
commit98f3a1b90795d7216de0d56157868d174317f91a
treefa97d798f6f13ad9d818a0bebf69575f8fbaac4d
parentb642435333b23ab01d526bd854d118704434fc88
usb: gadget: mass_storage: remove >= 0 check for unsigned type

| In file included from drivers/usb/gadget/acm_ms.c:43:
| f_mass_storage.c:2199:18: warning: comparison of unsigned expression >= 0 is always true  [-Wtautological-compare]
|         if (common->lun >= 0 && common->lun < common->nluns)
|             ~~~~~~~~~~~ ^  ~

common->lun is defined as "unsigned int" so its value is always >= 0.
It is assigned via cbw->Lun which is defined as u8 so it is also not
abused as -1.

[ mina86@mina86.com : make lun unsigned int and use %u in DBG() macro for it ]

Signed-off-by: Michal Nazarewicz <mina86@mina86.com>
Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
Signed-off-by: Felipe Balbi <balbi@ti.com>
drivers/usb/gadget/f_mass_storage.c