All of lore.kernel.org
 help / color / mirror / Atom feed
From: Marek Vasut <marex@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] travis: turn off dtc unit address warnings
Date: Thu, 21 Apr 2016 14:51:20 +0200	[thread overview]
Message-ID: <5718CCC8.5010806@denx.de> (raw)
In-Reply-To: <1461242884-5282-1-git-send-email-hs@denx.de>

On 04/21/2016 02:48 PM, Heiko Schocher wrote:
> suppress a lot of
> "reg or ranges property, but no unit name" warnings,
> through the dtc compiler flag "-Wno-unit_address_vs_reg".
> 
> If all DTS are fixed, we can remove this flag again.
> 
> Signed-off-by: Heiko Schocher <hs@denx.de>
> ---
> There is also a solution to suppress warnings from
> the dtc compiler by the "-q" flag, but that would
> suppress all warnings. Not realy what I want.

Yep

> With this patch and patch:
> http://patchwork.ozlabs.org/patch/609150/
> 
> travis build passes, see:
> https://travis-ci.org/hsdenx/u-boot/builds/124723016
> 
> 
>  arch/arc/dts/Makefile        | 2 ++
>  arch/arm/dts/Makefile        | 3 ++-
>  arch/microblaze/dts/Makefile | 2 ++
>  arch/mips/dts/Makefile       | 3 ++-
>  arch/nios2/dts/Makefile      | 2 ++
>  arch/powerpc/dts/Makefile    | 2 ++
>  arch/sandbox/dts/Makefile    | 2 ++
>  arch/x86/dts/Makefile        | 2 ++
>  8 files changed, 16 insertions(+), 2 deletions(-)

Isn't there some common place in scripts/ or so where we can disable
this warning using an one-liner ?

> diff --git a/arch/arc/dts/Makefile b/arch/arc/dts/Makefile
> index d4772ec..de8a1e1 100644
> --- a/arch/arc/dts/Makefile
> +++ b/arch/arc/dts/Makefile
> @@ -9,6 +9,8 @@ dtb-$(CONFIG_TARGET_TB100) +=  abilis_tb100.dtb
>  targets += $(dtb-y)
>  
>  DTC_FLAGS += -R 4 -p 0x1000
> +# Disable noisy checks by default
> +DTC_FLAGS += -Wno-unit_address_vs_reg
>  
>  PHONY += dtbs
>  dtbs: $(addprefix $(obj)/, $(dtb-y))
> diff --git a/arch/arm/dts/Makefile b/arch/arm/dts/Makefile
> index aa31fd9..e2da4c6 100644
> --- a/arch/arm/dts/Makefile
> +++ b/arch/arm/dts/Makefile
> @@ -234,7 +234,8 @@ dtb-$(CONFIG_SOC_KEYSTONE) += k2hk-evm.dtb \
>  targets += $(dtb-y)
>  
>  # Add any required device tree compiler flags here
> -DTC_FLAGS +=
> +# Disable noisy checks by default
> +DTC_FLAGS += -Wno-unit_address_vs_reg
>  
>  PHONY += dtbs
>  dtbs: $(addprefix $(obj)/, $(dtb-y))
> diff --git a/arch/microblaze/dts/Makefile b/arch/microblaze/dts/Makefile
> index 5e70d9e..0ccd3ee 100644
> --- a/arch/microblaze/dts/Makefile
> +++ b/arch/microblaze/dts/Makefile
> @@ -7,6 +7,8 @@ dtb-y += microblaze-generic.dtb
>  targets += $(dtb-y)
>  
>  DTC_FLAGS += -R 4 -p 0x1000
> +# Disable noisy checks by default
> +DTC_FLAGS += -Wno-unit_address_vs_reg
>  
>  PHONY += dtbs
>  dtbs: $(addprefix $(obj)/, $(dtb-y))
> diff --git a/arch/mips/dts/Makefile b/arch/mips/dts/Makefile
> index b513918..c254942 100644
> --- a/arch/mips/dts/Makefile
> +++ b/arch/mips/dts/Makefile
> @@ -7,7 +7,8 @@ dtb-$(CONFIG_TARGET_PIC32MZDASK) += pic32mzda_sk.dtb
>  targets += $(dtb-y)
>  
>  # Add any required device tree compiler flags here
> -DTC_FLAGS +=
> +# Disable noisy checks by default
> +DTC_FLAGS += -Wno-unit_address_vs_reg
>  
>  PHONY += dtbs
>  dtbs: $(addprefix $(obj)/, $(dtb-y))
> diff --git a/arch/nios2/dts/Makefile b/arch/nios2/dts/Makefile
> index f22fabf..7af6e39 100644
> --- a/arch/nios2/dts/Makefile
> +++ b/arch/nios2/dts/Makefile
> @@ -7,6 +7,8 @@ dtb-y += $(CONFIG_DEFAULT_DEVICE_TREE:"%"=%).dtb
>  targets += $(dtb-y)
>  
>  DTC_FLAGS += -R 4 -p 0x1000
> +# Disable noisy checks by default
> +DTC_FLAGS += -Wno-unit_address_vs_reg
>  
>  PHONY += dtbs
>  dtbs: $(addprefix $(obj)/, $(dtb-y))
> diff --git a/arch/powerpc/dts/Makefile b/arch/powerpc/dts/Makefile
> index 80b4c0c..908155e 100644
> --- a/arch/powerpc/dts/Makefile
> +++ b/arch/powerpc/dts/Makefile
> @@ -9,6 +9,8 @@ dtb-$(CONFIG_TARGET_XILINX_PPC405_GENERIC) += xilinx-ppc405-generic.dtb
>  targets += $(dtb-y)
>  
>  DTC_FLAGS += -R 4 -p 0x1000
> +# Disable noisy checks by default
> +DTC_FLAGS += -Wno-unit_address_vs_reg
>  
>  PHONY += dtbs
>  dtbs: $(addprefix $(obj)/, $(dtb-y))
> diff --git a/arch/sandbox/dts/Makefile b/arch/sandbox/dts/Makefile
> index 0197569..ba94157 100644
> --- a/arch/sandbox/dts/Makefile
> +++ b/arch/sandbox/dts/Makefile
> @@ -8,6 +8,8 @@ dtb-$(CONFIG_UT_DM) += test.dtb
>  targets += $(dtb-y)
>  
>  DTC_FLAGS += -R 4 -p 0x1000
> +# Disable noisy checks by default
> +DTC_FLAGS += -Wno-unit_address_vs_reg
>  
>  PHONY += dtbs
>  dtbs: $(addprefix $(obj)/, $(dtb-y))
> diff --git a/arch/x86/dts/Makefile b/arch/x86/dts/Makefile
> index 23156bb..562b21a 100644
> --- a/arch/x86/dts/Makefile
> +++ b/arch/x86/dts/Makefile
> @@ -19,6 +19,8 @@ dtb-y += bayleybay.dtb \
>  targets += $(dtb-y)
>  
>  DTC_FLAGS += -R 4 -p 0x1000
> +# Disable noisy checks by default
> +DTC_FLAGS += -Wno-unit_address_vs_reg
>  
>  PHONY += dtbs
>  dtbs: $(addprefix $(obj)/, $(dtb-y))
> 


-- 
Best regards,
Marek Vasut

  reply	other threads:[~2016-04-21 12:51 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-04-21 12:48 [U-Boot] [PATCH] travis: turn off dtc unit address warnings Heiko Schocher
2016-04-21 12:51 ` Marek Vasut [this message]
2016-04-21 13:17   ` Heiko Schocher
2016-04-21 13:25     ` Marek Vasut
2016-04-21 13:33       ` Tom Rini
2016-04-21 13:35       ` Simon Glass
2016-04-21 13:50         ` Marek Vasut
2016-04-21 13:53           ` Tom Rini
2016-04-21 14:11             ` Heiko Schocher
2016-04-21 16:20               ` 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=5718CCC8.5010806@denx.de \
    --to=marex@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.