Patchwork [U-Boot,1/4] tegra: remove some cruft from CONFIG_EXTRA_ENV_SETTINGS

login
register
mail settings
Submitter Stephen Warren
Date May 15, 2012, 10:16 p.m.
Message ID <1337120173-3479-1-git-send-email-swarren@wwwdotorg.org>
Download mbox | patch
Permalink /patch/159462/
State Accepted
Delegated to: Tom Warren
Headers show

Comments

Stephen Warren - May 15, 2012, 10:16 p.m.
From: Stephen Warren <swarren@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 <swarren@nvidia.com>
---
 include/configs/tegra2-common.h |    3 ---
 1 files changed, 0 insertions(+), 3 deletions(-)

Patch

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" \
-	"smpflag=smp\0" \
 	TEGRA2_DEVICE_SETTINGS
 
 #define CONFIG_LOADADDR		0x408000	/* def. location for kernel */