From mboxrd@z Thu Jan  1 00:00:00 1970
From: Masahiro Yamada <yamada.m@jp.panasonic.com>
Date: Wed, 19 Mar 2014 12:16:53 +0900
Subject: [U-Boot] [RFC PATCH 04/17] kconfig: add defconfig files for all
	boards
In-Reply-To: <5327B8C6.9090100@wwwdotorg.org>
References: <1395046392-1212-5-git-send-email-yamada.m@jp.panasonic.com>
	<5327B8C6.9090100@wwwdotorg.org>
Message-ID: <20140319121653.7A5D.AA925319@jp.panasonic.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Hi Stephen,

> > diff --git a/configs/beaver_defconfig b/configs/beaver_defconfig
> > new file mode 100644
> > index 0000000..fbf00f3
> > --- /dev/null
> > +++ b/configs/beaver_defconfig
> > @@ -0,0 +1,10 @@
> > +CONFIG_SPL=y
> > +CONFIG_ARM=y
> > +CONFIG_SYS_CPU="armv7"
> > +CONFIG_SOC_DIR=y
> > +CONFIG_SYS_SOC="tegra30"
> > +CONFIG_SYS_BOARD="beaver"
> > +CONFIG_VENDOR_DIR=y
> > +CONFIG_SYS_VENDOR="nvidia"
> > +CONFIG_SYS_CONFIG_NAME="beaver"
> > +CONFIG_BOARD_MAINTAINER="Tom Warren <twarren@nvidia.com>:Stephen Warren <swarren@nvidia.com>"
> 
> This is odd; defconfig in the Linux kernel is for defining values for
> user-editable configuration options. However, at least
> CONFIG_BOARD_MAINTAINERS is a property of the board port, not something
> the a user should be editing.

In U-Boot, each board and its maintainer are tightly coupled.
So, Albert chose to merge boards.cfg and MAINTAINERS in commit 27af930e9a.
I am following that concept.
That's why I added maintainers information to defconfig files
rather than keeping it in a seperated file.

Anyway, this item is for open discussion.
Do you have any idea instead of this?


> Other values in this file feel very odd
> too. For example, I would expect selecting SYS_SOC=tegra30 it
> automatically select SYS_VENDOR, SYS_CPU, ARM, SPL, ...

This is really really Tegra-specific matter.

You can send a follow-up patch
to add arch/arm/cpu/armv7/tegra/Kconfig
and desecribe them "by yourself"
if you believe it is necessary.
(although I doubt it.)

Please do not get me wrong.
This series provides the basic infrastructure.
Expecting SoC-specific issues to this series is not fair.
Tegra is not everything.

BTW, tegra30 cannot automatically select SYS_VENDOR.
It might be "nvidia", or "avionic-design".


Best Regards
Masahiro Yamada