diff mbox

u-boot: update to version 2014.07

Message ID 1410665660-25145-1-git-send-email-denis@denix.org
State Accepted
Commit a8692184d25cb7d8cb3b617e1dfb901fe1b0ca3d
Headers show

Commit Message

Denys Dmytriyenko Sept. 14, 2014, 3:34 a.m. UTC
From: Denys Dmytriyenko <denys@ti.com>

* Update U-boot and corresponding tools to version 2014.07
* Small cleanups with indentation and consolidation of u-boot.inc

Signed-off-by: Denys Dmytriyenko <denys@ti.com>
---
 ...h-Add-use-DEFAULT_LINUX_BOOT_ENV-environm.patch | 74 ----------------------
 ...2013.07.bb => u-boot-fw-utils-cross_2014.07.bb} | 22 +++----
 ...utils_2013.07.bb => u-boot-fw-utils_2014.07.bb} | 24 +++----
 ...kimage_2013.07.bb => u-boot-mkimage_2014.07.bb} | 19 +++---
 meta/recipes-bsp/u-boot/u-boot.inc                 | 11 +++-
 .../{u-boot_2013.07.bb => u-boot_2014.07.bb}       | 24 ++-----
 6 files changed, 45 insertions(+), 129 deletions(-)
 delete mode 100644 meta/recipes-bsp/u-boot/files/0001-am335x_evm.h-Add-use-DEFAULT_LINUX_BOOT_ENV-environm.patch
 rename meta/recipes-bsp/u-boot/{u-boot-fw-utils-cross_2013.07.bb => u-boot-fw-utils-cross_2014.07.bb} (49%)
 rename meta/recipes-bsp/u-boot/{u-boot-fw-utils_2013.07.bb => u-boot-fw-utils_2014.07.bb} (40%)
 rename meta/recipes-bsp/u-boot/{u-boot-mkimage_2013.07.bb => u-boot-mkimage_2014.07.bb} (51%)
 rename meta/recipes-bsp/u-boot/{u-boot_2013.07.bb => u-boot_2014.07.bb} (12%)

Comments

Ross Burton Sept. 14, 2014, 9:40 p.m. UTC | #1
On 14 September 2014 04:34, Denys Dmytriyenko <denis@denix.org> wrote:
> From: Denys Dmytriyenko <denys@ti.com>
>
> * Update U-boot and corresponding tools to version 2014.07
> * Small cleanups with indentation and consolidation of u-boot.inc

This missed the feature freeze, so can you justify merging this after
the freeze or can it wait until 1.8 opens?

Ross
Otavio Salvador Sept. 15, 2014, 1:33 p.m. UTC | #2
On Sun, Sep 14, 2014 at 6:40 PM, Burton, Ross <ross.burton@intel.com> wrote:
> On 14 September 2014 04:34, Denys Dmytriyenko <denis@denix.org> wrote:
>> From: Denys Dmytriyenko <denys@ti.com>
>>
>> * Update U-boot and corresponding tools to version 2014.07
>> * Small cleanups with indentation and consolidation of u-boot.inc
>
> This missed the feature freeze, so can you justify merging this after
> the freeze or can it wait until 1.8 opens?

I am using 2014.07 in meta-fsl-arm for a while and I support the merge
of this for 1.8.

However I'd love to see a incremental patchset where the indenting
fixes are done /before/ or /after/ the update. It is hard to identify
the real changes.

Another thing I think we can improve is to add the other licenses in
http://git.denx.de/u-boot.git/?p=u-boot.git;a=tree;f=Licenses;h=0d48cfcee2993e1353b3d75d9f6f1c29e7eb8fea;hb=HEAD
to the tracking of licenses.
Otavio Salvador Sept. 15, 2014, 1:33 p.m. UTC | #3
On Mon, Sep 15, 2014 at 10:33 AM, Otavio Salvador
<otavio@ossystems.com.br> wrote:
> On Sun, Sep 14, 2014 at 6:40 PM, Burton, Ross <ross.burton@intel.com> wrote:
>> On 14 September 2014 04:34, Denys Dmytriyenko <denis@denix.org> wrote:
>>> From: Denys Dmytriyenko <denys@ti.com>
>>>
>>> * Update U-boot and corresponding tools to version 2014.07
>>> * Small cleanups with indentation and consolidation of u-boot.inc
>>
>> This missed the feature freeze, so can you justify merging this after
>> the freeze or can it wait until 1.8 opens?
>
> I am using 2014.07 in meta-fsl-arm for a while and I support the merge
> of this for 1.8.

I meant 1.7 here sorry.
Ross Burton Sept. 15, 2014, 2:21 p.m. UTC | #4
On 15 September 2014 14:33, Otavio Salvador <otavio@ossystems.com.br> wrote:
> I am using 2014.07 in meta-fsl-arm for a while and I support the merge
> of this for 1.8.

But as we're post-freeze, endorsement doesn't qualify as a reason to
break the freeze...

(queued in my ross/unfreeze branch for when we've thawed)

Ross
Otavio Salvador Sept. 15, 2014, 2:33 p.m. UTC | #5
On Mon, Sep 15, 2014 at 11:21 AM, Burton, Ross <ross.burton@intel.com> wrote:
> On 15 September 2014 14:33, Otavio Salvador <otavio@ossystems.com.br> wrote:
>> I am using 2014.07 in meta-fsl-arm for a while and I support the merge
>> of this for 1.8.
>
> But as we're post-freeze, endorsement doesn't qualify as a reason to
> break the freeze...
>
> (queued in my ross/unfreeze branch for when we've thawed)

I think it is a way better base for work but I really don't care as I
use my own recipe in fsl-arm and so does TI.

Either way Denys please rework the patch so it is easier to review. We
decide for merging or not later when it is good.
Ross Burton Sept. 15, 2014, 2:36 p.m. UTC | #6
On 15 September 2014 15:33, Otavio Salvador <otavio@ossystems.com.br> wrote:
> I think it is a way better base for work but I really don't care as I
> use my own recipe in fsl-arm and so does TI.

And that's the problem really isn't it. :)  Why do you both use your
own recipe, instead of keeping the u-boot in oe-core up to date?

Ross
Otavio Salvador Sept. 15, 2014, 2:41 p.m. UTC | #7
On Mon, Sep 15, 2014 at 11:36 AM, Burton, Ross <ross.burton@intel.com> wrote:
> On 15 September 2014 15:33, Otavio Salvador <otavio@ossystems.com.br> wrote:
>> I think it is a way better base for work but I really don't care as I
>> use my own recipe in fsl-arm and so does TI.
>
> And that's the problem really isn't it. :)  Why do you both use your
> own recipe, instead of keeping the u-boot in oe-core up to date?

Well we both require patches; this does not mean we ought to not keep
OE-Core in sync (or close to).

This is the reason why I advocate for the inclusion. Denys were
working on this update for a while but he must have been pulled out of
it due work commitments as usual ... so it ended sent now.
Denys Dmytriyenko Sept. 16, 2014, 11:23 p.m. UTC | #8
On Mon, Sep 15, 2014 at 11:41:22AM -0300, Otavio Salvador wrote:
> On Mon, Sep 15, 2014 at 11:36 AM, Burton, Ross <ross.burton@intel.com> wrote:
> > On 15 September 2014 15:33, Otavio Salvador <otavio@ossystems.com.br> wrote:
> >> I think it is a way better base for work but I really don't care as I
> >> use my own recipe in fsl-arm and so does TI.
> >
> > And that's the problem really isn't it. :)  Why do you both use your
> > own recipe, instead of keeping the u-boot in oe-core up to date?

The way we use it in meta-ti, is to include u-boot.inc file and add any SRCREV 
bumps or patches on top. So, the base gets reused.


> Well we both require patches; this does not mean we ought to not keep
> OE-Core in sync (or close to).
> 
> This is the reason why I advocate for the inclusion. Denys were
> working on this update for a while but he must have been pulled out of
> it due work commitments as usual ... so it ended sent now.

Correct. I meant to finish and submit it last month, but got pulled into 
resolving other critical issues, hence the delay, sorry.

I've submitted a v2 version that splits the changes into 2 patches - cleanup 
and update. Please provide comments.
Ross Burton Nov. 5, 2014, 2:56 p.m. UTC | #9
On 14 September 2014 04:34, Denys Dmytriyenko <denis@denix.org> wrote:

> From: Denys Dmytriyenko <denys@ti.com>
>
> * Update U-boot and corresponding tools to version 2014.07
> * Small cleanups with indentation and consolidation of u-boot.inc
>
> Signed-off-by: Denys Dmytriyenko <denys@ti.com>
>

So u-boot-mkimage is failing to build for the target case:

| DEBUG: Executing shell function do_compile
| NOTE: make -j 8 HOSTCC=arm-poky-linux-gnueabi-gcc  -march=armv5te -marm
-mthumb-interwork --sysroot=/data/poky-master/tmp/sysroots/qemuarm
HOSTLD=arm-poky-linux-gnueabi-ld
--sysroot=/data/poky-master/tmp/sysroots/qemuarm  HOSTLDFLAGS=-Wl,-O1
-Wl,--hash-style=gnu -Wl,--as-needed HOSTSTRIP=true tools-only
|   CHK     include/config/uboot.release
|   HOSTCC  scripts/basic/fixdep
|   CHK     include/generated/timestamp_autogenerated.h
|   UPD     include/generated/timestamp_autogenerated.h
|   UPD     include/config/uboot.release
|   CHK     include/generated/version_autogenerated.h
|   UPD     include/generated/version_autogenerated.h
| scripts/basic/fixdep: 1: scripts/basic/fixdep: Syntax error: word
unexpected (expecting ")")

I suspect the real problem is:

$ file fixdep
fixdep: ELF 32-bit LSB executable, ARM, version 1 (SYSV), dynamically
linked (uses shared libs), for GNU/Linux 2.6.32,
BuildID[sha1]=0xb4ea84a189f10b4f3e08fb10f8e490135e2b920c, not stripped

It appears to have been built using the target compiler, not the host.

Ross
Ross Burton Nov. 5, 2014, 3:09 p.m. UTC | #10
On 5 November 2014 14:56, Burton, Ross <ross.burton@intel.com> wrote:

> | scripts/basic/fixdep: 1: scripts/basic/fixdep: Syntax error: word
> unexpected (expecting ")")
>

The autobuilder failed with a more obvious error (and also has more logs
available if required):

| /bin/sh: scripts/basic/fixdep:
/opt/poky/1.7/sysroots/i686-pokysdk-linux/lib/ld-linux.so.2: bad ELF
interpreter: No such file or directory

https://autobuilder.yoctoproject.org/main/builders/nightly-multilib/builds/87/steps/BuildImages_3/logs/stdio

Ross
Denys Dmytriyenko Nov. 7, 2014, 7:41 p.m. UTC | #11
On Wed, Nov 05, 2014 at 02:56:23PM +0000, Burton, Ross wrote:
> On 14 September 2014 04:34, Denys Dmytriyenko <denis@denix.org> wrote:
> 
> > From: Denys Dmytriyenko <denys@ti.com>
> >
> > * Update U-boot and corresponding tools to version 2014.07
> > * Small cleanups with indentation and consolidation of u-boot.inc
> >
> > Signed-off-by: Denys Dmytriyenko <denys@ti.com>
> >
> 
> So u-boot-mkimage is failing to build for the target case:
> 
> | DEBUG: Executing shell function do_compile
> | NOTE: make -j 8 HOSTCC=arm-poky-linux-gnueabi-gcc  -march=armv5te -marm
> -mthumb-interwork --sysroot=/data/poky-master/tmp/sysroots/qemuarm
> HOSTLD=arm-poky-linux-gnueabi-ld
> --sysroot=/data/poky-master/tmp/sysroots/qemuarm  HOSTLDFLAGS=-Wl,-O1
> -Wl,--hash-style=gnu -Wl,--as-needed HOSTSTRIP=true tools-only
> |   CHK     include/config/uboot.release
> |   HOSTCC  scripts/basic/fixdep
> |   CHK     include/generated/timestamp_autogenerated.h
> |   UPD     include/generated/timestamp_autogenerated.h
> |   UPD     include/config/uboot.release
> |   CHK     include/generated/version_autogenerated.h
> |   UPD     include/generated/version_autogenerated.h
> | scripts/basic/fixdep: 1: scripts/basic/fixdep: Syntax error: word
> unexpected (expecting ")")
> 
> I suspect the real problem is:
> 
> $ file fixdep
> fixdep: ELF 32-bit LSB executable, ARM, version 1 (SYSV), dynamically
> linked (uses shared libs), for GNU/Linux 2.6.32,
> BuildID[sha1]=0xb4ea84a189f10b4f3e08fb10f8e490135e2b920c, not stripped
> 
> It appears to have been built using the target compiler, not the host.

Ross,

I'm a bit lost here - as you said above, this is a target build and yet you 
say it shouldn't use the target compiler. Can you please explain what you 
mean?
Denys Dmytriyenko Nov. 7, 2014, 7:43 p.m. UTC | #12
On Wed, Nov 05, 2014 at 03:09:51PM +0000, Burton, Ross wrote:
> On 5 November 2014 14:56, Burton, Ross <ross.burton@intel.com> wrote:
> 
> > | scripts/basic/fixdep: 1: scripts/basic/fixdep: Syntax error: word
> > unexpected (expecting ")")
> >
> 
> The autobuilder failed with a more obvious error (and also has more logs
> available if required):
> 
> | /bin/sh: scripts/basic/fixdep:
> /opt/poky/1.7/sysroots/i686-pokysdk-linux/lib/ld-linux.so.2: bad ELF
> interpreter: No such file or directory
> 
> https://autobuilder.yoctoproject.org/main/builders/nightly-multilib/builds/87/steps/BuildImages_3/logs/stdio

And I also tried looking at the above URL, but couldn't find any of the 
mentionend error messages there. Not sure if the build number got re-used in 
the last couple days... Any other pointers you have to look at the logs?
Ross Burton Nov. 7, 2014, 8:43 p.m. UTC | #13
On 7 November 2014 19:41, Denys Dmytriyenko <denis@denix.org> wrote:

> > | scripts/basic/fixdep: 1: scripts/basic/fixdep: Syntax error: word
> > unexpected (expecting ")")
> >
> > I suspect the real problem is:
> >
> > $ file fixdep
> > fixdep: ELF 32-bit LSB executable, ARM, version 1 (SYSV), dynamically
> > linked (uses shared libs), for GNU/Linux 2.6.32,
> > BuildID[sha1]=0xb4ea84a189f10b4f3e08fb10f8e490135e2b920c, not stripped
> >
> > It appears to have been built using the target compiler, not the host.
>
> Ross,
>
> I'm a bit lost here - as you said above, this is a target build and yet you
> say it shouldn't use the target compiler. Can you please explain what you
> mean?


Yes, it's building a binary (fixdep) and then executing it.  That binary
needs to be built with the host compiler (BUILD_CC).

Easily replicated for me with "MACHINE=qemuarm bitbake u-boot-mkimage".

Ross
Otavio Salvador Nov. 7, 2014, 8:52 p.m. UTC | #14
On Fri, Nov 7, 2014 at 6:43 PM, Burton, Ross <ross.burton@intel.com> wrote:
>
> On 7 November 2014 19:41, Denys Dmytriyenko <denis@denix.org> wrote:
>>
>> > | scripts/basic/fixdep: 1: scripts/basic/fixdep: Syntax error: word
>> > unexpected (expecting ")")
>> >
>> > I suspect the real problem is:
>> >
>> > $ file fixdep
>> > fixdep: ELF 32-bit LSB executable, ARM, version 1 (SYSV), dynamically
>> > linked (uses shared libs), for GNU/Linux 2.6.32,
>> > BuildID[sha1]=0xb4ea84a189f10b4f3e08fb10f8e490135e2b920c, not stripped
>> >
>> > It appears to have been built using the target compiler, not the host.
>>
>> Ross,
>>
>> I'm a bit lost here - as you said above, this is a target build and yet
>> you
>> say it shouldn't use the target compiler. Can you please explain what you
>> mean?
>
>
> Yes, it's building a binary (fixdep) and then executing it.  That binary
> needs to be built with the host compiler (BUILD_CC).
>
> Easily replicated for me with "MACHINE=qemuarm bitbake u-boot-mkimage".

We should move to 2014.10 version. If people wish I can prepare and
send those patches as I have those working on meta-fsl-arm for a
while.
Denys Dmytriyenko Nov. 7, 2014, 8:54 p.m. UTC | #15
On Fri, Nov 07, 2014 at 08:43:38PM +0000, Burton, Ross wrote:
> On 7 November 2014 19:41, Denys Dmytriyenko <denis@denix.org> wrote:
> 
> > > | scripts/basic/fixdep: 1: scripts/basic/fixdep: Syntax error: word
> > > unexpected (expecting ")")
> > >
> > > I suspect the real problem is:
> > >
> > > $ file fixdep
> > > fixdep: ELF 32-bit LSB executable, ARM, version 1 (SYSV), dynamically
> > > linked (uses shared libs), for GNU/Linux 2.6.32,
> > > BuildID[sha1]=0xb4ea84a189f10b4f3e08fb10f8e490135e2b920c, not stripped
> > >
> > > It appears to have been built using the target compiler, not the host.
> >
> > Ross,
> >
> > I'm a bit lost here - as you said above, this is a target build and yet you
> > say it shouldn't use the target compiler. Can you please explain what you
> > mean?
> 
> 
> Yes, it's building a binary (fixdep) and then executing it.  That binary
> needs to be built with the host compiler (BUILD_CC).
> 
> Easily replicated for me with "MACHINE=qemuarm bitbake u-boot-mkimage".

Ah, got it. Didn't realize, as I build -native version. Give me a sec to fix 
it. BTW, 2014.10 is out and I've been using it for a bit now - planning on 
sending the update soon...
Denys Dmytriyenko Nov. 7, 2014, 10:30 p.m. UTC | #16
On Fri, Nov 07, 2014 at 03:54:45PM -0500, Denys Dmytriyenko wrote:
> On Fri, Nov 07, 2014 at 08:43:38PM +0000, Burton, Ross wrote:
> > On 7 November 2014 19:41, Denys Dmytriyenko <denis@denix.org> wrote:
> > 
> > > > | scripts/basic/fixdep: 1: scripts/basic/fixdep: Syntax error: word
> > > > unexpected (expecting ")")
> > > >
> > > > I suspect the real problem is:
> > > >
> > > > $ file fixdep
> > > > fixdep: ELF 32-bit LSB executable, ARM, version 1 (SYSV), dynamically
> > > > linked (uses shared libs), for GNU/Linux 2.6.32,
> > > > BuildID[sha1]=0xb4ea84a189f10b4f3e08fb10f8e490135e2b920c, not stripped
> > > >
> > > > It appears to have been built using the target compiler, not the host.
> > >
> > > Ross,
> > >
> > > I'm a bit lost here - as you said above, this is a target build and yet you
> > > say it shouldn't use the target compiler. Can you please explain what you
> > > mean?
> > 
> > 
> > Yes, it's building a binary (fixdep) and then executing it.  That binary
> > needs to be built with the host compiler (BUILD_CC).
> > 
> > Easily replicated for me with "MACHINE=qemuarm bitbake u-boot-mkimage".
> 
> Ah, got it. Didn't realize, as I build -native version. Give me a sec to fix 
> it. BTW, 2014.10 is out and I've been using it for a bit now - planning on 
> sending the update soon...

Ross,

I've just sent an RFC patch to allow building mkimage for the target. I'd like 
to collect feedback for it first, but feel free to squash it with the previous 
patch, if it's accepted (or commit separately - your choice). The update to 
2014.10 will follow shortly, I just want this version to get through first, as 
it has been in limbo for a few months...
Ross Burton Nov. 9, 2014, 10:57 p.m. UTC | #17
On 7 November 2014 22:30, Denys Dmytriyenko <denis@denix.org> wrote:

> I've just sent an RFC patch to allow building mkimage for the target. I'd
> like
> to collect feedback for it first, but feel free to squash it with the
> previous
> patch, if it's accepted (or commit separately - your choice). The update to
> 2014.10 will follow shortly, I just want this version to get through
> first, as
> it has been in limbo for a few months...
>

Thanks Denys, confirming that his works for me too.

Ross
diff mbox

Patch

diff --git a/meta/recipes-bsp/u-boot/files/0001-am335x_evm.h-Add-use-DEFAULT_LINUX_BOOT_ENV-environm.patch b/meta/recipes-bsp/u-boot/files/0001-am335x_evm.h-Add-use-DEFAULT_LINUX_BOOT_ENV-environm.patch
deleted file mode 100644
index 77e35bb..0000000
--- a/meta/recipes-bsp/u-boot/files/0001-am335x_evm.h-Add-use-DEFAULT_LINUX_BOOT_ENV-environm.patch
+++ /dev/null
@@ -1,74 +0,0 @@ 
-From 5701384cea4a829b772bf7a96a74825b58c22385 Mon Sep 17 00:00:00 2001
-From: Denys Dmytriyenko <denys@ti.com>
-Date: Thu, 17 Apr 2014 12:25:40 -0400
-Subject: [PATCH] am335x_evm.h: Add, use DEFAULT_LINUX_BOOT_ENV environment
- string
-
-Modified version of the patch currently being reviewed for mainline:
-http://patchwork.ozlabs.org/patch/334861/
-
-To deal with a reoccurring problem properly we need to specify addresses
-for the Linux kernel, Flatted Device Tree and ramdisk that obey the
-constraints within the kernel's Documentation/arm/Booting file but also
-make sure that we relocate things within a valid address range.
-
-Signed-off-by: Denys Dmytriyenko <denys@ti.com>
-Signed-off-by: Tom Rini <trini@ti.com>
-
-Upstream-Status: Pending
----
- include/configs/am335x_evm.h | 31 ++++++++++++++++++++++++++-----
- 1 file changed, 26 insertions(+), 5 deletions(-)
-
-diff --git a/include/configs/am335x_evm.h b/include/configs/am335x_evm.h
-index c5a6d4b..01e32b3 100644
---- a/include/configs/am335x_evm.h
-+++ b/include/configs/am335x_evm.h
-@@ -54,10 +54,7 @@
- #define CONFIG_ENV_VARS_UBOOT_RUNTIME_CONFIG
- #ifndef CONFIG_SPL_BUILD
- #define CONFIG_EXTRA_ENV_SETTINGS \
--	"loadaddr=0x80200000\0" \
--	"fdtaddr=0x80F80000\0" \
--	"fdt_high=0xffffffff\0" \
--	"rdaddr=0x81000000\0" \
-+	DEFAULT_LINUX_BOOT_ENV \
- 	"bootdir=/boot\0" \
- 	"bootfile=uImage\0" \
- 	"fdtfile=undefined\0" \
-@@ -197,7 +194,31 @@
- #define CONFIG_SYS_MEMTEST_END		(CONFIG_SYS_MEMTEST_START \
- 					+ (8 * 1024 * 1024))
- 
--#define CONFIG_SYS_LOAD_ADDR		0x81000000 /* Default load address */
-+/*
-+ * Our DDR memory always starts at 0x80000000 and U-Boot shall have
-+ * relocated itself to higher in memory by the time this value is used.
-+ * However, set this to a 32MB offset to allow for easier Linux kernel
-+ * booting as the default is often used as the kernel load address.
-+ */
-+#define CONFIG_SYS_LOAD_ADDR		0x82000000 /* Default load address */
-+
-+/*
-+ * We setup defaults based on constraints from the Linux kernel, which should
-+ * also be safe elsewhere.  We have the default load at 32MB into DDR (for
-+ * the kernel), FDT above 128MB (the maximum location for the end of the
-+ * kernel), and the ramdisk 512KB above that (allowing for hopefully never
-+ * seen large trees).  We say all of this must be within the first 256MB
-+ * as that will normally be within the kernel lowmem and thus visible via
-+ * bootm_size and we only run on platforms with 256MB or more of memory.
-+ */
-+#define DEFAULT_LINUX_BOOT_ENV \
-+	"loadaddr=0x82000000\0" \
-+	"kernel_addr_r=0x82000000\0" \
-+	"fdtaddr=0x88000000\0" \
-+	"fdt_addr_r=0x88000000\0" \
-+	"rdaddr=0x88080000\0" \
-+	"ramdisk_addr_r=0x88080000\0" \
-+	"bootm_size=0x10000000\0"
- 
- #define CONFIG_MMC
- #define CONFIG_GENERIC_MMC
--- 
-1.9.2
-
diff --git a/meta/recipes-bsp/u-boot/u-boot-fw-utils-cross_2013.07.bb b/meta/recipes-bsp/u-boot/u-boot-fw-utils-cross_2014.07.bb
similarity index 49%
rename from meta/recipes-bsp/u-boot/u-boot-fw-utils-cross_2013.07.bb
rename to meta/recipes-bsp/u-boot/u-boot-fw-utils-cross_2014.07.bb
index e616397..d1f1f9a 100644
--- a/meta/recipes-bsp/u-boot/u-boot-fw-utils-cross_2013.07.bb
+++ b/meta/recipes-bsp/u-boot/u-boot-fw-utils-cross_2014.07.bb
@@ -1,15 +1,15 @@ 
 SUMMARY = "U-Boot bootloader fw_printenv/setenv utilities"
 LICENSE = "GPLv2+"
-LIC_FILES_CHKSUM = "file://COPYING;md5=1707d6db1d42237583f50183a5651ecb"
+LIC_FILES_CHKSUM = "file://Licenses/README;md5=025bf9f768cbcb1a165dbe1a110babfb"
 SECTION = "bootloader"
 DEPENDS = "mtd-utils"
 
-# This revision corresponds to the tag "v2013.07"
+# This revision corresponds to the tag "v2014.07"
 # We use the revision in order to avoid having to fetch it from the
 # repo during parse
-SRCREV = "62c175fbb8a0f9a926c88294ea9f7e88eb898f6c"
+SRCREV = "524123a70761110c5cf3ccc5f52f6d4da071b959"
 
-PV = "v2013.07+git${SRCPV}"
+PV = "v2014.07+git${SRCPV}"
 
 SRC_URI = "git://git.denx.de/u-boot.git;branch=master;protocol=git"
 
@@ -17,22 +17,22 @@  S = "${WORKDIR}/git"
 
 inherit uboot-config cross
 
-EXTRA_OEMAKE = 'CROSS_COMPILE=${TARGET_PREFIX} CC="${TARGET_PREFIX}gcc ${TOOLCHAIN_OPTIONS}"'
+EXTRA_OEMAKE = 'CROSS_COMPILE=${TARGET_PREFIX} CC="${TARGET_PREFIX}gcc ${TARGET_CC_ARCH} ${TOOLCHAIN_OPTIONS}"'
 
 do_compile () {
-  oe_runmake ${UBOOT_MACHINE}
-  oe_runmake env
+	oe_runmake ${UBOOT_MACHINE}
+	oe_runmake env
 }
 
 do_install () {
-  install -d ${D}${bindir_cross}
-  install -m 755 ${S}/tools/env/fw_printenv ${D}${bindir_cross}/fw_printenv
-  install -m 755 ${S}/tools/env/fw_printenv ${D}${bindir_cross}/fw_setenv
+	install -d ${D}${bindir_cross}
+	install -m 755 ${S}/tools/env/fw_printenv ${D}${bindir_cross}/fw_printenv
+	install -m 755 ${S}/tools/env/fw_printenv ${D}${bindir_cross}/fw_setenv
 }
 
 SYSROOT_PREPROCESS_FUNCS = "uboot_fw_utils_cross"
 uboot_fw_utils_cross() {
-    sysroot_stage_dir ${D}${bindir_cross} ${SYSROOT_DESTDIR}${bindir_cross}
+	sysroot_stage_dir ${D}${bindir_cross} ${SYSROOT_DESTDIR}${bindir_cross}
 }
 
 PACKAGE_ARCH = "${MACHINE_ARCH}"
diff --git a/meta/recipes-bsp/u-boot/u-boot-fw-utils_2013.07.bb b/meta/recipes-bsp/u-boot/u-boot-fw-utils_2014.07.bb
similarity index 40%
rename from meta/recipes-bsp/u-boot/u-boot-fw-utils_2013.07.bb
rename to meta/recipes-bsp/u-boot/u-boot-fw-utils_2014.07.bb
index 89fed8b..a626c95 100644
--- a/meta/recipes-bsp/u-boot/u-boot-fw-utils_2013.07.bb
+++ b/meta/recipes-bsp/u-boot/u-boot-fw-utils_2014.07.bb
@@ -1,35 +1,35 @@ 
 SUMMARY = "U-Boot bootloader fw_printenv/setenv utilities"
 LICENSE = "GPLv2+"
-LIC_FILES_CHKSUM = "file://COPYING;md5=1707d6db1d42237583f50183a5651ecb"
+LIC_FILES_CHKSUM = "file://Licenses/README;md5=025bf9f768cbcb1a165dbe1a110babfb"
 SECTION = "bootloader"
 DEPENDS = "mtd-utils"
 
-# This revision corresponds to the tag "v2013.07"
+# This revision corresponds to the tag "v2014.07"
 # We use the revision in order to avoid having to fetch it from the
 # repo during parse
-SRCREV = "62c175fbb8a0f9a926c88294ea9f7e88eb898f6c"
+SRCREV = "524123a70761110c5cf3ccc5f52f6d4da071b959"
 
-PV = "v2013.07+git${SRCPV}"
+PV = "v2014.07+git${SRCPV}"
 
 SRC_URI = "git://git.denx.de/u-boot.git;branch=master;protocol=git"
 
 S = "${WORKDIR}/git"
 
-EXTRA_OEMAKE = 'HOSTCC="${CC}" HOSTSTRIP="true"'
+INSANE_SKIP_${PN} = "already-stripped"
 
 inherit uboot-config
 
 do_compile () {
-  oe_runmake ${UBOOT_MACHINE}
-  oe_runmake env
+	oe_runmake ${UBOOT_MACHINE}
+	oe_runmake env
 }
 
 do_install () {
-  install -d ${D}${base_sbindir}
-  install -d ${D}${sysconfdir}
-  install -m 755 ${S}/tools/env/fw_printenv ${D}${base_sbindir}/fw_printenv
-  install -m 755 ${S}/tools/env/fw_printenv ${D}${base_sbindir}/fw_setenv
-  install -m 0644 ${S}/tools/env/fw_env.config ${D}${sysconfdir}/fw_env.config
+	install -d ${D}${base_sbindir}
+	install -d ${D}${sysconfdir}
+	install -m 755 ${S}/tools/env/fw_printenv ${D}${base_sbindir}/fw_printenv
+	install -m 755 ${S}/tools/env/fw_printenv ${D}${base_sbindir}/fw_setenv
+	install -m 0644 ${S}/tools/env/fw_env.config ${D}${sysconfdir}/fw_env.config
 }
 
 PACKAGE_ARCH = "${MACHINE_ARCH}"
diff --git a/meta/recipes-bsp/u-boot/u-boot-mkimage_2013.07.bb b/meta/recipes-bsp/u-boot/u-boot-mkimage_2014.07.bb
similarity index 51%
rename from meta/recipes-bsp/u-boot/u-boot-mkimage_2013.07.bb
rename to meta/recipes-bsp/u-boot/u-boot-mkimage_2014.07.bb
index 93c8102..b9fad1b 100644
--- a/meta/recipes-bsp/u-boot/u-boot-mkimage_2013.07.bb
+++ b/meta/recipes-bsp/u-boot/u-boot-mkimage_2014.07.bb
@@ -1,14 +1,14 @@ 
 SUMMARY = "U-Boot bootloader image creation tool"
 LICENSE = "GPLv2+"
-LIC_FILES_CHKSUM = "file://COPYING;md5=1707d6db1d42237583f50183a5651ecb"
+LIC_FILES_CHKSUM = "file://Licenses/README;md5=025bf9f768cbcb1a165dbe1a110babfb"
 SECTION = "bootloader"
 
-# This revision corresponds to the tag "v2013.07"
+# This revision corresponds to the tag "v2014.07"
 # We use the revision in order to avoid having to fetch it from the
 # repo during parse
-SRCREV = "62c175fbb8a0f9a926c88294ea9f7e88eb898f6c"
+SRCREV = "524123a70761110c5cf3ccc5f52f6d4da071b959"
 
-PV = "v2013.07+git${SRCPV}"
+PV = "v2014.07+git${SRCPV}"
 
 SRC_URI = "git://git.denx.de/u-boot.git;branch=master;protocol=git"
 
@@ -17,16 +17,13 @@  S = "${WORKDIR}/git"
 EXTRA_OEMAKE = 'HOSTCC="${CC}" HOSTLD="${LD}" HOSTLDFLAGS="${LDFLAGS}" HOSTSTRIP=true'
 
 do_compile () {
-  # Make sure the recompile is OK
-  rm -f ${B}/tools/.depend
-
-  oe_runmake tools
+	oe_runmake tools-only
 }
 
 do_install () {
-  install -d ${D}${bindir}
-  install -m 0755 tools/mkimage ${D}${bindir}/uboot-mkimage
-  ln -sf uboot-mkimage ${D}${bindir}/mkimage
+	install -d ${D}${bindir}
+	install -m 0755 tools/mkimage ${D}${bindir}/uboot-mkimage
+	ln -sf uboot-mkimage ${D}${bindir}/mkimage
 }
 
 BBCLASSEXTEND = "native nativesdk"
diff --git a/meta/recipes-bsp/u-boot/u-boot.inc b/meta/recipes-bsp/u-boot/u-boot.inc
index 53c87c2..c695b73 100644
--- a/meta/recipes-bsp/u-boot/u-boot.inc
+++ b/meta/recipes-bsp/u-boot/u-boot.inc
@@ -3,6 +3,15 @@  HOMEPAGE = "http://www.denx.de/wiki/U-Boot/WebHome"
 SECTION = "bootloaders"
 PROVIDES = "virtual/bootloader"
 
+LICENSE = "GPLv2+"
+LIC_FILES_CHKSUM = "file://Licenses/README;md5=025bf9f768cbcb1a165dbe1a110babfb"
+
+SRC_URI = "git://git.denx.de/u-boot.git;branch=master"
+
+S = "${WORKDIR}/git"
+
+PACKAGE_ARCH = "${MACHINE_ARCH}"
+
 inherit uboot-config deploy
 
 EXTRA_OEMAKE = 'CROSS_COMPILE=${TARGET_PREFIX} CC="${TARGET_PREFIX}gcc ${TOOLCHAIN_OPTIONS}"'
@@ -14,7 +23,7 @@  UBOOT_LOCALVERSION ?= ""
 
 # Some versions of u-boot use .bin and others use .img.  By default use .bin
 # but enable individual recipes to change this value.
-UBOOT_SUFFIX ?= "bin"
+UBOOT_SUFFIX ??= "bin"
 UBOOT_IMAGE ?= "u-boot-${MACHINE}-${PV}-${PR}.${UBOOT_SUFFIX}"
 UBOOT_BINARY ?= "u-boot.${UBOOT_SUFFIX}"
 UBOOT_SYMLINK ?= "u-boot-${MACHINE}.${UBOOT_SUFFIX}"
diff --git a/meta/recipes-bsp/u-boot/u-boot_2013.07.bb b/meta/recipes-bsp/u-boot/u-boot_2014.07.bb
similarity index 12%
rename from meta/recipes-bsp/u-boot/u-boot_2013.07.bb
rename to meta/recipes-bsp/u-boot/u-boot_2014.07.bb
index f8a8856..44cc52e 100644
--- a/meta/recipes-bsp/u-boot/u-boot_2013.07.bb
+++ b/meta/recipes-bsp/u-boot/u-boot_2014.07.bb
@@ -1,25 +1,9 @@ 
 require u-boot.inc
 
-# To build u-boot for your machine, provide the following lines in your machine
-# config, replacing the assignments as appropriate for your machine.
-# UBOOT_MACHINE = "omap3_beagle_config"
-# UBOOT_ENTRYPOINT = "0x80008000"
-# UBOOT_LOADADDRESS = "0x80008000"
+DEPENDS += "dtc-native"
 
-LICENSE = "GPLv2+"
-LIC_FILES_CHKSUM = "file://COPYING;md5=1707d6db1d42237583f50183a5651ecb \
-                    file://README;beginline=1;endline=22;md5=78b195c11cb6ef63e6985140db7d7bab"
-
-# This revision corresponds to the tag "v2013.07"
+# This revision corresponds to the tag "v2014.07"
 # We use the revision in order to avoid having to fetch it from the repo during parse
-SRCREV = "62c175fbb8a0f9a926c88294ea9f7e88eb898f6c"
-
-PV = "v2013.07+git${SRCPV}"
-
-SRC_URI = "git://git.denx.de/u-boot.git;branch=master \
-           file://0001-am335x_evm.h-Add-use-DEFAULT_LINUX_BOOT_ENV-environm.patch \
-"
-
-S = "${WORKDIR}/git"
+SRCREV = "524123a70761110c5cf3ccc5f52f6d4da071b959"
 
-PACKAGE_ARCH = "${MACHINE_ARCH}"
+PV = "v2014.07+git${SRCPV}"