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

Tom Rini
2016-04-21T13:33:39Z

On Thu, Apr 21, 2016 at 03:25:20PM +0200, Marek Vasut 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.

Agreed.

--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20160421/306e97a8/attachment.sig>