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

[Simon Glass]

Hi,

On 21 April 2016 at 07:25, Marek Vasut <marex at 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 at 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