[PATCH] u-boot: update to 2015.01

[PATCH] u-boot: update to 2015.01

[Denys Dmytriyenko]

From: Denys Dmytriyenko <denys@ti.com>

Signed-off-by: Denys Dmytriyenko <denys@ti.com>
---
 ...2014.07.bb => u-boot-fw-utils-cross_2015.01.bb} | 11 ++++---
 .../0001-tools-env-fix-build-error.patch           | 36 ++++++++++++++++++++++
 ...utils_2014.07.bb => u-boot-fw-utils_2015.01.bb} | 12 +++++---
 ...kimage_2014.07.bb => u-boot-mkimage_2015.01.bb} | 20 ++++++------
 meta/recipes-bsp/u-boot/u-boot.inc                 |  2 +-
 .../{u-boot_2014.07.bb => u-boot_2015.01.bb}       |  6 ++--
 6 files changed, 62 insertions(+), 25 deletions(-)
 rename meta/recipes-bsp/u-boot/{u-boot-fw-utils-cross_2014.07.bb => u-boot-fw-utils-cross_2015.01.bb} (76%)
 create mode 100644 meta/recipes-bsp/u-boot/u-boot-fw-utils/0001-tools-env-fix-build-error.patch
 rename meta/recipes-bsp/u-boot/{u-boot-fw-utils_2014.07.bb => u-boot-fw-utils_2015.01.bb} (69%)
 rename meta/recipes-bsp/u-boot/{u-boot-mkimage_2014.07.bb => u-boot-mkimage_2015.01.bb} (45%)
 rename meta/recipes-bsp/u-boot/{u-boot_2014.07.bb => u-boot_2015.01.bb} (50%)

diff --git a/meta/recipes-bsp/u-boot/u-boot-fw-utils-cross_2014.07.bb b/meta/recipes-bsp/u-boot/u-boot-fw-utils-cross_2015.01.bb
similarity index 76%
rename from meta/recipes-bsp/u-boot/u-boot-fw-utils-cross_2014.07.bb
rename to meta/recipes-bsp/u-boot/u-boot-fw-utils-cross_2015.01.bb
index d1f1f9a..e826b89 100644
--- a/meta/recipes-bsp/u-boot/u-boot-fw-utils-cross_2014.07.bb
+++ b/meta/recipes-bsp/u-boot/u-boot-fw-utils-cross_2015.01.bb
@@ -1,17 +1,18 @@
 SUMMARY = "U-Boot bootloader fw_printenv/setenv utilities"
 LICENSE = "GPLv2+"
-LIC_FILES_CHKSUM = "file://Licenses/README;md5=025bf9f768cbcb1a165dbe1a110babfb"
+LIC_FILES_CHKSUM = "file://Licenses/README;md5=c7383a594871c03da76b3707929d2919"
 SECTION = "bootloader"
 DEPENDS = "mtd-utils"
 
-# This revision corresponds to the tag "v2014.07"
+# This revision corresponds to the tag "v2015.01"
 # We use the revision in order to avoid having to fetch it from the
 # repo during parse
-SRCREV = "524123a70761110c5cf3ccc5f52f6d4da071b959"
+SRCREV = "92fa7f53f1f3f03296f8ffb14bdf1baefab83368"
 
-PV = "v2014.07+git${SRCPV}"
+PV = "v2015.01+git${SRCPV}"
 
-SRC_URI = "git://git.denx.de/u-boot.git;branch=master;protocol=git"
+SRC_URI = "git://git.denx.de/u-boot.git;branch=master;protocol=git \
+           file://0001-tools-env-fix-build-error.patch"
 
 S = "${WORKDIR}/git"
 
diff --git a/meta/recipes-bsp/u-boot/u-boot-fw-utils/0001-tools-env-fix-build-error.patch b/meta/recipes-bsp/u-boot/u-boot-fw-utils/0001-tools-env-fix-build-error.patch
new file mode 100644
index 0000000..381b505
--- /dev/null
+++ b/meta/recipes-bsp/u-boot/u-boot-fw-utils/0001-tools-env-fix-build-error.patch
@@ -0,0 +1,36 @@
+From ee2d75513452aa6d5306fd380104adc8a2f6d8f2 Mon Sep 17 00:00:00 2001
+From: Masahiro Yamada <yamada.m@jp.panasonic.com>
+Date: Wed, 3 Dec 2014 10:22:50 +0900
+Subject: [PATCH] tools: env: fix build error
+
+Since CONFIG_SYS_ARCH, CONFIG_SYS_CPU, ... were moved to Kconfig,
+tools/env/fw_printenv fails to build if CONFIG_ENV_VARS_UBOOT_CONFIG
+is defined.
+(I do not think this is the right way to fix the problem, but
+for now I do not have enough time to take a close look.)
+
+Upstream-Status: Submitted [http://patchwork.ozlabs.org/patch/417192/]
+
+Signed-off-by: Masahiro Yamada <yamada.m@jp.panasonic.com>
+Reported-by: Denys Dmytriyenko <denys@ti.com>
+---
+ tools/env/fw_env.c | 3 +++
+ 1 file changed, 3 insertions(+)
+
+diff --git a/tools/env/fw_env.c b/tools/env/fw_env.c
+index 1173eea..698fe51 100644
+--- a/tools/env/fw_env.c
++++ b/tools/env/fw_env.c
+@@ -8,6 +8,9 @@
+  * SPDX-License-Identifier:	GPL-2.0+
+  */
+ 
++/* FIXME: Do not include this */
++#include <linux/kconfig.h>
++
+ #include <errno.h>
+ #include <env_flags.h>
+ #include <fcntl.h>
+-- 
+2.2.0
+
diff --git a/meta/recipes-bsp/u-boot/u-boot-fw-utils_2014.07.bb b/meta/recipes-bsp/u-boot/u-boot-fw-utils_2015.01.bb
similarity index 69%
rename from meta/recipes-bsp/u-boot/u-boot-fw-utils_2014.07.bb
rename to meta/recipes-bsp/u-boot/u-boot-fw-utils_2015.01.bb
index a626c95..50d1974 100644
--- a/meta/recipes-bsp/u-boot/u-boot-fw-utils_2014.07.bb
+++ b/meta/recipes-bsp/u-boot/u-boot-fw-utils_2015.01.bb
@@ -1,21 +1,23 @@
 SUMMARY = "U-Boot bootloader fw_printenv/setenv utilities"
 LICENSE = "GPLv2+"
-LIC_FILES_CHKSUM = "file://Licenses/README;md5=025bf9f768cbcb1a165dbe1a110babfb"
+LIC_FILES_CHKSUM = "file://Licenses/README;md5=c7383a594871c03da76b3707929d2919"
 SECTION = "bootloader"
 DEPENDS = "mtd-utils"
 
-# This revision corresponds to the tag "v2014.07"
+# This revision corresponds to the tag "v2015.01"
 # We use the revision in order to avoid having to fetch it from the
 # repo during parse
-SRCREV = "524123a70761110c5cf3ccc5f52f6d4da071b959"
+SRCREV = "92fa7f53f1f3f03296f8ffb14bdf1baefab83368"
 
-PV = "v2014.07+git${SRCPV}"
+PV = "v2015.01+git${SRCPV}"
 
-SRC_URI = "git://git.denx.de/u-boot.git;branch=master;protocol=git"
+SRC_URI = "git://git.denx.de/u-boot.git;branch=master;protocol=git \
+           file://0001-tools-env-fix-build-error.patch"
 
 S = "${WORKDIR}/git"
 
 INSANE_SKIP_${PN} = "already-stripped"
+EXTRA_OEMAKE = 'CROSS_COMPILE=${TARGET_PREFIX} CC="${CC}"'
 
 inherit uboot-config
 
diff --git a/meta/recipes-bsp/u-boot/u-boot-mkimage_2014.07.bb b/meta/recipes-bsp/u-boot/u-boot-mkimage_2015.01.bb
similarity index 45%
rename from meta/recipes-bsp/u-boot/u-boot-mkimage_2014.07.bb
rename to meta/recipes-bsp/u-boot/u-boot-mkimage_2015.01.bb
index eabf680..1bfdf9d 100644
--- a/meta/recipes-bsp/u-boot/u-boot-mkimage_2014.07.bb
+++ b/meta/recipes-bsp/u-boot/u-boot-mkimage_2015.01.bb
@@ -1,28 +1,26 @@
 SUMMARY = "U-Boot bootloader image creation tool"
 LICENSE = "GPLv2+"
-LIC_FILES_CHKSUM = "file://Licenses/README;md5=025bf9f768cbcb1a165dbe1a110babfb"
+LIC_FILES_CHKSUM = "file://Licenses/README;md5=c7383a594871c03da76b3707929d2919"
 SECTION = "bootloader"
 
-# This revision corresponds to the tag "v2014.07"
+DEPENDS = "openssl"
+
+# This revision corresponds to the tag "v2015.01"
 # We use the revision in order to avoid having to fetch it from the
 # repo during parse
-SRCREV = "524123a70761110c5cf3ccc5f52f6d4da071b959"
+SRCREV = "92fa7f53f1f3f03296f8ffb14bdf1baefab83368"
 
-PV = "v2014.07+git${SRCPV}"
+PV = "v2015.01+git${SRCPV}"
 
 SRC_URI = "git://git.denx.de/u-boot.git;branch=master;protocol=git"
 
 S = "${WORKDIR}/git"
 
-EXTRA_OEMAKE = 'HOSTCC="${CC}" HOSTLD="${LD}" HOSTLDFLAGS="${LDFLAGS}" HOSTSTRIP=true'
+EXTRA_OEMAKE = 'CROSS_COMPILE="${TARGET_PREFIX}" CC="${CC}" STRIP=true'
 
 do_compile () {
-	# Make sure the recompile is OK
-	rm -f ${B}/tools/.depend
-
-	make HOSTCC="${BUILD_CC}" HOSTLD="${BUILD_LD}" HOSTLDFLAGS="${BUILD_LDFLAGS}" HOSTSTRIP=true dot-config=0 scripts_basic
-	sed 's/^tools-only: scripts_basic /tools-only: /' -i Makefile
-	oe_runmake tools-only
+	oe_runmake sandbox_defconfig
+	oe_runmake cross_tools NO_SDL=1
 }
 
 do_install () {
diff --git a/meta/recipes-bsp/u-boot/u-boot.inc b/meta/recipes-bsp/u-boot/u-boot.inc
index c695b73..4edab84 100644
--- a/meta/recipes-bsp/u-boot/u-boot.inc
+++ b/meta/recipes-bsp/u-boot/u-boot.inc
@@ -4,7 +4,7 @@ SECTION = "bootloaders"
 PROVIDES = "virtual/bootloader"
 
 LICENSE = "GPLv2+"
-LIC_FILES_CHKSUM = "file://Licenses/README;md5=025bf9f768cbcb1a165dbe1a110babfb"
+LIC_FILES_CHKSUM = "file://Licenses/README;md5=c7383a594871c03da76b3707929d2919"
 
 SRC_URI = "git://git.denx.de/u-boot.git;branch=master"
 
diff --git a/meta/recipes-bsp/u-boot/u-boot_2014.07.bb b/meta/recipes-bsp/u-boot/u-boot_2015.01.bb
similarity index 50%
rename from meta/recipes-bsp/u-boot/u-boot_2014.07.bb
rename to meta/recipes-bsp/u-boot/u-boot_2015.01.bb
index 44cc52e..6feac31 100644
--- a/meta/recipes-bsp/u-boot/u-boot_2014.07.bb
+++ b/meta/recipes-bsp/u-boot/u-boot_2015.01.bb
@@ -2,8 +2,8 @@ require u-boot.inc
 
 DEPENDS += "dtc-native"
 
-# This revision corresponds to the tag "v2014.07"
+# This revision corresponds to the tag "v2015.01"
 # We use the revision in order to avoid having to fetch it from the repo during parse
-SRCREV = "524123a70761110c5cf3ccc5f52f6d4da071b959"
+SRCREV = "92fa7f53f1f3f03296f8ffb14bdf1baefab83368"
 
-PV = "v2014.07+git${SRCPV}"
+PV = "v2015.01+git${SRCPV}"
-- 
2.2.0

Re: [PATCH] u-boot: update to 2015.01

[Denys Dmytriyenko]

Ping.


On Tue, Jan 13, 2015 at 11:09:51AM -0500, Denys Dmytriyenko wrote:
> From: Denys Dmytriyenko <denys@ti.com>
> 
> Signed-off-by: Denys Dmytriyenko <denys@ti.com>
> ---
>  ...2014.07.bb => u-boot-fw-utils-cross_2015.01.bb} | 11 ++++---
>  .../0001-tools-env-fix-build-error.patch           | 36 ++++++++++++++++++++++
>  ...utils_2014.07.bb => u-boot-fw-utils_2015.01.bb} | 12 +++++---
>  ...kimage_2014.07.bb => u-boot-mkimage_2015.01.bb} | 20 ++++++------
>  meta/recipes-bsp/u-boot/u-boot.inc                 |  2 +-
>  .../{u-boot_2014.07.bb => u-boot_2015.01.bb}       |  6 ++--
>  6 files changed, 62 insertions(+), 25 deletions(-)
>  rename meta/recipes-bsp/u-boot/{u-boot-fw-utils-cross_2014.07.bb => u-boot-fw-utils-cross_2015.01.bb} (76%)
>  create mode 100644 meta/recipes-bsp/u-boot/u-boot-fw-utils/0001-tools-env-fix-build-error.patch
>  rename meta/recipes-bsp/u-boot/{u-boot-fw-utils_2014.07.bb => u-boot-fw-utils_2015.01.bb} (69%)
>  rename meta/recipes-bsp/u-boot/{u-boot-mkimage_2014.07.bb => u-boot-mkimage_2015.01.bb} (45%)
>  rename meta/recipes-bsp/u-boot/{u-boot_2014.07.bb => u-boot_2015.01.bb} (50%)
> 
> diff --git a/meta/recipes-bsp/u-boot/u-boot-fw-utils-cross_2014.07.bb b/meta/recipes-bsp/u-boot/u-boot-fw-utils-cross_2015.01.bb
> similarity index 76%
> rename from meta/recipes-bsp/u-boot/u-boot-fw-utils-cross_2014.07.bb
> rename to meta/recipes-bsp/u-boot/u-boot-fw-utils-cross_2015.01.bb
> index d1f1f9a..e826b89 100644
> --- a/meta/recipes-bsp/u-boot/u-boot-fw-utils-cross_2014.07.bb
> +++ b/meta/recipes-bsp/u-boot/u-boot-fw-utils-cross_2015.01.bb
> @@ -1,17 +1,18 @@
>  SUMMARY = "U-Boot bootloader fw_printenv/setenv utilities"
>  LICENSE = "GPLv2+"
> -LIC_FILES_CHKSUM = "file://Licenses/README;md5=025bf9f768cbcb1a165dbe1a110babfb"
> +LIC_FILES_CHKSUM = "file://Licenses/README;md5=c7383a594871c03da76b3707929d2919"
>  SECTION = "bootloader"
>  DEPENDS = "mtd-utils"
>  
> -# This revision corresponds to the tag "v2014.07"
> +# This revision corresponds to the tag "v2015.01"
>  # We use the revision in order to avoid having to fetch it from the
>  # repo during parse
> -SRCREV = "524123a70761110c5cf3ccc5f52f6d4da071b959"
> +SRCREV = "92fa7f53f1f3f03296f8ffb14bdf1baefab83368"
>  
> -PV = "v2014.07+git${SRCPV}"
> +PV = "v2015.01+git${SRCPV}"
>  
> -SRC_URI = "git://git.denx.de/u-boot.git;branch=master;protocol=git"
> +SRC_URI = "git://git.denx.de/u-boot.git;branch=master;protocol=git \
> +           file://0001-tools-env-fix-build-error.patch"
>  
>  S = "${WORKDIR}/git"
>  
> diff --git a/meta/recipes-bsp/u-boot/u-boot-fw-utils/0001-tools-env-fix-build-error.patch b/meta/recipes-bsp/u-boot/u-boot-fw-utils/0001-tools-env-fix-build-error.patch
> new file mode 100644
> index 0000000..381b505
> --- /dev/null
> +++ b/meta/recipes-bsp/u-boot/u-boot-fw-utils/0001-tools-env-fix-build-error.patch
> @@ -0,0 +1,36 @@
> +From ee2d75513452aa6d5306fd380104adc8a2f6d8f2 Mon Sep 17 00:00:00 2001
> +From: Masahiro Yamada <yamada.m@jp.panasonic.com>
> +Date: Wed, 3 Dec 2014 10:22:50 +0900
> +Subject: [PATCH] tools: env: fix build error
> +
> +Since CONFIG_SYS_ARCH, CONFIG_SYS_CPU, ... were moved to Kconfig,
> +tools/env/fw_printenv fails to build if CONFIG_ENV_VARS_UBOOT_CONFIG
> +is defined.
> +(I do not think this is the right way to fix the problem, but
> +for now I do not have enough time to take a close look.)
> +
> +Upstream-Status: Submitted [http://patchwork.ozlabs.org/patch/417192/]
> +
> +Signed-off-by: Masahiro Yamada <yamada.m@jp.panasonic.com>
> +Reported-by: Denys Dmytriyenko <denys@ti.com>
> +---
> + tools/env/fw_env.c | 3 +++
> + 1 file changed, 3 insertions(+)
> +
> +diff --git a/tools/env/fw_env.c b/tools/env/fw_env.c
> +index 1173eea..698fe51 100644
> +--- a/tools/env/fw_env.c
> ++++ b/tools/env/fw_env.c
> +@@ -8,6 +8,9 @@
> +  * SPDX-License-Identifier:	GPL-2.0+
> +  */
> + 
> ++/* FIXME: Do not include this */
> ++#include <linux/kconfig.h>
> ++
> + #include <errno.h>
> + #include <env_flags.h>
> + #include <fcntl.h>
> +-- 
> +2.2.0
> +
> diff --git a/meta/recipes-bsp/u-boot/u-boot-fw-utils_2014.07.bb b/meta/recipes-bsp/u-boot/u-boot-fw-utils_2015.01.bb
> similarity index 69%
> rename from meta/recipes-bsp/u-boot/u-boot-fw-utils_2014.07.bb
> rename to meta/recipes-bsp/u-boot/u-boot-fw-utils_2015.01.bb
> index a626c95..50d1974 100644
> --- a/meta/recipes-bsp/u-boot/u-boot-fw-utils_2014.07.bb
> +++ b/meta/recipes-bsp/u-boot/u-boot-fw-utils_2015.01.bb
> @@ -1,21 +1,23 @@
>  SUMMARY = "U-Boot bootloader fw_printenv/setenv utilities"
>  LICENSE = "GPLv2+"
> -LIC_FILES_CHKSUM = "file://Licenses/README;md5=025bf9f768cbcb1a165dbe1a110babfb"
> +LIC_FILES_CHKSUM = "file://Licenses/README;md5=c7383a594871c03da76b3707929d2919"
>  SECTION = "bootloader"
>  DEPENDS = "mtd-utils"
>  
> -# This revision corresponds to the tag "v2014.07"
> +# This revision corresponds to the tag "v2015.01"
>  # We use the revision in order to avoid having to fetch it from the
>  # repo during parse
> -SRCREV = "524123a70761110c5cf3ccc5f52f6d4da071b959"
> +SRCREV = "92fa7f53f1f3f03296f8ffb14bdf1baefab83368"
>  
> -PV = "v2014.07+git${SRCPV}"
> +PV = "v2015.01+git${SRCPV}"
>  
> -SRC_URI = "git://git.denx.de/u-boot.git;branch=master;protocol=git"
> +SRC_URI = "git://git.denx.de/u-boot.git;branch=master;protocol=git \
> +           file://0001-tools-env-fix-build-error.patch"
>  
>  S = "${WORKDIR}/git"
>  
>  INSANE_SKIP_${PN} = "already-stripped"
> +EXTRA_OEMAKE = 'CROSS_COMPILE=${TARGET_PREFIX} CC="${CC}"'
>  
>  inherit uboot-config
>  
> diff --git a/meta/recipes-bsp/u-boot/u-boot-mkimage_2014.07.bb b/meta/recipes-bsp/u-boot/u-boot-mkimage_2015.01.bb
> similarity index 45%
> rename from meta/recipes-bsp/u-boot/u-boot-mkimage_2014.07.bb
> rename to meta/recipes-bsp/u-boot/u-boot-mkimage_2015.01.bb
> index eabf680..1bfdf9d 100644
> --- a/meta/recipes-bsp/u-boot/u-boot-mkimage_2014.07.bb
> +++ b/meta/recipes-bsp/u-boot/u-boot-mkimage_2015.01.bb
> @@ -1,28 +1,26 @@
>  SUMMARY = "U-Boot bootloader image creation tool"
>  LICENSE = "GPLv2+"
> -LIC_FILES_CHKSUM = "file://Licenses/README;md5=025bf9f768cbcb1a165dbe1a110babfb"
> +LIC_FILES_CHKSUM = "file://Licenses/README;md5=c7383a594871c03da76b3707929d2919"
>  SECTION = "bootloader"
>  
> -# This revision corresponds to the tag "v2014.07"
> +DEPENDS = "openssl"
> +
> +# This revision corresponds to the tag "v2015.01"
>  # We use the revision in order to avoid having to fetch it from the
>  # repo during parse
> -SRCREV = "524123a70761110c5cf3ccc5f52f6d4da071b959"
> +SRCREV = "92fa7f53f1f3f03296f8ffb14bdf1baefab83368"
>  
> -PV = "v2014.07+git${SRCPV}"
> +PV = "v2015.01+git${SRCPV}"
>  
>  SRC_URI = "git://git.denx.de/u-boot.git;branch=master;protocol=git"
>  
>  S = "${WORKDIR}/git"
>  
> -EXTRA_OEMAKE = 'HOSTCC="${CC}" HOSTLD="${LD}" HOSTLDFLAGS="${LDFLAGS}" HOSTSTRIP=true'
> +EXTRA_OEMAKE = 'CROSS_COMPILE="${TARGET_PREFIX}" CC="${CC}" STRIP=true'
>  
>  do_compile () {
> -	# Make sure the recompile is OK
> -	rm -f ${B}/tools/.depend
> -
> -	make HOSTCC="${BUILD_CC}" HOSTLD="${BUILD_LD}" HOSTLDFLAGS="${BUILD_LDFLAGS}" HOSTSTRIP=true dot-config=0 scripts_basic
> -	sed 's/^tools-only: scripts_basic /tools-only: /' -i Makefile
> -	oe_runmake tools-only
> +	oe_runmake sandbox_defconfig
> +	oe_runmake cross_tools NO_SDL=1
>  }
>  
>  do_install () {
> diff --git a/meta/recipes-bsp/u-boot/u-boot.inc b/meta/recipes-bsp/u-boot/u-boot.inc
> index c695b73..4edab84 100644
> --- a/meta/recipes-bsp/u-boot/u-boot.inc
> +++ b/meta/recipes-bsp/u-boot/u-boot.inc
> @@ -4,7 +4,7 @@ SECTION = "bootloaders"
>  PROVIDES = "virtual/bootloader"
>  
>  LICENSE = "GPLv2+"
> -LIC_FILES_CHKSUM = "file://Licenses/README;md5=025bf9f768cbcb1a165dbe1a110babfb"
> +LIC_FILES_CHKSUM = "file://Licenses/README;md5=c7383a594871c03da76b3707929d2919"
>  
>  SRC_URI = "git://git.denx.de/u-boot.git;branch=master"
>  
> diff --git a/meta/recipes-bsp/u-boot/u-boot_2014.07.bb b/meta/recipes-bsp/u-boot/u-boot_2015.01.bb
> similarity index 50%
> rename from meta/recipes-bsp/u-boot/u-boot_2014.07.bb
> rename to meta/recipes-bsp/u-boot/u-boot_2015.01.bb
> index 44cc52e..6feac31 100644
> --- a/meta/recipes-bsp/u-boot/u-boot_2014.07.bb
> +++ b/meta/recipes-bsp/u-boot/u-boot_2015.01.bb
> @@ -2,8 +2,8 @@ require u-boot.inc
>  
>  DEPENDS += "dtc-native"
>  
> -# This revision corresponds to the tag "v2014.07"
> +# This revision corresponds to the tag "v2015.01"
>  # We use the revision in order to avoid having to fetch it from the repo during parse
> -SRCREV = "524123a70761110c5cf3ccc5f52f6d4da071b959"
> +SRCREV = "92fa7f53f1f3f03296f8ffb14bdf1baefab83368"
>  
> -PV = "v2014.07+git${SRCPV}"
> +PV = "v2015.01+git${SRCPV}"
> -- 
> 2.2.0
> 
> -- 
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [PATCH] u-boot: update to 2015.01

[Otavio Salvador]

On Tue, Jan 13, 2015 at 2:09 PM, Denys Dmytriyenko <denis@denix.org> wrote:
> From: Denys Dmytriyenko <denys@ti.com>
>
> Signed-off-by: Denys Dmytriyenko <denys@ti.com>
...
> +++ b/meta/recipes-bsp/u-boot/u-boot-fw-utils/0001-tools-env-fix-build-error.patch
> @@ -0,0 +1,36 @@
> +From ee2d75513452aa6d5306fd380104adc8a2f6d8f2 Mon Sep 17 00:00:00 2001
> +From: Masahiro Yamada <yamada.m@jp.panasonic.com>
> +Date: Wed, 3 Dec 2014 10:22:50 +0900
> +Subject: [PATCH] tools: env: fix build error
> +
> +Since CONFIG_SYS_ARCH, CONFIG_SYS_CPU, ... were moved to Kconfig,
> +tools/env/fw_printenv fails to build if CONFIG_ENV_VARS_UBOOT_CONFIG
> +is defined.
> +(I do not think this is the right way to fix the problem, but
> +for now I do not have enough time to take a close look.)
> +
> +Upstream-Status: Submitted [http://patchwork.ozlabs.org/patch/417192/]
> +
> +Signed-off-by: Masahiro Yamada <yamada.m@jp.panasonic.com>
> +Reported-by: Denys Dmytriyenko <denys@ti.com>

Is it necessary? In meta-fsl-arm we don't have this patch and get the
tools built.

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

Re: [PATCH] u-boot: update to 2015.01

[Denys Dmytriyenko]

On Wed, Jan 21, 2015 at 11:17:19AM -0200, Otavio Salvador wrote:
> On Tue, Jan 13, 2015 at 2:09 PM, Denys Dmytriyenko <denis@denix.org> wrote:
> > From: Denys Dmytriyenko <denys@ti.com>
> >
> > Signed-off-by: Denys Dmytriyenko <denys@ti.com>
> ...
> > +++ b/meta/recipes-bsp/u-boot/u-boot-fw-utils/0001-tools-env-fix-build-error.patch
> > @@ -0,0 +1,36 @@
> > +From ee2d75513452aa6d5306fd380104adc8a2f6d8f2 Mon Sep 17 00:00:00 2001
> > +From: Masahiro Yamada <yamada.m@jp.panasonic.com>
> > +Date: Wed, 3 Dec 2014 10:22:50 +0900
> > +Subject: [PATCH] tools: env: fix build error
> > +
> > +Since CONFIG_SYS_ARCH, CONFIG_SYS_CPU, ... were moved to Kconfig,
> > +tools/env/fw_printenv fails to build if CONFIG_ENV_VARS_UBOOT_CONFIG
> > +is defined.
> > +(I do not think this is the right way to fix the problem, but
> > +for now I do not have enough time to take a close look.)
> > +
> > +Upstream-Status: Submitted [http://patchwork.ozlabs.org/patch/417192/]
> > +
> > +Signed-off-by: Masahiro Yamada <yamada.m@jp.panasonic.com>
> > +Reported-by: Denys Dmytriyenko <denys@ti.com>
> 
> Is it necessary? In meta-fsl-arm we don't have this patch and get the
> tools built.

Really? The patch above is rather self-explanatory. The issue was confirmed by 
U-boot maintainers and the workaround provided by one of the core developers.
I'm not familiar with meta-fsl-arm, but I guess you can grep your platform's 
defconfig to see if CONFIG_ENV_VARS_UBOOT_CONFIG and others are defined and 
whether your platforms are affected or not. It does affect our platforms 
(among many others), including beaglebone.

-- 
Denys

Re: [PATCH] u-boot: update to 2015.01

[Otavio Salvador]

On Wed, Jan 21, 2015 at 5:11 PM, Denys Dmytriyenko <denis@denix.org> wrote:
> On Wed, Jan 21, 2015 at 11:17:19AM -0200, Otavio Salvador wrote:
>> On Tue, Jan 13, 2015 at 2:09 PM, Denys Dmytriyenko <denis@denix.org> wrote:
>> > From: Denys Dmytriyenko <denys@ti.com>
>> >
>> > Signed-off-by: Denys Dmytriyenko <denys@ti.com>
>> ...
>> > +++ b/meta/recipes-bsp/u-boot/u-boot-fw-utils/0001-tools-env-fix-build-error.patch
>> > @@ -0,0 +1,36 @@
>> > +From ee2d75513452aa6d5306fd380104adc8a2f6d8f2 Mon Sep 17 00:00:00 2001
>> > +From: Masahiro Yamada <yamada.m@jp.panasonic.com>
>> > +Date: Wed, 3 Dec 2014 10:22:50 +0900
>> > +Subject: [PATCH] tools: env: fix build error
>> > +
>> > +Since CONFIG_SYS_ARCH, CONFIG_SYS_CPU, ... were moved to Kconfig,
>> > +tools/env/fw_printenv fails to build if CONFIG_ENV_VARS_UBOOT_CONFIG
>> > +is defined.
>> > +(I do not think this is the right way to fix the problem, but
>> > +for now I do not have enough time to take a close look.)
>> > +
>> > +Upstream-Status: Submitted [http://patchwork.ozlabs.org/patch/417192/]
>> > +
>> > +Signed-off-by: Masahiro Yamada <yamada.m@jp.panasonic.com>
>> > +Reported-by: Denys Dmytriyenko <denys@ti.com>
>>
>> Is it necessary? In meta-fsl-arm we don't have this patch and get the
>> tools built.
>
> Really? The patch above is rather self-explanatory. The issue was confirmed by
> U-boot maintainers and the workaround provided by one of the core developers.
> I'm not familiar with meta-fsl-arm, but I guess you can grep your platform's
> defconfig to see if CONFIG_ENV_VARS_UBOOT_CONFIG and others are defined and
> whether your platforms are affected or not. It does affect our platforms
> (among many others), including beaglebone.

I think this is for the cross tool only.

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

Re: [PATCH] u-boot: update to 2015.01

[Denys Dmytriyenko]

On Wed, Jan 21, 2015 at 06:11:23PM -0200, Otavio Salvador wrote:
> On Wed, Jan 21, 2015 at 5:11 PM, Denys Dmytriyenko <denis@denix.org> wrote:
> > On Wed, Jan 21, 2015 at 11:17:19AM -0200, Otavio Salvador wrote:
> >> On Tue, Jan 13, 2015 at 2:09 PM, Denys Dmytriyenko <denis@denix.org> wrote:
> >> > From: Denys Dmytriyenko <denys@ti.com>
> >> >
> >> > Signed-off-by: Denys Dmytriyenko <denys@ti.com>
> >> ...
> >> > +++ b/meta/recipes-bsp/u-boot/u-boot-fw-utils/0001-tools-env-fix-build-error.patch
> >> > @@ -0,0 +1,36 @@
> >> > +From ee2d75513452aa6d5306fd380104adc8a2f6d8f2 Mon Sep 17 00:00:00 2001
> >> > +From: Masahiro Yamada <yamada.m@jp.panasonic.com>
> >> > +Date: Wed, 3 Dec 2014 10:22:50 +0900
> >> > +Subject: [PATCH] tools: env: fix build error
> >> > +
> >> > +Since CONFIG_SYS_ARCH, CONFIG_SYS_CPU, ... were moved to Kconfig,
> >> > +tools/env/fw_printenv fails to build if CONFIG_ENV_VARS_UBOOT_CONFIG
> >> > +is defined.
> >> > +(I do not think this is the right way to fix the problem, but
> >> > +for now I do not have enough time to take a close look.)
> >> > +
> >> > +Upstream-Status: Submitted [http://patchwork.ozlabs.org/patch/417192/]
> >> > +
> >> > +Signed-off-by: Masahiro Yamada <yamada.m@jp.panasonic.com>
> >> > +Reported-by: Denys Dmytriyenko <denys@ti.com>
> >>
> >> Is it necessary? In meta-fsl-arm we don't have this patch and get the
> >> tools built.
> >
> > Really? The patch above is rather self-explanatory. The issue was confirmed by
> > U-boot maintainers and the workaround provided by one of the core developers.
> > I'm not familiar with meta-fsl-arm, but I guess you can grep your platform's
> > defconfig to see if CONFIG_ENV_VARS_UBOOT_CONFIG and others are defined and
> > whether your platforms are affected or not. It does affect our platforms
> > (among many others), including beaglebone.
> 
> I think this is for the cross tool only.

Versus what?

-- 
Denys

Re: [PATCH] u-boot: update to 2015.01

[Raphael Philipe]

Denys, I tested your patch and It seems not to be working for
u-boot-fw-utils-cross

$ bitbake u-boot-fw-utils-cross
Loading cache: 100%
|########################################################################################################################################################################|
ETA:  00:00:00
Loaded 1921 entries from dependency cache.
Parsing recipes: 100%
|######################################################################################################################################################################|
Time: 00:00:00
Parsing of 1451 .bb files complete (1450 cached, 1 parsed). 1914
targets, 85 skipped, 0 masked, 0 errors.
NOTE: Resolving any missing task queue dependencies

Build Configuration:
BB_VERSION        = "1.25.0"
BUILD_SYS         = "x86_64-linux"
NATIVELSBSTRING   = "Ubuntu-14.04"
TARGET_SYS        = "arm-poky-linux-gnueabi"
MACHINE           = "beaglebone"
DISTRO            = "poky"
DISTRO_VERSION    = "1.7"
TUNE_FEATURES     = "arm armv7a vfp neon callconvention-hard cortexa8"
TARGET_FPU        = "vfp-neon"
meta
meta-yocto
meta-yocto-bsp    = "master:45d090024d7b88c434fef2e88317946449e32c2c"
meta-oe           = "mymaster:5b4413ccf329a981bee976c9fcde5d2872f0defb"

NOTE: Preparing RunQueue
NOTE: Executing SetScene Tasks
NOTE: Executing RunQueue Tasks
ERROR: Function failed: do_compile (log file is located at
/home/rapphil/work/poky/build/tmp/work/beaglebone-poky-linux-gnueabi/u-boot-fw-utils-cross/v2015.01+gitAUTOINC+92fa7f53f1-r0/temp/log.do_compile.31549)
ERROR: Logfile of failure stored in:
/home/rapphil/work/poky/build/tmp/work/beaglebone-poky-linux-gnueabi/u-boot-fw-utils-cross/v2015.01+gitAUTOINC+92fa7f53f1-r0/temp/log.do_compile.31549
Log data follows:
| DEBUG: Executing shell function do_compile
| NOTE: make -j 4 CROSS_COMPILE=arm-poky-linux-gnueabi-
CC=arm-poky-linux-gnueabi-gcc  -march=armv7-a -mfloat-abi=hard
-mfpu=neon -mtune=cortex-a8  am335x_evm_config
|   HOSTCC  scripts/basic/fixdep
|   HOSTCC  scripts/kconfig/conf.o
|   HOSTCC  scripts/kconfig/zconf.tab.o
|   HOSTLD  scripts/kconfig/conf
| #
| # configuration written to .config
| #
| #
| # configuration written to spl/.config
| #
| NOTE: make -j 4 CROSS_COMPILE=arm-poky-linux-gnueabi-
CC=arm-poky-linux-gnueabi-gcc  -march=armv7-a -mfloat-abi=hard
-mfpu=neon -mtune=cortex-a8  env
| scripts/kconfig/conf --silentoldconfig Kconfig
| scripts/kconfig/conf --silentoldconfig Kconfig
|   CHK     include/config.h
|   GEN     include/autoconf.mk
|   GEN     include/autoconf.mk.dep
|   GEN     spl/include/autoconf.mk
|   HOSTCC  tools/env/fw_env.o
|   HOSTCC  tools/env/fw_env_main.o
|   HOSTCC  tools/env/crc32.o
|   HOSTCC  tools/env/ctype.o
| tools/env/fw_env_main.c:30:19: fatal error: fcntl.h: No such file or directory
|  #include <fcntl.h>
|                    ^
| compilation terminated.
| In file included from tools/env/crc32.c:1:0:
| tools/env/../../lib/crc32.c:12:23: fatal error: arpa/inet.h: No such
file or directory
|  #include <arpa/inet.h>
|                        ^
| compilation terminated.
| make[1]: *** [tools/env/fw_env_main.o] Error 1
| make[1]: *** Waiting for unfinished jobs....
| make[1]: *** [tools/env/crc32.o] Error 1
| tools/env/fw_env.c:16:19: fatal error: fcntl.h: No such file or directory
|  #include <fcntl.h>
|                    ^
| compilation terminated.
| make[1]: *** [tools/env/fw_env.o] Error 1
| make: *** [env] Error 2
| ERROR: oe_runmake failed
| WARNING: exit code 1 from a shell command.
| ERROR: Function failed: do_compile (log file is located at
/home/rapphil/work/poky/build/tmp/work/beaglebone-poky-linux-gnueabi/u-boot-fw-utils-cross/v2015.01+gitAUTOINC+92fa7f53f1-r0/temp/log.do_compile.31549)
ERROR: Task 6 (/home/rapphil/work/poky/meta/recipes-bsp/u-boot/u-boot-fw-utils-cross_2015.01.bb,
do_compile) failed with exit code '1'
NOTE: Tasks Summary: Attempted 658 tasks of which 656 didn't need to
be rerun and 1 failed.
No currently running tasks (658 of 661)

Summary: 1 task failed:
  /home/rapphil/work/poky/meta/recipes-bsp/u-boot/u-boot-fw-utils-cross_2015.01.bb,
do_compile

On Wed, Jan 21, 2015 at 6:11 PM, Otavio Salvador
<otavio@ossystems.com.br> wrote:
> On Wed, Jan 21, 2015 at 5:11 PM, Denys Dmytriyenko <denis@denix.org> wrote:
>> On Wed, Jan 21, 2015 at 11:17:19AM -0200, Otavio Salvador wrote:
>>> On Tue, Jan 13, 2015 at 2:09 PM, Denys Dmytriyenko <denis@denix.org> wrote:
>>> > From: Denys Dmytriyenko <denys@ti.com>
>>> >
>>> > Signed-off-by: Denys Dmytriyenko <denys@ti.com>
>>> ...
>>> > +++ b/meta/recipes-bsp/u-boot/u-boot-fw-utils/0001-tools-env-fix-build-error.patch
>>> > @@ -0,0 +1,36 @@
>>> > +From ee2d75513452aa6d5306fd380104adc8a2f6d8f2 Mon Sep 17 00:00:00 2001
>>> > +From: Masahiro Yamada <yamada.m@jp.panasonic.com>
>>> > +Date: Wed, 3 Dec 2014 10:22:50 +0900
>>> > +Subject: [PATCH] tools: env: fix build error
>>> > +
>>> > +Since CONFIG_SYS_ARCH, CONFIG_SYS_CPU, ... were moved to Kconfig,
>>> > +tools/env/fw_printenv fails to build if CONFIG_ENV_VARS_UBOOT_CONFIG
>>> > +is defined.
>>> > +(I do not think this is the right way to fix the problem, but
>>> > +for now I do not have enough time to take a close look.)
>>> > +
>>> > +Upstream-Status: Submitted [http://patchwork.ozlabs.org/patch/417192/]
>>> > +
>>> > +Signed-off-by: Masahiro Yamada <yamada.m@jp.panasonic.com>
>>> > +Reported-by: Denys Dmytriyenko <denys@ti.com>
>>>
>>> Is it necessary? In meta-fsl-arm we don't have this patch and get the
>>> tools built.
>>
>> Really? The patch above is rather self-explanatory. The issue was confirmed by
>> U-boot maintainers and the workaround provided by one of the core developers.
>> I'm not familiar with meta-fsl-arm, but I guess you can grep your platform's
>> defconfig to see if CONFIG_ENV_VARS_UBOOT_CONFIG and others are defined and
>> whether your platforms are affected or not. It does affect our platforms
>> (among many others), including beaglebone.
>
> I think this is for the cross tool only.
>
> --
> 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
> --
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [PATCH] u-boot: update to 2015.01

[Denys Dmytriyenko]

On Wed, Jan 21, 2015 at 06:25:35PM -0200, Raphael Philipe wrote:
> Denys, I tested your patch and It seems not to be working for
> u-boot-fw-utils-cross

Did the old version work for you?


> $ bitbake u-boot-fw-utils-cross
> Loading cache: 100%
> |########################################################################################################################################################################|
> ETA:  00:00:00
> Loaded 1921 entries from dependency cache.
> Parsing recipes: 100%
> |######################################################################################################################################################################|
> Time: 00:00:00
> Parsing of 1451 .bb files complete (1450 cached, 1 parsed). 1914
> targets, 85 skipped, 0 masked, 0 errors.
> NOTE: Resolving any missing task queue dependencies
> 
> Build Configuration:
> BB_VERSION        = "1.25.0"
> BUILD_SYS         = "x86_64-linux"
> NATIVELSBSTRING   = "Ubuntu-14.04"
> TARGET_SYS        = "arm-poky-linux-gnueabi"
> MACHINE           = "beaglebone"
> DISTRO            = "poky"
> DISTRO_VERSION    = "1.7"
> TUNE_FEATURES     = "arm armv7a vfp neon callconvention-hard cortexa8"
> TARGET_FPU        = "vfp-neon"
> meta
> meta-yocto
> meta-yocto-bsp    = "master:45d090024d7b88c434fef2e88317946449e32c2c"
> meta-oe           = "mymaster:5b4413ccf329a981bee976c9fcde5d2872f0defb"
> 
> NOTE: Preparing RunQueue
> NOTE: Executing SetScene Tasks
> NOTE: Executing RunQueue Tasks
> ERROR: Function failed: do_compile (log file is located at
> /home/rapphil/work/poky/build/tmp/work/beaglebone-poky-linux-gnueabi/u-boot-fw-utils-cross/v2015.01+gitAUTOINC+92fa7f53f1-r0/temp/log.do_compile.31549)
> ERROR: Logfile of failure stored in:
> /home/rapphil/work/poky/build/tmp/work/beaglebone-poky-linux-gnueabi/u-boot-fw-utils-cross/v2015.01+gitAUTOINC+92fa7f53f1-r0/temp/log.do_compile.31549
> Log data follows:
> | DEBUG: Executing shell function do_compile
> | NOTE: make -j 4 CROSS_COMPILE=arm-poky-linux-gnueabi-
> CC=arm-poky-linux-gnueabi-gcc  -march=armv7-a -mfloat-abi=hard
> -mfpu=neon -mtune=cortex-a8  am335x_evm_config
> |   HOSTCC  scripts/basic/fixdep
> |   HOSTCC  scripts/kconfig/conf.o
> |   HOSTCC  scripts/kconfig/zconf.tab.o
> |   HOSTLD  scripts/kconfig/conf
> | #
> | # configuration written to .config
> | #
> | #
> | # configuration written to spl/.config
> | #
> | NOTE: make -j 4 CROSS_COMPILE=arm-poky-linux-gnueabi-
> CC=arm-poky-linux-gnueabi-gcc  -march=armv7-a -mfloat-abi=hard
> -mfpu=neon -mtune=cortex-a8  env
> | scripts/kconfig/conf --silentoldconfig Kconfig
> | scripts/kconfig/conf --silentoldconfig Kconfig
> |   CHK     include/config.h
> |   GEN     include/autoconf.mk
> |   GEN     include/autoconf.mk.dep
> |   GEN     spl/include/autoconf.mk
> |   HOSTCC  tools/env/fw_env.o
> |   HOSTCC  tools/env/fw_env_main.o
> |   HOSTCC  tools/env/crc32.o
> |   HOSTCC  tools/env/ctype.o
> | tools/env/fw_env_main.c:30:19: fatal error: fcntl.h: No such file or directory
> |  #include <fcntl.h>
> |                    ^
> | compilation terminated.
> | In file included from tools/env/crc32.c:1:0:
> | tools/env/../../lib/crc32.c:12:23: fatal error: arpa/inet.h: No such
> file or directory
> |  #include <arpa/inet.h>
> |                        ^
> | compilation terminated.
> | make[1]: *** [tools/env/fw_env_main.o] Error 1
> | make[1]: *** Waiting for unfinished jobs....
> | make[1]: *** [tools/env/crc32.o] Error 1
> | tools/env/fw_env.c:16:19: fatal error: fcntl.h: No such file or directory
> |  #include <fcntl.h>
> |                    ^
> | compilation terminated.
> | make[1]: *** [tools/env/fw_env.o] Error 1
> | make: *** [env] Error 2
> | ERROR: oe_runmake failed
> | WARNING: exit code 1 from a shell command.
> | ERROR: Function failed: do_compile (log file is located at
> /home/rapphil/work/poky/build/tmp/work/beaglebone-poky-linux-gnueabi/u-boot-fw-utils-cross/v2015.01+gitAUTOINC+92fa7f53f1-r0/temp/log.do_compile.31549)
> ERROR: Task 6 (/home/rapphil/work/poky/meta/recipes-bsp/u-boot/u-boot-fw-utils-cross_2015.01.bb,
> do_compile) failed with exit code '1'
> NOTE: Tasks Summary: Attempted 658 tasks of which 656 didn't need to
> be rerun and 1 failed.
> No currently running tasks (658 of 661)
> 
> Summary: 1 task failed:
>   /home/rapphil/work/poky/meta/recipes-bsp/u-boot/u-boot-fw-utils-cross_2015.01.bb,
> do_compile
> 
> On Wed, Jan 21, 2015 at 6:11 PM, Otavio Salvador
> <otavio@ossystems.com.br> wrote:
> > On Wed, Jan 21, 2015 at 5:11 PM, Denys Dmytriyenko <denis@denix.org> wrote:
> >> On Wed, Jan 21, 2015 at 11:17:19AM -0200, Otavio Salvador wrote:
> >>> On Tue, Jan 13, 2015 at 2:09 PM, Denys Dmytriyenko <denis@denix.org> wrote:
> >>> > From: Denys Dmytriyenko <denys@ti.com>
> >>> >
> >>> > Signed-off-by: Denys Dmytriyenko <denys@ti.com>
> >>> ...
> >>> > +++ b/meta/recipes-bsp/u-boot/u-boot-fw-utils/0001-tools-env-fix-build-error.patch
> >>> > @@ -0,0 +1,36 @@
> >>> > +From ee2d75513452aa6d5306fd380104adc8a2f6d8f2 Mon Sep 17 00:00:00 2001
> >>> > +From: Masahiro Yamada <yamada.m@jp.panasonic.com>
> >>> > +Date: Wed, 3 Dec 2014 10:22:50 +0900
> >>> > +Subject: [PATCH] tools: env: fix build error
> >>> > +
> >>> > +Since CONFIG_SYS_ARCH, CONFIG_SYS_CPU, ... were moved to Kconfig,
> >>> > +tools/env/fw_printenv fails to build if CONFIG_ENV_VARS_UBOOT_CONFIG
> >>> > +is defined.
> >>> > +(I do not think this is the right way to fix the problem, but
> >>> > +for now I do not have enough time to take a close look.)
> >>> > +
> >>> > +Upstream-Status: Submitted [http://patchwork.ozlabs.org/patch/417192/]
> >>> > +
> >>> > +Signed-off-by: Masahiro Yamada <yamada.m@jp.panasonic.com>
> >>> > +Reported-by: Denys Dmytriyenko <denys@ti.com>
> >>>
> >>> Is it necessary? In meta-fsl-arm we don't have this patch and get the
> >>> tools built.
> >>
> >> Really? The patch above is rather self-explanatory. The issue was confirmed by
> >> U-boot maintainers and the workaround provided by one of the core developers.
> >> I'm not familiar with meta-fsl-arm, but I guess you can grep your platform's
> >> defconfig to see if CONFIG_ENV_VARS_UBOOT_CONFIG and others are defined and
> >> whether your platforms are affected or not. It does affect our platforms
> >> (among many others), including beaglebone.
> >
> > I think this is for the cross tool only.
> >
> > --
> > 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
> > --
> > _______________________________________________
> > Openembedded-core mailing list
> > Openembedded-core@lists.openembedded.org
> > http://lists.openembedded.org/mailman/listinfo/openembedded-core
> 

Re: [PATCH] u-boot: update to 2015.01

[Raphael Philipe]

I just checked 2014.07, and it does not work too. Same output.

Sorry,am I missing something?

On Wed, Jan 21, 2015 at 6:28 PM, Denys Dmytriyenko <denis@denix.org> wrote:
> On Wed, Jan 21, 2015 at 06:25:35PM -0200, Raphael Philipe wrote:
>> Denys, I tested your patch and It seems not to be working for
>> u-boot-fw-utils-cross
>
> Did the old version work for you?
>
>
>> $ bitbake u-boot-fw-utils-cross
>> Loading cache: 100%
>> |########################################################################################################################################################################|
>> ETA:  00:00:00
>> Loaded 1921 entries from dependency cache.
>> Parsing recipes: 100%
>> |######################################################################################################################################################################|
>> Time: 00:00:00
>> Parsing of 1451 .bb files complete (1450 cached, 1 parsed). 1914
>> targets, 85 skipped, 0 masked, 0 errors.
>> NOTE: Resolving any missing task queue dependencies
>>
>> Build Configuration:
>> BB_VERSION        = "1.25.0"
>> BUILD_SYS         = "x86_64-linux"
>> NATIVELSBSTRING   = "Ubuntu-14.04"
>> TARGET_SYS        = "arm-poky-linux-gnueabi"
>> MACHINE           = "beaglebone"
>> DISTRO            = "poky"
>> DISTRO_VERSION    = "1.7"
>> TUNE_FEATURES     = "arm armv7a vfp neon callconvention-hard cortexa8"
>> TARGET_FPU        = "vfp-neon"
>> meta
>> meta-yocto
>> meta-yocto-bsp    = "master:45d090024d7b88c434fef2e88317946449e32c2c"
>> meta-oe           = "mymaster:5b4413ccf329a981bee976c9fcde5d2872f0defb"
>>
>> NOTE: Preparing RunQueue
>> NOTE: Executing SetScene Tasks
>> NOTE: Executing RunQueue Tasks
>> ERROR: Function failed: do_compile (log file is located at
>> /home/rapphil/work/poky/build/tmp/work/beaglebone-poky-linux-gnueabi/u-boot-fw-utils-cross/v2015.01+gitAUTOINC+92fa7f53f1-r0/temp/log.do_compile.31549)
>> ERROR: Logfile of failure stored in:
>> /home/rapphil/work/poky/build/tmp/work/beaglebone-poky-linux-gnueabi/u-boot-fw-utils-cross/v2015.01+gitAUTOINC+92fa7f53f1-r0/temp/log.do_compile.31549
>> Log data follows:
>> | DEBUG: Executing shell function do_compile
>> | NOTE: make -j 4 CROSS_COMPILE=arm-poky-linux-gnueabi-
>> CC=arm-poky-linux-gnueabi-gcc  -march=armv7-a -mfloat-abi=hard
>> -mfpu=neon -mtune=cortex-a8  am335x_evm_config
>> |   HOSTCC  scripts/basic/fixdep
>> |   HOSTCC  scripts/kconfig/conf.o
>> |   HOSTCC  scripts/kconfig/zconf.tab.o
>> |   HOSTLD  scripts/kconfig/conf
>> | #
>> | # configuration written to .config
>> | #
>> | #
>> | # configuration written to spl/.config
>> | #
>> | NOTE: make -j 4 CROSS_COMPILE=arm-poky-linux-gnueabi-
>> CC=arm-poky-linux-gnueabi-gcc  -march=armv7-a -mfloat-abi=hard
>> -mfpu=neon -mtune=cortex-a8  env
>> | scripts/kconfig/conf --silentoldconfig Kconfig
>> | scripts/kconfig/conf --silentoldconfig Kconfig
>> |   CHK     include/config.h
>> |   GEN     include/autoconf.mk
>> |   GEN     include/autoconf.mk.dep
>> |   GEN     spl/include/autoconf.mk
>> |   HOSTCC  tools/env/fw_env.o
>> |   HOSTCC  tools/env/fw_env_main.o
>> |   HOSTCC  tools/env/crc32.o
>> |   HOSTCC  tools/env/ctype.o
>> | tools/env/fw_env_main.c:30:19: fatal error: fcntl.h: No such file or directory
>> |  #include <fcntl.h>
>> |                    ^
>> | compilation terminated.
>> | In file included from tools/env/crc32.c:1:0:
>> | tools/env/../../lib/crc32.c:12:23: fatal error: arpa/inet.h: No such
>> file or directory
>> |  #include <arpa/inet.h>
>> |                        ^
>> | compilation terminated.
>> | make[1]: *** [tools/env/fw_env_main.o] Error 1
>> | make[1]: *** Waiting for unfinished jobs....
>> | make[1]: *** [tools/env/crc32.o] Error 1
>> | tools/env/fw_env.c:16:19: fatal error: fcntl.h: No such file or directory
>> |  #include <fcntl.h>
>> |                    ^
>> | compilation terminated.
>> | make[1]: *** [tools/env/fw_env.o] Error 1
>> | make: *** [env] Error 2
>> | ERROR: oe_runmake failed
>> | WARNING: exit code 1 from a shell command.
>> | ERROR: Function failed: do_compile (log file is located at
>> /home/rapphil/work/poky/build/tmp/work/beaglebone-poky-linux-gnueabi/u-boot-fw-utils-cross/v2015.01+gitAUTOINC+92fa7f53f1-r0/temp/log.do_compile.31549)
>> ERROR: Task 6 (/home/rapphil/work/poky/meta/recipes-bsp/u-boot/u-boot-fw-utils-cross_2015.01.bb,
>> do_compile) failed with exit code '1'
>> NOTE: Tasks Summary: Attempted 658 tasks of which 656 didn't need to
>> be rerun and 1 failed.
>> No currently running tasks (658 of 661)
>>
>> Summary: 1 task failed:
>>   /home/rapphil/work/poky/meta/recipes-bsp/u-boot/u-boot-fw-utils-cross_2015.01.bb,
>> do_compile
>>
>> On Wed, Jan 21, 2015 at 6:11 PM, Otavio Salvador
>> <otavio@ossystems.com.br> wrote:
>> > On Wed, Jan 21, 2015 at 5:11 PM, Denys Dmytriyenko <denis@denix.org> wrote:
>> >> On Wed, Jan 21, 2015 at 11:17:19AM -0200, Otavio Salvador wrote:
>> >>> On Tue, Jan 13, 2015 at 2:09 PM, Denys Dmytriyenko <denis@denix.org> wrote:
>> >>> > From: Denys Dmytriyenko <denys@ti.com>
>> >>> >
>> >>> > Signed-off-by: Denys Dmytriyenko <denys@ti.com>
>> >>> ...
>> >>> > +++ b/meta/recipes-bsp/u-boot/u-boot-fw-utils/0001-tools-env-fix-build-error.patch
>> >>> > @@ -0,0 +1,36 @@
>> >>> > +From ee2d75513452aa6d5306fd380104adc8a2f6d8f2 Mon Sep 17 00:00:00 2001
>> >>> > +From: Masahiro Yamada <yamada.m@jp.panasonic.com>
>> >>> > +Date: Wed, 3 Dec 2014 10:22:50 +0900
>> >>> > +Subject: [PATCH] tools: env: fix build error
>> >>> > +
>> >>> > +Since CONFIG_SYS_ARCH, CONFIG_SYS_CPU, ... were moved to Kconfig,
>> >>> > +tools/env/fw_printenv fails to build if CONFIG_ENV_VARS_UBOOT_CONFIG
>> >>> > +is defined.
>> >>> > +(I do not think this is the right way to fix the problem, but
>> >>> > +for now I do not have enough time to take a close look.)
>> >>> > +
>> >>> > +Upstream-Status: Submitted [http://patchwork.ozlabs.org/patch/417192/]
>> >>> > +
>> >>> > +Signed-off-by: Masahiro Yamada <yamada.m@jp.panasonic.com>
>> >>> > +Reported-by: Denys Dmytriyenko <denys@ti.com>
>> >>>
>> >>> Is it necessary? In meta-fsl-arm we don't have this patch and get the
>> >>> tools built.
>> >>
>> >> Really? The patch above is rather self-explanatory. The issue was confirmed by
>> >> U-boot maintainers and the workaround provided by one of the core developers.
>> >> I'm not familiar with meta-fsl-arm, but I guess you can grep your platform's
>> >> defconfig to see if CONFIG_ENV_VARS_UBOOT_CONFIG and others are defined and
>> >> whether your platforms are affected or not. It does affect our platforms
>> >> (among many others), including beaglebone.
>> >
>> > I think this is for the cross tool only.
>> >
>> > --
>> > 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
>> > --
>> > _______________________________________________
>> > Openembedded-core mailing list
>> > Openembedded-core@lists.openembedded.org
>> > http://lists.openembedded.org/mailman/listinfo/openembedded-core
>>

Re: [PATCH] u-boot: update to 2015.01

[Denys Dmytriyenko]

On Wed, Jan 21, 2015 at 06:31:32PM -0200, Raphael Philipe wrote:
> I just checked 2014.07, and it does not work too. Same output.
> 
> Sorry,am I missing something?

Looks like host-specific issue - it calls HOSTCC in there.


> On Wed, Jan 21, 2015 at 6:28 PM, Denys Dmytriyenko <denis@denix.org> wrote:
> > On Wed, Jan 21, 2015 at 06:25:35PM -0200, Raphael Philipe wrote:
> >> Denys, I tested your patch and It seems not to be working for
> >> u-boot-fw-utils-cross
> >
> > Did the old version work for you?
> >
> >
> >> $ bitbake u-boot-fw-utils-cross
> >> Loading cache: 100%
> >> |########################################################################################################################################################################|
> >> ETA:  00:00:00
> >> Loaded 1921 entries from dependency cache.
> >> Parsing recipes: 100%
> >> |######################################################################################################################################################################|
> >> Time: 00:00:00
> >> Parsing of 1451 .bb files complete (1450 cached, 1 parsed). 1914
> >> targets, 85 skipped, 0 masked, 0 errors.
> >> NOTE: Resolving any missing task queue dependencies
> >>
> >> Build Configuration:
> >> BB_VERSION        = "1.25.0"
> >> BUILD_SYS         = "x86_64-linux"
> >> NATIVELSBSTRING   = "Ubuntu-14.04"
> >> TARGET_SYS        = "arm-poky-linux-gnueabi"
> >> MACHINE           = "beaglebone"
> >> DISTRO            = "poky"
> >> DISTRO_VERSION    = "1.7"
> >> TUNE_FEATURES     = "arm armv7a vfp neon callconvention-hard cortexa8"
> >> TARGET_FPU        = "vfp-neon"
> >> meta
> >> meta-yocto
> >> meta-yocto-bsp    = "master:45d090024d7b88c434fef2e88317946449e32c2c"
> >> meta-oe           = "mymaster:5b4413ccf329a981bee976c9fcde5d2872f0defb"
> >>
> >> NOTE: Preparing RunQueue
> >> NOTE: Executing SetScene Tasks
> >> NOTE: Executing RunQueue Tasks
> >> ERROR: Function failed: do_compile (log file is located at
> >> /home/rapphil/work/poky/build/tmp/work/beaglebone-poky-linux-gnueabi/u-boot-fw-utils-cross/v2015.01+gitAUTOINC+92fa7f53f1-r0/temp/log.do_compile.31549)
> >> ERROR: Logfile of failure stored in:
> >> /home/rapphil/work/poky/build/tmp/work/beaglebone-poky-linux-gnueabi/u-boot-fw-utils-cross/v2015.01+gitAUTOINC+92fa7f53f1-r0/temp/log.do_compile.31549
> >> Log data follows:
> >> | DEBUG: Executing shell function do_compile
> >> | NOTE: make -j 4 CROSS_COMPILE=arm-poky-linux-gnueabi-
> >> CC=arm-poky-linux-gnueabi-gcc  -march=armv7-a -mfloat-abi=hard
> >> -mfpu=neon -mtune=cortex-a8  am335x_evm_config
> >> |   HOSTCC  scripts/basic/fixdep
> >> |   HOSTCC  scripts/kconfig/conf.o
> >> |   HOSTCC  scripts/kconfig/zconf.tab.o
> >> |   HOSTLD  scripts/kconfig/conf
> >> | #
> >> | # configuration written to .config
> >> | #
> >> | #
> >> | # configuration written to spl/.config
> >> | #
> >> | NOTE: make -j 4 CROSS_COMPILE=arm-poky-linux-gnueabi-
> >> CC=arm-poky-linux-gnueabi-gcc  -march=armv7-a -mfloat-abi=hard
> >> -mfpu=neon -mtune=cortex-a8  env
> >> | scripts/kconfig/conf --silentoldconfig Kconfig
> >> | scripts/kconfig/conf --silentoldconfig Kconfig
> >> |   CHK     include/config.h
> >> |   GEN     include/autoconf.mk
> >> |   GEN     include/autoconf.mk.dep
> >> |   GEN     spl/include/autoconf.mk
> >> |   HOSTCC  tools/env/fw_env.o
> >> |   HOSTCC  tools/env/fw_env_main.o
> >> |   HOSTCC  tools/env/crc32.o
> >> |   HOSTCC  tools/env/ctype.o
> >> | tools/env/fw_env_main.c:30:19: fatal error: fcntl.h: No such file or directory
> >> |  #include <fcntl.h>
> >> |                    ^
> >> | compilation terminated.
> >> | In file included from tools/env/crc32.c:1:0:
> >> | tools/env/../../lib/crc32.c:12:23: fatal error: arpa/inet.h: No such
> >> file or directory
> >> |  #include <arpa/inet.h>
> >> |                        ^
> >> | compilation terminated.
> >> | make[1]: *** [tools/env/fw_env_main.o] Error 1
> >> | make[1]: *** Waiting for unfinished jobs....
> >> | make[1]: *** [tools/env/crc32.o] Error 1
> >> | tools/env/fw_env.c:16:19: fatal error: fcntl.h: No such file or directory
> >> |  #include <fcntl.h>
> >> |                    ^
> >> | compilation terminated.
> >> | make[1]: *** [tools/env/fw_env.o] Error 1
> >> | make: *** [env] Error 2
> >> | ERROR: oe_runmake failed
> >> | WARNING: exit code 1 from a shell command.
> >> | ERROR: Function failed: do_compile (log file is located at
> >> /home/rapphil/work/poky/build/tmp/work/beaglebone-poky-linux-gnueabi/u-boot-fw-utils-cross/v2015.01+gitAUTOINC+92fa7f53f1-r0/temp/log.do_compile.31549)
> >> ERROR: Task 6 (/home/rapphil/work/poky/meta/recipes-bsp/u-boot/u-boot-fw-utils-cross_2015.01.bb,
> >> do_compile) failed with exit code '1'
> >> NOTE: Tasks Summary: Attempted 658 tasks of which 656 didn't need to
> >> be rerun and 1 failed.
> >> No currently running tasks (658 of 661)
> >>
> >> Summary: 1 task failed:
> >>   /home/rapphil/work/poky/meta/recipes-bsp/u-boot/u-boot-fw-utils-cross_2015.01.bb,
> >> do_compile
> >>
> >> On Wed, Jan 21, 2015 at 6:11 PM, Otavio Salvador
> >> <otavio@ossystems.com.br> wrote:
> >> > On Wed, Jan 21, 2015 at 5:11 PM, Denys Dmytriyenko <denis@denix.org> wrote:
> >> >> On Wed, Jan 21, 2015 at 11:17:19AM -0200, Otavio Salvador wrote:
> >> >>> On Tue, Jan 13, 2015 at 2:09 PM, Denys Dmytriyenko <denis@denix.org> wrote:
> >> >>> > From: Denys Dmytriyenko <denys@ti.com>
> >> >>> >
> >> >>> > Signed-off-by: Denys Dmytriyenko <denys@ti.com>
> >> >>> ...
> >> >>> > +++ b/meta/recipes-bsp/u-boot/u-boot-fw-utils/0001-tools-env-fix-build-error.patch
> >> >>> > @@ -0,0 +1,36 @@
> >> >>> > +From ee2d75513452aa6d5306fd380104adc8a2f6d8f2 Mon Sep 17 00:00:00 2001
> >> >>> > +From: Masahiro Yamada <yamada.m@jp.panasonic.com>
> >> >>> > +Date: Wed, 3 Dec 2014 10:22:50 +0900
> >> >>> > +Subject: [PATCH] tools: env: fix build error
> >> >>> > +
> >> >>> > +Since CONFIG_SYS_ARCH, CONFIG_SYS_CPU, ... were moved to Kconfig,
> >> >>> > +tools/env/fw_printenv fails to build if CONFIG_ENV_VARS_UBOOT_CONFIG
> >> >>> > +is defined.
> >> >>> > +(I do not think this is the right way to fix the problem, but
> >> >>> > +for now I do not have enough time to take a close look.)
> >> >>> > +
> >> >>> > +Upstream-Status: Submitted [http://patchwork.ozlabs.org/patch/417192/]
> >> >>> > +
> >> >>> > +Signed-off-by: Masahiro Yamada <yamada.m@jp.panasonic.com>
> >> >>> > +Reported-by: Denys Dmytriyenko <denys@ti.com>
> >> >>>
> >> >>> Is it necessary? In meta-fsl-arm we don't have this patch and get the
> >> >>> tools built.
> >> >>
> >> >> Really? The patch above is rather self-explanatory. The issue was confirmed by
> >> >> U-boot maintainers and the workaround provided by one of the core developers.
> >> >> I'm not familiar with meta-fsl-arm, but I guess you can grep your platform's
> >> >> defconfig to see if CONFIG_ENV_VARS_UBOOT_CONFIG and others are defined and
> >> >> whether your platforms are affected or not. It does affect our platforms
> >> >> (among many others), including beaglebone.
> >> >
> >> > I think this is for the cross tool only.
> >> >
> >> > --
> >> > 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
> >> > --
> >> > _______________________________________________
> >> > Openembedded-core mailing list
> >> > Openembedded-core@lists.openembedded.org
> >> > http://lists.openembedded.org/mailman/listinfo/openembedded-core
> >>
> 

Re: [PATCH] u-boot: update to 2015.01

[Raphael Philipe]

Instead of HOSTCC, what is called  in your log.do_compile  ?

Maybe it is a Yocto bug. My Linux distro is supported though.
http://www.yoctoproject.org/docs/1.7/ref-manual/ref-manual.html#detailed-supported-distros.




On Wed, Jan 21, 2015 at 6:35 PM, Denys Dmytriyenko <denis@denix.org> wrote:
> On Wed, Jan 21, 2015 at 06:31:32PM -0200, Raphael Philipe wrote:
>> I just checked 2014.07, and it does not work too. Same output.
>>
>> Sorry,am I missing something?
>
> Looks like host-specific issue - it calls HOSTCC in there.
>
>
>> On Wed, Jan 21, 2015 at 6:28 PM, Denys Dmytriyenko <denis@denix.org> wrote:
>> > On Wed, Jan 21, 2015 at 06:25:35PM -0200, Raphael Philipe wrote:
>> >> Denys, I tested your patch and It seems not to be working for
>> >> u-boot-fw-utils-cross
>> >
>> > Did the old version work for you?
>> >
>> >
>> >> $ bitbake u-boot-fw-utils-cross
>> >> Loading cache: 100%
>> >> |########################################################################################################################################################################|
>> >> ETA:  00:00:00
>> >> Loaded 1921 entries from dependency cache.
>> >> Parsing recipes: 100%
>> >> |######################################################################################################################################################################|
>> >> Time: 00:00:00
>> >> Parsing of 1451 .bb files complete (1450 cached, 1 parsed). 1914
>> >> targets, 85 skipped, 0 masked, 0 errors.
>> >> NOTE: Resolving any missing task queue dependencies
>> >>
>> >> Build Configuration:
>> >> BB_VERSION        = "1.25.0"
>> >> BUILD_SYS         = "x86_64-linux"
>> >> NATIVELSBSTRING   = "Ubuntu-14.04"
>> >> TARGET_SYS        = "arm-poky-linux-gnueabi"
>> >> MACHINE           = "beaglebone"
>> >> DISTRO            = "poky"
>> >> DISTRO_VERSION    = "1.7"
>> >> TUNE_FEATURES     = "arm armv7a vfp neon callconvention-hard cortexa8"
>> >> TARGET_FPU        = "vfp-neon"
>> >> meta
>> >> meta-yocto
>> >> meta-yocto-bsp    = "master:45d090024d7b88c434fef2e88317946449e32c2c"
>> >> meta-oe           = "mymaster:5b4413ccf329a981bee976c9fcde5d2872f0defb"
>> >>
>> >> NOTE: Preparing RunQueue
>> >> NOTE: Executing SetScene Tasks
>> >> NOTE: Executing RunQueue Tasks
>> >> ERROR: Function failed: do_compile (log file is located at
>> >> /home/rapphil/work/poky/build/tmp/work/beaglebone-poky-linux-gnueabi/u-boot-fw-utils-cross/v2015.01+gitAUTOINC+92fa7f53f1-r0/temp/log.do_compile.31549)
>> >> ERROR: Logfile of failure stored in:
>> >> /home/rapphil/work/poky/build/tmp/work/beaglebone-poky-linux-gnueabi/u-boot-fw-utils-cross/v2015.01+gitAUTOINC+92fa7f53f1-r0/temp/log.do_compile.31549
>> >> Log data follows:
>> >> | DEBUG: Executing shell function do_compile
>> >> | NOTE: make -j 4 CROSS_COMPILE=arm-poky-linux-gnueabi-
>> >> CC=arm-poky-linux-gnueabi-gcc  -march=armv7-a -mfloat-abi=hard
>> >> -mfpu=neon -mtune=cortex-a8  am335x_evm_config
>> >> |   HOSTCC  scripts/basic/fixdep
>> >> |   HOSTCC  scripts/kconfig/conf.o
>> >> |   HOSTCC  scripts/kconfig/zconf.tab.o
>> >> |   HOSTLD  scripts/kconfig/conf
>> >> | #
>> >> | # configuration written to .config
>> >> | #
>> >> | #
>> >> | # configuration written to spl/.config
>> >> | #
>> >> | NOTE: make -j 4 CROSS_COMPILE=arm-poky-linux-gnueabi-
>> >> CC=arm-poky-linux-gnueabi-gcc  -march=armv7-a -mfloat-abi=hard
>> >> -mfpu=neon -mtune=cortex-a8  env
>> >> | scripts/kconfig/conf --silentoldconfig Kconfig
>> >> | scripts/kconfig/conf --silentoldconfig Kconfig
>> >> |   CHK     include/config.h
>> >> |   GEN     include/autoconf.mk
>> >> |   GEN     include/autoconf.mk.dep
>> >> |   GEN     spl/include/autoconf.mk
>> >> |   HOSTCC  tools/env/fw_env.o
>> >> |   HOSTCC  tools/env/fw_env_main.o
>> >> |   HOSTCC  tools/env/crc32.o
>> >> |   HOSTCC  tools/env/ctype.o
>> >> | tools/env/fw_env_main.c:30:19: fatal error: fcntl.h: No such file or directory
>> >> |  #include <fcntl.h>
>> >> |                    ^
>> >> | compilation terminated.
>> >> | In file included from tools/env/crc32.c:1:0:
>> >> | tools/env/../../lib/crc32.c:12:23: fatal error: arpa/inet.h: No such
>> >> file or directory
>> >> |  #include <arpa/inet.h>
>> >> |                        ^
>> >> | compilation terminated.
>> >> | make[1]: *** [tools/env/fw_env_main.o] Error 1
>> >> | make[1]: *** Waiting for unfinished jobs....
>> >> | make[1]: *** [tools/env/crc32.o] Error 1
>> >> | tools/env/fw_env.c:16:19: fatal error: fcntl.h: No such file or directory
>> >> |  #include <fcntl.h>
>> >> |                    ^
>> >> | compilation terminated.
>> >> | make[1]: *** [tools/env/fw_env.o] Error 1
>> >> | make: *** [env] Error 2
>> >> | ERROR: oe_runmake failed
>> >> | WARNING: exit code 1 from a shell command.
>> >> | ERROR: Function failed: do_compile (log file is located at
>> >> /home/rapphil/work/poky/build/tmp/work/beaglebone-poky-linux-gnueabi/u-boot-fw-utils-cross/v2015.01+gitAUTOINC+92fa7f53f1-r0/temp/log.do_compile.31549)
>> >> ERROR: Task 6 (/home/rapphil/work/poky/meta/recipes-bsp/u-boot/u-boot-fw-utils-cross_2015.01.bb,
>> >> do_compile) failed with exit code '1'
>> >> NOTE: Tasks Summary: Attempted 658 tasks of which 656 didn't need to
>> >> be rerun and 1 failed.
>> >> No currently running tasks (658 of 661)
>> >>
>> >> Summary: 1 task failed:
>> >>   /home/rapphil/work/poky/meta/recipes-bsp/u-boot/u-boot-fw-utils-cross_2015.01.bb,
>> >> do_compile
>> >>
>> >> On Wed, Jan 21, 2015 at 6:11 PM, Otavio Salvador
>> >> <otavio@ossystems.com.br> wrote:
>> >> > On Wed, Jan 21, 2015 at 5:11 PM, Denys Dmytriyenko <denis@denix.org> wrote:
>> >> >> On Wed, Jan 21, 2015 at 11:17:19AM -0200, Otavio Salvador wrote:
>> >> >>> On Tue, Jan 13, 2015 at 2:09 PM, Denys Dmytriyenko <denis@denix.org> wrote:
>> >> >>> > From: Denys Dmytriyenko <denys@ti.com>
>> >> >>> >
>> >> >>> > Signed-off-by: Denys Dmytriyenko <denys@ti.com>
>> >> >>> ...
>> >> >>> > +++ b/meta/recipes-bsp/u-boot/u-boot-fw-utils/0001-tools-env-fix-build-error.patch
>> >> >>> > @@ -0,0 +1,36 @@
>> >> >>> > +From ee2d75513452aa6d5306fd380104adc8a2f6d8f2 Mon Sep 17 00:00:00 2001
>> >> >>> > +From: Masahiro Yamada <yamada.m@jp.panasonic.com>
>> >> >>> > +Date: Wed, 3 Dec 2014 10:22:50 +0900
>> >> >>> > +Subject: [PATCH] tools: env: fix build error
>> >> >>> > +
>> >> >>> > +Since CONFIG_SYS_ARCH, CONFIG_SYS_CPU, ... were moved to Kconfig,
>> >> >>> > +tools/env/fw_printenv fails to build if CONFIG_ENV_VARS_UBOOT_CONFIG
>> >> >>> > +is defined.
>> >> >>> > +(I do not think this is the right way to fix the problem, but
>> >> >>> > +for now I do not have enough time to take a close look.)
>> >> >>> > +
>> >> >>> > +Upstream-Status: Submitted [http://patchwork.ozlabs.org/patch/417192/]
>> >> >>> > +
>> >> >>> > +Signed-off-by: Masahiro Yamada <yamada.m@jp.panasonic.com>
>> >> >>> > +Reported-by: Denys Dmytriyenko <denys@ti.com>
>> >> >>>
>> >> >>> Is it necessary? In meta-fsl-arm we don't have this patch and get the
>> >> >>> tools built.
>> >> >>
>> >> >> Really? The patch above is rather self-explanatory. The issue was confirmed by
>> >> >> U-boot maintainers and the workaround provided by one of the core developers.
>> >> >> I'm not familiar with meta-fsl-arm, but I guess you can grep your platform's
>> >> >> defconfig to see if CONFIG_ENV_VARS_UBOOT_CONFIG and others are defined and
>> >> >> whether your platforms are affected or not. It does affect our platforms
>> >> >> (among many others), including beaglebone.
>> >> >
>> >> > I think this is for the cross tool only.
>> >> >
>> >> > --
>> >> > 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
>> >> > --
>> >> > _______________________________________________
>> >> > Openembedded-core mailing list
>> >> > Openembedded-core@lists.openembedded.org
>> >> > http://lists.openembedded.org/mailman/listinfo/openembedded-core
>> >>
>>

Re: [PATCH] u-boot: update to 2015.01

[Raphael Philipe]

Hey Denys, I tested u-boot-fw-utils-cross_2014.07.bb in Ubuntu 12.04
and got the same error .

Can you confirm that this recipe works for you?

On Wed, Jan 21, 2015 at 6:20 PM, Denys Dmytriyenko <denis@denix.org> wrote:
> On Wed, Jan 21, 2015 at 06:11:23PM -0200, Otavio Salvador wrote:
>> On Wed, Jan 21, 2015 at 5:11 PM, Denys Dmytriyenko <denis@denix.org> wrote:
>> > On Wed, Jan 21, 2015 at 11:17:19AM -0200, Otavio Salvador wrote:
>> >> On Tue, Jan 13, 2015 at 2:09 PM, Denys Dmytriyenko <denis@denix.org> wrote:
>> >> > From: Denys Dmytriyenko <denys@ti.com>
>> >> >
>> >> > Signed-off-by: Denys Dmytriyenko <denys@ti.com>
>> >> ...
>> >> > +++ b/meta/recipes-bsp/u-boot/u-boot-fw-utils/0001-tools-env-fix-build-error.patch
>> >> > @@ -0,0 +1,36 @@
>> >> > +From ee2d75513452aa6d5306fd380104adc8a2f6d8f2 Mon Sep 17 00:00:00 2001
>> >> > +From: Masahiro Yamada <yamada.m@jp.panasonic.com>
>> >> > +Date: Wed, 3 Dec 2014 10:22:50 +0900
>> >> > +Subject: [PATCH] tools: env: fix build error
>> >> > +
>> >> > +Since CONFIG_SYS_ARCH, CONFIG_SYS_CPU, ... were moved to Kconfig,
>> >> > +tools/env/fw_printenv fails to build if CONFIG_ENV_VARS_UBOOT_CONFIG
>> >> > +is defined.
>> >> > +(I do not think this is the right way to fix the problem, but
>> >> > +for now I do not have enough time to take a close look.)
>> >> > +
>> >> > +Upstream-Status: Submitted [http://patchwork.ozlabs.org/patch/417192/]
>> >> > +
>> >> > +Signed-off-by: Masahiro Yamada <yamada.m@jp.panasonic.com>
>> >> > +Reported-by: Denys Dmytriyenko <denys@ti.com>
>> >>
>> >> Is it necessary? In meta-fsl-arm we don't have this patch and get the
>> >> tools built.
>> >
>> > Really? The patch above is rather self-explanatory. The issue was confirmed by
>> > U-boot maintainers and the workaround provided by one of the core developers.
>> > I'm not familiar with meta-fsl-arm, but I guess you can grep your platform's
>> > defconfig to see if CONFIG_ENV_VARS_UBOOT_CONFIG and others are defined and
>> > whether your platforms are affected or not. It does affect our platforms
>> > (among many others), including beaglebone.
>>
>> I think this is for the cross tool only.
>
> Versus what?
>
> --
> Denys
> --
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [PATCH] u-boot: update to 2015.01

[Burton, Ross]

[-- Attachment #1: Type: text/plain, Size: 466 bytes --]

On 22 January 2015 at 09:50, Raphael Philipe <rapphil@gmail.com> wrote:

> Hey Denys, I tested u-boot-fw-utils-cross_2014.07.bb in Ubuntu 12.04
> and got the same error .
>
> Can you confirm that this recipe works for you?
>

I can confirm that it doesn't work on the autobuilder:

https://autobuilder.yoctoproject.org/main/builders/nightly-fsl-ppc/builds/164/steps/BuildImages/logs/stdio

Didn't we discuss exactly this thing a few months ago?

Ross

[-- Attachment #2: Type: text/html, Size: 1302 bytes --]

Re: [PATCH] u-boot: update to 2015.01

[Denys Dmytriyenko]

On Thu, Jan 22, 2015 at 12:05:02PM +0000, Burton, Ross wrote:
> On 22 January 2015 at 09:50, Raphael Philipe <rapphil@gmail.com> wrote:
> 
> > Hey Denys, I tested u-boot-fw-utils-cross_2014.07.bb in Ubuntu 12.04

^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

> > and got the same error .
> >
> > Can you confirm that this recipe works for you?
> >
> 
> I can confirm that it doesn't work on the autobuilder:
> 
> https://autobuilder.yoctoproject.org/main/builders/nightly-fsl-ppc/builds/164/steps/BuildImages/logs/stdio

Ross,

Please note that he's talking about current 2014.07 version, not the update! 
Can you confirm that it works or fails on the autobuilder?


> Didn't we discuss exactly this thing a few months ago?

If you are talking about u-boot-fw-utils-cross specifically, then no, I don't 
recall. Please provide a link.
But I think you are mistaking it with u-boot-mkimage recipe...

-- 
Denys

Re: [PATCH] u-boot: update to 2015.01

[Burton, Ross]

[-- Attachment #1: Type: text/plain, Size: 315 bytes --]

On 22 January 2015 at 14:57, Denys Dmytriyenko <denis@denix.org> wrote:

> Please note that he's talking about current 2014.07 version, not the
> update!
> Can you confirm that it works or fails on the autobuilder?
>

Sorry, got lost in the thread.  The old version works fine, the upgrade
breaks.

Ross

[-- Attachment #2: Type: text/html, Size: 716 bytes --]

Re: [PATCH] u-boot: update to 2015.01

[Raphael Philipe]

Ross,

Is u-boot-fw-utils-cross that you are talking about?

On Thu, Jan 22, 2015 at 1:02 PM, Burton, Ross <ross.burton@intel.com> wrote:
>
> On 22 January 2015 at 14:57, Denys Dmytriyenko <denis@denix.org> wrote:
>>
>> Please note that he's talking about current 2014.07 version, not the
>> update!
>> Can you confirm that it works or fails on the autobuilder?
>
>
> Sorry, got lost in the thread.  The old version works fine, the upgrade
> breaks.
>
> Ross

Re: [PATCH] u-boot: update to 2015.01

[Denys Dmytriyenko]

On Thu, Jan 22, 2015 at 03:02:06PM +0000, Burton, Ross wrote:
> On 22 January 2015 at 14:57, Denys Dmytriyenko <denis@denix.org> wrote:
> 
> > Please note that he's talking about current 2014.07 version, not the
> > update!
> > Can you confirm that it works or fails on the autobuilder?
> >
> 
> Sorry, got lost in the thread.  The old version works fine, the upgrade
> breaks.

That's not the case for the rest of us, unfortunately...

I'm suspecting sstate got something to do with the fact that it still works 
for the old version on autobuilder.


Let me emphasise it - this specific u-boot-fw-utils-cross recipe used to 
build fine, but now fails in the same exact way regardless of the version, 
either current 2014.07 or the updated 2015.01.

So, it seems to me that fixing it is rather out of scope of this version 
update patch... :)

I'm thinking of 2 short-term workarounds - don't update this particular recipe 
to the new version; or remove it altogether. I'm sure Otavio wouldn't be happy 
either way, but fixing it should probably be a separate effort...

-- 
Denys

Re: [PATCH] u-boot: update to 2015.01

[Otavio Salvador]

On Thu, Jan 22, 2015 at 1:17 PM, Denys Dmytriyenko <denis@denix.org> wrote:
> On Thu, Jan 22, 2015 at 03:02:06PM +0000, Burton, Ross wrote:
>> On 22 January 2015 at 14:57, Denys Dmytriyenko <denis@denix.org> wrote:
>>
>> > Please note that he's talking about current 2014.07 version, not the
>> > update!
>> > Can you confirm that it works or fails on the autobuilder?
>> >
>>
>> Sorry, got lost in the thread.  The old version works fine, the upgrade
>> breaks.
>
> That's not the case for the rest of us, unfortunately...
>
> I'm suspecting sstate got something to do with the fact that it still works
> for the old version on autobuilder.
>
>
> Let me emphasise it - this specific u-boot-fw-utils-cross recipe used to
> build fine, but now fails in the same exact way regardless of the version,
> either current 2014.07 or the updated 2015.01.
>
> So, it seems to me that fixing it is rather out of scope of this version
> update patch... :)
>
> I'm thinking of 2 short-term workarounds - don't update this particular recipe
> to the new version; or remove it altogether. I'm sure Otavio wouldn't be happy
> either way, but fixing it should probably be a separate effort...

We should have denied the update on previous versions. It has been
broken by your update to 2014.07:

Author: Denys Dmytriyenko <denys@ti.com>
Date:   Tue Sep 16 19:10:01 2014 -0400

    u-boot: update to version 2014.07

    u-boot doesn't really support building its tools for the target, as they are
    built with HOSTCC compiler, which is also used to compile fixdep utility
    that gets executed during the build. Since it might be beneficial to have a
    target version of mkimage, let's hack it to build fixdep in a separate step.

    Signed-off-by: Denys Dmytriyenko <denys@ti.com>
    Signed-off-by: Ross Burton <ross.burton@intel.com>
    Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>

and this has not been checked if it kept building at that time.

So I'm ok with this update to go and you work in fix it in a follow up
patch, but drop it is not acceptable. It has a clear use case.

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

Re: [PATCH] u-boot: update to 2015.01

[Denys Dmytriyenko]

On Thu, Jan 22, 2015 at 01:40:43PM -0200, Otavio Salvador wrote:
> On Thu, Jan 22, 2015 at 1:17 PM, Denys Dmytriyenko <denis@denix.org> wrote:
> > On Thu, Jan 22, 2015 at 03:02:06PM +0000, Burton, Ross wrote:
> >> On 22 January 2015 at 14:57, Denys Dmytriyenko <denis@denix.org> wrote:
> >>
> >> > Please note that he's talking about current 2014.07 version, not the
> >> > update!
> >> > Can you confirm that it works or fails on the autobuilder?
> >> >
> >>
> >> Sorry, got lost in the thread.  The old version works fine, the upgrade
> >> breaks.
> >
> > That's not the case for the rest of us, unfortunately...
> >
> > I'm suspecting sstate got something to do with the fact that it still works
> > for the old version on autobuilder.
> >
> >
> > Let me emphasise it - this specific u-boot-fw-utils-cross recipe used to
> > build fine, but now fails in the same exact way regardless of the version,
> > either current 2014.07 or the updated 2015.01.
> >
> > So, it seems to me that fixing it is rather out of scope of this version
> > update patch... :)
> >
> > I'm thinking of 2 short-term workarounds - don't update this particular recipe
> > to the new version; or remove it altogether. I'm sure Otavio wouldn't be happy
> > either way, but fixing it should probably be a separate effort...
> 
> We should have denied the update on previous versions. It has been
> broken by your update to 2014.07:
> 
> Author: Denys Dmytriyenko <denys@ti.com>
> Date:   Tue Sep 16 19:10:01 2014 -0400
> 
>     u-boot: update to version 2014.07
> 
>     u-boot doesn't really support building its tools for the target, as they are
>     built with HOSTCC compiler, which is also used to compile fixdep utility
>     that gets executed during the build. Since it might be beneficial to have a
>     target version of mkimage, let's hack it to build fixdep in a separate step.
> 
>     Signed-off-by: Denys Dmytriyenko <denys@ti.com>
>     Signed-off-by: Ross Burton <ross.burton@intel.com>
>     Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
> 
> and this has not been checked if it kept building at that time.

Yes, it was building that time! And Ross has proof of that with autobuilder 
logs. He wouldn't have accepted it if autobuilder was breaking. It was 
breaking on another recipe and I fixed that. But fw-utils-cross was fine.


> So I'm ok with this update to go and you work in fix it in a follow up
> patch, but drop it is not acceptable. It has a clear use case.

If it has such a clear use case, how do you use it when it's broken?

-- 
Denys

Re: [PATCH] u-boot: update to 2015.01

[Otavio Salvador]

On Thu, Jan 22, 2015 at 2:00 PM, Denys Dmytriyenko <denis@denix.org> wrote:
> On Thu, Jan 22, 2015 at 01:40:43PM -0200, Otavio Salvador wrote:
>> On Thu, Jan 22, 2015 at 1:17 PM, Denys Dmytriyenko <denis@denix.org> wrote:
>> > On Thu, Jan 22, 2015 at 03:02:06PM +0000, Burton, Ross wrote:
>> >> On 22 January 2015 at 14:57, Denys Dmytriyenko <denis@denix.org> wrote:
>> >>
>> >> > Please note that he's talking about current 2014.07 version, not the
>> >> > update!
>> >> > Can you confirm that it works or fails on the autobuilder?
>> >> >
>> >>
>> >> Sorry, got lost in the thread.  The old version works fine, the upgrade
>> >> breaks.
>> >
>> > That's not the case for the rest of us, unfortunately...
>> >
>> > I'm suspecting sstate got something to do with the fact that it still works
>> > for the old version on autobuilder.
>> >
>> >
>> > Let me emphasise it - this specific u-boot-fw-utils-cross recipe used to
>> > build fine, but now fails in the same exact way regardless of the version,
>> > either current 2014.07 or the updated 2015.01.
>> >
>> > So, it seems to me that fixing it is rather out of scope of this version
>> > update patch... :)
>> >
>> > I'm thinking of 2 short-term workarounds - don't update this particular recipe
>> > to the new version; or remove it altogether. I'm sure Otavio wouldn't be happy
>> > either way, but fixing it should probably be a separate effort...
>>
>> We should have denied the update on previous versions. It has been
>> broken by your update to 2014.07:
>>
>> Author: Denys Dmytriyenko <denys@ti.com>
>> Date:   Tue Sep 16 19:10:01 2014 -0400
>>
>>     u-boot: update to version 2014.07
>>
>>     u-boot doesn't really support building its tools for the target, as they are
>>     built with HOSTCC compiler, which is also used to compile fixdep utility
>>     that gets executed during the build. Since it might be beneficial to have a
>>     target version of mkimage, let's hack it to build fixdep in a separate step.
>>
>>     Signed-off-by: Denys Dmytriyenko <denys@ti.com>
>>     Signed-off-by: Ross Burton <ross.burton@intel.com>
>>     Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
>>
>> and this has not been checked if it kept building at that time.
>
> Yes, it was building that time! And Ross has proof of that with autobuilder
> logs. He wouldn't have accepted it if autobuilder was breaking. It was
> breaking on another recipe and I fixed that. But fw-utils-cross was fine.
>
>
>> So I'm ok with this update to go and you work in fix it in a follow up
>> patch, but drop it is not acceptable. It has a clear use case.
>
> If it has such a clear use case, how do you use it when it's broken?

Well a non commonly used one? I didn't expect often use would be a
trade metric here but ...

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

Re: [PATCH] u-boot: update to 2015.01

[Denys Dmytriyenko]

On Wed, Jan 21, 2015 at 03:20:05PM -0500, Denys Dmytriyenko wrote:
> On Wed, Jan 21, 2015 at 06:11:23PM -0200, Otavio Salvador wrote:
> > On Wed, Jan 21, 2015 at 5:11 PM, Denys Dmytriyenko <denis@denix.org> wrote:
> > > On Wed, Jan 21, 2015 at 11:17:19AM -0200, Otavio Salvador wrote:
> > >> On Tue, Jan 13, 2015 at 2:09 PM, Denys Dmytriyenko <denis@denix.org> wrote:
> > >> > From: Denys Dmytriyenko <denys@ti.com>
> > >> >
> > >> > Signed-off-by: Denys Dmytriyenko <denys@ti.com>
> > >> ...
> > >> > +++ b/meta/recipes-bsp/u-boot/u-boot-fw-utils/0001-tools-env-fix-build-error.patch
> > >> > @@ -0,0 +1,36 @@
> > >> > +From ee2d75513452aa6d5306fd380104adc8a2f6d8f2 Mon Sep 17 00:00:00 2001
> > >> > +From: Masahiro Yamada <yamada.m@jp.panasonic.com>
> > >> > +Date: Wed, 3 Dec 2014 10:22:50 +0900
> > >> > +Subject: [PATCH] tools: env: fix build error
> > >> > +
> > >> > +Since CONFIG_SYS_ARCH, CONFIG_SYS_CPU, ... were moved to Kconfig,
> > >> > +tools/env/fw_printenv fails to build if CONFIG_ENV_VARS_UBOOT_CONFIG
> > >> > +is defined.
> > >> > +(I do not think this is the right way to fix the problem, but
> > >> > +for now I do not have enough time to take a close look.)
> > >> > +
> > >> > +Upstream-Status: Submitted [http://patchwork.ozlabs.org/patch/417192/]
> > >> > +
> > >> > +Signed-off-by: Masahiro Yamada <yamada.m@jp.panasonic.com>
> > >> > +Reported-by: Denys Dmytriyenko <denys@ti.com>
> > >>
> > >> Is it necessary? In meta-fsl-arm we don't have this patch and get the
> > >> tools built.
> > >
> > > Really? The patch above is rather self-explanatory. The issue was confirmed by
> > > U-boot maintainers and the workaround provided by one of the core developers.
> > > I'm not familiar with meta-fsl-arm, but I guess you can grep your platform's
> > > defconfig to see if CONFIG_ENV_VARS_UBOOT_CONFIG and others are defined and
> > > whether your platforms are affected or not. It does affect our platforms
> > > (among many others), including beaglebone.
> > 
> > I think this is for the cross tool only.
> 
> Versus what?

Otavio,

Please clarify what exactly you meant with the statement above? Thanks.

-- 
Denys

Re: [PATCH] u-boot: update to 2015.01

[Otavio Salvador]

On Thu, Jan 22, 2015 at 2:07 PM, Denys Dmytriyenko <denis@denix.org> wrote:
> On Wed, Jan 21, 2015 at 03:20:05PM -0500, Denys Dmytriyenko wrote:
>> On Wed, Jan 21, 2015 at 06:11:23PM -0200, Otavio Salvador wrote:
>> > On Wed, Jan 21, 2015 at 5:11 PM, Denys Dmytriyenko <denis@denix.org> wrote:
>> > > On Wed, Jan 21, 2015 at 11:17:19AM -0200, Otavio Salvador wrote:
>> > >> On Tue, Jan 13, 2015 at 2:09 PM, Denys Dmytriyenko <denis@denix.org> wrote:
>> > >> > From: Denys Dmytriyenko <denys@ti.com>
>> > >> >
>> > >> > Signed-off-by: Denys Dmytriyenko <denys@ti.com>
>> > >> ...
>> > >> > +++ b/meta/recipes-bsp/u-boot/u-boot-fw-utils/0001-tools-env-fix-build-error.patch
>> > >> > @@ -0,0 +1,36 @@
>> > >> > +From ee2d75513452aa6d5306fd380104adc8a2f6d8f2 Mon Sep 17 00:00:00 2001
>> > >> > +From: Masahiro Yamada <yamada.m@jp.panasonic.com>
>> > >> > +Date: Wed, 3 Dec 2014 10:22:50 +0900
>> > >> > +Subject: [PATCH] tools: env: fix build error
>> > >> > +
>> > >> > +Since CONFIG_SYS_ARCH, CONFIG_SYS_CPU, ... were moved to Kconfig,
>> > >> > +tools/env/fw_printenv fails to build if CONFIG_ENV_VARS_UBOOT_CONFIG
>> > >> > +is defined.
>> > >> > +(I do not think this is the right way to fix the problem, but
>> > >> > +for now I do not have enough time to take a close look.)
>> > >> > +
>> > >> > +Upstream-Status: Submitted [http://patchwork.ozlabs.org/patch/417192/]
>> > >> > +
>> > >> > +Signed-off-by: Masahiro Yamada <yamada.m@jp.panasonic.com>
>> > >> > +Reported-by: Denys Dmytriyenko <denys@ti.com>
>> > >>
>> > >> Is it necessary? In meta-fsl-arm we don't have this patch and get the
>> > >> tools built.
>> > >
>> > > Really? The patch above is rather self-explanatory. The issue was confirmed by
>> > > U-boot maintainers and the workaround provided by one of the core developers.
>> > > I'm not familiar with meta-fsl-arm, but I guess you can grep your platform's
>> > > defconfig to see if CONFIG_ENV_VARS_UBOOT_CONFIG and others are defined and
>> > > whether your platforms are affected or not. It does affect our platforms
>> > > (among many others), including beaglebone.
>> >
>> > I think this is for the cross tool only.
>>
>> Versus what?
>
> Please clarify what exactly you meant with the statement above? Thanks.

The fw-utils require the machine configuration, right? So it depends
on the machine to be setup before running the build. This is the
reason it ends being a cross and not a native tool.

I think your patch does address part of the issue. We now need to
figure what has been lost when moving to the kbuild system.

I can try to help to debug it if you want...

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

Re: [PATCH] u-boot: update to 2015.01

[Denys Dmytriyenko]

On Thu, Jan 22, 2015 at 02:12:30PM -0200, Otavio Salvador wrote:
> On Thu, Jan 22, 2015 at 2:07 PM, Denys Dmytriyenko <denis@denix.org> wrote:
> > On Wed, Jan 21, 2015 at 03:20:05PM -0500, Denys Dmytriyenko wrote:
> >> On Wed, Jan 21, 2015 at 06:11:23PM -0200, Otavio Salvador wrote:
> >> > On Wed, Jan 21, 2015 at 5:11 PM, Denys Dmytriyenko <denis@denix.org> wrote:
> >> > > On Wed, Jan 21, 2015 at 11:17:19AM -0200, Otavio Salvador wrote:
> >> > >> On Tue, Jan 13, 2015 at 2:09 PM, Denys Dmytriyenko <denis@denix.org> wrote:
> >> > >> > From: Denys Dmytriyenko <denys@ti.com>
> >> > >> >
> >> > >> > Signed-off-by: Denys Dmytriyenko <denys@ti.com>
> >> > >> ...
> >> > >> > +++ b/meta/recipes-bsp/u-boot/u-boot-fw-utils/0001-tools-env-fix-build-error.patch
> >> > >> > @@ -0,0 +1,36 @@
> >> > >> > +From ee2d75513452aa6d5306fd380104adc8a2f6d8f2 Mon Sep 17 00:00:00 2001
> >> > >> > +From: Masahiro Yamada <yamada.m@jp.panasonic.com>
> >> > >> > +Date: Wed, 3 Dec 2014 10:22:50 +0900
> >> > >> > +Subject: [PATCH] tools: env: fix build error
> >> > >> > +
> >> > >> > +Since CONFIG_SYS_ARCH, CONFIG_SYS_CPU, ... were moved to Kconfig,
> >> > >> > +tools/env/fw_printenv fails to build if CONFIG_ENV_VARS_UBOOT_CONFIG
> >> > >> > +is defined.
> >> > >> > +(I do not think this is the right way to fix the problem, but
> >> > >> > +for now I do not have enough time to take a close look.)
> >> > >> > +
> >> > >> > +Upstream-Status: Submitted [http://patchwork.ozlabs.org/patch/417192/]
> >> > >> > +
> >> > >> > +Signed-off-by: Masahiro Yamada <yamada.m@jp.panasonic.com>
> >> > >> > +Reported-by: Denys Dmytriyenko <denys@ti.com>
> >> > >>
> >> > >> Is it necessary? In meta-fsl-arm we don't have this patch and get the
> >> > >> tools built.
> >> > >
> >> > > Really? The patch above is rather self-explanatory. The issue was confirmed by
> >> > > U-boot maintainers and the workaround provided by one of the core developers.
> >> > > I'm not familiar with meta-fsl-arm, but I guess you can grep your platform's
> >> > > defconfig to see if CONFIG_ENV_VARS_UBOOT_CONFIG and others are defined and
> >> > > whether your platforms are affected or not. It does affect our platforms
> >> > > (among many others), including beaglebone.
> >> >
> >> > I think this is for the cross tool only.
> >>
> >> Versus what?
> >
> > Please clarify what exactly you meant with the statement above? Thanks.
> 
> The fw-utils require the machine configuration, right?

Right.


> So it depends on the machine to be setup before running the build.

Correct.


> This is the reason it ends being a cross and not a native tool.

Well, partially correct. There are 2 versions of fw-utils recipe - target and 
cross. There is no native version of fw-utils and logically it doesn't even 
make sense.

Now, for both of the target and cross recipes we need the machine config:

do_compile () {
        oe_runmake ${UBOOT_MACHINE}
        oe_runmake env
}

So, your statement that the patch above is only needed for the cross is 
incorrect or misleading.


> I think your patch does address part of the issue. We now need to
> figure what has been lost when moving to the kbuild system.

The patch above is not mine, but comes from U-boot core developer who was 
instrumental in kbuild migration - Masahiro Yamada. There's currently no 
proper fix for the issue and the suggested workaround is a hack, but it is 
sufficient and safe enough to get the tools built, which is what we need. And 
since it may not be safe for the rest of U-boot, I only apply it to fw-utils 
recipes.


> I can try to help to debug it if you want...

You are more than welcome to lend a hand to U-boot developers in debugging and 
finding a proper fix for the problem... ;)

-- 
Denys

Re: [PATCH] u-boot: update to 2015.01

[Denys Dmytriyenko]

On Thu, Jan 22, 2015 at 07:50:28AM -0200, Raphael Philipe wrote:
> Hey Denys, I tested u-boot-fw-utils-cross_2014.07.bb in Ubuntu 12.04
> and got the same error .
> 
> Can you confirm that this recipe works for you?

Ok, trying to bisect the breakage of the current 2014.07 version of 
fw-utils-cross now:

Poky snapshot from mid-September, when 2014.07 update patch was submitted and 
when it was validated by the autobuilder - still works fine.

Since it was a feature freeze time before the release, the patch was not 
accepted and was queued for later.

Poky snapshot from mid-November, when the patch was finally applied - the 
build breaks. Again, it may not be showing the issue on the autobuilder due to 
an earlier successful sstate.

So, somewhere in between a change was introduced that breaks fw-utils-cross. 
BTW, neither 2014.07 update nor 2015.01 change anything to the way 
fw-utils-cross gets built.

-- 
Denys

Re: [PATCH] u-boot: update to 2015.01

[Denys Dmytriyenko]

On Thu, Jan 22, 2015 at 04:23:16PM -0500, Denys Dmytriyenko wrote:
> On Thu, Jan 22, 2015 at 07:50:28AM -0200, Raphael Philipe wrote:
> > Hey Denys, I tested u-boot-fw-utils-cross_2014.07.bb in Ubuntu 12.04
> > and got the same error .
> > 
> > Can you confirm that this recipe works for you?
> 
> Ok, trying to bisect the breakage of the current 2014.07 version of 
> fw-utils-cross now:
> 
> Poky snapshot from mid-September, when 2014.07 update patch was submitted and 
> when it was validated by the autobuilder - still works fine.
> 
> Since it was a feature freeze time before the release, the patch was not 
> accepted and was queued for later.
> 
> Poky snapshot from mid-November, when the patch was finally applied - the 
> build breaks. Again, it may not be showing the issue on the autobuilder due to 
> an earlier successful sstate.
> 
> So, somewhere in between a change was introduced that breaks fw-utils-cross. 
> BTW, neither 2014.07 update nor 2015.01 change anything to the way 
> fw-utils-cross gets built.

Heh, and the commit that breaks (on purpose!) building 2014.07 version of 
u-boot-fw-utils-cross is the one from Richard (which does make sense):

gcc: poison default sysroot path
Various pieces of the code assume that the --sysroot option gets passed
into the compiler tools. By having a "sane" default, we don't always
spot when this occurs and this can later show up as breakage in sstate,
or in usage of the external toolchain.

We've long since talked about poisoning the default such that it will
break unless the correct option is specified. This patch does just that.

If this patch causes something to fail to build, it most likely means
the various compiler flags and commands are not correctly being passed
through to the underlying piece of software and that there is a real
problem that needs fixing, its not the fault of this patch.

http://git.yoctoproject.org/cgit/cgit.cgi/poky/commit/?id=678e8798ebe0f4fd1bd347db136f1499b8fe00c9

-- 
Denys