From mboxrd@z Thu Jan 1 00:00:00 1970 From: Uwe =?iso-8859-1?Q?Kleine-K=F6nig?= Subject: Re: [PATCHv3 4/4] mc13xxx: Add i2c and spi drivers to Kconfig and Makefile Date: Mon, 20 Dec 2010 09:38:39 +0100 Message-ID: <20101220083839.GQ1940@pengutronix.de> References: <1292817055-17715-1-git-send-email-marc@cpdesign.com.au> <1292817055-17715-5-git-send-email-marc@cpdesign.com.au> Mime-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Transfer-Encoding: QUOTED-PRINTABLE Cc: linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, w.sang-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org, spi-devel-general-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org, linux-i2c-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Marc Reilly Return-path: Content-Disposition: inline In-Reply-To: <1292817055-17715-5-git-send-email-marc-DtE7ei5U7Kg0n/F98K4Iww@public.gmane.org> Sender: linux-i2c-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: linux-spi.vger.kernel.org On Mon, Dec 20, 2010 at 02:50:55PM +1100, Marc Reilly wrote: > Signed-off-by: Marc Reilly > --- > drivers/mfd/Kconfig | 22 ++++++++++++++-------- > drivers/mfd/Makefile | 2 ++ > 2 files changed, 16 insertions(+), 8 deletions(-) >=20 > diff --git a/drivers/mfd/Kconfig b/drivers/mfd/Kconfig > index 3a1493b..6980cf2 100644 > --- a/drivers/mfd/Kconfig > +++ b/drivers/mfd/Kconfig > @@ -424,20 +424,26 @@ config MFD_PCF50633 > facilities, and registers devices for the various functions > so that function-specific drivers can bind to them. > =20 > -config MFD_MC13783 > - tristate > - this needs at least a note in the commit log. And $(git grep MFD_MC13783 drivers) shows that you don't want to remove that yet. > config MFD_MC13XXX > - tristate "Support Freescale MC13783 and MC13892" > - depends on SPI_MASTER > + tristate "Support Freescale MC13XXX" Hmm, I'd prefer to have the supported numbers. Consider FSL releasing an MC13991. Then it's unclear if MFD_MC13XXX supports it. > + depends on SPI_MASTER || I2C > select MFD_CORE > - select MFD_MC13783 > help > Support for the Freescale (Atlas) PMIC and audio CODECs > MC13783 and MC13892. > - This driver provides common support for accessing the device, > + This driver provides common support for accessing the device, > additional drivers must be enabled in order to use the > - functionality of the device. > + functionality of these devices. > + > +config MFD_MC13XXX_SPI > + tristate "Support for MC13XXX via SPI" > + depends on SPI_MASTER > + select MFD_MC13XXX > + > +config MFD_MC13XXX_I2C > + tristate "Support for MC13XXX via I2C" and only mc13892 can do i2c, so writing MC13XXX doesn't make sense here= =2E > + depends on I2C > + select MFD_MC13XXX Hmm, that means that MFD_MC13XXX alone doesn't do anything useful, right? IMHO either do: config MFD_MC13XXX_SPI tristate config MFD_MC13XXX_I2C tristate config MFD_MC13XXX tristate "..." select MFD_MC13XXX_SPI if SPI_MASTER select MFD_MC13XXX_I2C if I2C ... or config MFD_MC13XXX tristate ... config MFD_MC13XXX_SPI tristate "..." select MFD_MC13XXX ... config MFD_MC13XXX_I2C tristate "..." select MFD_MC13XXX ... > config PCF50633_ADC > tristate "Support for NXP PCF50633 ADC" > diff --git a/drivers/mfd/Makefile b/drivers/mfd/Makefile > index f54b365..b7d774f 100644 > --- a/drivers/mfd/Makefile > +++ b/drivers/mfd/Makefile > @@ -42,6 +42,8 @@ obj-$(CONFIG_TWL4030_CODEC) +=3D twl4030-codec.o > obj-$(CONFIG_TWL6030_PWM) +=3D twl6030-pwm.o > =20 > obj-$(CONFIG_MFD_MC13XXX) +=3D mc13xxx-core.o > +obj-$(CONFIG_MFD_MC13XXX_SPI) +=3D mc13xxx-spi.o > +obj-$(CONFIG_MFD_MC13XXX_I2C) +=3D mc13xxx-i2c.o > =20 > obj-$(CONFIG_MFD_CORE) +=3D mfd-core.o Best regards Uwe --=20 Pengutronix e.K. | Uwe Kleine-K=F6nig = | Industrial Linux Solutions | http://www.pengutronix.de/= |