On Tue, Mar 7, 2017 at 6:10 PM, Tom Rini <tr...@konsulko.com> wrote: > On Wed, Mar 08, 2017 at 02:09:46AM +0900, Masahiro Yamada wrote: >> 2017-02-27 15:24 GMT+09:00 Masahiro Yamada <yamada.masah...@socionext.com>: >> > DTC 1.4.2 or later checks DT unit-address without reg property and >> > vice-versa, and generates lots of warnings. Fixing DT files will >> > take for a while. Until then, let's turn off the check unless >> > building with W=*. >> > >> > Introduce a new helper dtc-option to check if the option is supported >> > in order to suppress warnings on older versions. >> > >> > Signed-off-by: Masahiro Yamada <yamada.masah...@socionext.com> >> > --- >> > >> > This is one possible answer to Tom's: >> > https://www.mail-archive.com/u-boot@lists.denx.de/msg240328.html >> > >> > He fixed the problem on travis-ci by commit a0f3e3d, but it is still >> > annoying during the regular development. >> > Perhaps we may want to hide the warnings (at least hidden in Linux >> > Kernel by default). >> > Or, is it better to keep it noisy >> > to motivate people to fix their DT files? >> > I am not quite sure... >> > >> > Now I am sending this as RFC patch in case people may want to start >> > discussion. >> > >> > BTW, this is a counter-part of the patch I sent to the Kbuild sub-system >> > (https://patchwork.kernel.org/patch/9592747/) >> > because I want the U-Boot build system with Linux as much as possible. >> > Let's see if I will get possible opinions in the Kbuild review. >> >> I sent a similar patch for Linux, but >> the response was negative in the Kbuild subsystem. >> >> So, this patch will be our own, if applied. > > I think we should be proactive here and take this patch. Newer dtc will > be seen more widely I believe as various overlay related changes go in > upstream. Do you have anything you would change in a v2 non-RFC?
For reference they updated the kernel included dtc to upstream commit 0931cea3ba20 in 4.11 for overlay support [1] so I similarly updated the Fedora dtc to the same revision for consistency so I think this is a pragmatic approach. Peter [1] http://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=6f05afcbb031722ec1eff77dde188ff2edf8940e _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
