diff options
| author | Karl Palsson <karlp@etactica.com> | 2016-12-02 13:33:19 +0300 |
|---|---|---|
| committer | Karl Palsson <karlp@etactica.com> | 2016-12-19 14:40:39 +0300 |
| commit | cd9ae5e6d7f9825d102d23cb46a07ac7cf777703 (patch) | |
| tree | 84cfdbf23615436fb661644296d7298463b636f6 /ld | |
| parent | 950e06488578a0d613016739b4cd9436a6a06022 (diff) | |
eeprom: drop support from linker scripts.
While the NOLOAD variant sometimes worked with some toolchains, the
version in the generator scripts could never work, as neither the
startup code, nor gdb know how to load those sections properly.
Originally added in: eb18cc19cb392a69f8675c42776c644d1003ed89
The original scripts allowed you to place variables in eeprom space for
reading only. However, the last toolchain that generated working code
with this linker script was the gcc-arm-none-eabi-4_9-2014q4 release.
Subsequent releases treat the directives differently, and can lose track
of where variables are. One known symptom is constants getting bad
addresses, so for instance, "printf("asdfad")" will end up passing the
wrong address of the string constant into the eventual _write() call.
This commit removes the problematic directives until a more fully
correct system can be found that more properly follows the linker
script rules.
Diffstat (limited to 'ld')
| -rw-r--r-- | ld/linker.ld.S | 7 |
1 files changed, 0 insertions, 7 deletions
diff --git a/ld/linker.ld.S b/ld/linker.ld.S index e81ccc0c..51cdacd5 100644 --- a/ld/linker.ld.S +++ b/ld/linker.ld.S @@ -130,13 +130,6 @@ SECTIONS _ebss = .; } >ram -#if defined(_EEP) - .eep : { - *(.eeprom*) - . = ALIGN(4); - } >eep -#endif - #if defined(_CCM) .ccm : { *(.ccmram*) |