lkml.org 
[lkml]   [2015]   [Jan]   [27]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH] mmc: Add hardware dependencies for sdhci-pxav3 and sdhci-pxav2
From
Date
Hi Ulf,

Le Tuesday 27 January 2015 à 15:06 +0100, Ulf Hansson a écrit :
> On 26 January 2015 at 11:23, Jean Delvare <jdelvare@suse.de> wrote:
> > I seem to understand that the sdhci-pxav3 and sdhci-pxav2 drivers are
> > only needed on the MMP architecture. So add a hardware dependency on
> > ARCH_MMP, so that other users don't get to build useless drivers.
>
> I would rather see the default option to be N.
> Thus those configurations that needs this driver will have to select it.

This is a different question. The purpose of my patch is that people
configuring kernels for systems which just can't have these controllers,
are not asked about this driver at all. Changing the default to N would
not achieve that.

That being said, feel free to change the default to N if you want, but
to me (who knows nothing about MMP architecture and not much about MMC)
the current default values look sane.

Thanks,
Jean

> > Signed-off-by: Jean Delvare <jdelvare@suse.de>
> > Cc: Chris Ball <chris@printf.net>
> > Cc: Ulf Hansson <ulf.hansson@linaro.org>
> > Cc: Eric Miao <eric.y.miao@gmail.com>
> > Acked-by: Haojian Zhuang <haojian.zhuang@gmail.com>
> > ---
> > This patch was already sent on:
> > * 2014-04-23
> > * 2014-06-16
> >
> > drivers/mmc/host/Kconfig | 2 ++
> > 1 file changed, 2 insertions(+)
> >
> > --- linux-3.19-rc6.orig/drivers/mmc/host/Kconfig 2015-01-26 10:30:56.472182636 +0100
> > +++ linux-3.19-rc6/drivers/mmc/host/Kconfig 2015-01-26 11:13:33.669863314 +0100
> > @@ -228,6 +228,7 @@ config MMC_SDHCI_PXAV3
> > tristate "Marvell MMP2 SD Host Controller support (PXAV3)"
> > depends on CLKDEV_LOOKUP
> > depends on MMC_SDHCI_PLTFM
> > + depends on ARCH_MMP || COMPILE_TEST
> > default CPU_MMP2
> > help
> > This selects the Marvell(R) PXAV3 SD Host Controller.
> > @@ -240,6 +241,7 @@ config MMC_SDHCI_PXAV2
> > tristate "Marvell PXA9XX SD Host Controller support (PXAV2)"
> > depends on CLKDEV_LOOKUP
> > depends on MMC_SDHCI_PLTFM
> > + depends on ARCH_MMP || COMPILE_TEST
> > default CPU_PXA910
> > help
> > This selects the Marvell(R) PXAV2 SD Host Controller.
> >
> >
> > --
> > Jean Delvare
> > SUSE L3 Support




\
 
 \ /
  Last update: 2015-01-27 15:41    [W:0.095 / U:0.292 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site