On Friday 18 April 2008, Anton Vorontsov wrote: > On Thu, Apr 17, 2008 at 09:21:40PM -0500, Kumar Gala wrote: > > On Apr 17, 2008, at 5:41 PM, Anton Vorontsov wrote: > >> > >> No problem. Would you prefer this to go under drivers/gpio/ ? > > > > Yes that would be better. We actively worked on pull drivers out of > > arch/ppc back in the day.
I'm not sure I see the problem here. This is precisely the kind of *non-driver* code that normally (i.e. for all other Linux platforms I've looked at) *belongs* in the arch code ... what's the objection to doing it the same way other platforms do? And for that matter, why make it optional, via Kconfig, instead of always configuring it in, along with the GPIO interrupt support? A quick glance at one of the MPC83xx chips suggested that QE port IRQ capabilities would be very chip specific, so I'd expect that GPIO IRQ support wouldn't naturally move out of the arch tree. - Dave > Hi David, > > Do we need your Ack to go this through powerpc tree, and if so, could > you provide one? > > Thanks. > > Documentation/powerpc/booting-without-of.txt | 34 ++++--- > arch/powerpc/platforms/Kconfig | 2 + > drivers/gpio/Kconfig | 9 ++ > drivers/gpio/Makefile | 1 + > drivers/gpio/qe.c | 147 > ++++++++++++++++++++++++++ > 5 files changed, 180 insertions(+), 13 deletions(-) > create mode 100644 drivers/gpio/qe.c _______________________________________________ Linuxppc-dev mailing list Linuxppc-dev@ozlabs.org https://ozlabs.org/mailman/listinfo/linuxppc-dev