From mboxrd@z Thu Jan  1 00:00:00 1970
From: =?utf-8?Q?Beno=C3=AEt_Th=C3=A9baudeau?= <benoit.thebaudeau@advansee.com>
Date: Tue, 21 Aug 2012 14:58:44 +0200 (CEST)
Subject: [U-Boot] [PATCH 3/3] mx35: Clean up lowlevel_init
In-Reply-To: <5033250C.1050400@denx.de>
Message-ID: <1908084914.2642035.1345553924445.JavaMail.root@advansee.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Hi Stefano, Fabio,

> On 20/08/2012 22:25, Fabio Estevam wrote:
> > Hi Stefano,
> > 
> > On Mon, Aug 20, 2012 at 4:42 PM, stefano babic <sbabic@denx.de>
> > wrote:
> > 
> >> With the flea3 I have made some clean-up, making the
> >> lowLevel_init.S
> >> essential. Same changes were not reported back to the mx35pdk. I
> >> am
> >> going also to put in a common place the functions to setup the DDR
> >> controller ( board_setup_sdram_bank() in the flea3 directory).
> >> This
> >> should allow to factorize some other code, reducing the
> >> lowlevel_init.S.
> > 
> > Maybe even better would be to convert the init from lowlevel_init.S
> > into C code, like was done on mx31.
> > 
> > It is much easier to debug/maintain C code then assemby.
> 
> Yes, agree.
> 
> Regards,
> Stefano

Yes, but we must also not forget the size constraints for SPLs.

Best regards,
Beno?t