[linux-sunxi] Re: [PATCH 01/11] pinctrl: sunxi: create irq/pin mapping during init
From: Chen-Yu Tsai <hidden>
Date: 2014-05-27 14:21:04
Also in:
linux-devicetree, linux-mmc, linux-wireless
Possibly related (same subject, not in this thread)
- 2014-05-28 · Re: Re: [PATCH 01/11] pinctrl: sunxi: create irq/pin mapping during init · Linus Walleij <hidden>
- 2014-05-27 · Re: [PATCH 01/11] pinctrl: sunxi: create irq/pin mapping during init · Linus Walleij <hidden>
- 2014-05-27 · Re: Re: [PATCH 01/11] pinctrl: sunxi: create irq/pin mapping during init · Chen-Yu Tsai <hidden>
- 2014-05-27 · Re: [PATCH 01/11] pinctrl: sunxi: create irq/pin mapping during init · Maxime Ripard <hidden>
- 2014-05-26 · [PATCH 01/11] pinctrl: sunxi: create irq/pin mapping during init · Hans de Goede <hidden>
On Tue, May 27, 2014 at 10:11 PM, Linus Walleij [off-list ref] wrote:
On Mon, May 26, 2014 at 9:47 AM, Hans de Goede [off-list ref] wrote:quoted
From: Chen-Yu Tsai <redacted> The irq/pin mapping is used to lookup the pin to mux to the irq function when the irq is enabled. It is created when gpio_to_irq is called. Creating the mapping during init allows us to map the interrupts directly from the device tree. Signed-off-by: Chen-Yu Tsai <redacted> Signed-off-by: Hans de Goede <redacted>This is exactly correct. Patch applied.
Thanks.
I tried to hack the sunxi driver to even use the gpiolib irqchip helpers but exactly this complex map thing in ->irq_array[] git me stuck. If any of you guys could be so nice to take a stab at this (compare to other commits converting drivers to use gpiolibs irqchip helpers) then I'd be very happy.
We still need to rework the whole irqchip stuff for sun6i/sun8i, which have 1 parent interrupt per EINT capable pin bank. Earlier discussions with Maxime (on IRC IIRC) suggested moving this over to gpiolib irqchip helpers might be more work than using our own irqchip implementation, requiring 1 gpiochip per pin group? (not sure) ChenYu