Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1758302AbdCUUF4 (ORCPT ); Tue, 21 Mar 2017 16:05:56 -0400 Received: from mail.kernel.org ([198.145.29.136]:51598 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757383AbdCUUFy (ORCPT ); Tue, 21 Mar 2017 16:05:54 -0400 MIME-Version: 1.0 In-Reply-To: References: <20170321161958.9658-1-robh@kernel.org> From: Rob Herring Date: Tue, 21 Mar 2017 14:55:22 -0500 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [PATCH 0/3] dtc updates for 4.12 To: Olof Johansson Cc: "devicetree@vger.kernel.org" , "linux-kernel@vger.kernel.org" , Frank Rowand , Mark Rutland , "arm@kernel.org" Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1330 Lines: 31 On Tue, Mar 21, 2017 at 2:15 PM, Olof Johansson wrote: > On Tue, Mar 21, 2017 at 9:19 AM, Rob Herring 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