From mboxrd@z Thu Jan 1 00:00:00 1970 From: Heiko Schocher Date: Mon, 27 Feb 2017 06:29:41 +0100 Subject: [U-Boot] [PATCH] travis-ci: Temporarily disable using a newer device tree compiler In-Reply-To: References: <1488157279-23171-1-git-send-email-trini@konsulko.com> <20170227013720.GH20531@bill-the-cat> Message-ID: <58B3B945.8070702@denx.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hello Robert, Am 27.02.2017 um 05:34 schrieb Robert Nelson: > On Sun, Feb 26, 2017 at 7:37 PM, Tom Rini 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 >> >> 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