[U-Boot] [PATCH] travis-ci: Temporarily disable using a newer device tree compiler

Heiko Schocher hs at denx.de
Mon Feb 27 05:29:41 UTC 2017


Hello Robert,

Am 27.02.2017 um 05:34 schrieb Robert Nelson:
> On Sun, Feb 26, 2017 at 7:37 PM, Tom Rini <trini at konsulko.com> wrote:
>> On Sun, Feb 26, 2017 at 08:01:19PM -0500, Tom Rini wrote:
>>
>>> For a long while dtc has warned about various constructs.  This is now
>>> leading to log file size being exceeded in travis, and as the majority
>>> of these errors need to be fixed in the kernel, switch to using the
>>> stock device-tree-compiler package.
>>>
>>> Signed-off-by: Tom Rini <trini at konsulko.com>
>>
>> Applied to u-boot/master, thanks!
>>
>> But I'm not happy to have to apply it, either.  I do not wish to break
>> travis-ci, but sun7i is now exceeding the allowed stdout size.  And
>> these are problems that need to be fixed in the kernel first, rather
>> than here first.  The patches to fix u-boot local issues (such as x86
>> never got picked up either, but likely need to be respun.  I expect that
>> once the new DTC version is tagged and populated upstream in the kernel
>> as well, we will be able to re-enable the changes we had here.
>
> This was fixed (aka a hidden bandaid) in the linux kernel via:
>
> https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/scripts/Makefile.lib?id=bc553986a2f7c56d0de811485d5312ea29692d5d

Heh... IIRC, I tried to fix a lot of this messages (in U-Boot), around
a year ago, and we decided to not apply them in u-boot, instead they
should go away via kernel resyncs, when the warnings are fixed in the
kernel!

Also I think, I tried a similiar approach as the commit you mentioned...
Ah, yes, here:
http://lists.denx.de/pipermail/u-boot/2016-April/252408.html

We decided (for u-boot) it would be better to fix the problems and
not supress the warning, but it seems, the problems get not really fixed
in the kernel ...

I wasn;t aware of the above patch in the kernel... thanks for pointing!

@Tom:
So, I feel, the problems get not fixed fast in the kernel (if ever),
may we want to cherry-pick this patch from the kernel (or make a
similiar patch for U-Boot)?

bye,
Heiko
-- 
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany


More information about the U-Boot mailing list