AMD Geode CS553X GPIO driver

Deepak Saxena dsaxena at laptop.org
Mon Dec 29 17:21:09 EST 2008


On Dec 27 2008, at 17:22, Alessandro Zummo was caught saying:
> On Sat, 27 Dec 2008 08:48:45 -0700 (MST)
> jordan at cosmicpenguin.net wrote:
> 
> > I'll ack this to live in geode_32.c with the naming change and Deepak's
> > blessing.
> 
>  ok. so geode_32. will instantiate both the platform device and the platform driver?
>  or should the driver go in the forthcoming drivers/platform/x86 while keeping
>  the device in geode_32.c ?

I would think the driver should go in drivers/gpio like your current patch
has it, but I've not been following any discussions on driver tree layout.
The device should go in either geode_32.c or more likely in a platform 
specific place like arch/x86/kernel/olpc.c.

Question about the patch. What is the purpose of the gpio_base member in
the platform data?  

~Deepak


-- 
 Deepak Saxena - Kernel Developer, One Laptop Per Child
   _____   __o                                   (o>
------    -\<,  Give One Laptop, Get One Laptop  //\
 ----- ( )/ ( )  http://www.amazon.com/xo        V_/_
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~



More information about the Linux-geode mailing list