[U-Boot] [PATCH 03/26] ARM: cp15: setup mmu and enable dcache

[V, Aneesh]

Hello Heiko,

> -----Original Message-----
> From: u-boot-bounces at lists.denx.de [mailto:u-boot-
> bounces at lists.denx.de] On Behalf Of Heiko Schocher
> Sent: Wednesday, August 11, 2010 11:46 PM
> To: U-Boot user list
> Cc: Alessandro Rubini
> Subject: [U-Boot] [PATCH 03/26] ARM: cp15: setup mmu and enable
> dcache
> 
> 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.

This is rather strange. I thought the linker would have done better. 
However, I could overcome this problem by making 'done' a non-zero
initialized variable. Can't you try that?
I think the 'done' bit is quite useful. I see that the dcache_enable()
is called from multiple places. 

I tested your patch on OMAP4430(Cortex-A9). It works fine

Best regards,
Aneesh