[U-Boot] Makefile: Update minimum dtc version to 1.4.3

[Marek Vasut]

On 09/24/2017 02:52 PM, Tom Rini wrote:
> On Sun, Sep 24, 2017 at 02:07:27PM +0200, Marek Vasut wrote:
>> On 09/24/2017 01:36 PM, Tom Rini wrote:
>>> On Sun, Sep 24, 2017 at 10:28:18AM +0200, Marek Vasut wrote:
>>>> On 09/24/2017 05:22 AM, Tom Rini wrote:
>>>>> On Sat, Sep 23, 2017 at 09:07:02PM -0500, Robert Nelson wrote:
>>>>>> On Sat, Sep 23, 2017 at 8:49 PM, Bin Meng <bmeng.cn at gmail.com> wrote:
>>>>>>> Hi Tom,
>>>>>>>
>>>>>>> On Sun, Sep 24, 2017 at 12:10 AM, Tom Rini <trini at konsulko.com> wrote:
>>>>>>>> On Sat, Sep 23, 2017 at 03:04:39PM +0200, Marek Vasut wrote:
>>>>>>>>> On 09/23/2017 02:56 PM, Tom Rini wrote:
>>>>>>>>>> On Sat, Sep 23, 2017 at 12:39:44PM +0200, Marek Vasut wrote:
>>>>>>>>>>> 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 <sjg at chromium.org>
>>>>>>>>>>>>> Reported-by: Stephen Warren <swarren at wwwdotorg.org>
>>>>>>>>>>>>> Signed-off-by: Tom Rini <trini at konsulko.com>
>>>>>>>>>>>>
>>>>>>>>>>>> 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 ?
>>>>>>>>
>>>>>>>> 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).
>>>>
>>>> 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.
> 
> 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.

-- 
Best regards,
Marek Vasut