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

Masahiro Yamada
2014-10-06T10:54:02Z

Hi Ian

On Mon, 06 Oct 2014 09:27:19 +0100
Ian Campbell <ijc@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@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@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.