[PATCH v2] ARM: LPC32xx: ADC support
Wolfram Sang
w.sang at pengutronix.de
Tue Feb 14 17:09:31 UTC 2012
On Sun, Feb 12, 2012 at 09:09:05PM +0100, Roland Stigge wrote:
> This patch adds a 3-channel ADC driver for the LPC32xx ARM SoC
>
> Signed-off-by: Roland Stigge <stigge at antcom.de>
You need to split the mach-specific stuff and the driver, then submit
them seperately via the respective maintainers.
> Index: linux-arm-soc/arch/arm/mach-lpc32xx/phy3250.c
> ===================================================================
> --- linux-arm-soc.orig/arch/arm/mach-lpc32xx/phy3250.c 2012-02-12 20:04:49.000000000 +0100
> +++ linux-arm-soc/arch/arm/mach-lpc32xx/phy3250.c 2012-02-12 20:17:13.000000000 +0100
> @@ -252,6 +252,9 @@
> &lpc32xx_i2c2_device,
> &lpc32xx_watchdog_device,
> &lpc32xx_gpio_led_device,
> +#if defined(CONFIG_LPC32XX_ADC)
> + &lpc32xx_adc_device,
> +#endif
Probably personal taste, yet I'd skip the #ifdef.
The rest of the mach stuff looks fine to me.
Thanks,
Wolfram
--
Pengutronix e.K. | Wolfram Sang |
Industrial Linux Solutions | http://www.pengutronix.de/ |
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://driverdev.linuxdriverproject.org/pipermail/driverdev-devel/attachments/20120214/6cbbb1b9/attachment.asc>
More information about the devel
mailing list