Hi Linus, On 11.10.2013 13:11, Linus Walleij wrote: > On Tue, Oct 1, 2013 at 3:55 PM, Daniel Mack <zonque@xxxxxxxxx> wrote: >> +Example: >> + >> + wake_up { >> + compatible = "gpio-wakeup"; >> + gpios = <&gpio0 19 0>; >> + }; > > This will not work if that GPIO chip is not capable of supporting > interrupts on that GPIO line right? Of course. It's the IRQ that wakes up the system, after all :) > We have recently had a very long discussion about this: such > GPIO chips will also be marked "interrupt-controller" and you > should be able to just state interrupt-parent and > interrupts = <>; for this. (And it should accept an array.) Well, but that's not very intuitive. I was aiming for a drop-in replacement for what I currently use for this purpose: a fake GPIO key input device which is marked as wakeup source (linux,wakeup). Schematics and everything else use GPIO notation, and so should the DTB binding here. > Since what the driver will then eventually provide is to > flag an IRQ line as wakeup, I wonder if this should not just > simply go into the interrupt core, or atleast of/irq.c. But for that, the IRQ line must be requested exclusively and handled as well, no? If not, how would you handle cases where an interrupt is marked as wakeup source by the core, but used by another driver which calls disable_irq_wake() on it for whatever reason? Thanks, Daniel -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html