Patchwork [meta-fsl-arm,v2,3/6] u-boot-fslc: Update to v2014.01 based release

login
register
mail settings
Submitter Otavio Salvador
Date March 7, 2014, 2:52 p.m.
Message ID <1394203965-9775-3-git-send-email-otavio@ossystems.com.br>
Download mbox | patch
Permalink /patch/68269/
State Changes Requested
Delegated to: Otavio Salvador
Headers show

Comments

Otavio Salvador - March 7, 2014, 2:52 p.m.
This updates U-Boot to a v2014.01 based release. It includes following
patches on top of prestine v2014.01 tag:

3fc5045 wandboard: add Future Eletronics 7" WVGA LCD extension board
5aebc94 mx6sl: Adjust fdt_addr to avoid address overlapping
95a98c2 mx6sabresd: Use mmcblk0 for CONFIG_MMCROOT
cd0b4d9 MX6: Add SION flag to ENET_REF_CLK PAD

Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
---

Changes in v2: None

 ...boot-fslc-mxsboot_2013.10.bb => u-boot-fslc-mxsboot_2014.01.bb} | 6 +++---
 recipes-bsp/u-boot/u-boot-fslc_2013.10.bb                          | 7 -------
 recipes-bsp/u-boot/u-boot-fslc_2014.01.bb                          | 7 +++++++
 3 files changed, 10 insertions(+), 10 deletions(-)
 rename recipes-bsp/u-boot/{u-boot-fslc-mxsboot_2013.10.bb => u-boot-fslc-mxsboot_2014.01.bb} (87%)
 delete mode 100644 recipes-bsp/u-boot/u-boot-fslc_2013.10.bb
 create mode 100644 recipes-bsp/u-boot/u-boot-fslc_2014.01.bb
Daiane Angolini - March 10, 2014, 2:14 p.m.
On Fri, Mar 7, 2014 at 11:52 AM, Otavio Salvador
<otavio@ossystems.com.br> wrote:
> This updates U-Boot to a v2014.01 based release. It includes following
> patches on top of prestine v2014.01 tag:
>
> 3fc5045 wandboard: add Future Eletronics 7" WVGA LCD extension board
> 5aebc94 mx6sl: Adjust fdt_addr to avoid address overlapping
> 95a98c2 mx6sabresd: Use mmcblk0 for CONFIG_MMCROOT
> cd0b4d9 MX6: Add SION flag to ENET_REF_CLK PAD
>
> Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
> ---
>
> Changes in v2: None

I tested if for
imx53qsb
imx28evk
imx6qsabresd


**U-Boot 2014.01 (Mar 10 2014 - 09:02:44)
**Board: MX53 LOCO
**
**U-Boot 2014.01 (Mar 07 2014 - 17:12:59)
**CPU:   Freescale i.MX28 rev1.2 at 454 MHz
**
**U-Boot 2014.01 (Mar 10 2014 - 09:10:12)
**CPU:   Freescale i.MX6Q rev1.2 at 792 MHz

And realized the log for imx53qsb differs from other boards =P


I'm OK to push/merge

Daiane


>
>  ...boot-fslc-mxsboot_2013.10.bb => u-boot-fslc-mxsboot_2014.01.bb} | 6 +++---
>  recipes-bsp/u-boot/u-boot-fslc_2013.10.bb                          | 7 -------
>  recipes-bsp/u-boot/u-boot-fslc_2014.01.bb                          | 7 +++++++
>  3 files changed, 10 insertions(+), 10 deletions(-)
>  rename recipes-bsp/u-boot/{u-boot-fslc-mxsboot_2013.10.bb => u-boot-fslc-mxsboot_2014.01.bb} (87%)
>  delete mode 100644 recipes-bsp/u-boot/u-boot-fslc_2013.10.bb
>  create mode 100644 recipes-bsp/u-boot/u-boot-fslc_2014.01.bb
>
> diff --git a/recipes-bsp/u-boot/u-boot-fslc-mxsboot_2013.10.bb b/recipes-bsp/u-boot/u-boot-fslc-mxsboot_2014.01.bb
> similarity index 87%
> rename from recipes-bsp/u-boot/u-boot-fslc-mxsboot_2013.10.bb
> rename to recipes-bsp/u-boot/u-boot-fslc-mxsboot_2014.01.bb
> index 3009ddf..2c0848d 100644
> --- a/recipes-bsp/u-boot/u-boot-fslc-mxsboot_2013.10.bb
> +++ b/recipes-bsp/u-boot/u-boot-fslc-mxsboot_2014.01.bb
> @@ -5,10 +5,10 @@ SECTION = "bootloader"
>  DEPENDS = "openssl"
>  PROVIDES = "u-boot-mxsboot"
>
> -PV = "v2013.10"
> +PV = "v2014.01"
>
> -SRCREV = "079e214888279518ce061c71238a74a0c3db2c28"
> -SRCBRANCH = "patches-2013.10"
> +SRCREV = "bce40d36187be1d9165a9a6b9549c4989dcc48ed"
> +SRCBRANCH = "patches-2014.01"
>  SRC_URI = "git://github.com/Freescale/u-boot-imx.git;branch=${SRCBRANCH}"
>
>  S = "${WORKDIR}/git"
> diff --git a/recipes-bsp/u-boot/u-boot-fslc_2013.10.bb b/recipes-bsp/u-boot/u-boot-fslc_2013.10.bb
> deleted file mode 100644
> index e7f100d..0000000
> --- a/recipes-bsp/u-boot/u-boot-fslc_2013.10.bb
> +++ /dev/null
> @@ -1,7 +0,0 @@
> -include u-boot-fslc.inc
> -
> -PV = "v2013.10"
> -
> -SRCBRANCH = "patches-2013.10"
> -
> -SRCREV = "31d023bf8f15e6407557f5273c35c6ee3466a77f"
> diff --git a/recipes-bsp/u-boot/u-boot-fslc_2014.01.bb b/recipes-bsp/u-boot/u-boot-fslc_2014.01.bb
> new file mode 100644
> index 0000000..ec0ab00
> --- /dev/null
> +++ b/recipes-bsp/u-boot/u-boot-fslc_2014.01.bb
> @@ -0,0 +1,7 @@
> +include u-boot-fslc.inc
> +
> +PV = "v2014.01"
> +
> +SRCBRANCH = "patches-2014.01"
> +
> +SRCREV = "bce40d36187be1d9165a9a6b9549c4989dcc48ed"
> --
> 1.9.0
>
> _______________________________________________
> meta-freescale mailing list
> meta-freescale@yoctoproject.org
> https://lists.yoctoproject.org/listinfo/meta-freescale
John Weber - March 10, 2014, 8:28 p.m.

Fabio Estevam - March 10, 2014, 8:35 p.m.
On Mon, Mar 10, 2014 at 5:28 PM, John Weber <rjohnweber@gmail.com> wrote:
> Hi Otavio,
>
>
> On 3/7/14, 8:52 AM, Otavio Salvador wrote:
>
> This updates U-Boot to a v2014.01 based release. It includes following
> patches on top of prestine v2014.01 tag:
>
> 3fc5045 wandboard: add Future Eletronics 7" WVGA LCD extension board

I remember I reported issues with this patch when it was posted to the
U-boot list, so I am not sure why it was applied here.

> I've done a top-level check on this on wandboard-solo.  It boots, which is
> good.  But, I'm getting these i2c failure messages, which I believe to be
> due to a lack of an attached Future display panel:

The pristine U-boot 2014.01 does not contain i2c support for Wandboard...

>
> U-Boot 2014.01 (Mar 10 2014 - 14:49:56)

So Yocto should not tag it as 2014.01 since there are patches added on
top of 2014.01.

Regards,

Fabio Estevam
Otavio Salvador - March 11, 2014, 1:47 p.m.
Hello John,

On Mon, Mar 10, 2014 at 5:28 PM, John Weber <rjohnweber@gmail.com> wrote:
...
> *** Warning - bad CRC, using default environment
>
> wait_for_sr_state: failed sr=81 cr=a0 state=2020
> i2c_init_transfer: failed for chip 0x10 retry=0
> wait_for_sr_state: failed sr=81 cr=a0 state=2020
> i2c_init_transfer: failed for chip 0x10 retry=1
> wait_for_sr_state: failed sr=81 cr=a0 state=2020
> i2c_init_transfer: failed for chip 0x10 retry=2
> i2c_init_transfer: give up i2c_regs=021a0000
> No panel detected: default to HDMI
> Display: HDMI (1024x768)
...
> i2c_init_transfer: failed for chip 0x10 retry=0
> wait_for_sr_state: failed sr=81 cr=a0 state=2020
> i2c_init_transfer: failed for chip 0x10 retry=1
> wait_for_sr_state: failed sr=81 cr=a0 state=2020
> i2c_init_transfer: failed for chip 0x10 retry=2
> i2c_init_transfer: give up i2c_regs=021a0000

I will look at it and try to fix it.
Otavio Salvador - March 11, 2014, 1:49 p.m.
On Mon, Mar 10, 2014 at 5:35 PM, Fabio Estevam <festevam@gmail.com> wrote:
> On Mon, Mar 10, 2014 at 5:28 PM, John Weber <rjohnweber@gmail.com> wrote:
>> Hi Otavio,
>>
>>
>> On 3/7/14, 8:52 AM, Otavio Salvador wrote:
>>
>> This updates U-Boot to a v2014.01 based release. It includes following
>> patches on top of prestine v2014.01 tag:
>>
>> 3fc5045 wandboard: add Future Eletronics 7" WVGA LCD extension board
>
> I remember I reported issues with this patch when it was posted to the
> U-boot list, so I am not sure why it was applied here.

I might be missed it; I don't recall seeing it. Anyway I will work on this.

>> I've done a top-level check on this on wandboard-solo.  It boots, which is
>> good.  But, I'm getting these i2c failure messages, which I believe to be
>> due to a lack of an attached Future display panel:
>
> The pristine U-boot 2014.01 does not contain i2c support for Wandboard...

That's why the recipe is called u-boot-fslc.
Fabio Estevam - March 11, 2014, 3:02 p.m.
Hi Otavio,

On Tue, Mar 11, 2014 at 10:49 AM, Otavio Salvador
<otavio@ossystems.com.br> wrote:

>> The pristine U-boot 2014.01 does not contain i2c support for Wandboard...
>
> That's why the recipe is called u-boot-fslc.

Then we shouldn't read "U-boot 2014.01".

If we are helping someone with an issue and ask the person to post the
log and we read "U-boot 2014.01" then we understand that they are
using the pure mainline version of U-boot 2014.01.

However, in this case there are are patches added on top of 2014.01,
so the tag should be somewhat different to reflect that extra patches
were added and avoid confusion.

Regards,

Fabio Estevam
Otavio Salvador - March 11, 2014, 3:06 p.m.
On Tue, Mar 11, 2014 at 12:02 PM, Fabio Estevam <festevam@gmail.com> wrote:
> On Tue, Mar 11, 2014 at 10:49 AM, Otavio Salvador
> <otavio@ossystems.com.br> wrote:
>
>>> The pristine U-boot 2014.01 does not contain i2c support for Wandboard...
>>
>> That's why the recipe is called u-boot-fslc.
>
> Then we shouldn't read "U-boot 2014.01".
>
> If we are helping someone with an issue and ask the person to post the
> log and we read "U-boot 2014.01" then we understand that they are
> using the pure mainline version of U-boot 2014.01.
>
> However, in this case there are are patches added on top of 2014.01,
> so the tag should be somewhat different to reflect that extra patches
> were added and avoid confusion.

I will see if LOCALVERSION can be used for it. In any case I will work
on the Wandboard fix before sending it again.

Patch

diff --git a/recipes-bsp/u-boot/u-boot-fslc-mxsboot_2013.10.bb b/recipes-bsp/u-boot/u-boot-fslc-mxsboot_2014.01.bb
similarity index 87%
rename from recipes-bsp/u-boot/u-boot-fslc-mxsboot_2013.10.bb
rename to recipes-bsp/u-boot/u-boot-fslc-mxsboot_2014.01.bb
index 3009ddf..2c0848d 100644
--- a/recipes-bsp/u-boot/u-boot-fslc-mxsboot_2013.10.bb
+++ b/recipes-bsp/u-boot/u-boot-fslc-mxsboot_2014.01.bb
@@ -5,10 +5,10 @@  SECTION = "bootloader"
 DEPENDS = "openssl"
 PROVIDES = "u-boot-mxsboot"
 
-PV = "v2013.10"
+PV = "v2014.01"
 
-SRCREV = "079e214888279518ce061c71238a74a0c3db2c28"
-SRCBRANCH = "patches-2013.10"
+SRCREV = "bce40d36187be1d9165a9a6b9549c4989dcc48ed"
+SRCBRANCH = "patches-2014.01"
 SRC_URI = "git://github.com/Freescale/u-boot-imx.git;branch=${SRCBRANCH}"
 
 S = "${WORKDIR}/git"
diff --git a/recipes-bsp/u-boot/u-boot-fslc_2013.10.bb b/recipes-bsp/u-boot/u-boot-fslc_2013.10.bb
deleted file mode 100644
index e7f100d..0000000
--- a/recipes-bsp/u-boot/u-boot-fslc_2013.10.bb
+++ /dev/null
@@ -1,7 +0,0 @@ 
-include u-boot-fslc.inc
-
-PV = "v2013.10"
-
-SRCBRANCH = "patches-2013.10"
-
-SRCREV = "31d023bf8f15e6407557f5273c35c6ee3466a77f"
diff --git a/recipes-bsp/u-boot/u-boot-fslc_2014.01.bb b/recipes-bsp/u-boot/u-boot-fslc_2014.01.bb
new file mode 100644
index 0000000..ec0ab00
--- /dev/null
+++ b/recipes-bsp/u-boot/u-boot-fslc_2014.01.bb
@@ -0,0 +1,7 @@ 
+include u-boot-fslc.inc
+
+PV = "v2014.01"
+
+SRCBRANCH = "patches-2014.01"
+
+SRCREV = "bce40d36187be1d9165a9a6b9549c4989dcc48ed"