Skip to contentSkip to conversation

uboot-tools: bump to version 2017.09

PATCH New Fabio Estevam 6 participants 11 messages Nov 1, 2017 – Nov 6, 2017

lore
Share & tools
lore

Fabio Estevam

author
#0 lore raw

PATCH v2 1/1 uboot-tools: bump to version 2017.09

The 'env' target has been renamed to 'envtools' so change it accordingly.

Signed-off-by Fabio Estevam <[email protected]>

---
Changes since v1:
- Take the 'env' target rename into account

 package/uboot-tools/uboot-tools.hash | 2 +-
 package/uboot-tools/uboot-tools.mk   | 4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)
package/uboot-tools/uboot-tools.hash +1 -1
index 3ba8e34..82efc79 100644
@@ -1,2 +1,2 @@
 # Locally computed:
-sha256  5374bfdc8acb9a38c025371b1ff20f45e7533668e84e685d0df5d9e7c0e4feff  u-boot-2017.07.tar.bz2
+sha256  b2d15f2cf5f72e706025cde73d67247c6da8cd35f7e10891eefe7d9095089744  u-boot-2017.09.tar.bz2
package/uboot-tools/uboot-tools.mk +2 -2
index 4f39f4a..c936575 100644
@@ -4,7 +4,7 @@
 #
 ################################################################################
 
-UBOOT_TOOLS_VERSION = 2017.07
+UBOOT_TOOLS_VERSION = 2017.09
 UBOOT_TOOLS_SOURCE = u-boot-$(UBOOT_TOOLS_VERSION).tar.bz2
 UBOOT_TOOLS_SITE = ftp://ftp.denx.de/pub/u-boot
 UBOOT_TOOLS_LICENSE = GPL-2.0+

@@ -35,7 +35,7 @@ define UBOOT_TOOLS_BUILD_CMDS
 	$(TARGET_MAKE_ENV) $(MAKE) -C $(@D) $(UBOOT_TOOLS_MAKE_OPTS) \
 		CROSS_BUILD_TOOLS=y tools-only
 	$(TARGET_MAKE_ENV) $(MAKE) -C $(@D) $(UBOOT_TOOLS_MAKE_OPTS) \
-		env no-dot-config-targets=env
+		envtools no-dot-config-targets=envtools
 endef
 
 ifeq ($(BR2_PACKAGE_UBOOT_TOOLS_MKIMAGE),y)
Signature · 1 line
2.7.4
#2 lore raw

PATCH v2 1/1 uboot-tools: bump to version 2017.09

Hi Matthew,

Quoted 7 lines · Matthew Weber

Fabio,

On Wed, Nov 1, 2017 at 1:53 PM, Fabio Estevam <[email protected]> wrote:

Would you mind testing this patchset against this bump? I believe the patches will need rework if this series is applied first.
https://patchwork.ozlabs.org/patch/833049/

I applied your patch and u-boot tools still built fine.

#3 lore raw

PATCH v2 1/1 uboot-tools: bump to version 2017.09

In reply to Fabio Estevam
Quoted 41 lines · Fabio Estevam

The 'env' target has been renamed to 'envtools' so change it accordingly.

Signed-off-by Fabio Estevam <[email protected]>

---
Changes since v1:
- Take the 'env' target rename into account

 package/uboot-tools/uboot-tools.hash | 2 +-
 package/uboot-tools/uboot-tools.mk   | 4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)
package/uboot-tools/uboot-tools.hash +0 -1
index 3ba8e34..82efc79 100644
@@ -1,2 +1,2 @@
 # Locally computed:
-sha256

5374bfdc8acb9a38c025371b1ff20f45e7533668e84e685d0df5d9e7c0e4feff
u-boot-2017.07.tar.bz2
+sha256 b2d15f2cf5f72e706025cde73d67247c6da8cd35f7e10891eefe7d9095089744
u-boot-2017.09.tar.bz2

package/uboot-tools/uboot-tools.mk +1 -1
index 4f39f4a..c936575 100644
@@ -4,7 +4,7 @@
 #
 
 ################################################################################
 
-UBOOT_TOOLS_VERSION = 2017.07
+UBOOT_TOOLS_VERSION = 2017.09
 UBOOT_TOOLS_SOURCE = u-boot-$(UBOOT_TOOLS_VERSION).tar.bz2
 UBOOT_TOOLS_SITE = ftp://ftp.denx.de/pub/u-boot

UBOOT_TOOLS_LICENSE = GPL-2.0+

package/uboot-tools/uboot-tools.mk define UBOOT_TOOLS_BUILD_CMDS · line 41

Quoted 8 lines
package/uboot-tools/uboot-tools.mk +1 -1
@@ -35,7 +35,7 @@ define UBOOT_TOOLS_BUILD_CMDS
 	$(TARGET_MAKE_ENV) $(MAKE) -C $(@D) $(UBOOT_TOOLS_MAKE_OPTS) \
 		CROSS_BUILD_TOOLS=y tools-only
 	$(TARGET_MAKE_ENV) $(MAKE) -C $(@D) $(UBOOT_TOOLS_MAKE_OPTS) \
-		env no-dot-config-targets=env
+		envtools no-dot-config-targets=envtools
 endef
 
 ifeq ($(BR2_PACKAGE_UBOOT_TOOLS_MKIMAGE),y)

Thanks, this is similar to a local hack that I've been using. The build produces what I am expecting in the target image.

Tested-by Jan Kundr?t <[email protected]>
Reviewed-by Jan Kundr?t <[email protected]>
Applied
#4 lore raw

PATCH v2 1/1 uboot-tools: bump to version 2017.09

In reply to Fabio Estevam

Hello,

Quoted 7 lines · Fabio Estevam

The 'env' target has been renamed to 'envtools' so change it accordingly.

Signed-off-by Fabio Estevam <[email protected]>

---
Changes since v1:
- Take the 'env' target rename into account

Applied to master, thanks.

Thomas

Signature · 3 lines
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
#5 lore raw

PATCH v2 1/1 uboot-tools: bump to version 2017.09

In reply to Fabio Estevam

Hello,

Quoted 7 lines · Fabio Estevam

The 'env' target has been renamed to 'envtools' so change it accordingly.

Signed-off-by Fabio Estevam <[email protected]>

---
Changes since v1:
- Take the 'env' target rename into account

I'm not sure if it's this version bump, or Matthew Weber's patch about pylibfdt, but we're seeing new build failures with host-uboot-tools:

http://autobuild.buildroot.net/?reason=host-uboot-tools-2017.09

Could one of you have a look?

Thanks!

Thomas

Signature · 3 lines
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
#6 lore raw

PATCH v2 1/1 uboot-tools: bump to version 2017.09

Thomas, Fabio

Quoted 12 lines · Thomas Petazzoni

Hello,

Quoted 7 lines · Fabio Estevam

The 'env' target has been renamed to 'envtools' so change it accordingly.

Signed-off-by Fabio Estevam <[email protected]>

---
Changes since v1:
- Take the 'env' target rename into account

Quoted 7 lines

I'm not sure if it's this version bump, or Matthew Weber's patch about pylibfdt, but we're seeing new build failures with host-uboot-tools:

http://autobuild.buildroot.net/?reason=host-uboot-tools-2017.09

Could one of you have a look?

Sure

#7 lore raw

PATCH v2 1/1 uboot-tools: bump to version 2017.09

In reply to Thomas Petazzoni
Quoted 6 lines · Thomas Petazzoni

I'm not sure if it's this version bump, or Matthew Weber's patch about pylibfdt, but we're seeing new build failures with host-uboot-tools:

http://autobuild.buildroot.net/?reason=host-uboot-tools-2017.09

Could one of you have a look?

Hi,
I believe that this is caused by a change to host-dtc in commit b8c3e941731ddfc3728045142aa9081adeeeaee2. Since this commit, host-dtc has started installing its own libfdt into host/include/libfdt*. U-Boot uses #include directives in form of <system/ones>, not "user/ones", so it finds a copy of libfdt provided by dtc, not its own modified version. I sent a quick and dirty patch which fixes this earlier today:

http://lists.busybox.net/pipermail/buildroot/2017-November/206092.html

With kind regards,
Jan

#8 lore raw

PATCH v2 1/1 uboot-tools: bump to version 2017.09

Jan,

Quoted 18 lines · Jan Kundr?t
Quoted 7 lines · Thomas Petazzoni

I'm not sure if it's this version bump, or Matthew Weber's patch about pylibfdt, but we're seeing new build failures with host-uboot-tools:

http://autobuild.buildroot.net/?reason=host-uboot-tools-2017.09

Could one of you have a look?

Hi,
I believe that this is caused by a change to host-dtc in commit b8c3e941731ddfc3728045142aa9081adeeeaee2. Since this commit, host-dtc has started installing its own libfdt into host/include/libfdt*. U-Boot uses #include directives in form of <system/ones>, not "user/ones", so it finds a copy of libfdt provided by dtc, not its own modified version. I sent a quick and dirty patch which fixes this earlier today:

Quoted 2 lines

http://lists.busybox.net/pipermail/buildroot/2017-November/206092.html

Tested that patch and it resolves a rebuild without error on my end. Agree, unsure how we'd want to fix that one, upstream is a mess right now with anything tool/*fdt related...

Matt

#9 lore raw

PATCH v2 1/1 uboot-tools: bump to version 2017.09

In reply to Jan Kundrát

Hi Jan,

Quoted 16 lines · Jan Kundr?t
Quoted 6 lines · Thomas Petazzoni

I'm not sure if it's this version bump, or Matthew Weber's patch about pylibfdt, but we're seeing new build failures with host-uboot-tools:

http://autobuild.buildroot.net/?reason=host-uboot-tools-2017.09

Could one of you have a look?

Hi,
I believe that this is caused by a change to host-dtc in commit b8c3e941731ddfc3728045142aa9081adeeeaee2. Since this commit, host-dtc has started installing its own libfdt into host/include/libfdt*. U-Boot uses #include directives in form of <system/ones>, not "user/ones", so it finds a copy of libfdt provided by dtc, not its own modified version. I sent a quick and dirty patch which fixes this earlier today:

Quoted 1 line

http://lists.busybox.net/pipermail/buildroot/2017-November/206092.html

Tested your patch on my machine and I could build host-uboot-tools and host-qemu together without error.

Best Regards
Guo Ren

#10 lore raw

PATCH v2 1/1 uboot-tools: bump to version 2017.09

In reply to Jan Kundrát

Jan,

Quoted 18 lines · Jan Kundr?t
Quoted 7 lines · Thomas Petazzoni

I'm not sure if it's this version bump, or Matthew Weber's patch about pylibfdt, but we're seeing new build failures with host-uboot-tools:

http://autobuild.buildroot.net/?reason=host-uboot-tools-2017.09

Could one of you have a look?

Hi,
I believe that this is caused by a change to host-dtc in commit b8c3e941731ddfc3728045142aa9081adeeeaee2. Since this commit, host-dtc has started installing its own libfdt into host/include/libfdt*. U-Boot uses #include directives in form of <system/ones>, not "user/ones", so it finds a copy of libfdt provided by dtc, not its own modified version. I sent a quick and dirty patch which fixes this earlier today:

Quoted 2 lines

http://lists.busybox.net/pipermail/buildroot/2017-November/206092.html

Based on Jan's submission and Arnout's feedback

https://patchwork.ozlabs.org/patch/834861/
https://patchwork.ozlabs.org/patch/834862/

Matt