On Wed, Jul 08, 2015 at 10:32:54AM -0400, Bill Pringlemeir wrote: > > On 8 Jul 2015, shawn...@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 <ste...@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 -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
