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

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

[Vagrant Cascadian]

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
index b586803..b4cb9db 100644
--- a/include/configs/wandboard.h
+++ b/include/configs/wandboard.h
@@ -51,8 +51,6 @@
 #define CONFIG_CMD_BMODE
 #define CONFIG_CMD_SETEXPR
 
-#define CONFIG_BOOTDELAY		5
-
 #define CONFIG_SYS_MEMTEST_START	0x10000000
 #define CONFIG_SYS_MEMTEST_END		(CONFIG_SYS_MEMTEST_START + 500 * SZ_1M)
 #define CONFIG_LOADADDR			0x12000000
@@ -74,9 +72,6 @@
 #define CONFIG_CMD_MMC
 #define CONFIG_GENERIC_MMC
 #define CONFIG_BOUNCE_BUFFER
-#define CONFIG_CMD_EXT2
-#define CONFIG_CMD_FAT
-#define CONFIG_DOS_PARTITION
 
 /* USB Configs */
 #define CONFIG_CMD_USB
@@ -131,131 +126,34 @@
 #endif
 
 #define CONFIG_EXTRA_ENV_SETTINGS \
-	"script=boot.scr\0" \
-	"image=zImage\0" \
 	"console=ttymxc0\0" \
-	"splashpos=m,m\0" \
+	"fdtfile=" CONFIG_DEFAULT_FDT_FILE "\0" \
+	"fdt_addr_r=0x18000000\0"     \
 	"fdt_high=0xffffffff\0" \
 	"initrd_high=0xffffffff\0" \
-	"fdt_file=" CONFIG_DEFAULT_FDT_FILE "\0" \
-	"fdt_addr=0x18000000\0" \
-	"boot_fdt=try\0" \
-	"ip_dyn=yes\0" \
-	"mmcdev=" __stringify(CONFIG_SYS_MMC_ENV_DEV) "\0" \
-	"mmcpart=1\0" \
-	"mmcroot=/dev/mmcblk0p2 rootwait rw\0" \
-	"update_sd_firmware_filename=u-boot.imx\0" \
-	"update_sd_firmware=" \
-		"if test ${ip_dyn} = yes; then " \
-			"setenv get_cmd dhcp; " \
-		"else " \
-			"setenv get_cmd tftp; " \
-		"fi; " \
-		"if mmc dev ${mmcdev}; then "	\
-			"if ${get_cmd} ${update_sd_firmware_filename}; then " \
-				"setexpr fw_sz ${filesize} / 0x200; " \
-				"setexpr fw_sz ${fw_sz} + 1; "	\
-				"mmc write ${loadaddr} 0x2 ${fw_sz}; " \
-			"fi; "	\
-		"fi\0" \
-	"mmcargs=setenv bootargs console=${console},${baudrate} " \
-		"root=${mmcroot}; run videoargs\0" \
-	"videoargs=" \
-		"setenv nextcon 0; " \
-		"if hdmidet; then " \
-			"setenv bootargs ${bootargs} " \
-				"video=mxcfb${nextcon}:dev=hdmi,1280x720M at 60," \
-					"if=RGB24; " \
-			"setenv fbmen fbmem=28M; " \
-			"setexpr nextcon ${nextcon} + 1; " \
-		"else " \
-			"echo - no HDMI monitor;" \
-		"fi; " \
-		"i2c dev 1; " \
-		"if i2c probe 0x10; then " \
-			"setenv bootargs ${bootargs} " \
-				"video=mxcfb${nextcon}:dev=lcd,800x480 at 60," \
-					"if=RGB666,bpp=32; " \
-			"if test 0 -eq ${nextcon}; then " \
-				"setenv fbmem fbmem=10M; " \
-			"else " \
-				"setenv fbmem ${fbmem},10M; " \
-			"fi; " \
-			"setexpr nextcon ${nextcon} + 1; " \
-		"else " \
-			"echo '- no FWBADAPT-7WVGA-LCD-F07A-0102 display';" \
-		"fi; " \
-		"setenv bootargs ${bootargs} ${fbmem}\0" \
-	"loadbootscript=" \
-		"fatload mmc ${mmcdev}:${mmcpart} ${loadaddr} ${script};\0" \
-	"bootscript=echo Running bootscript from mmc ...; " \
-		"source\0" \
-	"loadimage=fatload mmc ${mmcdev}:${mmcpart} ${loadaddr} ${image}\0" \
-	"loadfdt=fatload mmc ${mmcdev}:${mmcpart} ${fdt_addr} ${fdt_file}\0" \
-	"mmcboot=echo Booting from mmc ...; " \
-		"run mmcargs; " \
-		"if test ${boot_fdt} = yes || test ${boot_fdt} = try; then " \
-			"if run loadfdt; then " \
-				"bootz ${loadaddr} - ${fdt_addr}; " \
-			"else " \
-				"if test ${boot_fdt} = try; then " \
-					"bootz; " \
-				"else " \
-					"echo WARN: Cannot load the DT; " \
-				"fi; " \
-			"fi; " \
-		"else " \
-			"bootz; " \
-		"fi;\0" \
-	"netargs=setenv bootargs console=${console},${baudrate} " \
-		"root=/dev/nfs " \
-	"ip=dhcp nfsroot=${serverip}:${nfsroot},v3,tcp\0" \
-		"netboot=echo Booting from net ...; " \
-		"run netargs; " \
-		"if test ${ip_dyn} = yes; then " \
-			"setenv get_cmd dhcp; " \
-		"else " \
-			"setenv get_cmd tftp; " \
-		"fi; " \
-		"${get_cmd} ${image}; " \
-		"if test ${boot_fdt} = yes || test ${boot_fdt} = try; then " \
-			"if ${get_cmd} ${fdt_addr} ${fdt_file}; then " \
-				"bootz ${loadaddr} - ${fdt_addr}; " \
-			"else " \
-				"if test ${boot_fdt} = try; then " \
-					"bootz; " \
-				"else " \
-					"echo WARN: Cannot load the DT; " \
-				"fi; " \
-			"fi; " \
-		"else " \
-			"bootz; " \
-		"fi;\0"
-
-#define CONFIG_BOOTCOMMAND \
-	   "mmc dev ${mmcdev}; if mmc rescan; then " \
-		   "if run loadbootscript; then " \
-			   "run bootscript; " \
-		   "else " \
-			   "if run loadimage; then " \
-				   "run mmcboot; " \
-			   "else run netboot; " \
-			   "fi; " \
-		   "fi; " \
-	   "else run netboot; fi"
+	"kernel_addr_r=" __stringify(CONFIG_LOADADDR) "\0" \
+	"pxe_addr_r=" __stringify(CONFIG_LOADADDR) "\0" \
+	"ramdisk_addr_r=0x13000000\0" \
+	"scriptaddr=" __stringify(CONFIG_LOADADDR) "\0" \
+	BOOTENV
+
+#define BOOT_TARGET_DEVICES(func) \
+	func(MMC, mmc, 0) \
+	func(MMC, mmc, 1) \
+	func(USB, usb, 0) \
+	func(PXE, pxe, na) \
+	func(DHCP, dhcp, na)
+
+#include <config_distro_defaults.h>
+#include <config_distro_bootcmd.h>
 
 /* Miscellaneous configurable options */
-#define CONFIG_SYS_LONGHELP
-#define CONFIG_SYS_HUSH_PARSER
-#define CONFIG_AUTO_COMPLETE
 #define CONFIG_SYS_CBSIZE		256
 #define CONFIG_SYS_MAXARGS	       16
 #define CONFIG_SYS_BARGSIZE CONFIG_SYS_CBSIZE
 
 #define CONFIG_SYS_LOAD_ADDR		CONFIG_LOADADDR
 
-#define CONFIG_CMDLINE_EDITING
-
 /* Physical Memory Map */
 #define CONFIG_NR_DRAM_BANKS		1
 #define PHYS_SDRAM			MMDC0_ARB_BASE_ADDR
@@ -278,9 +176,6 @@
 #define CONFIG_ENV_OFFSET		(6 * 64 * 1024)
 #define CONFIG_SYS_MMC_ENV_DEV		0
 
-#define CONFIG_OF_LIBFDT
-#define CONFIG_CMD_BOOTZ
-
 #ifndef CONFIG_SYS_DCACHE_OFF
 #define CONFIG_CMD_CACHE
 #endif
-- 
2.1.4

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

[Tom Rini]

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: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20150329/c2c9bf76/attachment.sig>

[U-Boot] [PATCH] Document the format of the console environment variable

[Karsten Merker]

config_distro_bootcmd.h defines a common boot environment for
multiple platforms.  Document the format of the console
environment variable to allow the use of generic boot scripts
on all supported platforms.

Signed-off-by: Karsten Merker <merker@debian.org>
---
 doc/README.distro |    9 +++++++++
 1 file changed, 9 insertions(+)

diff --git a/doc/README.distro b/doc/README.distro
index 0308a4c..3f7ee85 100644
--- a/doc/README.distro
+++ b/doc/README.distro
@@ -201,6 +201,15 @@ variables be set. Default values for these variables are often hard-coded into
 CONFIG_EXTRA_ENV_SETTINGS in the board's U-Boot configuration file, so that
 the user doesn't have to configure them.
 
+console:
+
+  Mandatory. The default console device to be passed to the Linux kernel.
+
+  The console device is specified in accordance with the guidelines
+  described in Documentation/serial-console.txt in the Linux kernel sources.
+  If the device is a serial port, the console variable must specify the
+  console baudrate in addition to the console device, e.g. "ttyS0,115200".
+
 fdt_addr:
 
   Mandatory for any system that provides the DTB in HW (e.g. ROM) and wishes
-- 
1.7.10.4

[U-Boot] [PATCH] Document the format of the console environment variable

[Hans de Goede]

Hi,

On 29-03-15 17:55, Karsten Merker wrote:
> config_distro_bootcmd.h defines a common boot environment for
> multiple platforms.  Document the format of the console
> environment variable to allow the use of generic boot scripts
> on all supported platforms.
>
> Signed-off-by: Karsten Merker <merker@debian.org>

Maybe add something that distros are not encouraged to use this ?

At least on Fedora we do not want to use it, as adding
console=${console} to bootargs on systems with both video-output
(e.g. hdmi out) and a serial port will cause boot messages to
only be shown on the serial port as that is where the /dev/console
then points.

Whereas when using the devicetree chosen/stdout-path property (which
can be set either by u-boot or in the dts with the kernel),
the kernel will use both /dev/tty0 (so the hdmi out) and the serial
port pointed to by chosen/stdout as console, showing boot messages
on both, and systemd will still automatically spawn a getty on the
serial console.

Regards,

Hans




> ---
>   doc/README.distro |    9 +++++++++
>   1 file changed, 9 insertions(+)
>
> diff --git a/doc/README.distro b/doc/README.distro
> index 0308a4c..3f7ee85 100644
> --- a/doc/README.distro
> +++ b/doc/README.distro
> @@ -201,6 +201,15 @@ variables be set. Default values for these variables are often hard-coded into
>   CONFIG_EXTRA_ENV_SETTINGS in the board's U-Boot configuration file, so that
>   the user doesn't have to configure them.
>
> +console:
> +
> +  Mandatory. The default console device to be passed to the Linux kernel.
> +
> +  The console device is specified in accordance with the guidelines
> +  described in Documentation/serial-console.txt in the Linux kernel sources.
> +  If the device is a serial port, the console variable must specify the
> +  console baudrate in addition to the console device, e.g. "ttyS0,115200".
> +
>   fdt_addr:
>
>     Mandatory for any system that provides the DTB in HW (e.g. ROM) and wishes
>

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

[XoD]

Any new of this ? it can be merged ?

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 ?

Thank you for working on this.

--
XoD

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
>
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot
>
>

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

[Vagrant Cascadian]

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>

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

[XoD]

Ok, thank you for your response.

I waiting SPL for wandboard too.
I like to add Wandboard support in OpenELEC (and Lakka) with SPL (to not
have separate img for Wandboard dual and Wandboard quad.)

XoD

2015-05-19 22:42 GMT+02:00 Vagrant Cascadian <vagrant@debian.org>:

> 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
>

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

[XoD]

Hello,
I have now tested upstream uboot-imx with OpenELEC with SPL suppport for
wandbard.

It's work great without your patch. but with your path, OpenELEC don't boot.

The default uboot config assume than kernel file name is zImage.
But OpenELEC use a filename KERNEL.
The actual version of OpenELEC use a uEnv.txt file with this line :
zImage=/KERNEL
bootfile=/KERNEL
But this no longer work with your patch.

How I can configure uboot to open a file named KERNEL instead of zImage (at
build time or at runtime) ?

Thank you.

2015-05-20 9:36 GMT+02:00 XoD <xoddark@gmail.com>:

> Ok, thank you for your response.
>
> I waiting SPL for wandboard too.
> I like to add Wandboard support in OpenELEC (and Lakka) with SPL (to not
> have separate img for Wandboard dual and Wandboard quad.)
>
> XoD
>
> 2015-05-19 22:42 GMT+02:00 Vagrant Cascadian <vagrant@debian.org>:
>
>> 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
>>
>
>

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

[Nikolay Dimitrov]

Hi XoD,

On 06/26/2015 02:54 PM, XoD wrote:
> Hello,
> I have now tested upstream uboot-imx with OpenELEC with SPL suppport for
> wandbard.
>
> It's work great without your patch. but with your path, OpenELEC don't boot.
>
> The default uboot config assume than kernel file name is zImage.
> But OpenELEC use a filename KERNEL.
> The actual version of OpenELEC use a uEnv.txt file with this line :
> zImage=/KERNEL
> bootfile=/KERNEL
> But this no longer work with your patch.
>
> How I can configure uboot to open a file named KERNEL instead of zImage (at
> build time or at runtime) ?

config_distro_bootcmd uses a cool feature called "extlinux", which
allows you to define a boot configuration without hacking in U-Boot
source/configuration. Here's how it works:

1. U-Boot scans a list of boot devices (mmc, usb, pxe, dhcp)
2. When it find a bootable local storage (mmc, usb) it looks for the
first bootable partition, or just first one if no partition was marked
as bootable
3. U-Boot looks for extlinux.conf in "/" and "/boot". This file
contains the actual boot configuration (kernel image, bootargs, etc)
that will be executed on boot. Here's how a simple extlinux.conf looks
like:

-----> cut <-----
default linux

label linux
kernel /boot/zImage
devicetree /boot/mydevicetree.dtb
append console=ttymxc1,115200 root=/dev/mmcblk0p1 rw
-----> cut <-----

4. U-Boot boots the "default" configuration.

Hope this helps.

Regards,
Nikolay

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

[XoD]

Ok, great, I use a little the file extlinux with Fedora on Wandboard.

Thank you for your detailed answer.

Le vendredi 26 juin 2015, Nikolay Dimitrov <picmaster@mail.bg> a ?crit :

> Hi XoD,
>
> On 06/26/2015 02:54 PM, XoD wrote:
>
>> Hello,
>> I have now tested upstream uboot-imx with OpenELEC with SPL suppport for
>> wandbard.
>>
>> It's work great without your patch. but with your path, OpenELEC don't
>> boot.
>>
>> The default uboot config assume than kernel file name is zImage.
>> But OpenELEC use a filename KERNEL.
>> The actual version of OpenELEC use a uEnv.txt file with this line :
>> zImage=/KERNEL
>> bootfile=/KERNEL
>> But this no longer work with your patch.
>>
>> How I can configure uboot to open a file named KERNEL instead of zImage
>> (at
>> build time or at runtime) ?
>>
>
> config_distro_bootcmd uses a cool feature called "extlinux", which
> allows you to define a boot configuration without hacking in U-Boot
> source/configuration. Here's how it works:
>
> 1. U-Boot scans a list of boot devices (mmc, usb, pxe, dhcp)
> 2. When it find a bootable local storage (mmc, usb) it looks for the
> first bootable partition, or just first one if no partition was marked
> as bootable
> 3. U-Boot looks for extlinux.conf in "/" and "/boot". This file
> contains the actual boot configuration (kernel image, bootargs, etc)
> that will be executed on boot. Here's how a simple extlinux.conf looks
> like:
>
> -----> cut <-----
> default linux
>
> label linux
> kernel /boot/zImage
> devicetree /boot/mydevicetree.dtb
> append console=ttymxc1,115200 root=/dev/mmcblk0p1 rw
> -----> cut <-----
>
> 4. U-Boot boots the "default" configuration.
>
> Hope this helps.
>
> Regards,
> Nikolay
>

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

[Otavio Salvador]

On Tue, Oct 27, 2015 at 5:13 PM, Vagrant Cascadian <vagrant@debian.org> wrote:
> On 2015-10-27, Vagrant Cascadian wrote:
>> This allows for more flexible and standardized boot across multiple
>> platforms. Remove most redundant legacy boot environment.
>
> Er, Bad commit message, this patch doesn't actually "Remove most
> redundant legacy boot environment" at all. Leftovers from an earlier
> attempt.

LOL, this was my complain a minute ago ;-)

-- 
Otavio Salvador                             O.S. Systems
http://www.ossystems.com.br        http://code.ossystems.com.br
Mobile: +55 (53) 9981-7854            Mobile: +1 (347) 903-9750

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

[Otavio Salvador]

On Tue, Oct 27, 2015 at 5:06 PM, Vagrant Cascadian <vagrant@debian.org> 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>

You need to remove the old environment script which is now duplicated.
Can you prepare a v2?

-- 
Otavio Salvador                             O.S. Systems
http://www.ossystems.com.br        http://code.ossystems.com.br
Mobile: +55 (53) 9981-7854            Mobile: +1 (347) 903-9750

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

[Vagrant Cascadian]

On 2015-10-27, Vagrant Cascadian wrote:
> This allows for more flexible and standardized boot across multiple
> platforms. Remove most redundant legacy boot environment.

Er, Bad commit message, this patch doesn't actually "Remove most
redundant legacy boot environment" at all. Leftovers from an earlier
attempt.

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/20151027/3c3a78ab/attachment.sig>

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

[Vagrant Cascadian]

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 | 18 ++++++++++++++++++
 1 file changed, 18 insertions(+)

diff --git a/include/configs/wandboard.h b/include/configs/wandboard.h
index 6e8aec2..67d5a6a 100644
--- a/include/configs/wandboard.h
+++ b/include/configs/wandboard.h
@@ -95,6 +95,7 @@
 	"fdtfile=undefined\0" \
 	"fdt_high=0xffffffff\0" \
 	"initrd_high=0xffffffff\0" \
+	"fdt_addr_r=0x18000000\0" \
 	"fdt_addr=0x18000000\0" \
 	"boot_fdt=try\0" \
 	"ip_dyn=yes\0" \
@@ -199,9 +200,23 @@
 			"setenv fdtfile imx6dl-wandboard-revb1.dtb; fi; " \
 		"if test $fdtfile = undefined; then " \
 			"echo WARNING: Could not determine dtb to use; fi; \0" \
+	"kernel_addr_r=" __stringify(CONFIG_LOADADDR) "\0" \
+	"pxe_addr_r=" __stringify(CONFIG_LOADADDR) "\0" \
+	"ramdisk_addr_r=0x13000000\0" \
+	"ramdiskaddr=0x13000000\0" \
+	"scriptaddr=" __stringify(CONFIG_LOADADDR) "\0" \
+	BOOTENV
+
+#define BOOT_TARGET_DEVICES(func) \
+	func(MMC, mmc, 0) \
+	func(MMC, mmc, 1) \
+	func(USB, usb, 0) \
+	func(PXE, pxe, na) \
+	func(DHCP, dhcp, na)
 
 #define CONFIG_BOOTCOMMAND \
 	   "run findfdt; " \
+	   "run distro_bootcmd; " \
 	   "mmc dev ${mmcdev}; if mmc rescan; then " \
 		   "if run loadbootscript; then " \
 			   "run bootscript; " \
@@ -213,6 +228,9 @@
 		   "fi; " \
 	   "else run netboot; fi"
 
+#include <config_distro_defaults.h>
+#include <config_distro_bootcmd.h>
+
 /* Physical Memory Map */
 #define CONFIG_NR_DRAM_BANKS		1
 #define PHYS_SDRAM			MMDC0_ARB_BASE_ADDR
-- 
2.1.4