[U-Boot] [PATCH 0/2] Fixes and simplification to omap3_pandora boot configuration

[U-Boot] [PATCH 0/2] Fixes and simplification to omap3_pandora boot configuration

[Vagrant Cascadian]

First patch fixes the syntax used for loading a boot script from mmc.

The second patch makes use of the "load" command provided by
CONFIG_CMD_FS_GENERIC.


Vagrant Cascadian (2):
  omap3_pandora: Fix mmc loading of autoboot script to use correct
    syntax.
  omap3_pandora: Switch to using "load" command to load the autoboot
    script.

 include/configs/omap3_pandora.h | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

-- 
2.9.3

[U-Boot] [PATCH 1/2] omap3_pandora: Fix mmc loading of autoboot script to use correct syntax.

[Vagrant Cascadian]

fatload/ext2load both require that the device and partition be
specified after specifying the device type. Specify the first
partition on mmc device 0, which is the only mmc device currently
configured on the pandora.

Signed-off-by: Vagrant Cascadian <vagrant@debian.org>
---

 include/configs/omap3_pandora.h | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/include/configs/omap3_pandora.h b/include/configs/omap3_pandora.h
index 380ec12..6ff9375 100644
--- a/include/configs/omap3_pandora.h
+++ b/include/configs/omap3_pandora.h
@@ -93,8 +93,8 @@
 	"mtdparts=" MTDPARTS_DEFAULT "\0" \
 
 #define CONFIG_BOOTCOMMAND \
-	"if mmc rescan && fatload mmc1 0 ${loadaddr} autoboot.scr || " \
-			"ext2load mmc1 0 ${loadaddr} autoboot.scr; then " \
+	"if mmc rescan && fatload mmc 0:1 ${loadaddr} autoboot.scr || " \
+			"ext2load mmc 0:1 ${loadaddr} autoboot.scr; then " \
 		"source ${loadaddr}; " \
 	"fi; " \
 	"ubi part boot && ubifsmount ubi:boot && " \
-- 
2.9.3

[U-Boot] [PATCH 2/2] omap3_pandora: Switch to using "load" command to load the autoboot script.

[Vagrant Cascadian]

CONFIG_CMD_FS_GENERIC is enabled; use it to load the autoboot script,
rather than first attempting with fatload and falling back to
ext2load.

Signed-off-by: Vagrant Cascadian <vagrant@debian.org>
---

 include/configs/omap3_pandora.h | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/include/configs/omap3_pandora.h b/include/configs/omap3_pandora.h
index 6ff9375..0d733bd 100644
--- a/include/configs/omap3_pandora.h
+++ b/include/configs/omap3_pandora.h
@@ -93,8 +93,7 @@
 	"mtdparts=" MTDPARTS_DEFAULT "\0" \
 
 #define CONFIG_BOOTCOMMAND \
-	"if mmc rescan && fatload mmc 0:1 ${loadaddr} autoboot.scr || " \
-			"ext2load mmc 0:1 ${loadaddr} autoboot.scr; then " \
+	"if mmc rescan && load mmc 0:1 ${loadaddr} autoboot.scr; then " \
 		"source ${loadaddr}; " \
 	"fi; " \
 	"ubi part boot && ubifsmount ubi:boot && " \
-- 
2.9.3

[U-Boot] [PATCH 1/2] omap3_pandora: Fix mmc loading of autoboot script to use correct syntax.

[Grazvydas Ignotas]

On Mon, Aug 29, 2016 at 10:56 AM, Vagrant Cascadian <vagrant@debian.org> wrote:
> fatload/ext2load both require that the device and partition be
> specified after specifying the device type. Specify the first
> partition on mmc device 0, which is the only mmc device currently
> configured on the pandora.
>
> Signed-off-by: Vagrant Cascadian <vagrant@debian.org>

Acked-by: Grazvydas Ignotas <notasas@gmail.com>

> ---
>
>  include/configs/omap3_pandora.h | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/include/configs/omap3_pandora.h b/include/configs/omap3_pandora.h
> index 380ec12..6ff9375 100644
> --- a/include/configs/omap3_pandora.h
> +++ b/include/configs/omap3_pandora.h
> @@ -93,8 +93,8 @@
>         "mtdparts=" MTDPARTS_DEFAULT "\0" \
>
>  #define CONFIG_BOOTCOMMAND \
> -       "if mmc rescan && fatload mmc1 0 ${loadaddr} autoboot.scr || " \
> -                       "ext2load mmc1 0 ${loadaddr} autoboot.scr; then " \
> +       "if mmc rescan && fatload mmc 0:1 ${loadaddr} autoboot.scr || " \
> +                       "ext2load mmc 0:1 ${loadaddr} autoboot.scr; then " \
>                 "source ${loadaddr}; " \
>         "fi; " \
>         "ubi part boot && ubifsmount ubi:boot && " \
> --
> 2.9.3
>

[U-Boot] [PATCH 2/2] omap3_pandora: Switch to using "load" command to load the autoboot script.

[Grazvydas Ignotas]

On Mon, Aug 29, 2016 at 10:56 AM, Vagrant Cascadian <vagrant@debian.org> wrote:
> CONFIG_CMD_FS_GENERIC is enabled; use it to load the autoboot script,
> rather than first attempting with fatload and falling back to
> ext2load.
>
> Signed-off-by: Vagrant Cascadian <vagrant@debian.org>

Acked-by: Grazvydas Ignotas <notasas@gmail.com>

> ---
>
>  include/configs/omap3_pandora.h | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/include/configs/omap3_pandora.h b/include/configs/omap3_pandora.h
> index 6ff9375..0d733bd 100644
> --- a/include/configs/omap3_pandora.h
> +++ b/include/configs/omap3_pandora.h
> @@ -93,8 +93,7 @@
>         "mtdparts=" MTDPARTS_DEFAULT "\0" \
>
>  #define CONFIG_BOOTCOMMAND \
> -       "if mmc rescan && fatload mmc 0:1 ${loadaddr} autoboot.scr || " \
> -                       "ext2load mmc 0:1 ${loadaddr} autoboot.scr; then " \
> +       "if mmc rescan && load mmc 0:1 ${loadaddr} autoboot.scr; then " \
>                 "source ${loadaddr}; " \
>         "fi; " \
>         "ubi part boot && ubifsmount ubi:boot && " \
> --
> 2.9.3
>

[U-Boot] [U-Boot, 1/2] omap3_pandora: Fix mmc loading of autoboot script to use correct syntax.

[Tom Rini]

On Mon, Aug 29, 2016 at 12:56:05AM -0700, Vagrant Cascadian wrote:

> fatload/ext2load both require that the device and partition be
> specified after specifying the device type. Specify the first
> partition on mmc device 0, which is the only mmc device currently
> configured on the pandora.
> 
> Signed-off-by: Vagrant Cascadian <vagrant@debian.org>
> Acked-by: Grazvydas Ignotas <notasas@gmail.com>

Applied to u-boot/master, thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20160907/8b71d4c7/attachment.sig>

[U-Boot] [U-Boot, 2/2] omap3_pandora: Switch to using "load" command to load the autoboot script.

[Tom Rini]

On Mon, Aug 29, 2016 at 12:56:06AM -0700, Vagrant Cascadian wrote:

> CONFIG_CMD_FS_GENERIC is enabled; use it to load the autoboot script,
> rather than first attempting with fatload and falling back to
> ext2load.
> 
> Signed-off-by: Vagrant Cascadian <vagrant@debian.org>
> Acked-by: Grazvydas Ignotas <notasas@gmail.com>

Applied to u-boot/master, thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20160907/ba2fc7bd/attachment.sig>