Re: [PATCH 11/9] move twl4030-gpio to drivers/gpio

[David Brownell <david-b@pacbell.net>]

On Wednesday 01 October 2008, Jean Delvare wrote:
> On Wed, 1 Oct 2008 08:58:12 -0700, David Brownell wrote:
> > On Wednesday 01 October 2008, Felipe Contreras wrote:
> > > Hmm, I'm seeing two different configs:
> > > CONFIG_TWL4030_GPIO
> > > CONFIG_GPIO_TWL4030
> > > 
> > > I guess the defconfigs are wrong.
> > 
> > Right, they should get fixed.
> > 
> > The same issue will come when drivers/i2c/chips/twl4030-core.c
> > moves to drivers/mfd/twl4030-core.c (...)
> 
> Yes, please! I think I'll kill drivers/i2c/chips soon.

But in the OMAP tree it's still chock-full of goodness:

~/kernel/beagle/drivers/i2c/chips$ ls
at24.c                Kconfig     pca9539.c      tps65010.c      twl4030-power.c
ds1682.c              lp5521.c    pcf8574.c      tsl2550.c       twl4030-poweroff.c
eeprom.c              Makefile    pcf8575.c      tsl2563.c       twl4030-pwrbutton.c
gpio_expander_omap.c  max6875.c   pcf8591.c      twl4030-core.c  twl4030-pwrirq.c
isp1301_omap.c        menelaus.c  tlv320aic23.c  twl4030-madc.c  twl4030-usb.c
~/kernel/beagle/drivers/i2c/chips$ 

So it's not yet ready to be removed ... at least there.  ;)

Although I certainly agree all that twl4030 [1] stuff should move.
It just takes time.

(Quick audit:   the AIC23 seems nearly ready to move to ALSA.
pcf8574/pcf8575/pca9539 have drivers/gpio versions, and that
gpio_expander_omap is yet another pcf8574 helper.  I'm not
sure the LED framework is ready for lp5521 yet (programmable
RGB led support).

- Dave

[1] same as http://focus.ti.com/docs/prod/folders/print/tps65950.html
    fyi ... programming manuals not yet public, but that data sheet
    may be interesting to you for a quick skim (8 pages).  As I2C
    chips go, it's quite powerful; block diagram on page 2.  ;)