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 15:50:28 +0200 [thread overview]
Message-ID: <5718DAA4.8020208@denx.de> (raw)
In-Reply-To: <CAPnjgZ1XHmUZr8t0=32NUH3517ZX=GDZ5g4SRD9dgTWr1XrGhA@mail.gmail.com>
On 04/21/2016 03:35 PM, Simon Glass wrote:
> 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?
My impression was that these warnings are just the result of
over-eagerness of DTC, that's why Linux prints them only if you
increase the W= (warning) verbosity. I might be wrong tho.
--
Best regards,
Marek Vasut
next prev parent reply other threads:[~2016-04-21 13:50 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
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 [this message]
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=5718DAA4.8020208@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.