[U-Boot] [PATCH 0/10 v2] i.MX6: move duplicated options to mx6_common to standardise mx6 config

[Stefano Babic]

Hi Peter,

On 22/05/2015 01:15, Peter Robinson wrote:
> Hi Stefano,
> 
> Sorry for the delay, we're on the pointy end of the Fedora 22 release
> and that's been taking my time.
> 

Do not worry :-)

>> On 11/05/2015 18:22, Peter Robinson wrote:

>> For example, tqma6 defines CONFIG_SYS_TEXT_BASE. I agree to have it in
>> mx6_common, we need a #undef CONFIG_SYS_TEXT_BASE in the board
>> configuration file for boards (like tqma6) that redefines it.
> 
> Is tqma6 a new board since I did the late patch, I thought any options
> I'd moved into the central were ones I removed over all the imx6
> boards
> 
>> CONFIG_BOOTDELAY is also set in config_distro_defaults, and it generates
>> also a warning - maybe we have to remove it from mx6_common.h
> 
> I was hoping eventually to be able to add the distro_defaults to the
> common so we got that across all the boards but I'd not got that far
> yet. Was basically trying to do the low hanging fruit first.

Agree on that. This can be done later.

> 
> In the interim would a #ifndef work?

Yes. After this big cleanup will be merged, it will be easier to
optimize further issues.

> 
>> Can you check these issues and repost a V3 on current u-boot-imx tree ?
> 
> Sure, will try and get the time over the weekend to rebase.

Thanks !

Best regards,
Stefano Babic


-- 
=====================================================================
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
=====================================================================