[U-Boot] [PATCH 18/20] Add Chrome OS config header
Stephen Warren
swarren at wwwdotorg.org
Thu Feb 26 00:28:26 CET 2015
On 02/17/2015 03:29 PM, Simon Glass wrote:
> From: Doug Anderson <dianders at chromium.org>
>
> This header includes useful scripts which can be used with any board that
> can boot Chrome OS.
>
> In particular, 'run nvboot' will boot a board without verified boot enabled.
I think this needs a bit of minification for an upstream U-Boot. In
particular, many of the environment variables overlap semantically or by
name with those from include/config_distro_*.h, and I'd like to see any
U-Boot for the Tegra Chromebooks (at least) support booting both
ChromeOS kernels and arbitrary distros using the environment from
config_distro_bootcmd.h.
> diff --git a/include/configs/chromeos.h b/include/configs/chromeos.h
> +/* Stringify a token */
> +#ifndef STRINGIFY
> +#define _STRINGIFY(x) #x
> +#define STRINGIFY(x) _STRINGIFY(x)
> +#endif
Shouldn't that be in some common header so it isn't ever duplicated?
> +#define CONFIG_CROS_FULL
There are a ton of macros in here without much in the way of
explanation. Shouldn't everything be documented in the README?
More information about the U-Boot
mailing list