[U-Boot] [PATCH for-next 2/3] sunxi: kconfig: Add top-level TARGET_SUNXI

Masahiro Yamada yamada.m at jp.panasonic.com
Mon Oct 6 12:54:02 CEST 2014


Hi Ian

On Mon, 06 Oct 2014 09:27:19 +0100
Ian Campbell <ijc at hellion.org.uk> wrote:
Hi Ian,

> On Mon, 2014-10-06 at 10:39 +0900, Masahiro Yamada wrote:
> > Hi Ian,
> > 
> > On Sat,  4 Oct 2014 09:48:11 +0100
> > Ian Campbell <ijc at hellion.org.uk> wrote:
> > 
> > > And make TARGET_SUN[457]I a choice variable under this.
> > > 
> > > configs updated with:
> > >     sed -i -e 's/^\+S:CONFIG_TARGET_SUN.I=y/+S:CONFIG_TARGET_SUNXI=y\n&/g' configs/*
> > > 
> > > Signed-off-by: Ian Campbell <ijc at hellion.org.uk>
> > 
> > 
> > Perhaps, is ARCH_SUNXI more familiar?
> 
> I've no idea ;-)
> 
> I think this is something which ought to be consistent within u-boot as
> a whole. Seems we have a mixture of ARCH_FOO (DAVINCI, VERSATILE,
> EXYNOS) and just FOO (TEGRA, ZYNQ). It does look like TARGET_FOO is all
> individual boards though, which would make it inappropriate for SUNXI or
> even SUN[4567]I.

SoC/board select menu clean-up is on the way.
I have to admit the naming convention is inconsistent now.

CONFIG_ARCH_{DAVINCI, VERSATILE, EXYNOS} were added recently.
If Xilinx/NVIDIA developers argree, we can rename
CONFIG_{TEGRA, ZYNQ} -> CONFIG_ARCH_{TEGRA, ZYNQ} at some point.



> Ultimately I'm happy to follow any guidance anyone has to give.
> 
> Ian.




More information about the U-Boot mailing list