Re: [PATCH v2 1/6] arm: davinci: Fix low level gpio irq handlers' argument

2011-07-11 Thread Ido Yariv
Hi Sekhar, On Mon, Jul 11, 2011 at 05:28:44PM +0530, Nori, Sekhar wrote: > Thanks for the bug fix. > > How about setting the handler data for bank IRQ in > davinci_gpio_irq_setup() to &chips[bank]? > > chips[bank].regs should give you the register base address > (g) and chips[bank].irq_base shou

RE: [PATCH v2 1/6] arm: davinci: Fix low level gpio irq handlers' argument

2011-07-11 Thread Nori, Sekhar
Hi Ido, On Sun, Jul 10, 2011 at 18:44:34, Ido Yariv wrote: > Commit 7416401 ("arm: davinci: Fix fallout from generic irq chip > conversion") introduced a bug, causing low level interrupt handlers to > get a bogus irq number as an argument. The gpio irq handler falsely > assumes that the handler da