From patchwork Wed Nov 22 22:12:32 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Rob Herring X-Patchwork-Id: 746137 Authentication-Results: smtp.subspace.kernel.org; dkim=none Received: from mail-io1-f49.google.com (mail-io1-f49.google.com [209.85.166.49]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 1FDB01B9; Wed, 22 Nov 2023 14:12:43 -0800 (PST) Received: by mail-io1-f49.google.com with SMTP id ca18e2360f4ac-7a93df91813so6579539f.2; Wed, 22 Nov 2023 14:12:43 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1700691162; x=1701295962; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=hU6Wnghj5KH1Ld3lLkNyhwgfAXJhDGyB0CCvRi7u8mk=; b=UthVPCrkPmuaSKUpj8hiQFvad2Bn3BhipezxF0q9MXsmZXF2YcmG67xt4d0EwJcAlt 1EGHrqmIS+C3trWjV+xrjV4+9QUxOzTmZ05uo0hlD/B7JSjAiLa026Tm62EmkbEda06t WEJk2EH85L7uFCpVtYQq/HFNuCoPQBCRgYecP5JKIbfu+v//01SyhKoA1f+f2HvjO/oL CSLwdjWd1S+RCnfJOMXHnHgVlmx7ayvUGGcqdkocLFPtW+u39r7HQMN8Rab2+0i8hnKz sza2ZakkiflBqx+x1KHtUkJmItDpQxZs0YSlT3q5a/8f6fs+pUblaILtz8AY2++7+5/Q INiA== X-Gm-Message-State: AOJu0YzL8w2dcOTnHFBHPGQZTRKtumb2Ljq4zhuVhxU5Kme2gs/ZYh27 jKitkDsZ8Z6nUMFEQugiY5x37Nh92w== X-Google-Smtp-Source: AGHT+IF8osr8TMYr2K7RbdJl1f0Qs0a8Vvy9ayPPg6W5HhHI5G6ri/6sDydnhfguLO7f2AvVQEKSig== X-Received: by 2002:a92:908:0:b0:359:d397:c806 with SMTP id y8-20020a920908000000b00359d397c806mr4100582ilg.18.1700691162319; Wed, 22 Nov 2023 14:12:42 -0800 (PST) Received: from herring.priv ([64.188.179.252]) by smtp.gmail.com with ESMTPSA id r3-20020a92cd83000000b00357ce043118sm159221ilb.79.2023.11.22.14.12.40 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 22 Nov 2023 14:12:41 -0800 (PST) Received: (nullmailer pid 2753836 invoked by uid 1000); Wed, 22 Nov 2023 22:12:33 -0000 From: Rob Herring Date: Wed, 22 Nov 2023 15:12:32 -0700 Subject: [PATCH v2 1/4] kbuild: Move dtc graph_child_address warning to W=2 Precedence: bulk X-Mailing-List: devicetree@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Message-Id: <20231122-dtc-warnings-v2-1-bd4087325392@kernel.org> References: <20231122-dtc-warnings-v2-0-bd4087325392@kernel.org> In-Reply-To: <20231122-dtc-warnings-v2-0-bd4087325392@kernel.org> To: Masahiro Yamada , Nathan Chancellor , Nick Desaulniers , Nicolas Schier , Krzysztof Kozlowski , Conor Dooley , Paul Walmsley , Palmer Dabbelt , Albert Ou , Krzysztof Kozlowski , Alim Akhtar , Conor Dooley Cc: linux-kbuild@vger.kernel.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, linux-riscv@lists.infradead.org, linux-arm-kernel@lists.infradead.org, linux-samsung-soc@vger.kernel.org X-Mailer: b4 0.13-dev There are cases where having a single child node with a unit-address is valid. For example, other child nodes may be optional or added via a DT overlay. This hasn't mattered until now, but as platforms become free of W=1 warnings, we want to enable W=1 by default on a per platform basis. So move the warning to W=2. In the process, the logic can be reworked a bit to avoid specifying a given warning in multiple places. Signed-off-by: Rob Herring --- scripts/Makefile.lib | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/scripts/Makefile.lib b/scripts/Makefile.lib index 68d0134bdbf9..a67f781ae8ee 100644 --- a/scripts/Makefile.lib +++ b/scripts/Makefile.lib @@ -340,8 +340,6 @@ quiet_cmd_gzip = GZIP $@ # DTC # --------------------------------------------------------------------------- DTC ?= $(objtree)/scripts/dtc/dtc -DTC_FLAGS += -Wno-interrupt_provider \ - -Wno-unique_unit_address # Disable noisy checks by default ifeq ($(findstring 1,$(KBUILD_EXTRA_WARN)),) @@ -357,9 +355,11 @@ endif ifneq ($(findstring 2,$(KBUILD_EXTRA_WARN)),) DTC_FLAGS += -Wnode_name_chars_strict \ - -Wproperty_name_chars_strict \ - -Winterrupt_provider \ - -Wunique_unit_address + -Wproperty_name_chars_strict +else +DTC_FLAGS += -Wno-interrupt_provider \ + -Wno-unique_unit_address \ + -Wno-graph_child_address endif DTC_FLAGS += $(DTC_FLAGS_$(basetarget)) From patchwork Wed Nov 22 22:12:33 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Rob Herring X-Patchwork-Id: 746136 Authentication-Results: smtp.subspace.kernel.org; dkim=none Received: from mail-io1-f54.google.com (mail-io1-f54.google.com [209.85.166.54]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 93881D53; Wed, 22 Nov 2023 14:12:48 -0800 (PST) Received: by mail-io1-f54.google.com with SMTP id ca18e2360f4ac-7b350130c3fso6485339f.3; Wed, 22 Nov 2023 14:12:48 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1700691168; x=1701295968; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=SvKVScUhvKKHq5yH2cV8DV0lJfU3kh2MuS6ZYH4ebh8=; b=HiFRz5NnDAw1OP/A8dBJikYWxNB6udr/LmxBA+xh6y6ObJQxLUFsF++WmF7XMkT6NU U4srEvw8Wn7/T+qqbFdUU7CALMDXGALWdNwHJZxnTd6oEwlrEx4RicSGoFwrmVb+H2sJ nPRe5TYdx59t4LWVAaQea4Kzcq0/4icA2XRPrAu5sUg0e8eJTBzG1pRaUiPInCVuADLe lXxh1NvMeLloZcr5NRSd4KM8ymrgfZ/UdrcE7KUHSt4MSBUOcwD5wVQBAf6itQiWJVZU 5Vdzv71F0CY4M1Lcakdk4UUE3MV4hjvBTT145+GMnN/E7w7VyUUQGSUY9bVopYtu8w3G FHJQ== X-Gm-Message-State: AOJu0YzrahSUME/P+8BubZTGpuQAGnnQs3pC6N7u6k+YOrGNUWvTNS0h t+3pAnw+fHxKaqPJf5fH9w== X-Google-Smtp-Source: AGHT+IGLO9xpcHV8HSJx4kSfogE9LNdREnwsjAbOU8PWDbp45WItqfhi9+N7Ml1bG8W7Xsf1CyrYyw== X-Received: by 2002:a05:6602:2c96:b0:792:8d16:91ef with SMTP id i22-20020a0566022c9600b007928d1691efmr4004136iow.18.1700691167790; Wed, 22 Nov 2023 14:12:47 -0800 (PST) Received: from herring.priv ([64.188.179.252]) by smtp.gmail.com with ESMTPSA id q10-20020a02a30a000000b0043a0aa909bfsm83341jai.159.2023.11.22.14.12.45 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 22 Nov 2023 14:12:47 -0800 (PST) Received: (nullmailer pid 2753838 invoked by uid 1000); Wed, 22 Nov 2023 22:12:33 -0000 From: Rob Herring Date: Wed, 22 Nov 2023 15:12:33 -0700 Subject: [PATCH v2 2/4] kbuild: Allow arch/platform override of dtc warning level Precedence: bulk X-Mailing-List: devicetree@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Message-Id: <20231122-dtc-warnings-v2-2-bd4087325392@kernel.org> References: <20231122-dtc-warnings-v2-0-bd4087325392@kernel.org> In-Reply-To: <20231122-dtc-warnings-v2-0-bd4087325392@kernel.org> To: Masahiro Yamada , Nathan Chancellor , Nick Desaulniers , Nicolas Schier , Krzysztof Kozlowski , Conor Dooley , Paul Walmsley , Palmer Dabbelt , Albert Ou , Krzysztof Kozlowski , Alim Akhtar , Conor Dooley Cc: linux-kbuild@vger.kernel.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, linux-riscv@lists.infradead.org, linux-arm-kernel@lists.infradead.org, linux-samsung-soc@vger.kernel.org X-Mailer: b4 0.13-dev In order to allow enabling extra dtc warnings on a per arch or platform basis, define a new variable, KBUILD_EXTRA_WARN_DTC, which contains the warning level and can be set by an arch or platform independently of KBUILD_EXTRA_WARN. The default value is KBUILD_EXTRA_WARN as before. The platform setting and command line setting are merged. For example, with a platform that defaults to W=1, one can run 'make W=2 dtbs' which will enable both level 1 and 2 warnings. Signed-off-by: Rob Herring --- scripts/Makefile.lib | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/scripts/Makefile.lib b/scripts/Makefile.lib index a67f781ae8ee..53a74e53e0ca 100644 --- a/scripts/Makefile.lib +++ b/scripts/Makefile.lib @@ -341,8 +341,10 @@ quiet_cmd_gzip = GZIP $@ # --------------------------------------------------------------------------- DTC ?= $(objtree)/scripts/dtc/dtc +KBUILD_EXTRA_WARN_DTC += $(KBUILD_EXTRA_WARN) + # Disable noisy checks by default -ifeq ($(findstring 1,$(KBUILD_EXTRA_WARN)),) +ifeq ($(findstring 1,$(KBUILD_EXTRA_WARN_DTC)),) DTC_FLAGS += -Wno-unit_address_vs_reg \ -Wno-avoid_unnecessary_addr_size \ -Wno-alias_paths \ @@ -353,7 +355,7 @@ DTC_FLAGS += \ -Wunique_unit_address_if_enabled endif -ifneq ($(findstring 2,$(KBUILD_EXTRA_WARN)),) +ifneq ($(findstring 2,$(KBUILD_EXTRA_WARN_DTC)),) DTC_FLAGS += -Wnode_name_chars_strict \ -Wproperty_name_chars_strict else