[U-Boot] [PATCH 4/8] travis-ci: Do not make buildman warnings fatal

Tom Rini
2016-10-21T11:19:00Z

On Fri, Oct 21, 2016 at 09:39:28AM +0200, Heiko Schocher wrote:
> Hello Tom,
>
> Am 20.10.2016 um 21:26 schrieb Tom Rini:
> >We currently will always see a number of warnings due to device tree
> >issues. These (and other warnings) should not make the build be marked
> >as failure so catch exit status 129 specifically and return 0 in that
> >case.
> >
> >Cc: Heiko Schocher <hs@denx.de>
> >Signed-off-by: Tom Rini <trini@konsulko.com>
>
> You mean here this "Warning (unit_address_vs_reg):" warnings?

And a few others as well, depending on toolchain.

> IIRC I posted some month ago a patch, which suppress them ... the
> patch was rejected, with the plausible reason, that we should fix
> this issues ... so I started and posted some dts fix patches, also
> rejected with the reason that this should come from linux syncs ...
>
> Hmm.. do we syncs for our DTS files with linux?

Yes. I suppose that for example for x86/sandbox we should take your
patches. But yes, for other platforms where we are not the
authoritative source the fixes need to be pushed to Linux first and
merged in. Some platforms sync more often with upstream than others in
this regard.

> Nevertheless: we should at least fix this for travis, so:
>
> Reviewed-by: Heiko Schocher <hs@denx.de>

Thanks!

--
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/20161021/d7b90dc0/attachment.sig>