Messages in this thread | | | Date | Wed, 20 Sep 2006 08:30:36 +0200 | From | Pierre Ossman <> | Subject | Re: [PATCH 2/2] [MMC] Driver for TI FlashMedia card reader - Kconfig/Makefile entries |
| |
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>
- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |