On Tue, Mar 21, 2017 at 2:15 PM, Olof Johansson <olof@xxxxxxxxx> wrote: > On Tue, Mar 21, 2017 at 9:19 AM, Rob Herring <robh@xxxxxxxxxx> wrote: >> This series syncs dtc with current mainline. The primary motivation is >> to pull in the new checks I've worked on. This gives lots of new >> warnings which are turned off by default. >> >> Arm-soc folks, I've left the PCI checks enabled as they are pretty much >> all real errors and there aren't that many (about 1200, but that's lots >> of duplicates). I have a patch for some of them. > > Yay for better warnings! > > But as with all changes of this kind, when new warnings are introduced > please give people some time to remove them without turning them on by > default. We had a cycle like that already. Which I quickly fixed and learned my lesson. That's why I tried getting this out earlier as well. > Main reason is that it becomes too hard to spot new warnings when they > come in if the logs are flooded with 1200 existing warnings. Yeah. Since almost everything is an include, the warnings really multiply. > How many of them do you have patches for, per chance? If we can see > them show up quickly the answer to the above might be a bit different. Okay, I lied or lost them... It looks like it is 79 unique warnings for arm. I'll take a stab at fixing them. Rob -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
