On 29/07/10 11:45, Heiko Schocher wrote:
> This has been tested on at91sam9263 and STN8815.
> Again, I didn't check if it has bad effects
> on non-arm926 cores.
> 
> Initially I had a "done" bit to only set up page tables
> at the beginning. However, since the aligmnent requirement
> was for the whole object file, this extra integer tool 16kB
> in BSS, so I chose to remove it.
> 
> Also, note not all boards use PHYS_SDRAM, but it looks like
> it's the most used name (more than CONFIG_SYS_DRAM_BASE for
> example).
> 
> rebased for full arm relocation from Heiko Schocher <h...@denx.de>
> 
> Signed-off-by: Alessandro Rubini <rub...@gnudd.com>
> ---
>  arch/arm/lib/cache-cp15.c |   38 ++++++++++++++++++++++++++++++++++++++
>  1 files changed, 38 insertions(+), 0 deletions(-)
> 
> diff --git a/arch/arm/lib/cache-cp15.c b/arch/arm/lib/cache-cp15.c
> index 62ed54f..11e64d8 100644
> --- a/arch/arm/lib/cache-cp15.c
> +++ b/arch/arm/lib/cache-cp15.c
> @@ -32,6 +32,36 @@ static void cp_delay (void)
>       /* copro seems to need some delay between reading and writing */
>       for (i = 0; i < 100; i++)
>               nop();
> +     asm volatile("" : : : "memory");
> +}
> +
> +/* to activate the MMU we need to set up virtual memory: use 1M areas in bss 
> */
> +static inline void mmu_setup(void)
> +{
> +     static u32 __attribute__((aligned(16384))) page_table[4096];
> +     int i;
> +     u32 reg;
> +
> +     /* Set up an identity-mapping for all 4GB, rw for everyone */
> +     for (i = 0; i < 4096; i++)
> +             page_table[i] = i << 20 | (3 << 10) | 0x12;
> +     /* Then, enable cacheable and bufferable for RAM only */
> +     for (i = PHYS_SDRAM >> 20;
> +          i < ( PHYS_SDRAM + PHYS_SDRAM_SIZE) >> 20;

As you noted, not all boards define these two symbols. I don't see where you
have added them in the config.h files. You do seem to have created a new
symbol CONFIG_SYS_SDRAM_BASE which you could use, but you would still need
a CONFIG_SYS_SDRAM_SIZE wouldn't you?

> +          i++) {
> +             page_table[i] = i << 20 | (3 << 10) | 0x1e;

These numbers ought to be defines, no?

The 0x1e will not work on da8xx as the data cache is broken. The d-cache can
still be used in write back mode, so the value 0x1a should be used. It would
be good to have symbols to define the caching modes: none, wr-thru', wr-back
or some such, similar to Linux.

> +     }
> +     /* Copy the page table address to cp15 */
> +     asm volatile("mcr p15, 0, %0, c2, c0, 0"
> +                  : : "r" (page_table) : "memory");
> +     /* Set the access control to all-supervisor */
> +     asm volatile("mcr p15, 0, %0, c3, c0, 0"
> +                  : : "r" (~0));
> +     /* and enable the mmu */
> +     reg = get_cr(); /* get control reg. */
> +     cp_delay();
> +     set_cr(reg | CR_M);
> +
>  }

I have previously tested this patch on da830 and it works fine, bar the two
issues above.

Nick.
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to