[U-Boot] [RFC][PATCH 00/19] arm: add full relocation / cache support
Heiko Schocher
hs at denx.de
Thu Aug 5 08:19:58 CEST 2010
Hello Matthias,
Matthias Weißer wrote:
> Am 29.07.2010 12:44, schrieb Heiko Schocher:
>> This patch series add full relocation and cache support for arm
>> based boards. I did this for arm1136, arm_cortexa8 and arm926ejs
>> based boards. As this change is not compatible to old code,
>> before this can go to mainline *all* plattforms and boards
>> have to be converted! As I don;t have access to all plattforms/
>> boards I need help here! Also I couldn;t test all boards,
>> so please test and report, send bugfixes!
>
> I just tested your patch set on my version of u-boot for MB86R01 from
> Fujitsu (arm926ejs based SoC). This is currently not available in
> mainline u-boot but current patches are available here
Thanks for testing!
> http://lists.denx.de/pipermail/u-boot/2010-August/074688.html
>
> The point is that the board doesn't boot after applying your patches and
> doing the changes to my board which are given at the end of this mail.
:-(
> The board runs through my low level init (so DDR RAM is up) and later on
> crashes in the first call to memset. I could not further debug this as I
Where is this memset()? The first after low level init is in:
arch/arm/lib/board.c board_init_f(), do you mean this?
If so, then something must be wrong with your memory setup.
> have to admit that I am not an expert with GDB + BDI2000 debugging.
> Maybe you can give me some hints what I am missing.
Hmm.. hard to say without debugging it. If you don;t mean with "crashes
in the first first memset" the function I above described, maybe
maybe your Ram gets not correct detected? Can you try to find out,
with what value dram_init() sets up gd->ram_size?
(Or you set this for testing to fix values?)
Hmmm... from where did your board boot? I tried it on the tx25
board, which boots from nand. Do you boot from a NOR flash?
If so you *must* change TEXT_BASE in config.mk (see:
doc/README.arm-relocation line 45) in your board directory
to where u-boot starts in flash!
Ah, yep, this seems to me the reason why it don;t work for you:
found in the patchseries you pointed to
http://lists.denx.de/pipermail/u-boot/2010-August/074688.html
board/syteco/jadecpu/config.mk
[...]
+TEXT_BASE = 0x46000000
change this to
(as in include/configs/jadecpu.h is defined the following:
+/*
+ * FLASH and environment organization
+ */
+#define CONFIG_SYS_FLASH_BASE 0x10000000
+#define CONFIG_SYS_MAX_FLASH_BANKS 1
+#define CONFIG_SYS_MAX_FLASH_SECT 256
+#define CONFIG_SYS_MONITOR_BASE CONFIG_SYS_FLASH_BASE
)
+TEXT_BASE = 0x10000000
and try it again.
> Changes made to the board code after applying your patches:
>
> diff --git a/include/configs/jadecpu.h b/include/configs/jadecpu.h
> index bfc60a6..24aa23d 100644
> --- a/include/configs/jadecpu.h
> +++ b/include/configs/jadecpu.h
> @@ -149,6 +149,10 @@
> #define PHYS_SDRAM 0x40000000 /* Start address of
> DDRRAM */
> #define PHYS_SDRAM_SIZE 0x08000000 /* 128 megs */
>
> +/* additions for new relocation code, must added to all boards */
> +#define CONFIG_SYS_SDRAM_BASE PHYS_SDRAM
> +#define CONFIG_SYS_INIT_SP_ADDR 0x01000000
> +
> /*
> * FLASH and environment organization
> */
>
> diff --git a/board/syteco/jadecpu/jadecpu.c
> b/board/syteco/jadecpu/jadecpu.c
> index 04d2f9d..bf96bcd 100644
> --- a/board/syteco/jadecpu/jadecpu.c
> +++ b/board/syteco/jadecpu/jadecpu.c
> @@ -154,12 +154,18 @@ int misc_init_r(void)
> */
> int dram_init(void)
> {
> - gd->bd->bi_dram[0].start = PHYS_SDRAM;
> - gd->bd->bi_dram[0].size = PHYS_SDRAM_SIZE;
> -
> + /* dram_init must store complete ramsize in gd->ram_size */
> + gd->ram_size = get_ram_size((volatile void *)PHYS_SDRAM,
> + PHYS_SDRAM_SIZE);
> return 0;
> }
>
> +void dram_init_banksize (void)
> +{
> + gd->bd->bi_dram[0].start = PHYS_SDRAM;
> + gd->bd->bi_dram[0].size = gd->ram_size;
> +}
> +
looks OK to me.
bye
Heiko
--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
More information about the U-Boot
mailing list