[coreboot] New patch to review for coreboot: d18b72c armv7: make bootblock linker script more explicit

David Hendricks (dhendrix@chromium.org) gerrit at coreboot.org
Sun Feb 10 00:47:49 CET 2013


David Hendricks (dhendrix at chromium.org) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/2344

-gerrit

commit d18b72c153ab7788f5ae36f58b2a6c2d2c1ae2b1
Author: David Hendricks <dhendrix at chromium.org>
Date:   Sat Feb 9 14:53:47 2013 -0800

    armv7: make bootblock linker script more explicit
    
    This adds a .bl1 and .start symbol that is placed at the beginning
    of the .rom section.
    
    The goal is to move the .id section in between the reset vector and
    bootblock_main.
    
    Change-Id: Ie732ce656d697c059cc0fa40c844b39f53fc214c
    Signed-off-by: David Hendricks <dhendrix at chromium.org>
---
 src/arch/armv7/bootblock.inc | 2 ++
 src/arch/armv7/bootblock.lds | 2 ++
 2 files changed, 4 insertions(+)

diff --git a/src/arch/armv7/bootblock.inc b/src/arch/armv7/bootblock.inc
index 25cc8c6..efe3d67 100644
--- a/src/arch/armv7/bootblock.inc
+++ b/src/arch/armv7/bootblock.inc
@@ -31,6 +31,7 @@
 
 #include <system.h>
 
+.section ".bl1", "a", %progbits
 _bl1:
 	/* For now we have to live with a first stage boot loader
 	 * on ARM, which is 8KB in size and it is prepended to the
@@ -38,6 +39,7 @@ _bl1:
 	 */
 	.skip (CONFIG_BL1_SIZE_KB * 1024)
 
+.section ".start", "a", %progbits
 .globl _start
 _start: b	reset
 	.balignl 16,0xdeadbeef
diff --git a/src/arch/armv7/bootblock.lds b/src/arch/armv7/bootblock.lds
index 9482cef..f45e6c0 100644
--- a/src/arch/armv7/bootblock.lds
+++ b/src/arch/armv7/bootblock.lds
@@ -30,6 +30,8 @@ SECTIONS
 	/* This section might be better named .setup */
 	.rom ROMLOC : {
 		_rom = .;
+		*(.bl1);
+		*(.start);
 		*(.text);
 		*(.text.*);
 		*(.rom.text);



More information about the coreboot mailing list