diff mbox series

[U-Boot,v3,1/8] powerpc/dts: Define '_end' symbol in mpc85xx U-Boot lds files

Message ID 1535990712-8359-2-git-send-email-jagdish.gediya@nxp.com
State Accepted
Commit 4d3294b1d5281f9c3f62b47ba17250eb0035df4d
Delegated to: York Sun
Headers show
Series Device tree support for PowerPC in U-Boot | expand

Commit Message

Jagdish Gediya Sept. 3, 2018, 4:05 p.m. UTC
'board_fdt_blob_setup' function sets up fdt blob at '&_end' so
define '_end' symbol in mpc85xx lds files.

Signed-off-by: Jagdish Gediya <jagdish.gediya@nxp.com>
---
Changes for v2:
	- Define '_end' symbol in lds file instead of defining new
	  'board_fdt_blob_setup' function using existing '_init_end' symbol.

Changes for v3:
	- Define '_end' symbol in spl lds files too.

 arch/powerpc/cpu/mpc85xx/u-boot-nand.lds     | 1 +
 arch/powerpc/cpu/mpc85xx/u-boot-nand_spl.lds | 1 +
 arch/powerpc/cpu/mpc85xx/u-boot-spl.lds      | 1 +
 arch/powerpc/cpu/mpc85xx/u-boot.lds          | 1 +
 4 files changed, 4 insertions(+)

Comments

Simon Glass Sept. 14, 2018, 10:53 a.m. UTC | #1
On 3 September 2018 at 18:05, Jagdish Gediya <jagdish.gediya@nxp.com> wrote:
> 'board_fdt_blob_setup' function sets up fdt blob at '&_end' so
> define '_end' symbol in mpc85xx lds files.
>
> Signed-off-by: Jagdish Gediya <jagdish.gediya@nxp.com>
> ---
> Changes for v2:
>         - Define '_end' symbol in lds file instead of defining new
>           'board_fdt_blob_setup' function using existing '_init_end' symbol.
>
> Changes for v3:
>         - Define '_end' symbol in spl lds files too.
>
>  arch/powerpc/cpu/mpc85xx/u-boot-nand.lds     | 1 +
>  arch/powerpc/cpu/mpc85xx/u-boot-nand_spl.lds | 1 +
>  arch/powerpc/cpu/mpc85xx/u-boot-spl.lds      | 1 +
>  arch/powerpc/cpu/mpc85xx/u-boot.lds          | 1 +
>  4 files changed, 4 insertions(+)

Reviewed-by: Simon Glass <sjg@chromium.org>
Bin Meng Sept. 17, 2018, 5:58 a.m. UTC | #2
On Mon, Sep 3, 2018 at 1:08 PM Jagdish Gediya <jagdish.gediya@nxp.com> wrote:
>
> 'board_fdt_blob_setup' function sets up fdt blob at '&_end' so
> define '_end' symbol in mpc85xx lds files.
>
> Signed-off-by: Jagdish Gediya <jagdish.gediya@nxp.com>
> ---
> Changes for v2:
>         - Define '_end' symbol in lds file instead of defining new
>           'board_fdt_blob_setup' function using existing '_init_end' symbol.
>
> Changes for v3:
>         - Define '_end' symbol in spl lds files too.
>
>  arch/powerpc/cpu/mpc85xx/u-boot-nand.lds     | 1 +
>  arch/powerpc/cpu/mpc85xx/u-boot-nand_spl.lds | 1 +
>  arch/powerpc/cpu/mpc85xx/u-boot-spl.lds      | 1 +
>  arch/powerpc/cpu/mpc85xx/u-boot.lds          | 1 +
>  4 files changed, 4 insertions(+)
>

Reviewed-by: Bin Meng <bmeng.cn@gmail.com>
York Sun Sept. 28, 2018, 3:50 p.m. UTC | #3
On 09/02/2018 10:08 PM, Jagdish Gediya wrote:
> 'board_fdt_blob_setup' function sets up fdt blob at '&_end' so
> define '_end' symbol in mpc85xx lds files.
> 
> Signed-off-by: Jagdish Gediya <jagdish.gediya@nxp.com>
> ---
> Changes for v2:
> 	- Define '_end' symbol in lds file instead of defining new
> 	  'board_fdt_blob_setup' function using existing '_init_end' symbol.
> 
> Changes for v3:
> 	- Define '_end' symbol in spl lds files too.
> 

This patch set has been applied to mpc85xx master, awaiting upstream.
Thanks.

York
diff mbox series

Patch

diff --git a/arch/powerpc/cpu/mpc85xx/u-boot-nand.lds b/arch/powerpc/cpu/mpc85xx/u-boot-nand.lds
index 6db6da1..75b0285 100644
--- a/arch/powerpc/cpu/mpc85xx/u-boot-nand.lds
+++ b/arch/powerpc/cpu/mpc85xx/u-boot-nand.lds
@@ -74,6 +74,7 @@  SECTIONS
   .data.init : { *(.data.init) }
   . = ALIGN(256);
   __init_end = .;
+  _end = .;
 
   .bootpg ADDR(.text) - 0x1000 :
   {
diff --git a/arch/powerpc/cpu/mpc85xx/u-boot-nand_spl.lds b/arch/powerpc/cpu/mpc85xx/u-boot-nand_spl.lds
index 8588d7c..a2193bf 100644
--- a/arch/powerpc/cpu/mpc85xx/u-boot-nand_spl.lds
+++ b/arch/powerpc/cpu/mpc85xx/u-boot-nand_spl.lds
@@ -42,6 +42,7 @@  SECTIONS
 	. = ALIGN(8);
 	__init_begin = .;
 	__init_end = .;
+	_end = .;
 #if defined(CONFIG_FSL_IFC) /* Restrict bootpg at 4K boundry for IFC */
 	.bootpg ADDR(.text) + 0x1000 :
 	{
diff --git a/arch/powerpc/cpu/mpc85xx/u-boot-spl.lds b/arch/powerpc/cpu/mpc85xx/u-boot-spl.lds
index 0495182..6dc8d99 100644
--- a/arch/powerpc/cpu/mpc85xx/u-boot-spl.lds
+++ b/arch/powerpc/cpu/mpc85xx/u-boot-spl.lds
@@ -55,6 +55,7 @@  SECTIONS
 	. = ALIGN(8);
 	__init_begin = .;
 	__init_end = .;
+	_end = .;
 #ifdef CONFIG_SPL_SKIP_RELOCATE
 	. = ALIGN(4);
 	__bss_start = .;
diff --git a/arch/powerpc/cpu/mpc85xx/u-boot.lds b/arch/powerpc/cpu/mpc85xx/u-boot.lds
index 14c31be..22bbac5 100644
--- a/arch/powerpc/cpu/mpc85xx/u-boot.lds
+++ b/arch/powerpc/cpu/mpc85xx/u-boot.lds
@@ -81,6 +81,7 @@  SECTIONS
   .data.init : { *(.data.init) }
   . = ALIGN(256);
   __init_end = .;
+  _end = .;
 
 #ifdef CONFIG_SYS_MPC85XX_NO_RESETVEC
   .bootpg ADDR(.text) - 0x1000 :