On Tue, Feb 26, 2013 at 08:31:27AM -0800, Andrey Smirnov wrote: > From: Andrey Smirnov <andreysm@charmander.(none)> > > This patch adds all necessary header files and Kbuild plumbing for the > core driver for Silicon Laboratories Si476x series of AM/FM tuner > chips. > > The driver as a whole is implemented as an MFD device and this patch > adds a core portion of it that provides all the necessary > functionality to the two other drivers that represent radio and audio > codec subsystems of the chip. Some small nits. First - this patch should be the final patch - as we otherwise fail to build the driver due to missing files (if it gets enabled). > > Signed-off-by: Andrey Smirnov <andrew.smirnov@xxxxxxxxx> > --- > drivers/mfd/Kconfig | 13 + > drivers/mfd/Makefile | 4 + > include/linux/mfd/si476x-core.h | 525 +++++++++++++++++++++++++++++++++++++++ > 3 files changed, 542 insertions(+) > create mode 100644 include/linux/mfd/si476x-core.h > > diff --git a/drivers/mfd/Kconfig b/drivers/mfd/Kconfig > index 1c0abd4..3214927 100644 > --- a/drivers/mfd/Kconfig > +++ b/drivers/mfd/Kconfig > @@ -970,6 +970,19 @@ config MFD_WL1273_CORE > driver connects the radio-wl1273 V4L2 module and the wl1273 > audio codec. > > +config MFD_SI476X_CORE > + tristate "Support for Silicon Laboratories 4761/64/68 AM/FM radio." > + depends on I2C > + select MFD_CORE > + default n The above line is redundant - drop it. > + help > + This is the core driver for the SI476x series of AM/FM > + radio. This MFD driver connects the radio-si476x V4L2 module > + and the si476x audio codec. > + > + To compile this driver as a module, choose M here: the > + module will be called si476x-core. > + > config MFD_OMAP_USB_HOST > bool "Support OMAP USBHS core and TLL driver" > depends on USB_EHCI_HCD_OMAP || USB_OHCI_HCD_OMAP3 > diff --git a/drivers/mfd/Makefile b/drivers/mfd/Makefile > index 8b977f8..bf7627b 100644 > --- a/drivers/mfd/Makefile > +++ b/drivers/mfd/Makefile > @@ -131,6 +131,10 @@ 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 > + > +si476x-core-objs := si476x-cmd.o si476x-prop.o si476x-i2c.o Please use: si476x-core-y := si476x-cmd.o si476x-prop.o si476x-i2c.o Functionality is the same - but the above is the recommended way today. It allows you to do stuff like: si476x-core-$(BLA) += foobar.o -- 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
