On 27.05.2014 10:07, Maxime Ripard wrote: > On Mon, May 26, 2014 at 09:47:57AM +0200, Hans de Goede wrote: >> From: Chen-Yu Tsai <w...@csie.org> >> >> The sunxi pinctrl irq chip driver does not support wakeup at the >> moment. Adding IRQCHIP_SKIP_SET_WAKE lets the irqs work with >> drivers using wakeup. >> >> Also add a name to the irq chip. >> >> Signed-off-by: Chen-Yu Tsai <w...@csie.org> Signed-off-by: Hans >> de Goede <hdego...@redhat.com> --- >> drivers/pinctrl/sunxi/pinctrl-sunxi.c | 2 ++ 1 file changed, 2 >> insertions(+) >> >> diff --git a/drivers/pinctrl/sunxi/pinctrl-sunxi.c >> b/drivers/pinctrl/sunxi/pinctrl-sunxi.c index db9ccd6..ec60c2e >> 100644 --- a/drivers/pinctrl/sunxi/pinctrl-sunxi.c +++ >> b/drivers/pinctrl/sunxi/pinctrl-sunxi.c @@ -646,6 +646,8 @@ >> static struct irq_chip sunxi_pinctrl_irq_chip = { .irq_mask_ack = >> sunxi_pinctrl_irq_mask_ack, .irq_unmask = >> sunxi_pinctrl_irq_unmask, .irq_set_type = >> sunxi_pinctrl_irq_set_type, + .name = "sunxi-pio", + >> .flags = >> IRQCHIP_SKIP_SET_WAKE, > > I'd rather see the name set to dev_name() or something like that. > This will not work that great with multiple pin controller > supporting interrupts.
Correct me if I'm wrong, but I've always thought that struct irq_chip describes a class of IRQ controllers, not particular instances. Best regards, Tomasz -- You received this message because you are subscribed to the Google Groups "linux-sunxi" group. To unsubscribe from this group and stop receiving emails from it, send an email to linux-sunxi+unsubscr...@googlegroups.com. For more options, visit https://groups.google.com/d/optout.