Re: [PATCH v6 1/9] mfd: Add header files and Kbuild plumbing for SI476x MFD core

[Sam Ravnborg <sam@ravnborg.org>]

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@gmail.com>
> ---
>  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