[U-Boot] [PATCH 0/6] Upstream environment changes being used in Yocto/OE

[U-Boot] [PATCH 0/6] Upstream environment changes being used in Yocto/OE

[Otavio Salvador]

This changes were being done in every version of U-Boot and it
makes sense to try to merge them upstream.

Please review them and ack/nack them.


Otavio Salvador (6):
  mx53loco: Change default environment to cope with OE changes
  mx6qsabrelite: Change default environment to cope with OE changes
  mx6slevk: Change default environment to cope with OE changes
  wandboard: Change default environment to cope with OE changes
  mx51evk: Change default environment to cope with OE changes
  mx53ard: Change default environment to cope with OE changes

 include/configs/mx51evk.h       | 4 ++--
 include/configs/mx53ard.h       | 4 ++--
 include/configs/mx53loco.h      | 4 ++--
 include/configs/mx6qsabrelite.h | 4 ++--
 include/configs/mx6slevk.h      | 4 ++--
 include/configs/wandboard.h     | 4 ++--
 6 files changed, 12 insertions(+), 12 deletions(-)

-- 
1.8.3.1

[U-Boot] [PATCH 1/6] mx53loco: Change default environment to cope with OE changes

[Otavio Salvador]

OpenEmbedded has change partitioning layout of generated image so it
does not raise warnings during the boot regarding unkown partition
being used for U-Boot.

Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
---
 include/configs/mx53loco.h | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/include/configs/mx53loco.h b/include/configs/mx53loco.h
index a4b610f..cc31e9b 100644
--- a/include/configs/mx53loco.h
+++ b/include/configs/mx53loco.h
@@ -124,8 +124,8 @@
 	"boot_fdt=try\0" \
 	"ip_dyn=yes\0" \
 	"mmcdev=0\0" \
-	"mmcpart=2\0" \
-	"mmcroot=/dev/mmcblk0p3 rw rootwait\0" \
+	"mmcpart=1\0" \
+	"mmcroot=/dev/mmcblk0p2 rw rootwait\0" \
 	"mmcargs=setenv bootargs console=ttymxc0,${baudrate} root=${mmcroot}\0" \
 	"loadbootscript=" \
 		"fatload mmc ${mmcdev}:${mmcpart} ${loadaddr} ${script};\0" \
-- 
1.8.3.1

[U-Boot] [PATCH 2/6] mx6qsabrelite: Change default environment to cope with OE changes

[Otavio Salvador]

OpenEmbedded has change partitioning layout of generated image so it
does not raise warnings during the boot regarding unkown partition
being used for U-Boot.

Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
---
 include/configs/mx6qsabrelite.h | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/include/configs/mx6qsabrelite.h b/include/configs/mx6qsabrelite.h
index b814418..c7db81d 100644
--- a/include/configs/mx6qsabrelite.h
+++ b/include/configs/mx6qsabrelite.h
@@ -170,8 +170,8 @@
 	"boot_fdt=try\0" \
 	"ip_dyn=yes\0" \
 	"mmcdev=0\0" \
-	"mmcpart=2\0" \
-	"mmcroot=/dev/mmcblk0p3 rootwait rw\0" \
+	"mmcpart=1\0" \
+	"mmcroot=/dev/mmcblk0p2 rootwait rw\0" \
 	"mmcargs=setenv bootargs console=${console},${baudrate} " \
 		"root=${mmcroot}\0" \
 	"loadbootscript=" \
-- 
1.8.3.1

[U-Boot] [PATCH 3/6] mx6slevk: Change default environment to cope with OE changes

[Otavio Salvador]

OpenEmbedded has change partitioning layout of generated image so it
does not raise warnings during the boot regarding unkown partition
being used for U-Boot.

Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
---
 include/configs/mx6slevk.h | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/include/configs/mx6slevk.h b/include/configs/mx6slevk.h
index 19dcdd6..55e3ad9 100644
--- a/include/configs/mx6slevk.h
+++ b/include/configs/mx6slevk.h
@@ -73,8 +73,8 @@
 	"boot_fdt=try\0" \
 	"ip_dyn=yes\0" \
 	"mmcdev=0\0" \
-	"mmcpart=2\0" \
-	"mmcroot=/dev/mmcblk0p3 rootwait rw\0" \
+	"mmcpart=1\0" \
+	"mmcroot=/dev/mmcblk0p2 rootwait rw\0" \
 	"mmcargs=setenv bootargs console=${console},${baudrate} " \
 		"root=${mmcroot}\0" \
 	"loadbootscript=" \
-- 
1.8.3.1

[U-Boot] [PATCH 4/6] wandboard: Change default environment to cope with OE changes

[Otavio Salvador]

OpenEmbedded has change partitioning layout of generated image so it
does not raise warnings during the boot regarding unkown partition
being used for U-Boot.

Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
---
 include/configs/wandboard.h | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/include/configs/wandboard.h b/include/configs/wandboard.h
index b2995d8..ee6bf21 100644
--- a/include/configs/wandboard.h
+++ b/include/configs/wandboard.h
@@ -120,8 +120,8 @@
 	"boot_fdt=try\0" \
 	"ip_dyn=yes\0" \
 	"mmcdev=" __stringify(CONFIG_SYS_MMC_ENV_DEV) "\0" \
-	"mmcpart=2\0" \
-	"mmcroot=/dev/mmcblk0p3 rootwait rw\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 " \
-- 
1.8.3.1

[U-Boot] [PATCH 5/6] mx51evk: Change default environment to cope with OE changes

[Otavio Salvador]

OpenEmbedded has change partitioning layout of generated image so it
does not raise warnings during the boot regarding unkown partition
being used for U-Boot.

Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
---
 include/configs/mx51evk.h | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/include/configs/mx51evk.h b/include/configs/mx51evk.h
index 13d1839..4383375d 100644
--- a/include/configs/mx51evk.h
+++ b/include/configs/mx51evk.h
@@ -162,8 +162,8 @@
 	"boot_fdt=try\0" \
 	"ip_dyn=yes\0" \
 	"mmcdev=0\0" \
-	"mmcpart=2\0" \
-	"mmcroot=/dev/mmcblk0p3 rootwait rw\0" \
+	"mmcpart=1\0" \
+	"mmcroot=/dev/mmcblk0p2 rootwait rw\0" \
 	"mmcargs=setenv bootargs console=ttymxc0,${baudrate} " \
 		"root=${mmcroot}\0" \
 	"loadbootscript=" \
-- 
1.8.3.1

[U-Boot] [PATCH 6/6] mx53ard: Change default environment to cope with OE changes

[Otavio Salvador]

OpenEmbedded has change partitioning layout of generated image so it
does not raise warnings during the boot regarding unkown partition
being used for U-Boot.

Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
---
 include/configs/mx53ard.h | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/include/configs/mx53ard.h b/include/configs/mx53ard.h
index b0a965f..fa160e4 100644
--- a/include/configs/mx53ard.h
+++ b/include/configs/mx53ard.h
@@ -118,8 +118,8 @@
 	"boot_fdt=try\0" \
 	"ip_dyn=yes\0" \
 	"mmcdev=" __stringify(CONFIG_SYS_MMC_ENV_DEV) "\0" \
-	"mmcpart=2\0" \
-	"mmcroot=/dev/mmcblk0p3 rootwait rw\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 " \
-- 
1.8.3.1

[U-Boot] [PATCH 0/6] Upstream environment changes being used in Yocto/OE

[Fabio Estevam]

On Fri, Jun 28, 2013 at 6:52 PM, Otavio Salvador
<otavio@ossystems.com.br> wrote:
> This changes were being done in every version of U-Boot and it
> makes sense to try to merge them upstream.
>
> Please review them and ack/nack them.

I think the series look good:

Reviewed-by: Fabio Estevam <fabio.estevam@freescale.com>

[U-Boot] [PATCH 0/6] Upstream environment changes being used in Yocto/OE

[Tom Rini]

On Fri, Jun 28, 2013 at 06:56:46PM -0300, Fabio Estevam wrote:

> On Fri, Jun 28, 2013 at 6:52 PM, Otavio Salvador
> <otavio@ossystems.com.br> wrote:
> > This changes were being done in every version of U-Boot and it
> > makes sense to try to merge them upstream.
> >
> > Please review them and ack/nack them.
> 
> I think the series look good:
> 
> Reviewed-by: Fabio Estevam <fabio.estevam@freescale.com>

We aren't breaking compat with other distros, are we?  In TI-land where
(for better or worse) we often have a FAT partition we're dealing with
too, we use (and can have OE spit out) a uEnv.txt file that's
user-editable and imported to the environment that would update these
variables.  I suspect there's not an easy analog here but I thought I'd
bring it up.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20130701/67431f42/attachment.pgp>

[U-Boot] [PATCH 0/6] Upstream environment changes being used in Yocto/OE

[Otavio Salvador]

On Mon, Jul 1, 2013 at 10:20 AM, Tom Rini <trini@ti.com> wrote:
> On Fri, Jun 28, 2013 at 06:56:46PM -0300, Fabio Estevam wrote:
>
>> On Fri, Jun 28, 2013 at 6:52 PM, Otavio Salvador
>> <otavio@ossystems.com.br> wrote:
>> > This changes were being done in every version of U-Boot and it
>> > makes sense to try to merge them upstream.
>> >
>> > Please review them and ack/nack them.
>>
>> I think the series look good:
>>
>> Reviewed-by: Fabio Estevam <fabio.estevam@freescale.com>
>
> We aren't breaking compat with other distros, are we?  In TI-land where
> (for better or worse) we often have a FAT partition we're dealing with
> too, we use (and can have OE spit out) a uEnv.txt file that's
> user-editable and imported to the environment that would update these
> variables.  I suspect there's not an easy analog here but I thought I'd
> bring it up.

We've not used uEnv.txt files for this in Freescale's Yocto support
but patched the U-Boot source code to make it work out of box. I think
these patches could go in now that Freescale is using Yocto more
extensivelly for ARM platforms so we avoid  same patches to be ported
for every new U-Boot release.

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

[U-Boot] [PATCH 0/6] Upstream environment changes being used in Yocto/OE

[Tom Rini]

On Mon, Jul 01, 2013 at 10:25:46AM -0300, Otavio Salvador wrote:
> On Mon, Jul 1, 2013 at 10:20 AM, Tom Rini <trini@ti.com> wrote:
> > On Fri, Jun 28, 2013 at 06:56:46PM -0300, Fabio Estevam wrote:
> >
> >> On Fri, Jun 28, 2013 at 6:52 PM, Otavio Salvador
> >> <otavio@ossystems.com.br> wrote:
> >> > This changes were being done in every version of U-Boot and it
> >> > makes sense to try to merge them upstream.
> >> >
> >> > Please review them and ack/nack them.
> >>
> >> I think the series look good:
> >>
> >> Reviewed-by: Fabio Estevam <fabio.estevam@freescale.com>
> >
> > We aren't breaking compat with other distros, are we?  In TI-land where
> > (for better or worse) we often have a FAT partition we're dealing with
> > too, we use (and can have OE spit out) a uEnv.txt file that's
> > user-editable and imported to the environment that would update these
> > variables.  I suspect there's not an easy analog here but I thought I'd
> > bring it up.
> 
> We've not used uEnv.txt files for this in Freescale's Yocto support
> but patched the U-Boot source code to make it work out of box. I think
> these patches could go in now that Freescale is using Yocto more
> extensivelly for ARM platforms so we avoid  same patches to be ported
> for every new U-Boot release.

Right well and good.  But part of the reason we use it here is that
meta-various-ti-things may use a different layout from
meta-community-oriented-distro and this lets us handle both with just a
tweak in the various-ti-things layer to add a text file.  Again, may not
apply here and that's fine.  Heck, we're trying to move more towards
always just using the community layout.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20130701/50f7315c/attachment.pgp>

[U-Boot] [PATCH 0/6] Upstream environment changes being used in Yocto/OE

[Otavio Salvador]

On Mon, Jul 1, 2013 at 10:30 AM, Tom Rini <trini@ti.com> wrote:
> On Mon, Jul 01, 2013 at 10:25:46AM -0300, Otavio Salvador wrote:
>> On Mon, Jul 1, 2013 at 10:20 AM, Tom Rini <trini@ti.com> wrote:
>> > On Fri, Jun 28, 2013 at 06:56:46PM -0300, Fabio Estevam wrote:
>> >
>> >> On Fri, Jun 28, 2013 at 6:52 PM, Otavio Salvador
>> >> <otavio@ossystems.com.br> wrote:
>> >> > This changes were being done in every version of U-Boot and it
>> >> > makes sense to try to merge them upstream.
>> >> >
>> >> > Please review them and ack/nack them.
>> >>
>> >> I think the series look good:
>> >>
>> >> Reviewed-by: Fabio Estevam <fabio.estevam@freescale.com>
>> >
>> > We aren't breaking compat with other distros, are we?  In TI-land where
>> > (for better or worse) we often have a FAT partition we're dealing with
>> > too, we use (and can have OE spit out) a uEnv.txt file that's
>> > user-editable and imported to the environment that would update these
>> > variables.  I suspect there's not an easy analog here but I thought I'd
>> > bring it up.
>>
>> We've not used uEnv.txt files for this in Freescale's Yocto support
>> but patched the U-Boot source code to make it work out of box. I think
>> these patches could go in now that Freescale is using Yocto more
>> extensivelly for ARM platforms so we avoid  same patches to be ported
>> for every new U-Boot release.
>
> Right well and good.  But part of the reason we use it here is that
> meta-various-ti-things may use a different layout from
> meta-community-oriented-distro and this lets us handle both with just a
> tweak in the various-ti-things layer to add a text file.  Again, may not
> apply here and that's fine.  Heck, we're trying to move more towards
> always just using the community layout.

Yes and we also have support to read a user provided file, but we
leave it for customizations not to use by default.

... yes, community layout is the way to go ;)

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

[U-Boot] [PATCH 0/6] Upstream environment changes being used in Yocto/OE

[Albert ARIBAUD]

Hi Fabio,

On Fri, 28 Jun 2013 18:56:46 -0300, Fabio Estevam <festevam@gmail.com>
wrote:

> On Fri, Jun 28, 2013 at 6:52 PM, Otavio Salvador
> <otavio@ossystems.com.br> wrote:
> > This changes were being done in every version of U-Boot and it
> > makes sense to try to merge them upstream.
> >
> > Please review them and ack/nack them.
> 
> I think the series look good:
> 
> Reviewed-by: Fabio Estevam <fabio.estevam@freescale.com>
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot

Cc:ing maintainers Jason Liu (mx53loco and mx6qsabrelite) and
Sefano Babic (mx51evk).

(/me thinks the MAINTAINERS file is going to endure a radical format
change after 2013.07 is out, in order to make it script-friendly.)

Amicalement,
-- 
Albert.

[U-Boot] [PATCH 5/6] mx51evk: Change default environment to cope with OE changes

[Stefano Babic]

On 28/06/2013 23:52, Otavio Salvador wrote:
> OpenEmbedded has change partitioning layout of generated image so it
> does not raise warnings during the boot regarding unkown partition
> being used for U-Boot.
> 
> Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
> ---
>  include/configs/mx51evk.h | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/include/configs/mx51evk.h b/include/configs/mx51evk.h
> index 13d1839..4383375d 100644
> --- a/include/configs/mx51evk.h
> +++ b/include/configs/mx51evk.h
> @@ -162,8 +162,8 @@
>  	"boot_fdt=try\0" \
>  	"ip_dyn=yes\0" \
>  	"mmcdev=0\0" \
> -	"mmcpart=2\0" \
> -	"mmcroot=/dev/mmcblk0p3 rootwait rw\0" \
> +	"mmcpart=1\0" \
> +	"mmcroot=/dev/mmcblk0p2 rootwait rw\0" \
>  	"mmcargs=setenv bootargs console=ttymxc0,${baudrate} " \
>  		"root=${mmcroot}\0" \
>  	"loadbootscript=" \
> 

Acked-by: Stefano Babic <sbabic@denx.de>

Best regards,
Stefano Babic

-- 
=====================================================================
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
=====================================================================

[U-Boot] [PATCH 0/6] Upstream environment changes being used in Yocto/OE

[Stefano Babic]

Hi Fabio, hi Jason,

On 28/06/2013 23:52, Otavio Salvador wrote:
> This changes were being done in every version of U-Boot and it
> makes sense to try to merge them upstream.
> 
> Please review them and ack/nack them.
> 
> 

these are small fixes in the default environment and I have no problem
to merge them for 2013.07, but I have not seen yet ACK or NACk for them.
If you think these should flow into the release, please send your ACK.

Thanks,
Stefano

-- 
=====================================================================
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
=====================================================================

[U-Boot] [PATCH 0/6] Upstream environment changes being used in Yocto/OE

[Fabio Estevam]

Hi Stefano,

On Fri, Jul 12, 2013 at 4:43 AM, Stefano Babic <sbabic@denx.de> wrote:
> Hi Fabio, hi Jason,
>
> On 28/06/2013 23:52, Otavio Salvador wrote:
>> This changes were being done in every version of U-Boot and it
>> makes sense to try to merge them upstream.
>>
>> Please review them and ack/nack them.
>>
>>
>
> these are small fixes in the default environment and I have no problem
> to merge them for 2013.07, but I have not seen yet ACK or NACk for them.
> If you think these should flow into the release, please send your ACK.

I am happy with this series:

Acked-by: Fabio Estevam <fabio.estevam@freescale.com>


I have also updated Jason's email on Cc (not sure if his Linaro's
email is still valid).

Thanks,

Fabio Estevam

[U-Boot] [PATCH 0/6] Upstream environment changes being used in Yocto/OE

[Stefano Babic]

On 28/06/2013 23:52, Otavio Salvador wrote:
> This changes were being done in every version of U-Boot and it
> makes sense to try to merge them upstream.
> 
> Please review them and ack/nack them.
> 
> 

Whole series applied to u-boot-imx (fix), thanks !

Best regards,
Stefano Babic

-- 
=====================================================================
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
=====================================================================