Hi Matti, On Fri, Dec 03, 2010 at 05:02:27PM +0200, Matti J. Aaltonen wrote: > This is the core of the WL1273 FM radio driver, it connects > the two child modules. The two child drivers are > drivers/media/radio/radio-wl1273.c and sound/soc/codecs/wl1273.c. > > The radio-wl1273 driver implements the V4L2 interface and communicates > with the device. The ALSA codec offers digital audio, without it only > analog audio is available. The driver looks much better now. If that goes through Mauro's tree, please add my: Acked-by: Samuel Ortiz <sameo@xxxxxxxxxxxxxxx> I just have one nitpick comment here: > diff --git a/drivers/mfd/Makefile b/drivers/mfd/Makefile > index f54b365..b59be8b 100644 > --- a/drivers/mfd/Makefile > +++ b/drivers/mfd/Makefile > @@ -81,3 +81,4 @@ obj-$(CONFIG_MFD_JANZ_CMODIO) += janz-cmodio.o > obj-$(CONFIG_MFD_JZ4740_ADC) += jz4740-adc.o > obj-$(CONFIG_MFD_TPS6586X) += tps6586x.o > obj-$(CONFIG_MFD_VX855) += vx855.o > +obj-$(CONFIG_MFD_WL1273_CORE) += wl1273-core.o > \ No newline at end of file Please add a new line here. Cheers, Samuel. -- Intel Open Source Technology Centre http://oss.intel.com/ -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html