[U-Boot] [PATCH 08/25] Blackfin: kill off useless initdram() usage

[Mike Frysinger]

On Thursday 17 December 2009 17:48:48 Wolfgang Denk wrote:
> Mike Frysinger wrote:
> > like this is an optional board-specific function that probes external
> > memory sizes by reading/writing to it.
> 
> Right, with at least two purposes: detecting memroy errors, and
> determining the available memory size.

the memory error checking is kind of lame.  i cant imagine it catching 
anything useful which the board crashing doesnt already tell you -- things are 
screwed.

> > on a Blackfin system, this is a pretty big waste.  memory must always be
> 
> Waste in terms of ... ?

if it's only detecting memory, then probing all the way down to the base is a 
waste of time.

> > mapped starting at address 0 which means once the highest address is
> > validated, things can return.  ignoring that, the Blackfin port has no
> > relocation support, so the link time addresses are fixed.
> 
> The memory test alone is a pretty good reason to add this. And don;t
> you have boards with multiple configurations / different RAM sizes
> where auto-sizing would be useful?

not really, and we havent had a customer ask for this kind of functionality.  
we do however get people always asking for faster loading times.
-mike
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part.
Url : http://lists.denx.de/pipermail/u-boot/attachments/20091217/916700c6/attachment.pgp