[coreboot] New patch to review for coreboot: dc92352 Simplify and make corrections to romstage.ld

[David Hendricks (dhendrix@chromium.org)]

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

-gerrit

commit dc9235245a4196d53cddaa9aaaa24cff96eb7aa1
Author: David Hendricks <dhendrix at chromium.org>
Date:   Mon Dec 17 20:41:32 2012 -0800

    Simplify and make corrections to romstage.ld
    
    This is still a work-in-progress, but it seems to work better than
    before and is less complicated...
    
    Change-Id: I6f730d017391f9ec4401cdfd34931c869df10a9e
    Signed-off-by: Ronald G. Minnich <rminnich at gmail.com>
    Signed-off-by: David Hendricks <dhendrix at chromium.org>
---
 src/arch/armv7/romstage.ld | 58 ++++++++++------------------------------------
 1 file changed, 12 insertions(+), 46 deletions(-)

diff --git a/src/arch/armv7/romstage.ld b/src/arch/armv7/romstage.ld
index 0a2f7f2..3603ac2 100644
--- a/src/arch/armv7/romstage.ld
+++ b/src/arch/armv7/romstage.ld
@@ -29,7 +29,7 @@
  * FIXME 2: Somehow linker didn't like CONFIG_SPL_MAX_SIZE and CONFIG_SPL_TEXT_BASE...
  */
 /* MEMORY { .sram : ORIGIN = 0x02023400, LENGTH = 0x3800 } */
-MEMORY { .sram : ORIGIN = 0x02023400, LENGTH = 0x10000 }
+/*MEMORY { .sram : ORIGIN = 0x02023400, LENGTH = 0x10000 }*/
 
 /* We use ELF as output format. So that we can debug the code in some form. */
 OUTPUT_FORMAT("elf32-littlearm", "elf32-littlearm", "elf32-littlearm")
@@ -41,56 +41,22 @@ SECTIONS
 {
 	. = ROMSTAGE_BASE;
 
-	/*
-	.rom . : {
+	.romtext . : {
 		_rom = .;
-		*(.rom.text);
-		*(.rom.data);
+		*(.text);
+	}
+
+	.romdata . : {
 		*(.rodata);
-		*(.rodata.*);
-		*(.rom.data.*);
+		*(.machine_param);
+		*(.data);
 		. = ALIGN(16);
 		_erom = .;
 	}
-	*/
 
-	/* First we place the code and read only data (typically const declared).
-	 * This could theoretically be placed in rom.
-	 */
-	.text : {
-		_text = .;
-		*(.text);
-		*(.text.*);
-		. = ALIGN(4);
-		_etext = .;
-	} >.sram
-
-	.rodata : {
-		_rodata = .;
-		. = ALIGN(4);
-		cpu_drivers = . ;
-		*(.rodata.cpu_driver)
-		ecpu_drivers = . ;
-		*(.rodata)
-		*(.rodata.*)
-		/* kevinh/Ispiri - Added an align, because the objcopy tool
-		 * incorrectly converts sections that are not long word aligned.
-		 */
-		 . = ALIGN(4);
-
-		_erodata = .;
-	} >.sram
-	/* After the code we place initialized data (typically initialized
-	 * global variables). This gets copied into ram by startup code.
-	 * __data_start and __data_end shows where in ram this should be placed,
-	 * whereas __data_loadstart and __data_loadend shows where in rom to
-	 * copy from.
-	 */
-	.data : {
-		_data = .;
-		*(.data)
-		_edata = .;
-	} >.sram
+	.xidx . : {
+		//*(.*.exidx);
+	}
 
 	__image_copy_end = .;
 
@@ -104,7 +70,7 @@ SECTIONS
 		*(.bss)
 		*(.sbss)
 		*(COMMON)
-	} >.sram
+	} 
 	_ebss = .;
 	_end = .;