diff mbox

[U-Boot] powerpc: Replace CONFIG_SYS_INIT_RAM_END with CONFIG_SYS_INIT_RAM_SIZE

Message ID 1459975917-2439-1-git-send-email-york.sun@nxp.com
State Accepted
Commit b39d1213e30717c435c8ed43411d573d435557cb
Delegated to: York Sun
Headers show

Commit Message

York Sun April 6, 2016, 8:51 p.m. UTC
CONFIG_SYS_INIT_RAM_SIZE may be used out of the board header file.
Some boards use CONFIG_SYS_INIT_RAM_END for the same purpose. To
unify the macros, use CONFIG_SYS_INIT_RAM_SIZE for all.

Signed-off-by: York Sun <york.sun@nxp.com>
CC: Mario Six <mario.six@gdsys.cc>

---

 include/configs/BSC9131RDB.h   |    4 ++--
 include/configs/BSC9132QDS.h   |    4 ++--
 include/configs/C29XPCIE.h     |    4 ++--
 include/configs/P1010RDB.h     |    4 ++--
 include/configs/a3m071.h       |    4 ++--
 include/configs/ac14xx.h       |    4 ++--
 include/configs/dlvision-10g.h |    4 ++--
 include/configs/io.h           |    4 ++--
 include/configs/iocon.h        |    4 ++--
 include/configs/o2dnt-common.h |    6 +++---
 10 files changed, 21 insertions(+), 21 deletions(-)

Comments

York Sun April 8, 2016, 4:08 p.m. UTC | #1
On 04/06/2016 01:52 PM, York Sun wrote:
> CONFIG_SYS_INIT_RAM_SIZE may be used out of the board header file.
> Some boards use CONFIG_SYS_INIT_RAM_END for the same purpose. To
> unify the macros, use CONFIG_SYS_INIT_RAM_SIZE for all.
> 
> Signed-off-by: York Sun <york.sun@nxp.com>
> CC: Mario Six <mario.six@gdsys.cc>
> 
> ---
> 
>  include/configs/BSC9131RDB.h   |    4 ++--
>  include/configs/BSC9132QDS.h   |    4 ++--
>  include/configs/C29XPCIE.h     |    4 ++--
>  include/configs/P1010RDB.h     |    4 ++--
>  include/configs/a3m071.h       |    4 ++--
>  include/configs/ac14xx.h       |    4 ++--
>  include/configs/dlvision-10g.h |    4 ++--
>  include/configs/io.h           |    4 ++--
>  include/configs/iocon.h        |    4 ++--
>  include/configs/o2dnt-common.h |    6 +++---
>  10 files changed, 21 insertions(+), 21 deletions(-)
> 


Stefan, Anatolij, Dirk,

Do you have any objection/comment on the proposed changes? I plan to merge it soon.

York
Stefan Roese April 8, 2016, 4:11 p.m. UTC | #2
On 08.04.2016 18:08, York Sun wrote:
> On 04/06/2016 01:52 PM, York Sun wrote:
>> CONFIG_SYS_INIT_RAM_SIZE may be used out of the board header file.
>> Some boards use CONFIG_SYS_INIT_RAM_END for the same purpose. To
>> unify the macros, use CONFIG_SYS_INIT_RAM_SIZE for all.
>>
>> Signed-off-by: York Sun <york.sun@nxp.com>
>> CC: Mario Six <mario.six@gdsys.cc>
>>
>> ---
>>
>>   include/configs/BSC9131RDB.h   |    4 ++--
>>   include/configs/BSC9132QDS.h   |    4 ++--
>>   include/configs/C29XPCIE.h     |    4 ++--
>>   include/configs/P1010RDB.h     |    4 ++--
>>   include/configs/a3m071.h       |    4 ++--
>>   include/configs/ac14xx.h       |    4 ++--
>>   include/configs/dlvision-10g.h |    4 ++--
>>   include/configs/io.h           |    4 ++--
>>   include/configs/iocon.h        |    4 ++--
>>   include/configs/o2dnt-common.h |    6 +++---
>>   10 files changed, 21 insertions(+), 21 deletions(-)
>>
>
>
> Stefan, Anatolij, Dirk,
>
> Do you have any objection/comment on the proposed changes? I plan
> to merge it soon.

Looks good to me (not tested on real hardware), so:

Acked-by: Stefan Roese <sr@denx.de>

Thanks,
Stefan
Anatolij Gustschin April 8, 2016, 4:23 p.m. UTC | #3
Hi York,

On Wed, 6 Apr 2016 13:51:57 -0700
York Sun york.sun@nxp.com wrote:

> CONFIG_SYS_INIT_RAM_SIZE may be used out of the board header file.
> Some boards use CONFIG_SYS_INIT_RAM_END for the same purpose. To
> unify the macros, use CONFIG_SYS_INIT_RAM_SIZE for all.
> 
> Signed-off-by: York Sun <york.sun@nxp.com>
> CC: Mario Six <mario.six@gdsys.cc>
> 
> ---
> 
>  include/configs/BSC9131RDB.h   |    4 ++--
>  include/configs/BSC9132QDS.h   |    4 ++--
>  include/configs/C29XPCIE.h     |    4 ++--
>  include/configs/P1010RDB.h     |    4 ++--
>  include/configs/a3m071.h       |    4 ++--
>  include/configs/ac14xx.h       |    4 ++--
>  include/configs/dlvision-10g.h |    4 ++--
>  include/configs/io.h           |    4 ++--
>  include/configs/iocon.h        |    4 ++--
>  include/configs/o2dnt-common.h |    6 +++---
>  10 files changed, 21 insertions(+), 21 deletions(-)

Acked-by: Anatolij Gustschin <agust@denx.de>

--
Anatolij
York Sun April 8, 2016, 10:46 p.m. UTC | #4
On 04/06/2016 01:52 PM, York Sun wrote:
> CONFIG_SYS_INIT_RAM_SIZE may be used out of the board header file.
> Some boards use CONFIG_SYS_INIT_RAM_END for the same purpose. To
> unify the macros, use CONFIG_SYS_INIT_RAM_SIZE for all.
> 
> Signed-off-by: York Sun <york.sun@nxp.com>
> CC: Mario Six <mario.six@gdsys.cc>
> 
> ---
> 
>  include/configs/BSC9131RDB.h   |    4 ++--
>  include/configs/BSC9132QDS.h   |    4 ++--
>  include/configs/C29XPCIE.h     |    4 ++--
>  include/configs/P1010RDB.h     |    4 ++--
>  include/configs/a3m071.h       |    4 ++--
>  include/configs/ac14xx.h       |    4 ++--
>  include/configs/dlvision-10g.h |    4 ++--
>  include/configs/io.h           |    4 ++--
>  include/configs/iocon.h        |    4 ++--
>  include/configs/o2dnt-common.h |    6 +++---
>  10 files changed, 21 insertions(+), 21 deletions(-)

Applied to u-boot-mpc85xx master. Awaiting upstream.

York
diff mbox

Patch

diff --git a/include/configs/BSC9131RDB.h b/include/configs/BSC9131RDB.h
index 9c32a01..e870ffe 100644
--- a/include/configs/BSC9131RDB.h
+++ b/include/configs/BSC9131RDB.h
@@ -215,9 +215,9 @@  extern unsigned long get_sdram_size(void);
 
 #define CONFIG_SYS_INIT_RAM_LOCK
 #define CONFIG_SYS_INIT_RAM_ADDR	0xffd00000	/* stack in RAM */
-#define CONFIG_SYS_INIT_RAM_END		0x00004000/* End of used area in RAM */
+#define CONFIG_SYS_INIT_RAM_SIZE	0x00004000/* End of used area in RAM */
 
-#define CONFIG_SYS_GBL_DATA_OFFSET	(CONFIG_SYS_INIT_RAM_END \
+#define CONFIG_SYS_GBL_DATA_OFFSET	(CONFIG_SYS_INIT_RAM_SIZE \
 						- GENERATED_GBL_DATA_SIZE)
 #define CONFIG_SYS_INIT_SP_OFFSET	CONFIG_SYS_GBL_DATA_OFFSET
 
diff --git a/include/configs/BSC9132QDS.h b/include/configs/BSC9132QDS.h
index 9f7ceb8..108e924 100644
--- a/include/configs/BSC9132QDS.h
+++ b/include/configs/BSC9132QDS.h
@@ -396,9 +396,9 @@  combinations. this should be removed later
 
 #define CONFIG_SYS_INIT_RAM_LOCK
 #define CONFIG_SYS_INIT_RAM_ADDR	0xffd00000	/* stack in RAM */
-#define CONFIG_SYS_INIT_RAM_END		0x00004000 /* End of used area in RAM */
+#define CONFIG_SYS_INIT_RAM_SIZE	0x00004000 /* End of used area in RAM */
 
-#define CONFIG_SYS_GBL_DATA_OFFSET	(CONFIG_SYS_INIT_RAM_END \
+#define CONFIG_SYS_GBL_DATA_OFFSET	(CONFIG_SYS_INIT_RAM_SIZE \
 						- GENERATED_GBL_DATA_SIZE)
 #define CONFIG_SYS_INIT_SP_OFFSET	CONFIG_SYS_GBL_DATA_OFFSET
 
diff --git a/include/configs/C29XPCIE.h b/include/configs/C29XPCIE.h
index 4cbc9ad..a7a5e8e 100644
--- a/include/configs/C29XPCIE.h
+++ b/include/configs/C29XPCIE.h
@@ -329,9 +329,9 @@ 
 
 #define CONFIG_SYS_INIT_RAM_LOCK
 #define CONFIG_SYS_INIT_RAM_ADDR	0xffd00000
-#define CONFIG_SYS_INIT_RAM_END		0x00004000
+#define CONFIG_SYS_INIT_RAM_SIZE	0x00004000
 
-#define CONFIG_SYS_GBL_DATA_OFFSET	(CONFIG_SYS_INIT_RAM_END \
+#define CONFIG_SYS_GBL_DATA_OFFSET	(CONFIG_SYS_INIT_RAM_SIZE \
 						- GENERATED_GBL_DATA_SIZE)
 #define CONFIG_SYS_INIT_SP_OFFSET	CONFIG_SYS_GBL_DATA_OFFSET
 
diff --git a/include/configs/P1010RDB.h b/include/configs/P1010RDB.h
index 81af871..249e11b 100644
--- a/include/configs/P1010RDB.h
+++ b/include/configs/P1010RDB.h
@@ -555,9 +555,9 @@  extern unsigned long get_sdram_size(void);
 
 #define CONFIG_SYS_INIT_RAM_LOCK
 #define CONFIG_SYS_INIT_RAM_ADDR	0xffd00000 /* stack in RAM */
-#define CONFIG_SYS_INIT_RAM_END		0x00004000 /* End of used area in RAM */
+#define CONFIG_SYS_INIT_RAM_SIZE	0x00004000 /* End of used area in RAM */
 
-#define CONFIG_SYS_GBL_DATA_OFFSET	(CONFIG_SYS_INIT_RAM_END \
+#define CONFIG_SYS_GBL_DATA_OFFSET	(CONFIG_SYS_INIT_RAM_SIZE \
 						- GENERATED_GBL_DATA_SIZE)
 #define CONFIG_SYS_INIT_SP_OFFSET	CONFIG_SYS_GBL_DATA_OFFSET
 
diff --git a/include/configs/a3m071.h b/include/configs/a3m071.h
index cf931a6..31348da 100644
--- a/include/configs/a3m071.h
+++ b/include/configs/a3m071.h
@@ -141,9 +141,9 @@ 
 
 /* Use SRAM until RAM will be available */
 #define CONFIG_SYS_INIT_RAM_ADDR	MPC5XXX_SRAM
-#define CONFIG_SYS_INIT_RAM_END		MPC5XXX_SRAM_SIZE
+#define CONFIG_SYS_INIT_RAM_SIZE	MPC5XXX_SRAM_SIZE
 
-#define CONFIG_SYS_GBL_DATA_OFFSET	(CONFIG_SYS_INIT_RAM_END - \
+#define CONFIG_SYS_GBL_DATA_OFFSET	(CONFIG_SYS_INIT_RAM_SIZE - \
 					 GENERATED_GBL_DATA_SIZE)
 #define CONFIG_SYS_INIT_SP_OFFSET	CONFIG_SYS_GBL_DATA_OFFSET
 
diff --git a/include/configs/ac14xx.h b/include/configs/ac14xx.h
index 750d8ff..57421c6 100644
--- a/include/configs/ac14xx.h
+++ b/include/configs/ac14xx.h
@@ -289,9 +289,9 @@ 
 
 /* Use SRAM for initial stack */
 #define CONFIG_SYS_INIT_RAM_ADDR	CONFIG_SYS_SRAM_BASE
-#define CONFIG_SYS_INIT_RAM_END		CONFIG_SYS_SRAM_SIZE
+#define CONFIG_SYS_INIT_RAM_SIZE	CONFIG_SYS_SRAM_SIZE
 
-#define CONFIG_SYS_GBL_DATA_OFFSET	(CONFIG_SYS_INIT_RAM_END - \
+#define CONFIG_SYS_GBL_DATA_OFFSET	(CONFIG_SYS_INIT_RAM_SIZE - \
 					 GENERATED_GBL_DATA_SIZE)
 #define CONFIG_SYS_INIT_SP_OFFSET	CONFIG_SYS_GBL_DATA_OFFSET
 
diff --git a/include/configs/dlvision-10g.h b/include/configs/dlvision-10g.h
index 9f9bcb8..bbeed76 100644
--- a/include/configs/dlvision-10g.h
+++ b/include/configs/dlvision-10g.h
@@ -238,10 +238,10 @@ 
 #define CONFIG_SYS_OCM_DATA_ADDR	0xF8000000
 #define CONFIG_SYS_OCM_DATA_SIZE	0x1000
 #define CONFIG_SYS_INIT_RAM_ADDR	CONFIG_SYS_OCM_DATA_ADDR /* in SDRAM */
-#define CONFIG_SYS_INIT_RAM_END	CONFIG_SYS_OCM_DATA_SIZE /* End of used area */
+#define CONFIG_SYS_INIT_RAM_SIZE	CONFIG_SYS_OCM_DATA_SIZE
 
 #define CONFIG_SYS_GBL_DATA_OFFSET \
-	(CONFIG_SYS_INIT_RAM_END - GENERATED_GBL_DATA_SIZE)
+	(CONFIG_SYS_INIT_RAM_SIZE - GENERATED_GBL_DATA_SIZE)
 #define CONFIG_SYS_INIT_SP_OFFSET	CONFIG_SYS_GBL_DATA_OFFSET
 
 /*
diff --git a/include/configs/io.h b/include/configs/io.h
index 1213fe2..f05eb0a 100644
--- a/include/configs/io.h
+++ b/include/configs/io.h
@@ -196,10 +196,10 @@ 
 #define CONFIG_SYS_OCM_DATA_ADDR	0xF8000000
 #define CONFIG_SYS_OCM_DATA_SIZE	0x1000
 #define CONFIG_SYS_INIT_RAM_ADDR	CONFIG_SYS_OCM_DATA_ADDR /* in SDRAM */
-#define CONFIG_SYS_INIT_RAM_END	CONFIG_SYS_OCM_DATA_SIZE /* End of used area */
+#define CONFIG_SYS_INIT_RAM_SIZE	CONFIG_SYS_OCM_DATA_SIZE
 
 #define CONFIG_SYS_GBL_DATA_OFFSET \
-	(CONFIG_SYS_INIT_RAM_END - GENERATED_GBL_DATA_SIZE)
+	(CONFIG_SYS_INIT_RAM_SIZE - GENERATED_GBL_DATA_SIZE)
 #define CONFIG_SYS_INIT_SP_OFFSET	CONFIG_SYS_GBL_DATA_OFFSET
 
 /*
diff --git a/include/configs/iocon.h b/include/configs/iocon.h
index 8bc89a0..f75ca76 100644
--- a/include/configs/iocon.h
+++ b/include/configs/iocon.h
@@ -249,10 +249,10 @@  int fpga_gpio_get(unsigned int bus, int pin);
 #define CONFIG_SYS_OCM_DATA_ADDR	0xF8000000
 #define CONFIG_SYS_OCM_DATA_SIZE	0x1000
 #define CONFIG_SYS_INIT_RAM_ADDR	CONFIG_SYS_OCM_DATA_ADDR /* in SDRAM */
-#define CONFIG_SYS_INIT_RAM_END	CONFIG_SYS_OCM_DATA_SIZE /* End of used area */
+#define CONFIG_SYS_INIT_RAM_SIZE	CONFIG_SYS_OCM_DATA_SIZE
 
 #define CONFIG_SYS_GBL_DATA_OFFSET \
-	(CONFIG_SYS_INIT_RAM_END - GENERATED_GBL_DATA_SIZE)
+	(CONFIG_SYS_INIT_RAM_SIZE - GENERATED_GBL_DATA_SIZE)
 #define CONFIG_SYS_INIT_SP_OFFSET	CONFIG_SYS_GBL_DATA_OFFSET
 
 /*
diff --git a/include/configs/o2dnt-common.h b/include/configs/o2dnt-common.h
index 73a44a7..4ffab8a 100644
--- a/include/configs/o2dnt-common.h
+++ b/include/configs/o2dnt-common.h
@@ -253,13 +253,13 @@ 
 #define CONFIG_SYS_INIT_RAM_ADDR	MPC5XXX_SRAM
 #ifdef CONFIG_POST
 /* preserve space for the post_word at end of on-chip SRAM */
-#define CONFIG_SYS_INIT_RAM_END		MPC5XXX_SRAM_POST_SIZE
+#define CONFIG_SYS_INIT_RAM_SIZE	MPC5XXX_SRAM_POST_SIZE
 #else
 /* End of used area in DPRAM */
-#define CONFIG_SYS_INIT_RAM_END		MPC5XXX_SRAM_SIZE
+#define CONFIG_SYS_INIT_RAM_SIZE	MPC5XXX_SRAM_SIZE
 #endif
 
-#define CONFIG_SYS_GBL_DATA_OFFSET	(CONFIG_SYS_INIT_RAM_END - \
+#define CONFIG_SYS_GBL_DATA_OFFSET	(CONFIG_SYS_INIT_RAM_SIZE - \
 					 GENERATED_GBL_DATA_SIZE)
 #define CONFIG_SYS_INIT_SP_OFFSET	CONFIG_SYS_GBL_DATA_OFFSET