diff mbox series

[v2,8/8] armv8: u-boot-spl.lds: mark __image_copy_start as symbol

Message ID 20220618121316.12061-9-alpernebiyasak@gmail.com
State Accepted
Commit e87da5704ffa6fc782d93d137fa30a37a5df3566
Delegated to: Simon Glass
Headers show
Series spl: binman: Fixes for BINMAN_SYMBOLS | expand

Commit Message

Alper Nebi Yasak June 18, 2022, 12:13 p.m. UTC
From: Peng Fan <peng.fan@nxp.com>

In arch/arm/lib/sections.c there is below code:
char __image_copy_start[0] __section(".__image_copy_start");
But actually 'objdump -t spl/u-boot-spl' not able to find out
symbol '__image_copy_start' for binman update image-pos/size.

So update link file

Signed-off-by: Peng Fan <peng.fan@nxp.com>
Reviewed-by: Tom Rini <trini@konsulko.com>
Reviewed-by: Alper Nebi Yasak <alpernebiyasak@gmail.com>
---
This is from Peng's i.MX8M binman symbols series [1], picked it onto
this series because it made more sense as a binman symbols fix.

[1] armv8: u-boot-spl.lds: mark __image_copy_start as symbol
https://lore.kernel.org/u-boot/20220603071715.15212-5-peng.fan@oss.nxp.com/

Changes in v2:
- Pick Peng's __image_copy_start fix

 arch/arm/cpu/armv8/u-boot-spl.lds | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Simon Glass June 28, 2022, 1:37 p.m. UTC | #1
From: Peng Fan <peng.fan@nxp.com>

In arch/arm/lib/sections.c there is below code:
char __image_copy_start[0] __section(".__image_copy_start");
But actually 'objdump -t spl/u-boot-spl' not able to find out
symbol '__image_copy_start' for binman update image-pos/size.

So update link file

Signed-off-by: Peng Fan <peng.fan@nxp.com>
Reviewed-by: Tom Rini <trini@konsulko.com>
Reviewed-by: Alper Nebi Yasak <alpernebiyasak@gmail.com>
---
This is from Peng's i.MX8M binman symbols series [1], picked it onto
this series because it made more sense as a binman symbols fix.

[1] armv8: u-boot-spl.lds: mark __image_copy_start as symbol
https://lore.kernel.org/u-boot/20220603071715.15212-5-peng.fan@oss.nxp.com/

Changes in v2:
- Pick Peng's __image_copy_start fix

 arch/arm/cpu/armv8/u-boot-spl.lds | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Applied to u-boot-dm, thanks!
diff mbox series

Patch

diff --git a/arch/arm/cpu/armv8/u-boot-spl.lds b/arch/arm/cpu/armv8/u-boot-spl.lds
index 730eb93dbc3b..9b1e7d462870 100644
--- a/arch/arm/cpu/armv8/u-boot-spl.lds
+++ b/arch/arm/cpu/armv8/u-boot-spl.lds
@@ -23,7 +23,7 @@  SECTIONS
 {
 	.text : {
 		. = ALIGN(8);
-		*(.__image_copy_start)
+		__image_copy_start = .;
 		CPUDIR/start.o (.text*)
 		*(.text*)
 	} >.sram