[U-Boot] ARM Versatile port - possibly inefficient code?

[Hebbar]

Hi,



>>Regardless of CFG_MAX_FLASH_BANKS value, the only first bank is
>>configured. Was it done deliberately or it's logical flaw? I think
>>is code could cloned on a multiple of other platforms.

ARM Versatile has only one flash bank.
What if a user has configured an incorrect number for flash banks. 
The driver should take care of this properly. Since CFG_MAX_FLASH_BANKS 
is in a different place & the flash.c is a different file, care is taken to
see 
that an incorrect bank is not considered for initialization

also,

>>panic ("configured too many flash banks!\n"); 
this error message clearly explains the mistake the user did.

Regards
Gururaja

-- 
View this message in context: http://www.nabble.com/-U-Boot--ARM-Versatile-port---possibly-inefficient-code--tp19736928p19739341.html
Sent from the Uboot - Users mailing list archive at Nabble.com.