All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v2] u-boot: Update to 2016.05 release
@ 2016-05-27 13:58 Andrew Shadura
  2016-05-27 14:00 ` Otavio Salvador
  0 siblings, 1 reply; 7+ messages in thread
From: Andrew Shadura @ 2016-05-27 13:58 UTC (permalink / raw)
  To: openembedded-core

Upgrade U-Boot to the latest upstream version.
Add a patch to unbreak build on some toolchains.

Signed-off-by: Andrew Shadura <andrew.shadura@collabora.co.uk>
---
Changes since v1:

 * Fixes Upstream-Status of the patch.
---
 ...utils_2016.03.bb => u-boot-fw-utils_2016.05.bb} |  4 +-
 ...kimage_2016.03.bb => u-boot-mkimage_2016.05.bb} |  4 +-
 .../0000-video-ipu_common-fix-build-error.patch    | 48 ++++++++++++++++++++++
 .../{u-boot_2016.03.bb => u-boot_2016.05.bb}       | 10 +++--
 4 files changed, 59 insertions(+), 7 deletions(-)
 rename meta/recipes-bsp/u-boot/{u-boot-fw-utils_2016.03.bb => u-boot-fw-utils_2016.05.bb} (94%)
 rename meta/recipes-bsp/u-boot/{u-boot-mkimage_2016.03.bb => u-boot-mkimage_2016.05.bb} (90%)
 create mode 100644 meta/recipes-bsp/u-boot/u-boot/0000-video-ipu_common-fix-build-error.patch
 rename meta/recipes-bsp/u-boot/{u-boot_2016.03.bb => u-boot_2016.05.bb} (38%)

diff --git a/meta/recipes-bsp/u-boot/u-boot-fw-utils_2016.03.bb b/meta/recipes-bsp/u-boot/u-boot-fw-utils_2016.05.bb
similarity index 94%
rename from meta/recipes-bsp/u-boot/u-boot-fw-utils_2016.03.bb
rename to meta/recipes-bsp/u-boot/u-boot-fw-utils_2016.05.bb
index 79f1548..94d9c13 100644
--- a/meta/recipes-bsp/u-boot/u-boot-fw-utils_2016.03.bb
+++ b/meta/recipes-bsp/u-boot/u-boot-fw-utils_2016.05.bb
@@ -7,9 +7,9 @@ DEPENDS = "mtd-utils"
 # This revision corresponds to the tag "v2016.03"
 # We use the revision in order to avoid having to fetch it from the
 # repo during parse
-SRCREV = "df61a74e6845ec9bdcdd48d2aff5e9c2c6debeaa"
+SRCREV = "aeaec0e682f45b9e0c62c522fafea353931f73ed"
 
-PV = "v2016.03+git${SRCPV}"
+PV = "v2016.05+git${SRCPV}"
 
 SRC_URI = "git://git.denx.de/u-boot.git;branch=master"
 
diff --git a/meta/recipes-bsp/u-boot/u-boot-mkimage_2016.03.bb b/meta/recipes-bsp/u-boot/u-boot-mkimage_2016.05.bb
similarity index 90%
rename from meta/recipes-bsp/u-boot/u-boot-mkimage_2016.03.bb
rename to meta/recipes-bsp/u-boot/u-boot-mkimage_2016.05.bb
index d5921a0..bdfcaf4 100644
--- a/meta/recipes-bsp/u-boot/u-boot-mkimage_2016.03.bb
+++ b/meta/recipes-bsp/u-boot/u-boot-mkimage_2016.05.bb
@@ -8,9 +8,9 @@ DEPENDS = "openssl"
 # This revision corresponds to the tag "v2016.03"
 # We use the revision in order to avoid having to fetch it from the
 # repo during parse
-SRCREV = "df61a74e6845ec9bdcdd48d2aff5e9c2c6debeaa"
+SRCREV = "aeaec0e682f45b9e0c62c522fafea353931f73ed"
 
-PV = "v2016.03+git${SRCPV}"
+PV = "v2016.05+git${SRCPV}"
 
 SRC_URI = "git://git.denx.de/u-boot.git;branch=master"
 
diff --git a/meta/recipes-bsp/u-boot/u-boot/0000-video-ipu_common-fix-build-error.patch b/meta/recipes-bsp/u-boot/u-boot/0000-video-ipu_common-fix-build-error.patch
new file mode 100644
index 0000000..822c30b
--- /dev/null
+++ b/meta/recipes-bsp/u-boot/u-boot/0000-video-ipu_common-fix-build-error.patch
@@ -0,0 +1,48 @@
+From c510f2e436008e55a50b063f2180cb1e63984224 Mon Sep 17 00:00:00 2001
+From: Peng Fan <van.freenix@gmail.com>
+Date: Thu, 28 Apr 2016 10:07:53 +0800
+Subject: [PATCH] video: ipu_common: fix build error
+
+Some toolchains fail to build
+"clk->rate = (u64)(clk->parent->rate * 16) / div;"
+And the cast usage is wrong.
+
+Use the following code to fix the issue,
+"
+  do_div(parent_rate, div);
+  clk->rate = parent_rate;
+"
+
+Upstream-Status: Backport
+
+Reported-by: Peter Robinson <pbrobinson@gmail.com>
+Signed-off-by: Peng Fan <van.freenix@gmail.com>
+Cc: Stefano Babic <sbabic@denx.de>
+Cc: Fabio Estevam <fabio.estevam@nxp.com>
+Cc: Tom Rini <trini@konsulko.com>
+Cc: Anatolij Gustschin <agust@denx.de>
+Cc: Peter Robinson <pbrobinson@gmail.com>
+Reviewed-by: Tom Rini <trini@konsulko.com>
+Tested-by: Peter Robinson <pbrobinson@gmail.com>
+---
+ drivers/video/ipu_common.c | 4 +++-
+ 1 file changed, 3 insertions(+), 1 deletion(-)
+
+diff --git a/drivers/video/ipu_common.c b/drivers/video/ipu_common.c
+index 36d4b23..5676a0f 100644
+--- a/drivers/video/ipu_common.c
++++ b/drivers/video/ipu_common.c
+@@ -352,7 +352,9 @@ static int ipu_pixel_clk_set_rate(struct clk *clk, unsigned long rate)
+ 	 */
+ 	__raw_writel((div / 16) << 16, DI_BS_CLKGEN1(clk->id));
+ 
+-	clk->rate = (u64)(clk->parent->rate * 16) / div;
++	do_div(parent_rate, div);
++
++	clk->rate = parent_rate;
+ 
+ 	return 0;
+ }
+-- 
+2.7.4
+
diff --git a/meta/recipes-bsp/u-boot/u-boot_2016.03.bb b/meta/recipes-bsp/u-boot/u-boot_2016.05.bb
similarity index 38%
rename from meta/recipes-bsp/u-boot/u-boot_2016.03.bb
rename to meta/recipes-bsp/u-boot/u-boot_2016.05.bb
index 836b0ce..57408e7 100644
--- a/meta/recipes-bsp/u-boot/u-boot_2016.03.bb
+++ b/meta/recipes-bsp/u-boot/u-boot_2016.05.bb
@@ -2,9 +2,13 @@ require u-boot.inc
 
 DEPENDS += "dtc-native"
 
-# This revision corresponds to the tag "v2016.03"
+# This revision corresponds to the tag "v2016.05"
 # We use the revision in order to avoid having to fetch it from the
 # repo during parse
-SRCREV = "df61a74e6845ec9bdcdd48d2aff5e9c2c6debeaa"
+SRCREV = "aeaec0e682f45b9e0c62c522fafea353931f73ed"
 
-PV = "v2016.03+git${SRCPV}"
+SRC_URI_append = "\
+    file://0000-video-ipu_common-fix-build-error.patch \
+"
+
+PV = "v2016.05+git${SRCPV}"
-- 
2.7.4



^ permalink raw reply related	[flat|nested] 7+ messages in thread

* Re: [PATCH v2] u-boot: Update to 2016.05 release
  2016-05-27 13:58 [PATCH v2] u-boot: Update to 2016.05 release Andrew Shadura
@ 2016-05-27 14:00 ` Otavio Salvador
  2016-06-07  8:21   ` Andrew Shadura
  0 siblings, 1 reply; 7+ messages in thread
From: Otavio Salvador @ 2016-05-27 14:00 UTC (permalink / raw)
  To: Andrew Shadura; +Cc: Patches and discussions about the oe-core layer

On Fri, May 27, 2016 at 10:58 AM, Andrew Shadura
<andrew.shadura@collabora.co.uk> wrote:
> Upgrade U-Boot to the latest upstream version.
> Add a patch to unbreak build on some toolchains.
>
> Signed-off-by: Andrew Shadura <andrew.shadura@collabora.co.uk>
> ---

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


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


^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH v2] u-boot: Update to 2016.05 release
  2016-05-27 14:00 ` Otavio Salvador
@ 2016-06-07  8:21   ` Andrew Shadura
  2016-06-10 15:33     ` Burton, Ross
  0 siblings, 1 reply; 7+ messages in thread
From: Andrew Shadura @ 2016-06-07  8:21 UTC (permalink / raw)
  To: Otavio Salvador; +Cc: Patches and discussions about the oe-core layer

On 27/05/16 16:00, Otavio Salvador wrote:
> On Fri, May 27, 2016 at 10:58 AM, Andrew Shadura
> <andrew.shadura@collabora.co.uk> wrote:
>> > Upgrade U-Boot to the latest upstream version.
>> > Add a patch to unbreak build on some toolchains.
>> >
>> > Signed-off-by: Andrew Shadura <andrew.shadura@collabora.co.uk>
>> > ---
> Acked-by: Otavio Salvador <otavio@ossystems.com.br>

Any news on this? It still hasn't been merged, even though the other
patch has been.

-- 
Cheers,
  Andrew


^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH v2] u-boot: Update to 2016.05 release
  2016-06-07  8:21   ` Andrew Shadura
@ 2016-06-10 15:33     ` Burton, Ross
  2016-06-10 21:35       ` Andrew Shadura
  0 siblings, 1 reply; 7+ messages in thread
From: Burton, Ross @ 2016-06-10 15:33 UTC (permalink / raw)
  To: Andrew Shadura
  Cc: Otavio Salvador, Patches and discussions about the oe-core layer

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

On 7 June 2016 at 09:21, Andrew Shadura <andrew.shadura@collabora.co.uk>
wrote:

> Any news on this? It still hasn't been merged, even though the other
> patch has been.
>

It's broken on hosts where the library loader doesn't  match where the OE
distro puts it:

| cat ./Licenses/gpl-2.0.txt | gzip -9 -c | \
| tools/bin2header license_gzip > ./include/license.h
| /bin/sh: 2: tools/bin2header: not found

$ ./bin2header
-bash: ./bin2header: No such file or directory

Classic symptom of target compiler flags being used in a build using
CC_FOR_BUILD.

Ross

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

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH v2] u-boot: Update to 2016.05 release
  2016-06-10 15:33     ` Burton, Ross
@ 2016-06-10 21:35       ` Andrew Shadura
  2016-06-13 12:33         ` Burton, Ross
  0 siblings, 1 reply; 7+ messages in thread
From: Andrew Shadura @ 2016-06-10 21:35 UTC (permalink / raw)
  To: Burton, Ross
  Cc: Otavio Salvador, Patches and discussions about the oe-core layer

On 10/06/16 17:33, Burton, Ross wrote:
> 
> On 7 June 2016 at 09:21, Andrew Shadura <andrew.shadura@collabora.co.uk
> <mailto:andrew.shadura@collabora.co.uk>> wrote:
> 
>     Any news on this? It still hasn't been merged, even though the other
>     patch has been.
> 
> 
> It's broken on hosts where the library loader doesn't  match where the
> OE distro puts it:
> 
> | cat ./Licenses/gpl-2.0.txt | gzip -9 -c | \
> | tools/bin2header license_gzip > ./include/license.h
> | /bin/sh: 2: tools/bin2header: not found
> 
> $ ./bin2header
> -bash: ./bin2header: No such file or directory
> 
> Classic symptom of target compiler flags being used in a build using
> CC_FOR_BUILD.

Is it something new in this version or is that something that's also
been broken for a while now?

-- 
Cheers,
  Andrew


^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH v2] u-boot: Update to 2016.05 release
  2016-06-10 21:35       ` Andrew Shadura
@ 2016-06-13 12:33         ` Burton, Ross
  2016-06-14 11:24           ` Andrew Shadura
  0 siblings, 1 reply; 7+ messages in thread
From: Burton, Ross @ 2016-06-13 12:33 UTC (permalink / raw)
  To: Andrew Shadura
  Cc: Otavio Salvador, Patches and discussions about the oe-core layer

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

On 10 June 2016 at 22:35, Andrew Shadura <andrew.shadura@collabora.co.uk>
wrote:

> Is it something new in this version or is that something that's also
> been broken for a while now?
>

It worked previously so it must be in the new version.

Ross

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

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH v2] u-boot: Update to 2016.05 release
  2016-06-13 12:33         ` Burton, Ross
@ 2016-06-14 11:24           ` Andrew Shadura
  0 siblings, 0 replies; 7+ messages in thread
From: Andrew Shadura @ 2016-06-14 11:24 UTC (permalink / raw)
  To: Burton, Ross
  Cc: Otavio Salvador, Patches and discussions about the oe-core layer


[-- Attachment #1.1: Type: text/plain, Size: 398 bytes --]

On 13/06/16 14:33, Burton, Ross wrote:
> 
> On 10 June 2016 at 22:35, Andrew Shadura <andrew.shadura@collabora.co.uk
> <mailto:andrew.shadura@collabora.co.uk>> wrote:
> 
>> Is it something new in this version or is that something that's also
>> been broken for a while now?
> 
> It worked previously so it must be in the new version.

Okay, I'll have a look.

-- 
Cheers,
  Andrew


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2016-06-14 11:24 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-05-27 13:58 [PATCH v2] u-boot: Update to 2016.05 release Andrew Shadura
2016-05-27 14:00 ` Otavio Salvador
2016-06-07  8:21   ` Andrew Shadura
2016-06-10 15:33     ` Burton, Ross
2016-06-10 21:35       ` Andrew Shadura
2016-06-13 12:33         ` Burton, Ross
2016-06-14 11:24           ` Andrew Shadura

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.