[U-Boot] [RFC PATCH] kbuild: turn of dtc unit address warnings by default

Simon Glass sjg at chromium.org
Fri Mar 3 04:52:56 UTC 2017


On 26 February 2017 at 23:24, Masahiro Yamada
<yamada.masahiro at socionext.com> wrote:
> 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.
>
>
>  Makefile                   | 2 +-
>  scripts/Kbuild.include     | 5 +++++
>  scripts/Makefile.extrawarn | 6 ++++++
>  3 files changed, 12 insertions(+), 1 deletion(-)

Seems like a good idea to me, at least until more are converted.

Reviewed-by: Simon Glass <sjg at chromium.org>


More information about the U-Boot mailing list