[PATCH 2/2] i.MX6ull evk support
Philipp Zabel
p.zabel at pengutronix.de
Tue Oct 10 07:04:02 PDT 2017
On Thu, 2017-09-28 at 12:13 +0200, Sascha Hauer wrote:
> On Thu, Sep 28, 2017 at 11:33:32AM +0200, Lucas Stach wrote:
> > Am Donnerstag, den 28.09.2017, 11:26 +0200 schrieb Sascha Hauer:
> > > On Thu, Sep 28, 2017 at 11:16:15AM +0200, Lucas Stach wrote:
> > > > Am Mittwoch, den 27.09.2017, 14:16 +0200 schrieb Sascha Hauer:
> > > > > diff --git a/arch/arm/mach-imx/Kconfig b/arch/arm/mach-
> > > > > imx/Kconfig
> > > > > index 92440e3a75..dab19a33ec 100644
> > > > > --- a/arch/arm/mach-imx/Kconfig
> > > > > +++ b/arch/arm/mach-imx/Kconfig
> > > > > @@ -162,6 +162,10 @@ config ARCH_IMX6UL
> > > > > bool
> > > > > select ARCH_IMX6
> > > > >
> > > > > +config ARCH_IMX6ULL
> > > > > + bool
> > > > > + select ARCH_IMX6
> > > > > +
> > > >
> > > > Do we really need this? Seems the MX6ULL is just a stripped down
> > > > version of the MX6UL.
> > >
> > > We probably do not need this, but I suggest to keep it anyway. I
> > > think
> > > it makes it a bit clearer that there are indeed i.MX6ul *and*
> > > i.MX6ull
> > > and both are supported.
> >
> > Yeah, your decision, but I'll notice that we don't have separate
> > Kconfig symbols for MX6Q/DL, which probably expose more software
> > visible differences than MX6UL/ULL.
>
> While refactoring the patches I realized that having a i.MX6ull function
> which just calls the corresponding i.MX6ul function doesn't look too
> nice when additionally we have a board which either has one of both
> SoCs calling the i.MX6ul version and needs a comment that this is the
> same as the i.MX6ull variant.
>
> I dropped the IMX6ULL symbol.
In next, ARCH_IMX6ULL is still separate from ARCH_IMX6UL, and it
currently breaks i.MX6ULL support when building a barebox without
any i.MX6UL board enabled, because of the missing clock driver.
I'd prefer to just:
----------8<----------
diff --git a/arch/arm/mach-imx/Kconfig b/arch/arm/mach-imx/Kconfig
index dab19a33ec..1578875e95 100644
--- a/arch/arm/mach-imx/Kconfig
+++ b/arch/arm/mach-imx/Kconfig
@@ -162,10 +162,6 @@ config ARCH_IMX6UL
bool
select ARCH_IMX6
-config ARCH_IMX6ULL
- bool
- select ARCH_IMX6
-
config ARCH_IMX7
bool
select CPU_V7
@@ -420,7 +416,7 @@ config MACH_FREESCALE_MX7_SABRESD
config MACH_NXP_IMX6ULL_EVK
bool "NXP i.MX6ull EVK Board"
- select ARCH_IMX6ULL
+ select ARCH_IMX6UL
endif
---------->8----------
regards
Philipp
More information about the barebox
mailing list