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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.