Thread (1 message) 1 message, 1 author, 2015-05-13
DORMANTno replies

[patch] ARM: pxa: pxa_cplds: signedness bug in probe

From: robert.jarzmik@free.fr (Robert Jarzmik)
Date: 2015-05-13 19:52:29
Also in: kernel-janitors

Dan Carpenter [off-list ref] 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
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help