[U-Boot] [PATCH] travis: turn off dtc unit address warnings

Heiko Schocher
2016-04-21T13:17:35Z

Hello Marek,

Am 21.04.2016 um 14:51 schrieb Marek Vasut:
> 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 ?

I don;t know ... but I prefer to disable this per arch .. so we can
enable the check back if one arch is fixed ...

bye,
Heiko
>
>> 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))
>>
>
>

--
DENX Software Engineering GmbH, Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany