[U-Boot] [PATCH] atmel: sam9g10: correct the text base and move into config.mk
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 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
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
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
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
participants (2)
-
Andreas Bießmann -
Bo Shen