Re: [PATCHv5 2/4] TPS65910: GPIO: Add GPIO driver

From: Samuel Ortiz
Date: Fri May 13 2011 - 14:42:04 EST


Hi Grant,

On Fri, Apr 29, 2011 at 04:06:51PM -0600, Grant Likely wrote:
> On Fri, Apr 29, 2011 at 04:23:20PM +0100, Liam Girdwood wrote:
> > On 27/04/11 16:51, Jorge Eduardo Candelaria wrote:
> > > From: Graeme Gregory <gg@xxxxxxxxxxxxxxx>
> > >
> > > TPS65910 has one configurable GPIO that can be used for several
> > > purposes. Subsequent versions of the TPS chip support more than
> > > one GPIO.
> > >
> > > Signed-off-by: Graeme Gregory <gg@xxxxxxxxxxxxxxx>
> > > Signed-off-by: Jorge Eduardo Candelaria <jedu@xxxxxxxxxxxxxxx>
> > > ---
> > > drivers/mfd/Makefile | 2 +-
> > > drivers/mfd/tps65910-gpio.c | 91 ++++++++++++++++++++++++++++++++++++++++++
> > > drivers/mfd/tps65910.c | 7 +++
> > > include/linux/mfd/tps65910.h | 2 +
> > > 4 files changed, 101 insertions(+), 1 deletions(-)
> > > create mode 100644 drivers/mfd/tps65910-gpio.c
> > >
> >
> > Grant, if you are happy with this I can take all via regulator.
>
> Yes, I'm okay with it.
Would you be ok with moving this one to drivers/gpio/ ? And in general, moving
most of the GPIO drivers under drivers/mfd/ back to drivers/gpio/ ?
David didn't want to take the basic MFD related GPIO drivers, that's why we
have several of them under mfd.

Cheers,
Samuel.

--
Intel Open Source Technology Centre
http://oss.intel.com/
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/