On Mon, Aug 7, 2017 at 11:53 AM, Sergei Shtylyov
[off-list ref] wrote:

On 8/7/2017 12:07 PM, Linus Walleij wrote:

quoted

quoted

quoted

quoted

of_irq_get() may return  any negative error number as well as 0 on
failure,
while the driver only checks for -EPROBE_DEFER, blithely continuing
with
the call to gpiochip_set_chained_irqchip() -- that function expects the
parent IRQ as *unsigned int*, so would probably do nothing  when a
large
IRQ number resulting from a conversion of a negative error number is
passed
to it, however passing 0 would probably work but the driver won't
receive
valid GPIO bank interrupts.

Check for 'ret <= 0' instead and return -ENXIO from the driver's probe
iff
of_irq_get() returned 0.

Fixes: f9367793293d ("pinctrl: sirf: add sirf atlas7 pinctrl and gpio
support")
Signed-off-by: Sergei Shtylyov <redacted>

Patch applied!

   Thanks! But to which branch? I'm not seeing it in linux-pinctrl.git...

devel.

   Still not seeing the patch, perhaps you forgot to push?

Nah just haven't had time to push yet...

My mailbox is filling up with new patches at the same pace that I
apply them it seems.

Linus