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

Vagrant Cascadian
2015-05-19T20:42:35Z

On 2015-05-19, XoD wrote:
> Any new of this ? it can be merged ?

I think my submitted patch was a bit too invasive by removing most of
the environment, and have reworked the patch to leave much of the
environment:

https://anonscm.debian.org/cgit/collab-maint/u-boot.git/tree/debian/patches/wandboard/config_distro_bootcmd.patch?h=experimental-2015.04

I haven't yet reworked it for resubmission, but I'd be happy to do so.

I was hoping to see the wandboard SPL support added before reworking, as
that will require a few minor changes to the config_distro_bootcmd patch
as well:

https://patchwork.ozlabs.org/patch/471092/

> I have tested and successfully boot a fedora on a Wandboard Quad with this.
>
> The improvement of the console variable management can be done with an
> other patch ?

It would be nice if switching to config_distro_bootcmd.h was not
dependent on sorting out the console variable switch... though I'd be
fine with switching the default console to include the baudrate as well.

> Thank you for working on this.

Thanks for testing!

live well,
vagrant

> 2015-03-29 15:05 GMT+02:00 Tom Rini <trini@konsulko.com>:
>
>> On Sat, Mar 28, 2015 at 02:15:38PM +0100, Karsten Merker wrote:
>> > On Fri, Mar 27, 2015 at 06:24:43PM -0700, Vagrant Cascadian wrote:
>> >
>> > > This allows for more flexible and standardized boot across multiple
>> > > platforms. Remove most redundant legacy boot environment.
>> > >
>> > > Cc: Otavio Salvador <otavio@ossystems.com.br>
>> > > Signed-off-by: Vagrant Cascadian <vagrant@debian.org>
>> > > ---
>> > >
>> > > include/configs/wandboard.h | 139
>> ++++++--------------------------------------
>> > > 1 file changed, 17 insertions(+), 122 deletions(-)
>> > >
>> > > diff --git a/include/configs/wandboard.h b/include/configs/wandboard.h
>> > [...]
>> > > #define CONFIG_EXTRA_ENV_SETTINGS \
>> > > - "script=boot.scr\0" \
>> > > - "image=zImage\0" \
>> > > "console=ttymxc0\0" \
>> >
>> > Hello,
>> >
>> > regarding the boot environment standardization there is still the
>> > open topic of standardizing the console variable format for
>> > serial consoles - most platforms include the console baudrate in
>> > the console variable (e.g. "console=ttyS0,115200") while some
>> > others, in particular the i.MX6 platforms, do not. This means
>> > that distributions like Debian currently need to add special-case
>> > handling for i.MX6-based platforms in their boot scripts which
>> > goes against the idea of having one generic boot script for all
>> > platforms that use config_distro_bootcmd.h.
>> >
>> > It would be nice if the i.MX6 platforms could - while adopting
>> > config_distro_bootcmd.h and thereby changing their default
>> > environment to a large extend - also change their console
>> > variable from console=ttymxc0 to console=ttymxc0,115200.
>>
>> Yes please. And Karsten can you do a patch that updates the README to
>> note that as an expectation? Thanks!
>>
>> --
>> Tom
-------------- 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/20150519/80107b92/attachment.sig>