[U-Boot] [PATCH v2 3/8] arm/km: add BootROM config file for memphis SDRAM

Valentin Longchamp valentin.longchamp at keymile.com
Thu May 5 08:28:36 CEST 2011


Wolfgang Denk wrote:
> Dear Valentin Longchamp,
> 
> In message <4DC1092E.4080804 at keymile.com> you wrote:
>>> You are just adding file here, whereas it is not being used. Is it like dead code to me.
>> Yes, it is not used with the u-boot build system, we have a small script 
>> that wraps around it.
> 
> As this "small script" is neither available here this _is_ dead code
> and will not be accepted.

Yes I completely agree. That's why we have some other config files that 
were initially posted for mainline inclusion and removed for this patch 
series.

> 
>> It is used by the mgcoge3un board, that is later introduced with this 
>> series.
> 
> Then please add it together with this board, then.

That's what I have done in the v3 series posted yesterday.

> 
>> Now I have a question about the marvell boards build: I see in the 
>> Makefile that there is a rule about this kwb file, which is exactly what 
>> we do in our additionnal build script:
> 
> Please either get rid of this "additionnal build script", or move it's
> function into the mainline U-Boot code.

I agree once again. I have asked Prafulla for additionnal information 
about the mkimage usage in the standard u-boot Makefile (because this is 
new for us in u-boot). If we can use this, we then can simplify the 
given build script (or even get rid of it). I first have to clarify 
things about this build script with my colleagues.

> 
> (Note: this might even be considered as a GPL violation which
> requires to include "scripts used to control compilation").
> 

Well, no violation intended here. This script can be published. But I 
have to clarify with my colleagues why exactly we have to script 
additionally to the standard u-boot Makefile (it may only be legacy 
because we used to work with an old u-boot before our mainlining effort).

Best Regards

-- 
Valentin Longchamp
Embedded Software Engineer
Hardware and Chip Integration
______________________________________
KEYMILE AG
Schwarzenburgstr. 73
CH-3097 Liebefeld
Phone +41 31 377 1318
Fax   +41 31 377 1212
valentin.longchamp at keymile.com
www.keymile.com
______________________________________
KEYMILE: A Specialist as a Partner


More information about the U-Boot mailing list