[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <200811221108.54331.david-b@pacbell.net>
Date: Sat, 22 Nov 2008 11:08:53 -0800
From: David Brownell <david-b@...bell.net>
To: Daniel Ribeiro <drwyrm@...il.com>,
Eric Miao <eric.y.miao@...il.com>, stefan@...enfreihafen.org
Cc: spi-devel-general@...ts.sourceforge.net, sameo@...nedhand.com,
linux-arm-kernel@...ts.arm.linux.org.uk,
linux-kernel@...r.kernel.org
Subject: Re: [spi-devel-general] [patch 05/14] mfd: PCAP2 driver
On Saturday 22 November 2008, Daniel Ribeiro wrote:
> Em Sáb, 2008-11-22 às 22:01 +0800, Eric Miao escreveu:
> > On Sat, Nov 22, 2008 at 1:25 PM, David Brownell <david-b@...bell.net> wrote:
> > > On Friday 21 November 2008, stefan@...enfreihafen.org wrote:
> > >> Since the last submission we are also using the spi subsystem and
> > >> pxa2xx-spi instead of ssp.c directly as before.
> > >
> > > I took a quick glance at this and it seemed like it should be
> > > able to build without depending on PXA ... should certainly
> > > not include <mach/ssp.h> or <mach/regs-ssp.h>, and it doesn't
> > > look like it needs PXA-specific stuff like <mach/mfp-pxa27x.h>
> > > either.
> > >
> > > It's also worth removing the reverse dependencies ("select X")
> > > from Kconfig; they don't work very well for the things which
> > > those dependencies rely on.
> > >
> > I'm wondering if it call fall into the regulator framework, along with
> > a hwmon driver with its attributes being exported by sysfs.
Make it work like other MFD drivers, with child devices
to encapsulate such functionality ... and drivers specific
to those functionalities.
Such as a regulator framework driver, an RTC, and so on.
> The ADC is also used by the touchscreen driver, and later will be used
> for accessory detection (usb otg, charger, EMU (audio and uart on usb
> port)).
ADC integration may be an interesting little puzzle.
> Currently the only user for the voltage regulators is the MMC, i plan to
> use the regulator subsystem when more users arise.
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists