Prevent .noinit section from incorrect placement for AVR.

When .data and .bss sections are empty .noinit section is placed at data
region's start. This will be incorrect for devices that has different
data start address than data region start in linker script.
The patch updates .noinit section's VMA to end of .bss section. So, .noinit
section will be placed at .data section address (-Tdata=<address>) when .data
and .bss sections are empty.

ld/

	* scripttempl/avr.sc (.noinit): Force .noinit VMA to end of .bss VMA.
	* scripttempl/avrtiny.sc (.noinit): Likewise.

diff --git a/ld/ChangeLog b/ld/ChangeLog
index c13a202bc29aef0c4bb8d27ead65cddbf1cee837..2462b30bf100f3465c62412e1c1c97612d18f8ac 100644 (file)
--- a/ld/ChangeLog
+++ b/ld/ChangeLog
@@ -1,3 +1,8 @@
+2016-01-22  Pitchumani Sivanupandi  <pitchumani.s@atmel.com>
+
+       * scripttempl/avr.sc (.noinit): Force .noinit VMA to end of .bss VMA.
+       * scripttempl/avrtiny.sc (.noinit): Likewise.
+
 2016-01-21  Nick Clifton  <nickc@redhat.com>
 
        PR ld/19453

diff --git a/ld/scripttempl/avr.sc b/ld/scripttempl/avr.sc
index de819875bdaa7166a772755533a07130a7ec059f..c26f4ead94d447cfe2e96d63796197b2ec5291f0 100644 (file)
--- a/ld/scripttempl/avr.sc
+++ b/ld/scripttempl/avr.sc
@@ -210,7 +210,7 @@ SECTIONS
   ${RELOCATING+ __data_load_end = __data_load_start + SIZEOF(.data); }
 
   /* Global data not cleared after reset.  */
-  .noinit ${RELOCATING-0}:
+  .noinit ${RELOCATING+ ADDR(.bss) + SIZEOF (.bss)} ${RELOCATING-0}: ${RELOCATING+ AT (ADDR (.noinit))}
   {
     ${RELOCATING+ PROVIDE (__noinit_start = .) ; }
     *(.noinit*)

diff --git a/ld/scripttempl/avrtiny.sc b/ld/scripttempl/avrtiny.sc
index 00daf11464995306a28f9b88a32783adc2156811..045bc4b26e0fe88fceb9ce2387595099050a3a9b 100644 (file)
--- a/ld/scripttempl/avrtiny.sc
+++ b/ld/scripttempl/avrtiny.sc
@@ -210,7 +210,7 @@ SECTIONS
   ${RELOCATING+ __data_load_end = __data_load_start + SIZEOF(.data); }
 
   /* Global data not cleared after reset.  */
-  .noinit ${RELOCATING-0}:
+  .noinit ${RELOCATING+ ADDR(.bss) + SIZEOF (.bss)} ${RELOCATING-0} : ${RELOCATING+ AT (ADDR (.noinit))}
   {
     ${RELOCATING+ PROVIDE (__noinit_start = .) ; }
     *(.noinit*)