[U-Boot] [PATCH] arm: imx: Switch mx6cuboxi to use config_distro_bootcmd.h.

Vagrant Cascadian
2015-11-12T17:39:07Z

On 2015-11-12, Stefano Babic wrote:
> On 27/10/2015 21:18, Vagrant Cascadian wrote:
>> This allows for more flexible and standardized boot across multiple
>> platforms.
>>
>> Remove redundant legacy boot environment.
>>
>
> I have applied it.

Thanks!

> However, the patch generates a warning due to
> CONFIG_BOOT_DELAY, that is defined twice. Firstly, it is defined in
> mx6_common.h (anyway, in this file is protected) and then in distro_default

Oops, forgot to file an updated patch to fix that...

>> +#define BOOT_TARGET_DEVICES(func) \
>> + func(MMC, mmc, 0) \
>> + func(USB, usb, 0) \
>> + func(PXE, pxe, na) \
>> + func(DHCP, dhcp, na)
>> +
>
>> +#include <config_distro_defaults.h>
>> +#include <config_distro_bootcmd.h>
>
> Maybe just #undef CONFIG_BOOT_DELAY before including distro files ?

Alternately, including config_distro_defaults.h could be moved before
including mx6_common.h. Either way works fine by me.

Same applies for the wandboard patch.

I just realized that the console setting for mx6cuboxi should also
include the baudrate, just like for the wandboard.

live well,
vagrant
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 818 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20151112/b04e2616/attachment.sig>