[U-Boot] [RFC PATCH] arm, davinci: Remove board specific code from da850_lowlevel.c

Heiko Schocher hs at denx.de
Wed Nov 9 06:37:45 CET 2011


Hello Christian,

Christian Riesch wrote:
> Signed-off-by: Christian Riesch <christian.riesch at omicron.at>
> ---
> 
> Hello Heiko,
> 
> On my board I cannot use your code in arch_cpu_init() in da850_lowlevel.c
> since I have different versions of my board with different input
> clock frequencies. Here u-boot should first determine the board
> revision number and then configure the SoC accordingly. Therefore I would 
> like to move all board-specific parts (and PLL and memory configuration is 
> board-specific since it depends on the memory chips and oscillators 
> deployed on the board) to board_early_init_f which is called right after 
> arch_cpu_init() and keep only a few initializiation steps in arch_cpu_init().

Ok, if you have other needs on your board we must change something.
But I want to prevent, that all the code you remove in your patch is
moved to board code, so all boards must (copy?) it. Instead you should
move it to a weak function, which you can replace for your specific
needs. (I thought the code was common enough ...)

What do you think?

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