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

Simon Glass
2016-04-21T13:35:07Z

Hi,

On 21 April 2016 at 07:25, Marek Vasut <marex@denx.de> wrote:
> On 04/21/2016 03:17 PM, Heiko Schocher wrote:
>> 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 ...
>
> In my opinion, we should stick to the same behavior Linux does.
> Ccing a few more people.

Wouldn't it be better to fix the problems?

Regards,
Simon