| Message ID | 1434668319-6651-5-git-send-email-stefan@agner.ch |
|---|---|
| State | New |
| Headers | show |
On Fri, Jun 19, 2015 at 12:58:37AM +0200, Stefan Agner wrote: > Enable the NAND Flash Controller driver which is part of the Vybrid > SoC by default. > > Signed-off-by: Stefan Agner <stefan@agner.ch> > --- > arch/arm/mach-imx/Kconfig | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/arch/arm/mach-imx/Kconfig b/arch/arm/mach-imx/Kconfig > index 3a3d3e9..9358135 100644 > --- a/arch/arm/mach-imx/Kconfig > +++ b/arch/arm/mach-imx/Kconfig > @@ -589,6 +589,7 @@ config SOC_VF610 > select PINCTRL_VF610 > select PL310_ERRATA_769419 if CACHE_L2X0 > select SMP_ON_UP if SMP > + select HAVE_NAND_VF610_NFC I'm not sure about the benefit of having this option, except we have to have this additional architecture patch. Shawn > > help > This enables support for Freescale Vybrid VF610 processor. > -- > 2.4.2 > > -- > 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/ >
On 8 Jul 2015, shawnguo@kernel.org wrote: > On Fri, Jun 19, 2015 at 12:58:37AM +0200, Stefan Agner wrote: >> Enable the NAND Flash Controller driver which is part of the Vybrid >> SoC by default. >> >> Signed-off-by: Stefan Agner <stefan@agner.ch> >> --- >> arch/arm/mach-imx/Kconfig | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/arch/arm/mach-imx/Kconfig b/arch/arm/mach-imx/Kconfig >> index 3a3d3e9..9358135 100644 >> --- a/arch/arm/mach-imx/Kconfig >> +++ b/arch/arm/mach-imx/Kconfig >>>> -589,6 +589,7 @@ config SOC_VF610 >> select PINCTRL_VF610 >> select PL310_ERRATA_769419 if CACHE_L2X0 >> select SMP_ON_UP if SMP >> + select HAVE_NAND_VF610_NFC > I'm not sure about the benefit of having this option, except we have > to have this additional architecture patch. For other SOC, like the PowerPC or ColdFire CPUs with this controller, it was a way to mark the controller as populated. Otherwise, the Kconfig entry to select the controller will pop up for everybody. Is there a better way to do this? >> >> help >> This enables support for Freescale Vybrid VF610 processor.
On Wed, Jul 08, 2015 at 10:32:54AM -0400, Bill Pringlemeir wrote: > >> diff --git a/arch/arm/mach-imx/Kconfig b/arch/arm/mach-imx/Kconfig > >> index 3a3d3e9..9358135 100644 > >> --- a/arch/arm/mach-imx/Kconfig > >> +++ b/arch/arm/mach-imx/Kconfig > >>>> -589,6 +589,7 @@ config SOC_VF610 > >> select PINCTRL_VF610 > >> select PL310_ERRATA_769419 if CACHE_L2X0 > >> select SMP_ON_UP if SMP > >> + select HAVE_NAND_VF610_NFC > > > I'm not sure about the benefit of having this option, except we have > > to have this additional architecture patch. > > For other SOC, like the PowerPC or ColdFire CPUs with this controller, > it was a way to mark the controller as populated. Otherwise, the > Kconfig entry to select the controller will pop up for everybody. > > Is there a better way to do this? use depends on SOC_VF610 instead. Sebastian
On Wed, Jul 08, 2015 at 10:32:54AM -0400, Bill Pringlemeir wrote: > > On 8 Jul 2015, shawnguo@kernel.org wrote: > > On Fri, Jun 19, 2015 at 12:58:37AM +0200, Stefan Agner wrote: > >> Enable the NAND Flash Controller driver which is part of the Vybrid > >> SoC by default. > >> > >> Signed-off-by: Stefan Agner <stefan@agner.ch> > >> --- > >> arch/arm/mach-imx/Kconfig | 1 + > >> 1 file changed, 1 insertion(+) > >> > >> diff --git a/arch/arm/mach-imx/Kconfig b/arch/arm/mach-imx/Kconfig > >> index 3a3d3e9..9358135 100644 > >> --- a/arch/arm/mach-imx/Kconfig > >> +++ b/arch/arm/mach-imx/Kconfig > >>>> -589,6 +589,7 @@ config SOC_VF610 > >> select PINCTRL_VF610 > >> select PL310_ERRATA_769419 if CACHE_L2X0 > >> select SMP_ON_UP if SMP > >> + select HAVE_NAND_VF610_NFC > > > I'm not sure about the benefit of having this option, except we have > > to have this additional architecture patch. > > For other SOC, like the PowerPC or ColdFire CPUs with this controller, > it was a way to mark the controller as populated. Otherwise, the > Kconfig entry to select the controller will pop up for everybody. IMHO, having the option pop up for everybody isn't really a problem. On the other hand, we can even compile test the driver on any architecture without the COMPILE_TEST dependency. But if MTD maintainer has a different opinion, we can still have a 'depends on SOC_VF610' to save this patch. Shawn
On 2015-07-09 03:55, Shawn Guo wrote: > On Wed, Jul 08, 2015 at 10:32:54AM -0400, Bill Pringlemeir wrote: >> >> On 8 Jul 2015, shawnguo@kernel.org wrote: >> > On Fri, Jun 19, 2015 at 12:58:37AM +0200, Stefan Agner wrote: >> >> Enable the NAND Flash Controller driver which is part of the Vybrid >> >> SoC by default. >> >> >> >> Signed-off-by: Stefan Agner <stefan@agner.ch> >> >> --- >> >> arch/arm/mach-imx/Kconfig | 1 + >> >> 1 file changed, 1 insertion(+) >> >> >> >> diff --git a/arch/arm/mach-imx/Kconfig b/arch/arm/mach-imx/Kconfig >> >> index 3a3d3e9..9358135 100644 >> >> --- a/arch/arm/mach-imx/Kconfig >> >> +++ b/arch/arm/mach-imx/Kconfig >> >>>> -589,6 +589,7 @@ config SOC_VF610 >> >> select PINCTRL_VF610 >> >> select PL310_ERRATA_769419 if CACHE_L2X0 >> >> select SMP_ON_UP if SMP >> >> + select HAVE_NAND_VF610_NFC >> >> > I'm not sure about the benefit of having this option, except we have >> > to have this additional architecture patch. >> >> For other SOC, like the PowerPC or ColdFire CPUs with this controller, >> it was a way to mark the controller as populated. Otherwise, the >> Kconfig entry to select the controller will pop up for everybody. > > IMHO, having the option pop up for everybody isn't really a problem. On > the other hand, we can even compile test the driver on any architecture > without the COMPILE_TEST dependency. I think the distro people don't like that too much. Afaik they enable all drivers as modules, and hence the NAND driver would end up as kernel module in x86 distros... Reminds me that I wanted to send a patch to add the dependencies of STE_MODEM_RPROC, I recently stumbled upon the remoteproc/ste_modem_rproc kernel modules on my distro, which both are actually not really useful on x86 :-) > But if MTD maintainer has a different opinion, we can still have a > 'depends on SOC_VF610' to save this patch. I think this is the better solution, and Sebastian suggested that too. I will implement that in the next patch revision... -- Stefan
diff --git a/arch/arm/mach-imx/Kconfig b/arch/arm/mach-imx/Kconfig index 3a3d3e9..9358135 100644 --- a/arch/arm/mach-imx/Kconfig +++ b/arch/arm/mach-imx/Kconfig @@ -589,6 +589,7 @@ config SOC_VF610 select PINCTRL_VF610 select PL310_ERRATA_769419 if CACHE_L2X0 select SMP_ON_UP if SMP + select HAVE_NAND_VF610_NFC help This enables support for Freescale Vybrid VF610 processor.
Enable the NAND Flash Controller driver which is part of the Vybrid SoC by default. Signed-off-by: Stefan Agner <stefan@agner.ch> --- arch/arm/mach-imx/Kconfig | 1 + 1 file changed, 1 insertion(+)