mbox series

[U-Boot,0/4] arm: odroid: Convert to distro_bootcmd.

Message ID 20180605211355.6433-1-vagrant@debian.org
Headers show
Series arm: odroid: Convert to distro_bootcmd. | expand

Message

Vagrant Cascadian June 5, 2018, 9:13 p.m. UTC
Switch odroid platform to use distro_bootcmd, adjusting to use
standardized environment variable names, and use the default
bootdelay.

The additional environment variables from distro_bootcmd requires
increasing the default environment size, which would break backwards
compatibility with saved environments. At Marek's suggestion, I've
bumped it well over the size needed (~4.5k vs. 16k) to give room for
future growth.

Variations on these patches have been in use in Debian's u-boot
packages since 2016.

A variation of this patch series was originally submitted to u-boot
over a year ago, and resent a few times since, with little to no
response.

Thanks for considering!


Vagrant Cascadian (4):
  arm: odroid: Use standard environment variable names kernel_addr_r,
    ramdisk_addr_r and fdt_addr_r.
  arm: odroid: Increase default env size in preparation for
    distro_bootcmd.
  arm: odroid: Enable distro_bootcmd support.
  arm: odroid: Inherit default value for bootdelay from distro_bootcmd.

 include/configs/odroid.h | 36 ++++++++++++++++++++----------------
 1 file changed, 20 insertions(+), 16 deletions(-)

Comments

Lukasz Majewski June 7, 2018, 7:47 a.m. UTC | #1
On Tue,  5 Jun 2018 14:13:51 -0700
Vagrant Cascadian <vagrant@debian.org> wrote:

> Switch odroid platform to use distro_bootcmd, adjusting to use
> standardized environment variable names, and use the default
> bootdelay.
> 
> The additional environment variables from distro_bootcmd requires
> increasing the default environment size, which would break backwards
> compatibility with saved environments. At Marek's suggestion, I've
> bumped it well over the size needed (~4.5k vs. 16k) to give room for
> future growth.
> 
> Variations on these patches have been in use in Debian's u-boot
> packages since 2016.
> 
> A variation of this patch series was originally submitted to u-boot
> over a year ago, and resent a few times since, with little to no
> response.
> 
> Thanks for considering!

For the whole series:

Acked-by: Lukasz Majewski <lukma@denx.de>

> 
> 
> Vagrant Cascadian (4):
>   arm: odroid: Use standard environment variable names kernel_addr_r,
>     ramdisk_addr_r and fdt_addr_r.
>   arm: odroid: Increase default env size in preparation for
>     distro_bootcmd.
>   arm: odroid: Enable distro_bootcmd support.
>   arm: odroid: Inherit default value for bootdelay from
> distro_bootcmd.
> 
>  include/configs/odroid.h | 36 ++++++++++++++++++++----------------
>  1 file changed, 20 insertions(+), 16 deletions(-)
> 




Best regards,

Lukasz Majewski

--

DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd@denx.de