From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stephen Warren Date: Mon, 21 May 2012 13:22:30 -0600 Subject: [U-Boot] [PATCH 1/2] Add env vars describing U-Boot target board In-Reply-To: <20120521190004.1648620715B@gemini.denx.de> References: <1337189793-25367-1-git-send-email-swarren@wwwdotorg.org> <20120521190004.1648620715B@gemini.denx.de> Message-ID: <4FBA95F6.2020009@wwwdotorg.org> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On 05/21/2012 01:00 PM, Wolfgang Denk wrote: > Dear Stephen Warren, > > In message <1337189793-25367-1-git-send-email-swarren@wwwdotorg.org> you wrote: >> From: Stephen Warren >> >> This can be useful for generic scripts. For example, rather than hard- >> coding a script to ext2load tegra-harmony.dtb, it could load >> ${board_soc}-${board_name}.dtb and hence not need adjustments to >> run on multiple boards. >> >> Signed-off-by: Stephen Warren >> --- >> Wolfgang, does this seem like a reasonable feature? If so, can we merge >> it through the Tegra tree, since the next patch enables this for Tegra. >> Thanks! >> >> common/Makefile | 9 +++++++++ >> common/env_common.c | 15 +++++++++++++++ >> common/env_embedded.c | 15 +++++++++++++++ >> 3 files changed, 39 insertions(+), 0 deletions(-) > > Please make this optional - I don't want to have this enforced on all > boards, whether they want it or not. Wolfgang, Thanks for taking a look. The variables are already optional; you need to set CONFIG_ENV_VAR_* in order for common/env_*.c to add those variables to the default environment. Is that OK?