Makefile: Update minimum dtc version to 1.4.3
PATCH 1/1 Makefile: Update minimum dtc version to 1.4.3
With support for overlays and calling the -@ flag to dtc we need to have at least 1.4.3 available now.
---
Simon, please make sure to grab this one too with Pantelis' series, thanks!
--- Makefile | 4 ++-- scripts/dtc-version.sh | 7 ++++--- 2 files changed, 6 insertions(+), 5 deletions(-)
Makefile +2 -2
@@ -1448,8 +1448,8 @@ System.map: u-boot @$(call SYSTEM_MAP,$<) > $@ checkdtc: - @if test $(call dtc-version) -lt 0104; then \ - echo '*** Your dtc is too old, please upgrade to dtc 1.4 or newer'; \ + @if test $(call dtc-version) -lt 010403; then \ + echo '*** Your dtc is too old, please upgrade to dtc 1.4.3 or newer'; \ false; \ fi
scripts/dtc-version.sh +4 -3
@@ -2,8 +2,8 @@ # # dtc-version dtc-command # -# Prints the dtc version of `dtc-command' in a canonical 4-digit form -# such as `0222' for binutils 2.22 +# Prints the dtc version of `dtc-command' in a canonical 6-digit form +# such as `010404' for dtc 1.4.4 # dtc="$*"
@@ -16,5 +16,6 @@ fi MAJOR=$($dtc -v | head -1 | awk '{print $NF}' | cut -d . -f 1) MINOR=$($dtc -v | head -1 | awk '{print $NF}' | cut -d . -f 2) +PATCH=$($dtc -v | head -1 | awk '{print $NF}' | cut -d . -f 3 | cut -d - -f 1) -printf "%02d%02d\\n" $MAJOR $MINOR +printf "%02d%02d%02d\\n" $MAJOR $MINOR $PATCH
Signature · 1 line
1.9.1
PATCH 1/1 Makefile: Update minimum dtc version to 1.4.3
Makefile System.map: u-boot · line 1454
Quoted 12 lines · Tom Rini
With support for overlays and calling the -@ flag to dtc we need to have at least 1.4.3 available now.
Makefile +2
-2checkdtc: - @if test $(call dtc-version) -lt 0104; then \ - echo '*** Your dtc is too old, please upgrade to dtc 1.4 or newer'; \ + @if test $(call dtc-version) -lt 010403; then \ + echo '*** Your dtc is too old, please upgrade to dtc 1.4.3 or newer'; \ false; \ fi
Can we add a comment here indicating that .travis.yml must be updated too if the version requirement changes, and similarly a comment in .travis.yml that mentions it must be kept in sync with the checkdtc rule in Makefile?
For users, it's a bit annoying that we need a newer dtc that is present in distros, without any deprecation warning from the Makefile for a couple releases before:-(
PATCH 1/1 Makefile: Update minimum dtc version to 1.4.3
Quoted 6 lines · Stephen Warren
Quoted 4 lines · Tom Rini
With support for overlays and calling the -@ flag to dtc we need to have at least 1.4.3 available now.
Makefile +0
-0
Quoted 12 lines
Makefile System.map: u-boot · line 1454
Quoted 7 lines
checkdtc:
Makefile +2
-2- @if test $(call dtc-version) -lt 0104; then \ - echo '*** Your dtc is too old, please upgrade to dtc 1.4 or newer'; \ + @if test $(call dtc-version) -lt 010403; then \ + echo '*** Your dtc is too old, please upgrade to dtc 1.4.3 or newer'; \ false; \ fiCan we add a comment here indicating that .travis.yml must be updated too if the version requirement changes, and similarly a comment in .travis.yml that mentions it must be kept in sync with the checkdtc rule in
Makefile?
I'm planning to follow up https://patchwork.ozlabs.org/patch/813888/
with one to do -b v1.4.3 so we'll be testing the minimum requirement in the future.
Quoted 3 lines
For users, it's a bit annoying that we need a newer dtc that is present in distros, without any deprecation warning from the
Makefilefor a couple releases before:-(
Well, the other ways to do this are "mirror" the kernel and include a copy of dtc, or have different version checks for if UT_OVERLAY is enabled or not.
Signature · 8 lines
Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: Digital signature URL: <http://lists.denx.de/pipermail/u-boot/attachments/20170915/ca0d9f9d/attachment.sig>
Quoted 6 lines · Tom Rini
With support for overlays and calling the -@ flag to dtc we need to have at least 1.4.3 available now.
Cc Simon Glass <[email protected]>Reported-by Stephen Warren <[email protected]>Signed-off-by Tom Rini <[email protected]>
Applied to u-boot/master, thanks!
Signature · 8 lines
Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: Digital signature URL: <http://lists.denx.de/pipermail/u-boot/attachments/20170922/628530cb/attachment.sig>
Quoted 10 lines · Tom Rini
Quoted 6 lines · Tom Rini
With support for overlays and calling the -@ flag to dtc we need to have at least 1.4.3 available now.
Cc Simon Glass <[email protected]>Reported-by Stephen Warren <[email protected]>Signed-off-by Tom Rini <[email protected]>Applied to u-boot/master, thanks!
Debian still only has 1.4.2 even in unstable, so u-boot/master no longer builds on any debian system.
Signature · 2 lines
Best regards, Marek Vasut
Quoted 6 lines · Marek Vasut
Quoted 5 lines · Tom Rini
Quoted 3 lines · Tom Rini
With support for overlays and calling the -@ flag to dtc we need to have at least 1.4.3 available now.
Quoted 8 lines
Quoted 5 lines
Quoted 3 lines
Cc Simon Glass <[email protected]>Reported-by Stephen Warren <[email protected]>Signed-off-by Tom Rini <[email protected]>Applied to u-boot/master, thanks!
Debian still only has 1.4.2 even in unstable, so u-boot/master no longer builds on any debian system.
Then perhaps we need to mirror the kernel and provide / build dtc, but allow for people to override it.
Signature · 8 lines
Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: Digital signature URL: <http://lists.denx.de/pipermail/u-boot/attachments/20170923/2c475425/attachment.sig>
Quoted 11 lines · Tom Rini
Quoted 10 lines · Marek Vasut
Quoted 9 lines · Tom Rini
Quoted 6 lines · Tom Rini
With support for overlays and calling the -@ flag to dtc we need to have at least 1.4.3 available now.
Cc Simon Glass <[email protected]>Reported-by Stephen Warren <[email protected]>Signed-off-by Tom Rini <[email protected]>
Quoted 7 lines
Quoted 4 lines
Quoted 1 line
Applied to u-boot/master, thanks!
Debian still only has 1.4.2 even in unstable, so u-boot/master no longer builds on any debian system.
Then perhaps we need to mirror the kernel and provide / build dtc, but allow for people to override it.
What about allowing old DTC version if DTOs are not in use ?
Signature · 2 lines
Best regards, Marek Vasut
Quoted 14 lines · Marek Vasut
Quoted 13 lines · Tom Rini
Quoted 12 lines · Marek Vasut
Quoted 10 lines · Tom Rini
Quoted 6 lines · Tom Rini
With support for overlays and calling the -@ flag to dtc we need to have at least 1.4.3 available now.
Cc Simon Glass <[email protected]>Reported-by Stephen Warren <[email protected]>Signed-off-by Tom Rini <[email protected]>Applied to u-boot/master, thanks!
Quoted 7 lines
Quoted 5 lines
Quoted 2 lines
Debian still only has 1.4.2 even in unstable, so u-boot/master no longer builds on any debian system.
Then perhaps we need to mirror the kernel and provide / build dtc, but allow for people to override it.
What about allowing old DTC version if DTOs are not in use ?
If you can re-work the logic for that, sure, thanks!
Signature · 8 lines
Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: Digital signature URL: <http://lists.denx.de/pipermail/u-boot/attachments/20170923/95f23fac/attachment.sig>
Quoted 18 lines · Tom Rini
Quoted 17 lines · Marek Vasut
Quoted 16 lines · Tom Rini
Quoted 14 lines · Marek Vasut
Quoted 10 lines · Tom Rini
Quoted 6 lines · Tom Rini
With support for overlays and calling the -@ flag to dtc we need to have at least 1.4.3 available now.
Cc Simon Glass <[email protected]>Reported-by Stephen Warren <[email protected]>Signed-off-by Tom Rini <[email protected]>Applied to u-boot/master, thanks!
Debian still only has 1.4.2 even in unstable, so u-boot/master no longer builds on any debian system.
Quoted 7 lines
Quoted 4 lines
Quoted 2 lines
Then perhaps we need to mirror the kernel and provide / build dtc, but allow for people to override it.
What about allowing old DTC version if DTOs are not in use ?
If you can re-work the logic for that, sure, thanks!
I would expect the author to fix such massive breakage or a revert.
Signature · 2 lines
Best regards, Marek Vasut
Hi Tom,
Quoted 18 lines · Tom Rini
Quoted 17 lines · Marek Vasut
Quoted 16 lines · Tom Rini
Quoted 14 lines · Marek Vasut
Quoted 10 lines · Tom Rini
Quoted 6 lines · Tom Rini
With support for overlays and calling the -@ flag to dtc we need to have at least 1.4.3 available now.
Cc Simon Glass <[email protected]>Reported-by Stephen Warren <[email protected]>Signed-off-by Tom Rini <[email protected]>Applied to u-boot/master, thanks!
Debian still only has 1.4.2 even in unstable, so u-boot/master no longer builds on any debian system.
Quoted 7 lines
Quoted 4 lines
Quoted 2 lines
Then perhaps we need to mirror the kernel and provide / build dtc, but allow for people to override it.
What about allowing old DTC version if DTOs are not in use ?
If you can re-work the logic for that, sure, thanks!
Ubuntu 16.04 LTS is still at dtc v1.4.0, so it's broken there too.
Regards,
Bin
Quoted 24 lines · Bin Meng
Hi Tom,
Quoted 21 lines · Tom Rini
Quoted 20 lines · Marek Vasut
Quoted 18 lines · Tom Rini
Quoted 6 lines · Marek Vasut
Quoted 5 lines · Tom Rini
Quoted 3 lines · Tom Rini
With support for overlays and calling the -@ flag to dtc we need to have at least 1.4.3 available now.
Quoted 8 lines
Quoted 5 lines
Quoted 3 lines
Cc Simon Glass <[email protected]>Reported-by Stephen Warren <[email protected]>Signed-off-by Tom Rini <[email protected]>Applied to u-boot/master, thanks!
Debian still only has 1.4.2 even in unstable, so u-boot/master no longer builds on any debian system.
Then perhaps we need to mirror the kernel and provide / build dtc, but allow for people to override it.
Quoted 6 lines
Quoted 4 lines
Quoted 1 line
What about allowing old DTC version if DTOs are not in use ?
If you can re-work the logic for that, sure, thanks!
Ubuntu 16.04 LTS is still at dtc v1.4.0, so it's broken there too.
Maybe it's just time to embed a copy of dtc into u-boot (like the kernel, especially since we use so many features...)?
the device-tree-compiler package in debian (and thus ubuntu) doesn't get too much love:
http://metadata.ftp-master.debian.org/changelogs/main/d/device-tree-compiler/unstable_changelog
Regards,
Signature · 2 lines
Robert Nelson https://rcn-ee.com/
Quoted 27 lines · Robert Nelson
Quoted 26 lines · Bin Meng
Hi Tom,
Quoted 23 lines · Tom Rini
Quoted 21 lines · Marek Vasut
Quoted 11 lines · Tom Rini
Quoted 10 lines · Marek Vasut
Quoted 9 lines · Tom Rini
Quoted 6 lines · Tom Rini
With support for overlays and calling the -@ flag to dtc we need to have at least 1.4.3 available now.
Cc Simon Glass <[email protected]>Reported-by Stephen Warren <[email protected]>Signed-off-by Tom Rini <[email protected]>Quoted 7 lines
Quoted 4 lines
Quoted 1 line
Applied to u-boot/master, thanks!
Debian still only has 1.4.2 even in unstable, so u-boot/master no longer builds on any debian system.
Then perhaps we need to mirror the kernel and provide / build dtc, but allow for people to override it.
What about allowing old DTC version if DTOs are not in use ?
Quoted 7 lines
Quoted 4 lines
Quoted 2 lines
If you can re-work the logic for that, sure, thanks!
Ubuntu 16.04 LTS is still at dtc v1.4.0, so it's broken there too.
Maybe it's just time to embed a copy of dtc into u-boot (like the kernel, especially since we use so many features...)?
That is my intention at this point, and I'm testing a small series to do that now (the last pain point looks to have been the patch that allows 'DTC' to be over-ridden for FreeBSD).
Signature · 8 lines
Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: Digital signature URL: <http://lists.denx.de/pipermail/u-boot/attachments/20170923/7bc32f00/attachment.sig>
Quoted 31 lines · Tom Rini
Quoted 30 lines · Robert Nelson
Quoted 29 lines · Bin Meng
Hi Tom,
Quoted 25 lines · Tom Rini
Quoted 14 lines · Marek Vasut
Quoted 13 lines · Tom Rini
Quoted 12 lines · Marek Vasut
Quoted 10 lines · Tom Rini
Quoted 6 lines · Tom Rini
With support for overlays and calling the -@ flag to dtc we need to have at least 1.4.3 available now.
Cc Simon Glass <[email protected]>Reported-by Stephen Warren <[email protected]>Signed-off-by Tom Rini <[email protected]>Applied to u-boot/master, thanks!
Quoted 7 lines
Quoted 5 lines
Quoted 2 lines
Debian still only has 1.4.2 even in unstable, so u-boot/master no longer builds on any debian system.
Then perhaps we need to mirror the kernel and provide / build dtc, but allow for people to override it.
What about allowing old DTC version if DTOs are not in use ?
If you can re-work the logic for that, sure, thanks!
Quoted 8 lines
Quoted 4 lines
Quoted 1 line
Ubuntu 16.04 LTS is still at dtc v1.4.0, so it's broken there too.
Maybe it's just time to embed a copy of dtc into u-boot (like the kernel, especially since we use so many features...)?
That is my intention at this point, and I'm testing a small series to do that now (the last pain point looks to have been the patch that allows 'DTC' to be over-ridden for FreeBSD).
cfr my comment about vendoring , I'd really hate to go down that path. Chrome does that and look what came out of it -- outdated patched packages in the chrome codebase, with debian trying hard to unbundle that stuff again.
Signature · 2 lines
Best regards, Marek Vasut
Quoted 37 lines · Marek Vasut
Quoted 36 lines · Tom Rini
Quoted 34 lines · Robert Nelson
Quoted 24 lines · Bin Meng
Hi Tom,
Quoted 21 lines · Tom Rini
Quoted 20 lines · Marek Vasut
Quoted 18 lines · Tom Rini
Quoted 6 lines · Marek Vasut
Quoted 5 lines · Tom Rini
Quoted 3 lines · Tom Rini
With support for overlays and calling the -@ flag to dtc we need to have at least 1.4.3 available now.
Quoted 8 lines
Quoted 5 lines
Quoted 3 lines
Cc Simon Glass <[email protected]>Reported-by Stephen Warren <[email protected]>Signed-off-by Tom Rini <[email protected]>Applied to u-boot/master, thanks!
Debian still only has 1.4.2 even in unstable, so u-boot/master no longer builds on any debian system.
Then perhaps we need to mirror the kernel and provide / build dtc, but allow for people to override it.
Quoted 6 lines
Quoted 4 lines
Quoted 1 line
What about allowing old DTC version if DTOs are not in use ?
If you can re-work the logic for that, sure, thanks!
Ubuntu 16.04 LTS is still at dtc v1.4.0, so it's broken there too.
Maybe it's just time to embed a copy of dtc into u-boot (like the kernel, especially since we use so many features...)?
Quoted 8 lines
Quoted 3 lines
That is my intention at this point, and I'm testing a small series to do that now (the last pain point looks to have been the patch that allows 'DTC' to be over-ridden for FreeBSD).
cfr my comment about vendoring , I'd really hate to go down that path. Chrome does that and look what came out of it -- outdated patched packages in the chrome codebase, with debian trying hard to unbundle that stuff again.
Yes, but we're going to do it all the same. We're tied to some of the tools we need and vendors aren't keeping pace with them either. dtc 1.4.0 (Ubuntu 16.04) is from 2013. dtc 1.4.3 is from last December.
And I plan on following the Linux Kernel model of just syncing only directly with the upstream project.
Signature · 8 lines
Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: Digital signature URL: <http://lists.denx.de/pipermail/u-boot/attachments/20170924/1f8ea221/attachment.sig>
Quoted 42 lines · Tom Rini
Quoted 41 lines · Marek Vasut
Quoted 39 lines · Tom Rini
Quoted 27 lines · Robert Nelson
Quoted 26 lines · Bin Meng
Hi Tom,
Quoted 23 lines · Tom Rini
Quoted 21 lines · Marek Vasut
Quoted 11 lines · Tom Rini
Quoted 10 lines · Marek Vasut
Quoted 9 lines · Tom Rini
Quoted 6 lines · Tom Rini
With support for overlays and calling the -@ flag to dtc we need to have at least 1.4.3 available now.
Cc Simon Glass <[email protected]>Reported-by Stephen Warren <[email protected]>Signed-off-by Tom Rini <[email protected]>Quoted 7 lines
Quoted 4 lines
Quoted 1 line
Applied to u-boot/master, thanks!
Debian still only has 1.4.2 even in unstable, so u-boot/master no longer builds on any debian system.
Then perhaps we need to mirror the kernel and provide / build dtc, but allow for people to override it.
What about allowing old DTC version if DTOs are not in use ?
Quoted 7 lines
Quoted 4 lines
Quoted 2 lines
If you can re-work the logic for that, sure, thanks!
Ubuntu 16.04 LTS is still at dtc v1.4.0, so it's broken there too.
Maybe it's just time to embed a copy of dtc into u-boot (like the kernel, especially since we use so many features...)?
That is my intention at this point, and I'm testing a small series to do that now (the last pain point looks to have been the patch that allows 'DTC' to be over-ridden for FreeBSD).
Quoted 8 lines
Quoted 4 lines
cfr my comment about vendoring , I'd really hate to go down that path. Chrome does that and look what came out of it -- outdated patched packages in the chrome codebase, with debian trying hard to unbundle that stuff again.
Yes, but we're going to do it all the same. We're tied to some of the tools we need and vendors aren't keeping pace with them either. dtc 1.4.0 (Ubuntu 16.04) is from 2013. dtc 1.4.3 is from last December.
The new DTC version is only mandated by DTO support, which is not used by anyone to my knowledge at this point. Whoever wants to use DTO, should have new enough DTC and I'm fine with the build failing if they don't.
This heavy-handed approach of imposing custom DTC build on everyone because of a feature noone uses is awful and I disagree with it. If there is a version check for DTC for builds with and without DTO, fine, I can live with that.
But bundling DTC only because of unused functionality, NAK.
Quoted 2 lines
And I plan on following the Linux Kernel model of just syncing only directly with the upstream project.
See above, this is heavy-handed and IMO unnecessary. In fact, I hope the kernel removes the DTC fork completely.
Signature · 2 lines
Best regards, Marek Vasut
Hi Marek, Tom,
Quoted 62 lines · Tom Rini
Quoted 42 lines · Marek Vasut
Quoted 41 lines · Tom Rini
Quoted 39 lines · Robert Nelson
Quoted 27 lines · Bin Meng
Quoted 26 lines
Hi Tom,
Quoted 23 lines · Tom Rini
Quoted 21 lines · Marek Vasut
Quoted 11 lines · Tom Rini
Quoted 10 lines · Marek Vasut
Quoted 9 lines · Tom Rini
Quoted 6 lines · Tom Rini
With support for overlays and calling the -@ flag to dtc we need to have at least 1.4.3 available now.
Cc Simon Glass <[email protected]>Reported-by Stephen Warren <[email protected]>Signed-off-by Tom Rini <[email protected]>Quoted 7 lines
Quoted 4 lines
Quoted 1 line
Applied to u-boot/master, thanks!
Debian still only has 1.4.2 even in unstable, so u-boot/master no longer builds on any debian system.
Then perhaps we need to mirror the kernel and provide / build dtc, but allow for people to override it.
What about allowing old DTC version if DTOs are not in use ?
Quoted 7 lines
Quoted 4 lines
Quoted 2 lines
If you can re-work the logic for that, sure, thanks!
Ubuntu 16.04 LTS is still at dtc v1.4.0, so it's broken there too.
Maybe it's just time to embed a copy of dtc into u-boot (like the kernel, especially since we use so many features...)?
That is my intention at this point, and I'm testing a small series to do that now (the last pain point looks to have been the patch that allows 'DTC' to be over-ridden for FreeBSD).
Quoted 8 lines
Quoted 4 lines
cfr my comment about vendoring , I'd really hate to go down that path. Chrome does that and look what came out of it -- outdated patched packages in the chrome codebase, with debian trying hard to unbundle that stuff again.
Yes, but we're going to do it all the same. We're tied to some of the tools we need and vendors aren't keeping pace with them either. dtc 1.4.0 (Ubuntu 16.04) is from 2013. dtc 1.4.3 is from last December.
The new DTC version is only mandated by DTO support, which is not used by anyone to my knowledge at this point. Whoever wants to use DTO, should have new enough DTC and I'm fine with the build failing if they don't.
This heavy-handed approach of imposing custom DTC build on everyone because of a feature noone uses is awful and I disagree with it. If there is a version check for DTC for builds with and without DTO, fine, I can live with that.
Quoted 8 lines
But bundling DTC only because of unused functionality, NAK.
Quoted 2 lines
And I plan on following the Linux Kernel model of just syncing only directly with the upstream project.
See above, this is heavy-handed and IMO unnecessary. In fact, I hope the kernel removes the DTC fork completely.
As a debian stable user (at almost all my machines).... I would like to have the dtc 1.4.3 being used only for DTBO enabled.
Maybe, it would be possible to have such check added (and remove it when distros keep up with dtc version)?
Signature · 9 lines
Best regards, Lukasz Majewski -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
Quoted 48 lines · Marek Vasut
Quoted 47 lines · Tom Rini
Quoted 45 lines · Marek Vasut
Quoted 31 lines · Tom Rini
Quoted 30 lines · Robert Nelson
Quoted 29 lines · Bin Meng
Hi Tom,
Quoted 25 lines · Tom Rini
Quoted 14 lines · Marek Vasut
Quoted 13 lines · Tom Rini
Quoted 12 lines · Marek Vasut
Quoted 10 lines · Tom Rini
Quoted 6 lines · Tom Rini
With support for overlays and calling the -@ flag to dtc we need to have at least 1.4.3 available now.
Cc Simon Glass <[email protected]>Reported-by Stephen Warren <[email protected]>Signed-off-by Tom Rini <[email protected]>Applied to u-boot/master, thanks!
Quoted 7 lines
Quoted 5 lines
Quoted 2 lines
Debian still only has 1.4.2 even in unstable, so u-boot/master no longer builds on any debian system.
Then perhaps we need to mirror the kernel and provide / build dtc, but allow for people to override it.
What about allowing old DTC version if DTOs are not in use ?
If you can re-work the logic for that, sure, thanks!
Quoted 8 lines
Quoted 4 lines
Quoted 1 line
Ubuntu 16.04 LTS is still at dtc v1.4.0, so it's broken there too.
Maybe it's just time to embed a copy of dtc into u-boot (like the kernel, especially since we use so many features...)?
That is my intention at this point, and I'm testing a small series to do that now (the last pain point looks to have been the patch that allows 'DTC' to be over-ridden for FreeBSD).
cfr my comment about vendoring , I'd really hate to go down that path. Chrome does that and look what came out of it -- outdated patched packages in the chrome codebase, with debian trying hard to unbundle that stuff again.
Quoted 8 lines
Quoted 3 lines
Yes, but we're going to do it all the same. We're tied to some of the tools we need and vendors aren't keeping pace with them either. dtc 1.4.0 (Ubuntu 16.04) is from 2013. dtc 1.4.3 is from last December.
The new DTC version is only mandated by DTO support, which is not used by anyone to my knowledge at this point. Whoever wants to use DTO, should have new enough DTC and I'm fine with the build failing if they don't.
Why yes, there is a chicken-and-egg problem with DTO support and adoption. Happily, Google has picked it up as part of how things are to be done with Android 8 so it should finally see wider adoption.
Signature · 8 lines
Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: Digital signature URL: <http://lists.denx.de/pipermail/u-boot/attachments/20170924/d5fc7dc8/attachment.sig>
Quoted 53 lines · Tom Rini
Quoted 52 lines · Marek Vasut
Quoted 50 lines · Tom Rini
Quoted 37 lines · Marek Vasut
Quoted 36 lines · Tom Rini
Quoted 34 lines · Robert Nelson
Quoted 24 lines · Bin Meng
Hi Tom,
Quoted 21 lines · Tom Rini
Quoted 20 lines · Marek Vasut
Quoted 18 lines · Tom Rini
Quoted 6 lines · Marek Vasut
Quoted 5 lines · Tom Rini
Quoted 3 lines · Tom Rini
With support for overlays and calling the -@ flag to dtc we need to have
at least 1.4.3 available now.Quoted 8 lines
Quoted 5 lines
Quoted 3 lines
Cc: Simon Glass <[email protected]>
Reported-by: Stephen Warren <[email protected]>
Signed-off-by: Tom Rini <[email protected]>Applied to u-boot/master, thanks!
Debian still only has 1.4.2 even in unstable, so u-boot/master no longer builds on any debian system.
Then perhaps we need to mirror the kernel and provide / build dtc, but allow for people to override it.
Quoted 6 lines
Quoted 4 lines
Quoted 1 line
What about allowing old DTC version if DTOs are not in use ?
If you can re-work the logic for that, sure, thanks!
Ubuntu 16.04 LTS is still at dtc v1.4.0, so it's broken there too.
Maybe it's just time to embed a copy of dtc into u-boot (like the kernel, especially since we use so many features...)?
Quoted 8 lines
Quoted 3 lines
That is my intention at this point, and I'm testing a small series to do that now (the last pain point looks to have been the patch that allows 'DTC' to be over-ridden for FreeBSD).
cfr my comment about vendoring , I'd really hate to go down that path. Chrome does that and look what came out of it -- outdated patched packages in the chrome codebase, with debian trying hard to unbundle that stuff again.
Yes, but we're going to do it all the same. We're tied to some of the tools we need and vendors aren't keeping pace with them either. dtc 1.4.0 (Ubuntu 16.04) is from 2013. dtc 1.4.3 is from last December.
Quoted 9 lines
Quoted 4 lines
The new DTC version is only mandated by DTO support, which is not used by anyone to my knowledge at this point. Whoever wants to use DTO, should have new enough DTC and I'm fine with the build failing if they don't.
Why yes, there is a chicken-and-egg problem with DTO support and adoption. Happily, Google has picked it up as part of how things are to be done with Android 8 so it should finally see wider adoption.
The most important part of my reply was deleted, so let me repost it here:
This heavy-handed approach of imposing custom DTC build on everyone because of a feature noone uses is awful and I disagree with it. If there is a version check for DTC for builds with and without DTO, fine, I can live with that.
But bundling DTC only because of unused functionality, NAK.
Signature · 2 lines
Best regards, Marek Vasut
Hi Marek,
Quoted 63 lines · Marek Vasut
Quoted 53 lines · Tom Rini
Quoted 52 lines · Marek Vasut
Quoted 50 lines · Tom Rini
Quoted 37 lines · Marek Vasut
Quoted 36 lines · Tom Rini
Quoted 34 lines · Robert Nelson
Quoted 24 lines · Bin Meng
Hi Tom,
Quoted 21 lines · Tom Rini
Quoted 20 lines · Marek Vasut
Quoted 18 lines · Tom Rini
Quoted 6 lines · Marek Vasut
Quoted 5 lines · Tom Rini
On Fri, Sep 15, 2017 at 01:15:25PM -0400, Tom Rini wrote:
With support for overlays and calling the -@ flag to dtc we need to have
at least 1.4.3 available now.Quoted 8 lines
Quoted 5 lines
Cc: Simon Glass <[email protected]>
Reported-by: Stephen Warren <[email protected]>
Signed-off-by: Tom Rini <[email protected]>Applied to u-boot/master, thanks!
Debian still only has 1.4.2 even in unstable, so u-boot/master no longer builds on any debian system.
Then perhaps we need to mirror the kernel and provide / build dtc, but allow for people to override it.
Quoted 6 lines
Quoted 4 lines
Quoted 1 line
What about allowing old DTC version if DTOs are not in use ?
If you can re-work the logic for that, sure, thanks!
Ubuntu 16.04 LTS is still at dtc v1.4.0, so it's broken there too.
Maybe it's just time to embed a copy of dtc into u-boot (like the kernel, especially since we use so many features...)?
Quoted 8 lines
Quoted 3 lines
That is my intention at this point, and I'm testing a small series to do that now (the last pain point looks to have been the patch that allows 'DTC' to be over-ridden for FreeBSD).
cfr my comment about vendoring , I'd really hate to go down that path. Chrome does that and look what came out of it -- outdated patched packages in the chrome codebase, with debian trying hard to unbundle that stuff again.
Yes, but we're going to do it all the same. We're tied to some of the tools we need and vendors aren't keeping pace with them either. dtc 1.4.0 (Ubuntu 16.04) is from 2013. dtc 1.4.3 is from last December.
Quoted 9 lines
Quoted 4 lines
The new DTC version is only mandated by DTO support, which is not used by anyone to my knowledge at this point. Whoever wants to use DTO, should have new enough DTC and I'm fine with the build failing if they don't.
Why yes, there is a chicken-and-egg problem with DTO support and adoption. Happily, Google has picked it up as part of how things are to be done with Android 8 so it should finally see wider adoption.
Quoted 8 lines
The most important part of my reply was deleted, so let me repost it here:
This heavy-handed approach of imposing custom DTC build on everyone because of a feature noone uses is awful and I disagree with it. If there is a version check for DTC for builds with and without DTO, fine, I can live with that.
But bundling DTC only because of unused functionality, NAK.
Yes I suppose that is a bit painful.
Perhaps we should add a Makefile option (like we do with BUILD_ROM) to enable DTO (and require a certain dtc). That flag can be set with travis, but off by default.
Regards,
Simon
Quoted 73 lines · Simon Glass
Hi Marek,
Quoted 70 lines · Marek Vasut
Quoted 53 lines · Tom Rini
Quoted 52 lines · Marek Vasut
Quoted 50 lines · Tom Rini
Quoted 37 lines · Marek Vasut
Quoted 36 lines · Tom Rini
Quoted 34 lines · Robert Nelson
Quoted 24 lines · Bin Meng
Hi Tom,
Quoted 21 lines · Tom Rini
Quoted 20 lines · Marek Vasut
Quoted 18 lines · Tom Rini
Quoted 14 lines · Marek Vasut
On 09/22/2017 04:20 PM, Tom Rini wrote:
On Fri, Sep 15, 2017 at 01:15:25PM -0400, Tom Rini wrote:With support for overlays and calling the -@ flag to dtc we need to have
at least 1.4.3 available now.Cc: Simon Glass <[email protected]>
Reported-by: Stephen Warren <[email protected]>
Signed-off-by: Tom Rini <[email protected]>Applied to u-boot/master, thanks!
Debian still only has 1.4.2 even in unstable, so u-boot/master no longer
builds on any debian system.Then perhaps we need to mirror the kernel and provide / build dtc, but allow for people to override it.
Quoted 6 lines
Quoted 4 lines
Quoted 1 line
What about allowing old DTC version if DTOs are not in use ?
If you can re-work the logic for that, sure, thanks!
Ubuntu 16.04 LTS is still at dtc v1.4.0, so it's broken there too.
Maybe it's just time to embed a copy of dtc into u-boot (like the kernel, especially since we use so many features...)?
Quoted 8 lines
Quoted 3 lines
That is my intention at this point, and I'm testing a small series to do that now (the last pain point looks to have been the patch that allows 'DTC' to be over-ridden for FreeBSD).
cfr my comment about vendoring , I'd really hate to go down that path. Chrome does that and look what came out of it -- outdated patched packages in the chrome codebase, with debian trying hard to unbundle that stuff again.
Yes, but we're going to do it all the same. We're tied to some of the tools we need and vendors aren't keeping pace with them either. dtc 1.4.0 (Ubuntu 16.04) is from 2013. dtc 1.4.3 is from last December.
Quoted 9 lines
Quoted 4 lines
The new DTC version is only mandated by DTO support, which is not used by anyone to my knowledge at this point. Whoever wants to use DTO, should have new enough DTC and I'm fine with the build failing if they don't.
Why yes, there is a chicken-and-egg problem with DTO support and adoption. Happily, Google has picked it up as part of how things are to be done with Android 8 so it should finally see wider adoption.
The most important part of my reply was deleted, so let me repost it here:
This heavy-handed approach of imposing custom DTC build on everyone because of a feature noone uses is awful and I disagree with it. If there is a version check for DTC for builds with and without DTO, fine, I can live with that.
Quoted 7 lines
Quoted 1 line
But bundling DTC only because of unused functionality, NAK.
Yes I suppose that is a bit painful.
Perhaps we should add a
Makefileoption (like we do withBUILD_ROM) to enable DTO (and require a certain dtc). That flag can be set with travis, but off by default.
Yes, agreed, that was my suggestion all along.
Quoted 3 lines
Regards,
Simon
Signature · 2 lines
Best regards, Marek Vasut
Quoted 11 lines · Tom Rini
Quoted 10 lines · Marek Vasut
Quoted 9 lines · Tom Rini
Quoted 6 lines · Tom Rini
With support for overlays and calling the -@ flag to dtc we need to have at least 1.4.3 available now.
Cc Simon Glass <[email protected]>Reported-by Stephen Warren <[email protected]>Signed-off-by Tom Rini <[email protected]>
Quoted 7 lines
Quoted 4 lines
Quoted 1 line
Applied to u-boot/master, thanks!
Debian still only has 1.4.2 even in unstable, so u-boot/master no longer builds on any debian system.
Then perhaps we need to mirror the kernel and provide / build dtc, but allow for people to override it.
This isn't too hard and I'm pushing that through Jenkins now to confirm I've got all the logic correct. I think this will be the least error prone method long term.
Signature · 8 lines
Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: Digital signature URL: <http://lists.denx.de/pipermail/u-boot/attachments/20170923/8e494909/attachment.sig>
Marek Vasut
NAKQuoted 17 lines · Tom Rini
Quoted 16 lines · Tom Rini
Quoted 14 lines · Marek Vasut
Quoted 10 lines · Tom Rini
Quoted 6 lines · Tom Rini
With support for overlays and calling the -@ flag to dtc we need to have at least 1.4.3 available now.
Cc Simon Glass <[email protected]>Reported-by Stephen Warren <[email protected]>Signed-off-by Tom Rini <[email protected]>Applied to u-boot/master, thanks!
Debian still only has 1.4.2 even in unstable, so u-boot/master no longer builds on any debian system.
Quoted 7 lines
Quoted 2 lines
Then perhaps we need to mirror the kernel and provide / build dtc, but allow for people to override it.
This isn't too hard and I'm pushing that through Jenkins now to confirm I've got all the logic correct. I think this will be the least error prone method long term.
NAK, this is the same thing which chromium does and what Debian is trying to stop -- vendoring. We will end up with patched custom DTC out of sync with upstream.
Signature · 2 lines
Best regards, Marek Vasut
Quoted 21 lines · Marek Vasut
Quoted 20 lines · Tom Rini
Quoted 18 lines · Tom Rini
Quoted 6 lines · Marek Vasut
Quoted 5 lines · Tom Rini
Quoted 3 lines · Tom Rini
With support for overlays and calling the -@ flag to dtc we need to have at least 1.4.3 available now.
Quoted 8 lines
Quoted 5 lines
Quoted 3 lines
Cc Simon Glass <[email protected]>Reported-by Stephen Warren <[email protected]>Signed-off-by Tom Rini <[email protected]>Applied to u-boot/master, thanks!
Debian still only has 1.4.2 even in unstable, so u-boot/master no longer builds on any debian system.
Then perhaps we need to mirror the kernel and provide / build dtc, but allow for people to override it.
Quoted 7 lines
Quoted 3 lines
This isn't too hard and I'm pushing that through Jenkins now to confirm I've got all the logic correct. I think this will be the least error prone method long term.
NAK, this is the same thing which chromium does and what Debian is trying to stop -- vendoring. We will end up with patched custom DTC out of sync with upstream.
Shrug, it's what Linux does, and what QEMU does, and probably what other projects do as well since apparently even Debian/Unstable is 2 releases and nearly a year behind (or, 9 months if you don't want to exaggerate).
Signature · 8 lines
Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: Digital signature URL: <http://lists.denx.de/pipermail/u-boot/attachments/20170923/3992814e/attachment.sig>
Quoted 26 lines · Tom Rini
Quoted 25 lines · Marek Vasut
Quoted 23 lines · Tom Rini
Quoted 11 lines · Tom Rini
Quoted 10 lines · Marek Vasut
Quoted 9 lines · Tom Rini
Quoted 6 lines · Tom Rini
With support for overlays and calling the -@ flag to dtc we need to have at least 1.4.3 available now.
Cc Simon Glass <[email protected]>Reported-by Stephen Warren <[email protected]>Signed-off-by Tom Rini <[email protected]>Quoted 7 lines
Quoted 4 lines
Quoted 1 line
Applied to u-boot/master, thanks!
Debian still only has 1.4.2 even in unstable, so u-boot/master no longer builds on any debian system.
Then perhaps we need to mirror the kernel and provide / build dtc, but allow for people to override it.
This isn't too hard and I'm pushing that through Jenkins now to confirm I've got all the logic correct. I think this will be the least error prone method long term.
Quoted 7 lines
Quoted 3 lines
NAK, this is the same thing which chromium does and what Debian is trying to stop -- vendoring. We will end up with patched custom DTC out of sync with upstream.
Shrug, it's what Linux does, and what QEMU does, and probably what other projects do as well since apparently even Debian/Unstable is 2 releases and nearly a year behind (or, 9 months if you don't want to exaggerate).
And it will likely stay that way for debian stable. I am really disappointed in the "Shrug" reaction to such massive breakage.
Signature · 2 lines
Best regards, Marek Vasut
Quoted 31 lines · Marek Vasut
Quoted 30 lines · Tom Rini
Quoted 28 lines · Marek Vasut
Quoted 17 lines · Tom Rini
Quoted 16 lines · Tom Rini
Quoted 14 lines · Marek Vasut
Quoted 10 lines · Tom Rini
Quoted 6 lines · Tom Rini
With support for overlays and calling the -@ flag to dtc we need to have at least 1.4.3 available now.
Cc Simon Glass <[email protected]>Reported-by Stephen Warren <[email protected]>Signed-off-by Tom Rini <[email protected]>Applied to u-boot/master, thanks!
Debian still only has 1.4.2 even in unstable, so u-boot/master no longer builds on any debian system.
Quoted 6 lines
Quoted 2 lines
Then perhaps we need to mirror the kernel and provide / build dtc, but allow for people to override it.
This isn't too hard and I'm pushing that through Jenkins now to confirm I've got all the logic correct. I think this will be the least error prone method long term.
NAK, this is the same thing which chromium does and what Debian is trying to stop -- vendoring. We will end up with patched custom DTC out of sync with upstream.
Quoted 6 lines
Quoted 3 lines
Shrug, it's what Linux does, and what QEMU does, and probably what other projects do as well since apparently even Debian/Unstable is 2 releases and nearly a year behind (or, 9 months if you don't want to exaggerate).
And it will likely stay that way for debian stable. I am really disappointed in the "Shrug" reaction to such massive breakage.
Nope and nor will it catch old-stable breaking in January when gcc-6.x or newer is required.
Signature · 8 lines
Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: Digital signature URL: <http://lists.denx.de/pipermail/u-boot/attachments/20170923/2de3a684/attachment.sig>
Quoted 36 lines · Tom Rini
Quoted 35 lines · Marek Vasut
Quoted 33 lines · Tom Rini
Quoted 21 lines · Marek Vasut
Quoted 20 lines · Tom Rini
Quoted 18 lines · Tom Rini
Quoted 6 lines · Marek Vasut
Quoted 5 lines · Tom Rini
Quoted 3 lines · Tom Rini
With support for overlays and calling the -@ flag to dtc we need to have at least 1.4.3 available now.
Quoted 8 lines
Quoted 5 lines
Quoted 3 lines
Cc Simon Glass <[email protected]>Reported-by Stephen Warren <[email protected]>Signed-off-by Tom Rini <[email protected]>Applied to u-boot/master, thanks!
Debian still only has 1.4.2 even in unstable, so u-boot/master no longer builds on any debian system.
Then perhaps we need to mirror the kernel and provide / build dtc, but allow for people to override it.
Quoted 7 lines
Quoted 3 lines
This isn't too hard and I'm pushing that through Jenkins now to confirm I've got all the logic correct. I think this will be the least error prone method long term.
NAK, this is the same thing which chromium does and what Debian is trying to stop -- vendoring. We will end up with patched custom DTC out of sync with upstream.
Shrug, it's what Linux does, and what QEMU does, and probably what other projects do as well since apparently even Debian/Unstable is 2 releases and nearly a year behind (or, 9 months if you don't want to exaggerate).
Quoted 6 lines
Quoted 2 lines
And it will likely stay that way for debian stable. I am really disappointed in the "Shrug" reaction to such massive breakage.
Nope and nor will it catch old-stable breaking in January when gcc-6.x or newer is required.
Above you have been discussing code duplication. This is not the right way to work with git.
The git way is to add
https://git.kernel.org/pub/scm/utils/dtc/dtc.git
as git submodule (https://git-scm.com/docs/git-submodule).
In our make file we can then use git submodule init, git submodule update, git verify-tag before we build a dtc with the appropriate version.
This way we can build dtc from sources but stay in sync with the upstream.
Regards
Heinrich
Quoted 47 lines · Heinrich Schuchardt
Quoted 36 lines · Tom Rini
Quoted 35 lines · Marek Vasut
Quoted 33 lines · Tom Rini
Quoted 21 lines · Marek Vasut
Quoted 20 lines · Tom Rini
Quoted 18 lines · Tom Rini
Quoted 6 lines · Marek Vasut
Quoted 5 lines · Tom Rini
Quoted 3 lines · Tom Rini
With support for overlays and calling the -@ flag to dtc we need to have at least 1.4.3 available now.
Quoted 8 lines
Quoted 5 lines
Quoted 3 lines
Cc Simon Glass <[email protected]>Reported-by Stephen Warren <[email protected]>Signed-off-by Tom Rini <[email protected]>Applied to u-boot/master, thanks!
Debian still only has 1.4.2 even in unstable, so u-boot/master no longer builds on any debian system.
Then perhaps we need to mirror the kernel and provide / build dtc, but allow for people to override it.
Quoted 7 lines
Quoted 3 lines
This isn't too hard and I'm pushing that through Jenkins now to confirm I've got all the logic correct. I think this will be the least error prone method long term.
NAK, this is the same thing which chromium does and what Debian is trying to stop -- vendoring. We will end up with patched custom DTC out of sync with upstream.
Shrug, it's what Linux does, and what QEMU does, and probably what other projects do as well since apparently even Debian/Unstable is 2 releases and nearly a year behind (or, 9 months if you don't want to exaggerate).
Quoted 6 lines
Quoted 2 lines
And it will likely stay that way for debian stable. I am really disappointed in the "Shrug" reaction to such massive breakage.
Nope and nor will it catch old-stable breaking in January when gcc-6.x or newer is required.
Above you have been discussing code duplication. This is not the right way to work with git.
Quoted 8 lines
The git way is to add
https://git.kernel.org/pub/scm/utils/dtc/dtc.git
as git submodule (https://git-scm.com/docs/git-submodule).In our make file we can then use git submodule init, git submodule update, git verify-tag before we build a dtc with the appropriate version.
This way we can build dtc from sources but stay in sync with the upstream.
+1
Quoted 10 lines
Regards
Heinrich
_______________________________________________
U-Boot mailing list
U-Boot at lists.denx.de
https://lists.denx.de/listinfo/u-boot
Signature · 9 lines
Best regards, Lukasz Majewski -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
Quoted 47 lines · Heinrich Schuchardt
Quoted 36 lines · Tom Rini
Quoted 35 lines · Marek Vasut
Quoted 33 lines · Tom Rini
Quoted 21 lines · Marek Vasut
Quoted 20 lines · Tom Rini
Quoted 18 lines · Tom Rini
Quoted 6 lines · Marek Vasut
Quoted 5 lines · Tom Rini
Quoted 3 lines · Tom Rini
With support for overlays and calling the -@ flag to dtc we need to have at least 1.4.3 available now.
Quoted 8 lines
Quoted 5 lines
Quoted 3 lines
Cc Simon Glass <[email protected]>Reported-by Stephen Warren <[email protected]>Signed-off-by Tom Rini <[email protected]>Applied to u-boot/master, thanks!
Debian still only has 1.4.2 even in unstable, so u-boot/master no longer builds on any debian system.
Then perhaps we need to mirror the kernel and provide / build dtc, but allow for people to override it.
Quoted 7 lines
Quoted 3 lines
This isn't too hard and I'm pushing that through Jenkins now to confirm I've got all the logic correct. I think this will be the least error prone method long term.
NAK, this is the same thing which chromium does and what Debian is trying to stop -- vendoring. We will end up with patched custom DTC out of sync with upstream.
Shrug, it's what Linux does, and what QEMU does, and probably what other projects do as well since apparently even Debian/Unstable is 2 releases and nearly a year behind (or, 9 months if you don't want to exaggerate).
Quoted 6 lines
Quoted 2 lines
And it will likely stay that way for debian stable. I am really disappointed in the "Shrug" reaction to such massive breakage.
Nope and nor will it catch old-stable breaking in January when gcc-6.x or newer is required.
Above you have been discussing code duplication. This is not the right way to work with git.
Quoted 8 lines
The git way is to add
https://git.kernel.org/pub/scm/utils/dtc/dtc.git
as git submodule (https://git-scm.com/docs/git-submodule).In our make file we can then use git submodule init, git submodule update, git verify-tag before we build a dtc with the appropriate version.
This way we can build dtc from sources but stay in sync with the upstream.
I'm not wading into the submodules vs repo debate. Happily the Linux Kernel already came up with a small script to just sync in what we need of the tool sources and to keep said sources in sync. The only divergence there between strict upstream is cherry-picking (essentially) of a few changes, rather than just doing a re-sync of the tool).
Signature · 8 lines
Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: Digital signature URL: <http://lists.denx.de/pipermail/u-boot/attachments/20170924/552a4b66/attachment.sig>
Quoted 9 lines · Tom Rini
Quoted 8 lines · Marek Vasut
Quoted 7 lines · Tom Rini
Quoted 6 lines · Marek Vasut
Quoted 5 lines · Tom Rini
Quoted 4 lines · Tom Rini
Quoted 3 lines · Marek Vasut
Quoted 2 lines · Tom Rini
On Fri, Sep 15, 2017 at 01:15:25PM -0400, Tom Rini wrote:
Quoted 8 lines
Quoted 8 lines
Quoted 8 lines
Quoted 8 lines
Quoted 8 lines
Quoted 8 lines
Quoted 8 lines
Quoted 8 lines
Quoted 6 lines
With support for overlays and calling the -@ flag to dtc we need to have at least 1.4.3 available now.
Cc Simon Glass <[email protected]>Reported-by Stephen Warren <[email protected]>Signed-off-by Tom Rini <[email protected]>Applied to u-boot/master, thanks!
It is unclear to me is why this was so urgent to push through while there was ongoing discussion raising some reasonable concerns...
Quoted 10 lines
Quoted 10 lines
Quoted 10 lines
Quoted 10 lines
Quoted 9 lines
Quoted 5 lines
Quoted 2 lines
Debian still only has 1.4.2 even in unstable, so u-boot/master no longer builds on any debian system.
Then perhaps we need to mirror the kernel and provide / build dtc, but allow for people to override it.
This isn't too hard and I'm pushing that through Jenkins now to confirm I've got all the logic correct. I think this will be the least error prone method long term.
Quoted 7 lines
Quoted 7 lines
Quoted 7 lines
Quoted 3 lines
NAK, this is the same thing which chromium does and what Debian is trying to stop -- vendoring. We will end up with patched custom DTC out of sync with upstream.
Shrug, it's what Linux does, and what QEMU does, and probably what other projects do as well since apparently even Debian/Unstable is 2 releases and nearly a year behind (or, 9 months if you don't want to exaggerate).
Given that dtc releases were:
1.4.0 on Sat Jun 22 13:06:27 2013 -0500,
1.4.1 on Wed Nov 12 14:31:44 2014 +1100,
1.4.2 on Sat Sep 3 21:02:46 2016 +1000,
1.4.3 on Tue Feb 28 16:10:59 2017 +1100,
1.4.4 on Fri Mar 10 11:22:18 2017 +1100
At least two releases are over a year apart; It's not like dtc is a high-velocity train here...
Being "2 releases and nearly a year" behind doesn't seem all that unreasonable... you could just as easily, and more accurately, describe it as "only two releases and almost 7 months behind."
Quoted 5 lines
Quoted 2 lines
And it will likely stay that way for debian stable. I am really disappointed in the "Shrug" reaction to such massive breakage.
Nope and nor will it catch old-stable breaking in January when gcc-6.x or newer is required.
Debian's oldstable would be unreasonable, sure.
There's a big difference between requiring gcc-6, which is present in the *current* Debian stable release, which will be supported for another 2-5+ years...
and in constrast, dtc 1.4.3, which was released not quite 7 months ago, while Debian had already frozen new upstream versions in preparation for a release 3 months later. The timing for dtc 1.4.3 just wasn't good for Debian stable; not through any fault of the maintainers.
For what it's worth, I've submitted some updated packaging to the device-tree-compiler (a.k.a. dtc) maintainers in Debian, including updating the version. Hopefully that will get uploaded one way or another soon.
Given that Debian in general prefers not to embed code copies, with u-boot, I'm hoping it will be fairly straightforward to use an external dtc, once the newer dtc version is available in Debian?
live well,
vagrant
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 832 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20170924/1d6f8b01/attachment.sig>