[PATCH v3][meta-fsl-arm-extra] imx6sl-warp: Set EXTRA_IMAGEDEPENDS to u-boot

[PATCH v3][meta-fsl-arm-extra] imx6sl-warp: Set EXTRA_IMAGEDEPENDS to u-boot

[Daiane Angolini]

Set EXTRA_IMAGEDEPENDS to "u-boot".  When user copies a .sdcard to the eMMC the
bootloader will not be updated it does not mean a fresh u-boot binary is not
helpful.

WaRP uses eMMC boot partition by default and the u-boot inside .sdcard image
does not override it. However, building u-boot binary anyway in order to provide
a binary to be used if an update is needed.

See how-to override eMMC boot partition on the following link:

   http://git.denx.de/?p=u-boot.git;a=blob;f=board/warp/README

It is only changed back to default value to ease u-boot manual update for the
user.

Signed-off-by: Daiane Angolini <daiane.angolini@freescale.com>
---
- Remove IMAGE_BOOTLOADER override
- Add EXTRA_IMAGEDEPENDS
- Change commit log accordingly

 conf/machine/imx6sl-warp.conf | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/conf/machine/imx6sl-warp.conf b/conf/machine/imx6sl-warp.conf
index 536e3d0..7c43903 100644
--- a/conf/machine/imx6sl-warp.conf
+++ b/conf/machine/imx6sl-warp.conf
@@ -27,8 +27,12 @@ include conf/machine/include/tune-cortexa9.inc
 
 SOC_FAMILY = "mx6:mx6sl"
 
-# Use eMMC boot partition by default
-IMAGE_BOOTLOADER ?= ""
+# WaRP use eMMC boot partition by default and the u-boot inside .sdcard image
+# does not override it. However, building u-boot binary anyway in order to provide
+# a binary to be used if an update is needed
+# see how-to override eMMC boot partition on the following link
+# http://git.denx.de/?p=u-boot.git;a=blob;f=board/warp/README
+EXTRA_IMAGEDEPENDS += "u-boot"
 
 UBOOT_MACHINE ?= "warp_defconfig"
 
-- 
1.9.1

Re: [PATCH v3][meta-fsl-arm-extra] imx6sl-warp: Set EXTRA_IMAGEDEPENDS to u-boot

[Fabio Estevam]

On Mon, Jul 6, 2015 at 1:22 PM, Daiane Angolini
<daiane.angolini@freescale.com> wrote:
> Set EXTRA_IMAGEDEPENDS to "u-boot".  When user copies a .sdcard to the eMMC the
> bootloader will not be updated it does not mean a fresh u-boot binary is not
> helpful.
>
> WaRP uses eMMC boot partition by default and the u-boot inside .sdcard image
> does not override it. However, building u-boot binary anyway in order to provide
> a binary to be used if an update is needed.

What about this?

"When user copies a .sdcard to the eMMC the bootloader will not be
updated, but it is still helpful to generate a U-boot binary and be
able to inspect U-boot source code."

>
> See how-to override eMMC boot partition on the following link:
>
>    http://git.denx.de/?p=u-boot.git;a=blob;f=board/warp/README
>
> It is only changed back to default value to ease u-boot manual update for the
> user.
>
> Signed-off-by: Daiane Angolini <daiane.angolini@freescale.com>
> ---
> - Remove IMAGE_BOOTLOADER override
> - Add EXTRA_IMAGEDEPENDS
> - Change commit log accordingly
>
>  conf/machine/imx6sl-warp.conf | 8 ++++++--
>  1 file changed, 6 insertions(+), 2 deletions(-)
>
> diff --git a/conf/machine/imx6sl-warp.conf b/conf/machine/imx6sl-warp.conf
> index 536e3d0..7c43903 100644
> --- a/conf/machine/imx6sl-warp.conf
> +++ b/conf/machine/imx6sl-warp.conf
> @@ -27,8 +27,12 @@ include conf/machine/include/tune-cortexa9.inc
>
>  SOC_FAMILY = "mx6:mx6sl"
>
> -# Use eMMC boot partition by default
> -IMAGE_BOOTLOADER ?= ""
> +# WaRP use eMMC boot partition by default and the u-boot inside .sdcard image

s/use/uses

> +# does not override it. However, building u-boot binary anyway in order to provide

/building/build

Other than that:

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