Re: [PATCH 2/2] [MMC] Driver for TI FlashMedia card reader - Kconfig/Makefile entries

[Pierre Ossman <drzeus-list@drzeus.cx>]

Alex Dubov wrote:
> Signed-off-by: Alex Dubov <oakad@yahoo.com>
> ---
>  drivers/misc/Kconfig  |   21 ++++++++++++++++++++-
>  drivers/misc/Makefile |    4 +++-
>  drivers/mmc/Kconfig   |   14 ++++++++++++++
>  drivers/mmc/Makefile  |    1 +
>  4 files changed, 38 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/misc/Kconfig b/drivers/misc/Kconfig
> index 7fc692a..7dfe4f3 100644
> --- a/drivers/misc/Kconfig
> +++ b/drivers/misc/Kconfig
> @@ -28,5 +28,24 @@ config IBM_ASM
>  
>  	  If unsure, say N.
>  
> -endmenu
> +config TIFM_CORE
> +	tristate "TI Flash Media interface support (EXPERIMENTAL)"
> +	depends on EXPERIMENTAL
>   
> +	help
> +	  If you want support for Texas Instruments(R) Flash Media adapters
> +	  you should select this option and than also choose an appropriate
> +	  host adapter and card format driver support.
> +	  
> +	  If unsure, say N.
>   

I believe most drivers do not have this, but either a comment about M
will build a module or nothing at all.

(Yes, yes, I know my drivers do not follow this. I've been meaning to
fix that. :))

> +
> +config TIFM_7XX1
> +	tristate "TI Flash Media PCI74xx/PCI76xx host adapter support (EXPERIMENTAL)"
> +	depends on PCI && TIFM_CORE && EXPERIMENTAL
> +	default TIFM_CORE
> +	help
> +	  This option enables support for Texas Instruments(R) PCI74xx and
> +	  PCI76xx families of Flash Media adapters, found in many laptops.
>  
> +	  If unsure, say N.
> +
>   

Same here. And you should also mention (again) that card drivers are
needed to do anything useful.

> +endmenu
> diff --git a/drivers/misc/Makefile b/drivers/misc/Makefile
> index 19c2b85..b1a9d90 100644
> --- a/drivers/misc/Makefile
> +++ b/drivers/misc/Makefile
> @@ -3,5 +3,7 @@ # Makefile for misc devices that really 
>  #
>  obj- := misc.o	# Dummy rule to force built-in.o to be made
>  
> -obj-$(CONFIG_IBM_ASM)	+= ibmasm/
> +obj-$(CONFIG_IBM_ASM)		+= ibmasm/
>  obj-$(CONFIG_HDPU_FEATURES)	+= hdpuftrs/
> +obj-$(CONFIG_TIFM_CORE)       	+= tifm_core.o
> +obj-$(CONFIG_TIFM_7XX1)       	+= tifm_7xx1.o
> diff --git a/drivers/mmc/Kconfig b/drivers/mmc/Kconfig
> index 45bcf09..aa08dd1 100644
> --- a/drivers/mmc/Kconfig
> +++ b/drivers/mmc/Kconfig
> @@ -109,4 +109,18 @@ config MMC_IMX
>  
>  	  If unsure, say N.
>  
> +comment "Texas Instruments Flash Media MMC/SD interface requires TIFM_CORE"
> +        depends on MMC != n && TIFM_CORE = n
> +	
> +config MMC_TIFM_SD
> +	tristate "TI Flash Media MMC/SD Interface support  (EXPERIMENTAL)"
> +	depends on TIFM_CORE && MMC && EXPERIMENTAL
> +	default TIFM_CORE
> +	help
> +	  This selects the Texas Instruments(R) Flash Media MMC/SD card
> +	  interface found in many laptops.
> +	  If you have a controller with this interface, say Y or M here.
> +
> +	  If unsure, say N.
> +
>   

Ditto. And until this depends/select business is sorted out, I'd prefer
a select on TIFM_CORE here.


Other than that...

Acked-by: Pierre Ossman <drzeus@drzeus.cx>