From: Heiko Schocher <hs@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] travis-ci: Temporarily disable using a newer device tree compiler
Date: Mon, 27 Feb 2017 06:29:41 +0100 [thread overview]
Message-ID: <58B3B945.8070702@denx.de> (raw)
In-Reply-To: <CAOCHtYj_EdOnUcmiAWZ2sb1BLivLGy=Xmm3UiGx5aaQ=AVFppw@mail.gmail.com>
Hello Robert,
Am 27.02.2017 um 05:34 schrieb Robert Nelson:
> On Sun, Feb 26, 2017 at 7:37 PM, Tom Rini <trini@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@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
next prev parent reply other threads:[~2017-02-27 5:29 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-02-27 1:01 [U-Boot] [PATCH] travis-ci: Temporarily disable using a newer device tree compiler Tom Rini
2017-02-27 1:37 ` Tom Rini
2017-02-27 4:34 ` Robert Nelson
2017-02-27 5:29 ` Heiko Schocher [this message]
2017-02-27 5:52 ` Robert Nelson
2017-02-27 15:46 ` Tom Rini
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=58B3B945.8070702@denx.de \
--to=hs@denx.de \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox