* [U-Boot] [PATCH] mpc85xx: configs: remove c=ffe from default environment
@ 2014-05-15 0:33 Kim Phillips
2014-05-15 4:50 ` York Sun
2014-05-20 15:41 ` York Sun
0 siblings, 2 replies; 4+ messages in thread
From: Kim Phillips @ 2014-05-15 0:33 UTC (permalink / raw)
To: u-boot
AFAICT, c=ffe does nothing and was a typo from the original commit
d17123696c6180ac8b74fbd318bf14652623e982 "powerpc/p4080: Add support
for the P4080DS board" and just kept on getting duplicated
in subsequently added board config files.
Signed-off-by: Kim Phillips <kim.phillips@freescale.com>
---
include/configs/B4860QDS.h | 3 +--
include/configs/P2041RDB.h | 3 +--
include/configs/T1040QDS.h | 3 +--
include/configs/T104xRDB.h | 3 +--
include/configs/T208xQDS.h | 3 +--
include/configs/T208xRDB.h | 3 +--
include/configs/T4240EMU.h | 3 +--
include/configs/T4240QDS.h | 3 +--
include/configs/corenet_ds.h | 3 +--
9 files changed, 9 insertions(+), 18 deletions(-)
diff --git a/include/configs/B4860QDS.h b/include/configs/B4860QDS.h
index df371b7..70cda74 100644
--- a/include/configs/B4860QDS.h
+++ b/include/configs/B4860QDS.h
@@ -824,8 +824,7 @@ unsigned long get_board_ddr_clk(void);
"ramdiskfile=b4860qds/ramdisk.uboot\0" \
"fdtaddr=c00000\0" \
"fdtfile=b4860qds/b4860qds.dtb\0" \
- "bdev=sda3\0" \
- "c=ffe\0"
+ "bdev=sda3\0"
/* For emulation this causes u-boot to jump to the start of the proof point
app code automatically */
diff --git a/include/configs/P2041RDB.h b/include/configs/P2041RDB.h
index 54e6493..e662322 100644
--- a/include/configs/P2041RDB.h
+++ b/include/configs/P2041RDB.h
@@ -713,8 +713,7 @@ unsigned long get_board_sys_clk(unsigned long dummy);
"ramdiskfile=p2041rdb/ramdisk.uboot\0" \
"fdtaddr=c00000\0" \
"fdtfile=p2041rdb/p2041rdb.dtb\0" \
- "bdev=sda3\0" \
- "c=ffe\0"
+ "bdev=sda3\0"
#define CONFIG_HDBOOT \
"setenv bootargs root=/dev/$bdev rw " \
diff --git a/include/configs/T1040QDS.h b/include/configs/T1040QDS.h
index 522653b..2215ac8 100644
--- a/include/configs/T1040QDS.h
+++ b/include/configs/T1040QDS.h
@@ -767,8 +767,7 @@ unsigned long get_board_ddr_clk(void);
"ramdiskfile=t1040qds/ramdisk.uboot\0" \
"fdtaddr=c00000\0" \
"fdtfile=t1040qds/t1040qds.dtb\0" \
- "bdev=sda3\0" \
- "c=ffe\0"
+ "bdev=sda3\0"
#define CONFIG_LINUX \
"setenv bootargs root=/dev/ram rw " \
diff --git a/include/configs/T104xRDB.h b/include/configs/T104xRDB.h
index 81b4f31..e564cb7 100644
--- a/include/configs/T104xRDB.h
+++ b/include/configs/T104xRDB.h
@@ -779,8 +779,7 @@
"ramdiskfile=" __stringify(RAMDISKFILE) "\0" \
"fdtaddr=c00000\0" \
"fdtfile=" __stringify(FDTFILE) "\0" \
- "bdev=sda3\0" \
- "c=ffe\0"
+ "bdev=sda3\0"
#define CONFIG_LINUX \
"setenv bootargs root=/dev/ram rw " \
diff --git a/include/configs/T208xQDS.h b/include/configs/T208xQDS.h
index 8bf0843..d2a9c6d 100644
--- a/include/configs/T208xQDS.h
+++ b/include/configs/T208xQDS.h
@@ -837,8 +837,7 @@ unsigned long get_board_ddr_clk(void);
"ramdiskfile=t2080qds/ramdisk.uboot\0" \
"fdtaddr=c00000\0" \
"fdtfile=t2080qds/t2080qds.dtb\0" \
- "bdev=sda3\0" \
- "c=ffe\0"
+ "bdev=sda3\0"
/*
* For emulation this causes u-boot to jump to the start of the
diff --git a/include/configs/T208xRDB.h b/include/configs/T208xRDB.h
index 0be0a0f..081e391 100644
--- a/include/configs/T208xRDB.h
+++ b/include/configs/T208xRDB.h
@@ -796,8 +796,7 @@ unsigned long get_board_ddr_clk(void);
"ramdiskfile=t2080rdb/ramdisk.uboot\0" \
"fdtaddr=c00000\0" \
"fdtfile=t2080rdb/t2080rdb.dtb\0" \
- "bdev=sda3\0" \
- "c=ffe\0"
+ "bdev=sda3\0"
/*
* For emulation this causes u-boot to jump to the start of the
diff --git a/include/configs/T4240EMU.h b/include/configs/T4240EMU.h
index ad5a9a6..53c69b0 100644
--- a/include/configs/T4240EMU.h
+++ b/include/configs/T4240EMU.h
@@ -128,8 +128,7 @@
"ramdiskfile=t4240emu/ramdisk.uboot\0" \
"fdtaddr=c00000\0" \
"fdtfile=t4240emu/t4240emu.dtb\0" \
- "bdev=sda3\0" \
- "c=ffe\0"
+ "bdev=sda3\0"
/*
* For emulation this causes u-boot to jump to the start of the proof point
diff --git a/include/configs/T4240QDS.h b/include/configs/T4240QDS.h
index e4eb30f..5e4065a 100644
--- a/include/configs/T4240QDS.h
+++ b/include/configs/T4240QDS.h
@@ -572,8 +572,7 @@ unsigned long get_board_ddr_clk(void);
"ramdiskfile=t4240qds/ramdisk.uboot\0" \
"fdtaddr=c00000\0" \
"fdtfile=t4240qds/t4240qds.dtb\0" \
- "bdev=sda3\0" \
- "c=ffe\0"
+ "bdev=sda3\0"
#define CONFIG_HVBOOT \
"setenv bootargs config-addr=0x60000000; " \
diff --git a/include/configs/corenet_ds.h b/include/configs/corenet_ds.h
index db6b9be..e288755 100644
--- a/include/configs/corenet_ds.h
+++ b/include/configs/corenet_ds.h
@@ -712,8 +712,7 @@
"ramdiskfile=p4080ds/ramdisk.uboot\0" \
"fdtaddr=c00000\0" \
"fdtfile=p4080ds/p4080ds.dtb\0" \
- "bdev=sda3\0" \
- "c=ffe\0"
+ "bdev=sda3\0"
#define CONFIG_HDBOOT \
"setenv bootargs root=/dev/$bdev rw " \
--
1.9.3
^ permalink raw reply related [flat|nested] 4+ messages in thread
* [U-Boot] [PATCH] mpc85xx: configs: remove c=ffe from default environment
2014-05-15 0:33 [U-Boot] [PATCH] mpc85xx: configs: remove c=ffe from default environment Kim Phillips
@ 2014-05-15 4:50 ` York Sun
2014-05-16 12:54 ` Edward L Swarthout
2014-05-20 15:41 ` York Sun
1 sibling, 1 reply; 4+ messages in thread
From: York Sun @ 2014-05-15 4:50 UTC (permalink / raw)
To: u-boot
On 05/14/2014 07:33 PM, Kim Phillips wrote:
> AFAICT, c=ffe does nothing and was a typo from the original commit
> d17123696c6180ac8b74fbd318bf14652623e982 "powerpc/p4080: Add support
> for the P4080DS board" and just kept on getting duplicated
> in subsequently added board config files.
>
> Signed-off-by: Kim Phillips <kim.phillips@freescale.com>
> ---
Thanks. I have been trying to get Kumar to respond but haven't heard from him.
It does look odd. I think we can delete them.
York
^ permalink raw reply [flat|nested] 4+ messages in thread
* [U-Boot] [PATCH] mpc85xx: configs: remove c=ffe from default environment
2014-05-15 4:50 ` York Sun
@ 2014-05-16 12:54 ` Edward L Swarthout
0 siblings, 0 replies; 4+ messages in thread
From: Edward L Swarthout @ 2014-05-16 12:54 UTC (permalink / raw)
To: u-boot
From: York Sun <yorksun@freescale.com>
>On 05/14/2014 07:33 PM, Kim Phillips wrote:
>> AFAICT, c=ffe does nothing and was a typo from the original commit
>> d17123696c6180ac8b74fbd318bf14652623e982 "powerpc/p4080: Add support
>> for the P4080DS board" and just kept on getting duplicated
>> in subsequently added board config files.
>
> Thanks. I have been trying to get Kumar to respond but haven't heard from him.
> It does look odd. I think we can delete them.
ACK. It was a convenience variable to access ccsrbar space from scripts,
but should have been changed to "c=fe" on p4080 and subsequent soc's.
Ed
^ permalink raw reply [flat|nested] 4+ messages in thread
* [U-Boot] [PATCH] mpc85xx: configs: remove c=ffe from default environment
2014-05-15 0:33 [U-Boot] [PATCH] mpc85xx: configs: remove c=ffe from default environment Kim Phillips
2014-05-15 4:50 ` York Sun
@ 2014-05-20 15:41 ` York Sun
1 sibling, 0 replies; 4+ messages in thread
From: York Sun @ 2014-05-20 15:41 UTC (permalink / raw)
To: u-boot
On 05/14/2014 05:33 PM, Kim Phillips wrote:
> AFAICT, c=ffe does nothing and was a typo from the original commit
> d17123696c6180ac8b74fbd318bf14652623e982 "powerpc/p4080: Add support
> for the P4080DS board" and just kept on getting duplicated
> in subsequently added board config files.
>
> Signed-off-by: Kim Phillips <kim.phillips@freescale.com>
> ---
Applied to u-boot-mpc85xx/master.
York
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2014-05-20 15:41 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-05-15 0:33 [U-Boot] [PATCH] mpc85xx: configs: remove c=ffe from default environment Kim Phillips
2014-05-15 4:50 ` York Sun
2014-05-16 12:54 ` Edward L Swarthout
2014-05-20 15:41 ` York Sun
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox