From mboxrd@z Thu Jan 1 00:00:00 1970 From: Heiko Schocher Date: Wed, 09 Nov 2011 13:27:42 +0100 Subject: [U-Boot] [PATCH] arm, davinci: make arch_cpu_init() in da850_lowlevel.c weak In-Reply-To: References: <1320830586-19124-1-git-send-email-christian.riesch@omicron.at> <20111109101217.2EAC613BE0E2@gemini.denx.de> <4EBA598B.5020206@denx.de> Message-ID: <4EBA71BE.7020007@denx.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hello Christian, Christian Riesch wrote: > Hello Heiko, > > On Wed, Nov 9, 2011 at 11:44 AM, Heiko Schocher wrote: >>> Heiko, Christian: please negotiate who performs which part of the >>> cleanup. But I expect that with proper symbolic names instead of the >>> hardwired constants the need for a wek function will go away. >> I rework this. > > I'd like to help but since we have different custom boards it's a bit > hard to test. Therefore I'd like to get this lowlevel initialization Do you plan to post your patches for your custom board to the ML? > working on the da850evm board. Since it boots from SPI I guess I'll > need an SPL for SPI flash, right? Could we try to clean it up in a way Yes. > so it nicely integrates with your SPL code in > arch/arm/cpu/arm926ejs/davinci/spl.c? I soon post the cleanup for the da850_lowlevel.c file, so you can do this based on that patch. You only have to adapt board_init_f() in arch/arm/cpu/arm926ejs/davinci/spl.c. There for the da850 case, arch_cpu_init() or for the spl case better called da850_lowlevel_init() from arch/arm/cpu/arm926ejs/davinci/da850_lowlevel.c must be called. bye, Heiko -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany