Stephen,

> -----Original Message-----
> From: Stephen Warren [mailto:swar...@wwwdotorg.org]
> Sent: Tuesday, May 15, 2012 3:16 PM
> To: Tom Warren
> Cc: Simon Glass; u-boot@lists.denx.de; Stephen Warren
> Subject: [PATCH 1/4] tegra: remove some cruft from CONFIG_EXTRA_ENV_SETTINGS
> 
> From: Stephen Warren <swar...@nvidia.com>
> 
> console isn't used by anything, and the kernel should be set appropriately
> by whatever script is booting the kernel, not imposed by the bootloader.
> 
> mem might be useful, but the current value is pretty bogus, since it
> includes nvmem options that make no sense for an upstream kernel, and
> equally should not be required for any downstream kernel. Either way, this
> is also best left to the kernel boot script.
> 
> smpflag isn't used by anything, and again was probably intended to be a
> kernel command-line option better set by the kernel boot script.
> 
> Signed-off-by: Stephen Warren <swar...@nvidia.com>
> ---
>  include/configs/tegra2-common.h |    3 ---
>  1 files changed, 0 insertions(+), 3 deletions(-)
> 
> diff --git a/include/configs/tegra2-common.h b/include/configs/tegra2-
> common.h index 068ce88..cdd3093 100644
> --- a/include/configs/tegra2-common.h
> +++ b/include/configs/tegra2-common.h
> @@ -140,9 +140,6 @@
>                                       "stderr=serial\0"
> 
>  #define CONFIG_EXTRA_ENV_SETTINGS \
> -     "console=ttyS0,115200n8\0" \
> -     "mem=" TEGRA2_SYSMEM "\0" \
You should remove the TEGRA2_SYSMEM #define from the board config files, too 
(seaboard.h, etc.)

Tom
> -     "smpflag=smp\0" \
>       TEGRA2_DEVICE_SETTINGS
> 
>  #define CONFIG_LOADADDR              0x408000        /* def. location for 
> kernel */
> --
> 1.7.0.4
-- 
nvpublic

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to