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 <[email protected]> > >>> --- > >>> 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
signature.asc
Description: Digital signature
_______________________________________________ U-Boot mailing list [email protected] http://lists.denx.de/mailman/listinfo/u-boot

