On 04/21/2016 03:35 PM, Simon Glass wrote: > Hi, > > On 21 April 2016 at 07:25, Marek Vasut <ma...@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 <h...@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 _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot