From mboxrd@z Thu Jan 1 00:00:00 1970 From: Vagrant Cascadian Date: Thu, 12 Nov 2015 11:39:07 -0600 Subject: [U-Boot] [PATCH] arm: imx: Switch mx6cuboxi to use config_distro_bootcmd.h. In-Reply-To: <5644C951.4080208@denx.de> References: <1445977102-16705-1-git-send-email-vagrant@debian.org> <5644C951.4080208@denx.de> Message-ID: <87egfvdsn8.fsf@aikidev.net> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de 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 >> +#include > > 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: