From mboxrd@z Thu Jan 1 00:00:00 1970 From: robert.jarzmik@free.fr (Robert Jarzmik) Date: Wed, 13 May 2015 21:52:29 +0200 Subject: [patch] ARM: pxa: pxa_cplds: signedness bug in probe In-Reply-To: <20150513113239.GA31617@mwanda> (Dan Carpenter's message of "Wed, 13 May 2015 14:32:39 +0300") References: <20150513113239.GA31617@mwanda> Message-ID: <87lhgsl0g2.fsf@belgarion.home> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Dan Carpenter writes: > "base_irq" needs to be signed for the error handling to work. Also we > can remove the initialization because we re-assign it later. Indeed. I suppose smatch found this one with : arch/arm/mach-pxa/pxa_cplds_irqs.c:126 cplds_probe() warn: unsigned 'base_irq' is never less than zero. I still don't see which tool found the "re-assign" part ... Anyway, thanks for this, I'll queue it in pxa/fixes. Cheers. -- Robert