Re: [PATCH v2] mfd: add TPS6586x driver

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: David Brownell
Date: Sunday, August 8, 2010 - 4:51 pm

> > I've left the GPIO code in the driver because of

I'd certainly prefer that.



I prefer drivers/GPIO code to be standalone chips
and SOC/ASIC/MFD GPIO support to stick together.

Classic example:  arch/arm/... almost every SOC
has its own GPIO support coupled with the rest
of its core code (GPIOs being widely used as IRQ
support, which is also core support). 


I prefer not seeing support for one chip end up
scattered throughout the source tree.  When one
of  the "sub-drivers" is very complicated (audio
and video come to mind) I object less, but that
kind of scattering still seems worth avoiding.

Some of Intel's platform chips aren't supported in
what I'd call very clean ways -- they're scattered,
with GPIO fragments in drivers/gpio (where I would
rather they not live, but I have no current notion
of better homes, lacking one directory tying all of
those MFD/SOC/Southbridge/... things together.

- Dave


- Dave



--
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
Re: [PATCH v2] mfd: add TPS6586x driver, David Brownell, (Sun Aug 8, 4:51 pm)