<div dir="ltr">I just noted that Otavio already sent a fix for this.<div><br></div><div>Please, review and apply his fix.</div><div><br></div><div>Cheers,</div><div>Anders</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">
On 18 September 2013 06:51, Anders Darander <span dir="ltr">&lt;<a href="mailto:anders.darander@gmail.com" target="_blank">anders.darander@gmail.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote"><div><div class="h5">On 18 September 2013 01:47, Saul Wold <span dir="ltr">&lt;<a href="mailto:sgw@linux.intel.com" target="_blank">sgw@linux.intel.com</a>&gt;</span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div>On 09/13/2013 12:21 PM, Anders Darander wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
The latest upgrade changed u-boot-fw-utils to u-boot-fw-utils-cross, which<br>
removed the on-target fw_printenv (and fw_setenv).<br>
<br>
Re-create the updated version of the u-boot-fw-utils recipe.<br>
<br>
U-Boot does try to strip the fw_printenv binary by default now. To avoid<br>
this, without patching the Makefile, we add HOSTSTRIP=true to EXTRA_OEMAKE.<br>
<br>
The new U-Boot do also require us to configure the build for a specific<br>
machine, thus we change the package arch.§<br>
<br>
Signed-off-by: Anders Darander &lt;<a href="mailto:anders@chargestorm.se" target="_blank">anders@chargestorm.se</a>&gt;<br>
---<br>
  meta/recipes-bsp/u-boot/<a href="http://u-boot-fw-utils_2013.07.bb" target="_blank">u-<u></u>boot-fw-utils_2013.07.bb</a> | 31 ++++++++++++++++++++++<br>
  1 file changed, 31 insertions(+)<br>
  create mode 100644 meta/recipes-bsp/u-boot/<a href="http://u-boot-fw-utils_2013.07.bb" target="_blank">u-<u></u>boot-fw-utils_2013.07.bb</a><br>
<br>
diff --git a/meta/recipes-bsp/u-boot/<a href="http://u-boot-fw-utils_2013.07.bb" target="_blank">u-<u></u>boot-fw-utils_2013.07.bb</a> b/meta/recipes-bsp/u-boot/<a href="http://u-boot-fw-utils_2013.07.bb" target="_blank">u-<u></u>boot-fw-utils_2013.07.bb</a><br>


new file mode 100644<br>
index 0000000..f97df71<br>
--- /dev/null<br>
+++ b/meta/recipes-bsp/u-boot/<a href="http://u-boot-fw-utils_2013.07.bb" target="_blank">u-<u></u>boot-fw-utils_2013.07.bb</a><br>
@@ -0,0 +1,31 @@<br>
+DESCRIPTION = &quot;U-boot bootloader fw_printenv/setenv utils&quot;<br>
+LICENSE = &quot;GPLv2+&quot;<br>
+LIC_FILES_CHKSUM = &quot;file://COPYING;md5=<u></u>1707d6db1d42237583f50183a5651e<u></u>cb&quot;<br>
+SECTION = &quot;bootloader&quot;<br>
+DEPENDS = &quot;mtd-utils&quot;<br>
+<br>
+# This revision corresponds to the tag &quot;v2013.07&quot;<br>
+# We use the revision in order to avoid having to fetch it from the<br>
+# repo during parse<br>
+SRCREV = &quot;<u></u>62c175fbb8a0f9a926c88294ea9f7e<u></u>88eb898f6c&quot;<br>
+<br>
+PV = &quot;v2013.07+git${SRCPV}&quot;<br>
+<br>
+SRC_URI = &quot;git://<a href="http://git.denx.de/u-boot.git;branch=master;protocol=git" target="_blank">git.denx.de/u-boot.git;<u></u>branch=master;protocol=git</a>&quot;<br>
+<br>
+S = &quot;${WORKDIR}/git&quot;<br>
+<br>
+EXTRA_OEMAKE = &#39;HOSTCC=&quot;${CC}&quot; HOSTSTRIP=&quot;true&quot;&#39;<br>
+<br>
+do_compile () {<br>
+  oe_runmake ${UBOOT_MACHINE}<br>
+  oe_runmake env<br>
+}<br>
+<br>
+do_install () {<br>
+  install -d ${D}${base_sbindir}<br>
+  install -m 755 ${S}/tools/env/fw_printenv ${D}${base_sbindir}/fw_<u></u>printenv<br>
+  install -m 755 ${S}/tools/env/fw_printenv ${D}${base_sbindir}/fw_setenv<br>
+}<br>
+<br>
+PACKAGE_ARCH = &quot;${MACHINE_ARCH}&quot;<br>
<br>
</blockquote>
<br></div></div>
I think this might need a COMPATIBLE_MACHINE or some check for correct systems<br></blockquote><div><br></div></div></div><div>Ah, you&#39;re correct. </div><div><br></div><div>I was looking too much on the old recipe and my working machine... As we&#39;re currently making the package machine specific, we definitely need this test. v2 on its way...</div>

<div><br></div><div>Cheers,</div><div>Anders</div><div> </div></div></div></div>
</blockquote></div><br><br clear="all"><div><br></div>-- <br>Anders Darander<br>EPO guidelines 1978: &quot;If the contribution to the known art resides<br>solely in a computer program then the subject matter is not<br>patentable in whatever manner it may be presented in the claims.&quot;
</div>