[U-Boot] [RFC PATCH] kbuild: turn of dtc unit address warnings by default
Masahiro Yamada
yamada.masahiro at socionext.com
Tue Mar 7 17:09:46 UTC 2017
2017-02-27 15:24 GMT+09:00 Masahiro Yamada <yamada.masahiro at 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.masahiro at 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 the necessity for this patch comes down to
how long we expect for fixing DT files.
The only difficulty is in our basic rule:
Send patches to Linux first, then import DT to U-Boot.
Without this, fixing the warnings would be pretty easy...
--
Best Regards
Masahiro Yamada
More information about the U-Boot
mailing list