From patchwork Thu Mar 17 22:54:43 2016 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Rob Herring X-Patchwork-Id: 64011 Delivered-To: patch@linaro.org Received: by 10.112.199.169 with SMTP id jl9csp741978lbc; Thu, 17 Mar 2016 15:55:10 -0700 (PDT) X-Received: by 10.66.234.104 with SMTP id ud8mr18823765pac.143.1458255310298; Thu, 17 Mar 2016 15:55:10 -0700 (PDT) Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id 12si15260236pfk.55.2016.03.17.15.55.10; Thu, 17 Mar 2016 15:55:10 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of devicetree-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of devicetree-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=devicetree-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S934691AbcCQWzI (ORCPT + 7 others); Thu, 17 Mar 2016 18:55:08 -0400 Received: from mail.kernel.org ([198.145.29.136]:34596 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933365AbcCQWzG (ORCPT ); Thu, 17 Mar 2016 18:55:06 -0400 Received: from mail.kernel.org (localhost [127.0.0.1]) by mail.kernel.org (Postfix) with ESMTP id 781D520218; Thu, 17 Mar 2016 22:55:05 +0000 (UTC) Received: from mail-yw0-f180.google.com (mail-yw0-f180.google.com [209.85.161.180]) (using TLSv1.2 with cipher AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id ACE8F20212; Thu, 17 Mar 2016 22:55:03 +0000 (UTC) Received: by mail-yw0-f180.google.com with SMTP id g127so119014028ywf.2; Thu, 17 Mar 2016 15:55:03 -0700 (PDT) X-Gm-Message-State: AD7BkJJW1N+92hhkxj3bz8F6r6xAoVmDZnUv9DXyK3pdSJHNAnqJHNTnIDtZKAgn9H/5LIxoJLq+rqTvBQxIBw== X-Received: by 10.37.15.130 with SMTP id 124mr5624158ybp.84.1458255303037; Thu, 17 Mar 2016 15:55:03 -0700 (PDT) MIME-Version: 1.0 Received: by 10.13.229.68 with HTTP; Thu, 17 Mar 2016 15:54:43 -0700 (PDT) In-Reply-To: References: <1457104420-18350-1-git-send-email-robh@kernel.org> From: Rob Herring Date: Thu, 17 Mar 2016 17:54:43 -0500 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [PATCH] scripts/dtc: Update to upstream version 53bf130b1cdd To: Olof Johansson Cc: "devicetree@vger.kernel.org" , "linux-kernel@vger.kernel.org" X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00, UNPARSEABLE_RELAY autolearn=unavailable version=3.3.1 X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on mail.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP Sender: devicetree-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: devicetree@vger.kernel.org On Tue, Mar 8, 2016 at 10:22 AM, Olof Johansson wrote: > Hi, > > On Mon, Mar 7, 2016 at 11:37 PM, Rob Herring wrote: >> On Mon, Mar 7, 2016 at 5:10 PM, Olof Johansson wrote: >>> Hi, >>> >>> On Fri, Mar 4, 2016 at 7:13 AM, Rob Herring wrote: >>>> Sync to upstream dtc commit 53bf130b1cdd ("libfdt: simplify >>>> fdt_node_check_compatible()"). This adds the following commits from >>>> upstream: >>>> >>>> 53bf130 libfdt: simplify fdt_node_check_compatible() >>>> c9d9121 Warn on node name unit-address presence/absence mismatch >>>> 2e53f9d Catch unsigned 32bit overflow when parsing flattened device tree offsets >>>> >>>> Signed-off-by: Rob Herring >>>> --- >>>> As usual, this is just an automated copy of upstream dtc into the kernel >>>> tree. The changeset is small enough that I have left it here. >>>> >>>> The main reason for this sync is to pick-up the new unit-address >>>> warnings. >>> >>> This spews a crazy amount of warnings on a multi_v7_defconfig build. >> >> Shocking, huh? And I've got more checks in the works. :) >> >>> I'd prefer to see most of those warnings fixed _before_ we introduce >>> it by default. Otherwise we just add a huge amount of noise that will >>> hide any real valid warnings that are now brought up. >> >> How do you propose to do that? If it is not enabled, then no one will >> see them nor care. I don't intend to fix everyone's stuff myself. > > Right, but pushing a change on everyone that makes the build > near-unusable as a tool to see if you're introducing a new error is > also not a good way to do it. However: > >> We could hide the check behind COMPILE_TEST perhaps. > > Putting it behind an option sounds like a suitable approach. Would it > be possible to plumb in under "make C=1" / "make C=2" instead? That's > closer in meaning to what you're doing here than COMPILE_TEST, which > is more about turning on drivers that might not make sense on your > platform to get compile coverage. C=1/2 is only designed to run on C files, so it would be a bit of work to extend it. However, "W=1" seems like a better match and easy to add: > This would make people aware of the tool, give them an easy way to run > it to do the cleanups. Nag maintainers when they post new code without > having paid attention to the old sources/cleanup, and with some amount > of time most of it might be fixed. I added the ability to do "make DTC=my/dtc", so there already is a way for people to run it if they want to. Of course, they'd have to fetch and build dtc compared to the above. Still, with either option it is dependent on people running something they don't likely run normally. Rob -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html diff --git a/scripts/Makefile.lib b/scripts/Makefile.lib index ad50d58..417d8c2 100644 --- a/scripts/Makefile.lib +++ b/scripts/Makefile.lib @@ -271,6 +271,10 @@ cmd_gzip = (cat $(filter-out FORCE,$^) | gzip -n -f -9 > $@) || \ # --------------------------------------------------------------------------- DTC ?= $(objtree)/scripts/dtc/dtc +ifeq ($(KBUILD_ENABLE_EXTRA_GCC_CHECKS),) +DTC_FLAGS += -Wno-unit_address_vs_reg +endif + # Generate an assembly file to wrap the output of the device tree compiler quiet_cmd_dt_S_dtb= DTB $@ cmd_dt_S_dtb= \