> On 03-12-2012 18:32, Sebastian Andrzej Siewior wrote: >> | 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. > >> [min...@mina86.com: make lun unsigned int and use %u in DBG() macro for it] >> Acked-by: Michal Nazarewicz <min...@mina86.com>
On Tue, Dec 04 2012, Sergei Shtylyov <sshtyl...@mvista.com> wrote: > Michal, shouldn't you rather sign off on the patch if it includes a chunk > from you anyway? Makes sense I guess, I'm fine either way: Signed-off-by: Michal Nazarewicz <min...@mina86.com> >> Signed-off-by: Sebastian Andrzej Siewior <bige...@linutronix.de> -- Best regards, _ _ .o. | Liege of Serenely Enlightened Majesty of o' \,=./ `o ..o | Computer Science, Michał “mina86” Nazarewicz (o o) ooo +----<email/xmpp: m...@google.com>--------------ooO--(_)--Ooo--
pgpkQCFCAIkJN.pgp
Description: PGP signature
