Hi Andreas,
On 8/23/2012 18:36, Andreas Bießmann wrote:
Dear Bo Shen,
On 23.08.2012 11:34, Bo Shen wrote:
This patch correct the text base for at91sam9g10ek board Move the text base define to config.mk
Signed-off-by: Bo Shen voice.shen@atmel.com
board/atmel/at91sam9261ek/config.mk | 1 + include/configs/at91sam9261ek.h | 1 - 2 files changed, 1 insertion(+), 1 deletion(-) create mode 100644 board/atmel/at91sam9261ek/config.mk
diff --git a/board/atmel/at91sam9261ek/config.mk b/board/atmel/at91sam9261ek/config.mk new file mode 100644 index 0000000..e554a45 --- /dev/null +++ b/board/atmel/at91sam9261ek/config.mk @@ -0,0 +1 @@ +CONFIG_SYS_TEXT_BASE = 0x23f00000
NAK, these configuration should go into the board config header or maybe the boards.cfg. For this board please read commit f7aea46d6ad9c257d2fbea7238cc8796aaa733f1
Thanks for your information. I will keep it and fix the bootstrap.
BRs, Bo Shen
Another point is, the 23f00000 is (possibly) not far enough from position the start.S code will relocate the u-boot (depends on size of gd_t, bd_t, malloc arena, some special env, a.s.o). That was changed in f7aea46 by intention. Please fix at91bootstrap instead (or start integrating at91 for common SPL framework in u-boot; still on my plan for this year but I couldn't manage to start until yet).
diff --git a/include/configs/at91sam9261ek.h b/include/configs/at91sam9261ek.h index 014437b..718ac60 100644 --- a/include/configs/at91sam9261ek.h +++ b/include/configs/at91sam9261ek.h @@ -52,7 +52,6 @@ #define CONFIG_DISPLAY_CPUINFO
#define CONFIG_ATMEL_LEGACY -#define CONFIG_SYS_TEXT_BASE 0x21f00000
/*
- Hardware drivers