From patchwork Tue Feb 25 18:48:21 2025 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Krzysztof Kozlowski X-Patchwork-Id: 13990588 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id B6A6AC021B2 for ; Tue, 25 Feb 2025 19:31:53 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:Content-Transfer-Encoding: MIME-Version:Message-ID:Date:Subject:Cc:To:From:Reply-To:Content-Type: Content-ID:Content-Description:Resent-Date:Resent-From:Resent-Sender: Resent-To:Resent-Cc:Resent-Message-ID:In-Reply-To:References:List-Owner; bh=xonXqxvKrBNVv9VbDpJk9UUk6NvAaABwloKGPZx8qJg=; b=NwOfCkN4zA74hd6VV1KgpdmIZp vkSQ8LbGYlzttetOPt01vFUC4m4gBUZ9e7raOpmOTmkx9/m9E2jLIUPTx1L0HQx6EGp945Upr6HjH WXKUodqWvEr58Y7SwIzbOMn/iSH8u22u1gUxfcWW+kr+TiuMqOWysmB8WWbNPmsHlEk/q7a2nlr1N JvXaAjNlBL+HIK6sii6aDi51Hi+wYiP+sT0j9p1HP1u+PcSNNdALIBe71vcDvzgG3NIX/rN32C510 T1J7OlecYHkKeQ4WiHAC6JwR8mGGD0B0VLQ04WLr8VE6uTIEgTcefS5iLqRu2fqZEv4vR8XhroQTC EHlg8Cug==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98 #2 (Red Hat Linux)) id 1tn0ef-000000019LP-1zmu; Tue, 25 Feb 2025 19:31:45 +0000 Received: from mail-ej1-x635.google.com ([2a00:1450:4864:20::635]) by bombadil.infradead.org with esmtps (Exim 4.98 #2 (Red Hat Linux)) id 1tmzym-000000011O8-0nI5 for linux-arm-kernel@lists.infradead.org; Tue, 25 Feb 2025 18:48:29 +0000 Received: by mail-ej1-x635.google.com with SMTP id a640c23a62f3a-abb9fac29d8so90797066b.0 for ; Tue, 25 Feb 2025 10:48:27 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1740509306; x=1741114106; darn=lists.infradead.org; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:from:to:cc:subject:date:message-id:reply-to; bh=xonXqxvKrBNVv9VbDpJk9UUk6NvAaABwloKGPZx8qJg=; b=uNa9RDAG0y06rKzqngLNX73+TQ0GYx8uf+DL1iEDxewTE5WtmjXn0t5zlo4kmSNR29 tqVppC26/M7LP98Z0FkH3u6pRCi7+TENH5Ff0Nj1YBOrafSWKdShu5pb5JRpPwP0SUMC SwCNqAabQ8C9D8xwCtElULnEuS5xklWMewz+oyjwGmTjJhgp89BLCV8AxpgodwtqChAZ PMnjVjpm/UqD0Fwttl6Fizlg84j6atxH4JYtzidMcn06zlG9bYdhoattFmmG/e6lYkbA ZOS2tyNX40eBLdWdyEqwKWCwRulDuHpOcjMlXdymaQDPRsrVAFwC96iIOtITk1uH/eYJ J1Nw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1740509306; x=1741114106; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=xonXqxvKrBNVv9VbDpJk9UUk6NvAaABwloKGPZx8qJg=; b=D1LuH+a/mQpg8p+uEYdW3ef5LsZWSUocGSKe2tE67cGE5NBMuHCVDYc+IlYF+KF675 VCLnyRrZW6+Vh+Gsx9irqwwTEOi7FaX6qcoOSASVAekSnwVajYN+cOfEFgMw8+lfLkp6 HCvJ2H0e4z4Vw+b0N++VMJhWsCVvlsbsytxrr6+ylq1IiAJNl68WBXIfVlLvsj4+V/SW rP7nqJgVLTHyEwauAifqj7SoMudm4vubehaSSxIv2D2Wn4Y99RTAmCVJzIoBfFlDmoqL amXWG1TZBmDQIOYtyf7LFyjAK2UAjCQiGq/pibygwrD5a0EAOoRWr9VSjkVyC8pmHmf9 zlPw== X-Forwarded-Encrypted: i=1; AJvYcCVpLVfHYaw4jX8AdMViAvHZnHjjNSg8R7ldDl85e8nabZ2Tcl3QySBRO0geMbmSBshgl594mFR7AbmM28HCwre4@lists.infradead.org X-Gm-Message-State: AOJu0YwD2ga370L5obsEvG/je8Y5SpUxituEDyUvLftEp3r5DKnp9R5u oZcDGAXG7Lbea+wIY5fAPRVqA4kiClEAa9CIcUftrEDKgt1Y7/r3f9NLi+RZvXc= X-Gm-Gg: ASbGncvto/s7/cywnkFQJzDOOGrU63VmT/jD7OynXjrPXiRJD1Wq5iGmW0xqrOSzj1V qB4dlx/sKfl8H1BZhwV/iMVXttBRPYjuF6gCUfBEGDzKA2Tl8/dwZi2DhpEMqDZlGYfNbPueTFP Qa13y2WcJiMMjQ3K+za07Dc1rQbBbe1doKKyoV5dG54/eEb7uKz0Plzb4yasi9nr7G3mJCk8pxD Nn0fv9CsUSjpEiVwBa9Tz+XCvlbTT+Sn3MOqnkotVPHqwnf3zR8XWd7ONvAlqx/Xua8ySUhZohd 2rjIsw2KS+EQKA0mZfg2sX85PeWF+q4BLb1fDSyQWmrfXewX/nDtU60VsKZZsOvW/Mvxgy+4YRg = X-Google-Smtp-Source: AGHT+IHgg2NfoJNIoVPeDVM+ZICkn7h+QLq/e1GcetAUMNdHAqmG1KBd1VC5Ah7e3zEn4CVbWsAd/g== X-Received: by 2002:a17:907:1c0d:b0:ab7:c28f:de59 with SMTP id a640c23a62f3a-abc09d2deb1mr534837166b.13.1740509306244; Tue, 25 Feb 2025 10:48:26 -0800 (PST) Received: from krzk-bin.. (78-11-220-99.static.ip.netia.com.pl. [78.11.220.99]) by smtp.gmail.com with ESMTPSA id a640c23a62f3a-abed1cdbf55sm185359466b.15.2025.02.25.10.48.24 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 25 Feb 2025 10:48:25 -0800 (PST) From: Krzysztof Kozlowski To: Rob Herring , Krzysztof Kozlowski , Conor Dooley , Arnd Bergmann , Jonathan Corbet , devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, soc@lists.linux.dev, workflows@vger.kernel.org, linux-doc@vger.kernel.org Cc: Krzysztof Kozlowski Subject: [PATCH 1/2] docs: dt: submitting-patches: Document sending DTS patches Date: Tue, 25 Feb 2025 19:48:21 +0100 Message-ID: <20250225184822.213296-1-krzysztof.kozlowski@linaro.org> X-Mailer: git-send-email 2.43.0 MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20250225_104828_227153_D525A033 X-CRM114-Status: GOOD ( 17.37 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Document two rules already widely used and enforced by DT maintainers and SoC platform maintainers: 1. DTS patches should be placed at the end of driver patchset to indicate no dependencies of driver code on DTS. 2. DTS patches should be applied via SoC platform maintainers, because it is a driver-independent hardware description. However some driver maintainers are reluctant to pick up portions of patchsets and prefer to take entire set at once. For such cases, the DTS portion should be split into separate patchset, so it will not end up in the driver subsystem integration tree. Signed-off-by: Krzysztof Kozlowski Reviewed-by: Laurent Pinchart --- .../bindings/submitting-patches.rst | 19 +++++++++++++++++-- 1 file changed, 17 insertions(+), 2 deletions(-) diff --git a/Documentation/devicetree/bindings/submitting-patches.rst b/Documentation/devicetree/bindings/submitting-patches.rst index a64f21a5f299..f3e23e69a638 100644 --- a/Documentation/devicetree/bindings/submitting-patches.rst +++ b/Documentation/devicetree/bindings/submitting-patches.rst @@ -54,11 +54,22 @@ I. For patch submitters followed as of commit bff5da4335256513497cc8c79f9a9d1665e09864 ("checkpatch: add DT compatible string documentation checks"). ] - 7) If a documented compatible string is not yet matched by the + 7) DTS is treated in general as driver-independent hardware description, thus + any DTS patches, regardless whether using existing or new bindings, should + be placed at the end of patchset to indicate no dependency of drivers on + the DTS. DTS will be anyway applied through separate tree or branch, so + different order would indicate the serie is non-bisectable. + + If a driver subsystem maintainer prefers to apply entire set, instead of + their relevant portion of patchset, please split the DTS patches into + separate patchset with a reference in changelog or cover letter to the + bindings submission on the mailing list. + + 8) If a documented compatible string is not yet matched by the driver, the documentation should also include a compatible string that is matched by the driver. - 8) Bindings are actively used by multiple projects other than the Linux + 9) Bindings are actively used by multiple projects other than the Linux Kernel, extra care and consideration may need to be taken when making changes to existing bindings. @@ -79,6 +90,10 @@ II. For kernel maintainers 3) For a series going though multiple trees, the binding patch should be kept with the driver using the binding. + 4) The DTS files should however never be applied via driver subsystem tree, + but always via platform SoC trees on dedicated branches (see also + Documentation/process/maintainer-soc.rst). + III. Notes ========== From patchwork Tue Feb 25 18:48:22 2025 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Krzysztof Kozlowski X-Patchwork-Id: 13990589 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id B3533C021BB for ; Tue, 25 Feb 2025 19:31:55 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:Content-Transfer-Encoding: MIME-Version:References:In-Reply-To:Message-ID:Date:Subject:Cc:To:From: Reply-To:Content-Type:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=/oN23gi/Cv1G/Zd8ve35VhHoD43rI+bSAAj2I1PX/Ag=; b=YxrCC9wHZlTTbqAzpuyjT2WanW 2wLdNQ9EpQCXD15hXBi7A8W3s9RHzH5lSFC3T5WCFQALoihFf9mVl9WE444qi6zWZHEAXUNR4U9ZP irBHi9JB+Xpu+TIHZFSRFNuEmfeqK5QTkYcwtn2T463WlKoUlF0uiy9rMQVRUStj0eZ/6J8yS8gYn PPBYzAJpHYhQyG42ny3lTayNnAP3Zah/AKXotZlP0d08FZwnn5sHDYJc08ODaM95cDndXv43WcsbO D1hPx45/K0xMGogn9Fb6gAjdpmOPDl3eht3pAJluZHriBTDiuLuMmnRLimluogiJkueNQ3/d/rn8S 3hHZpRuA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98 #2 (Red Hat Linux)) id 1tn0eg-000000019My-1244; Tue, 25 Feb 2025 19:31:46 +0000 Received: from mail-ej1-x62e.google.com ([2a00:1450:4864:20::62e]) by bombadil.infradead.org with esmtps (Exim 4.98 #2 (Red Hat Linux)) id 1tmzyn-000000011OP-272g for linux-arm-kernel@lists.infradead.org; Tue, 25 Feb 2025 18:48:30 +0000 Received: by mail-ej1-x62e.google.com with SMTP id a640c23a62f3a-abb9c8c4e59so99524966b.1 for ; Tue, 25 Feb 2025 10:48:29 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1740509308; x=1741114108; darn=lists.infradead.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=/oN23gi/Cv1G/Zd8ve35VhHoD43rI+bSAAj2I1PX/Ag=; b=GYp0+5YHRNOmiWtm3KiLC5ukfwvlsMrqlRhAdPSqIs2Ue4ng8NY5Ozmmub8VA5BtsV tGn0yDvR2RoQpRTBG+LSmDU15fJ8TDCMt22borWP39AB0MtkbO19RQpi2MSVyH1kO9Bf 4IciQvbbkPefJyOC+oOcPdJujTU+1Or21AFIeuFwdOKrcCz8VaYtG2pGl9PszReYx/aH ZJiyLVtuWzddigxPqPfwNY4M5nFDxHyEKKDFcigA5fN9X993oS9Er3crZH2ofJdRRdVb CWK+jMaj84xlm/KhGA1NxQTsmzMaI4+0jFBsONJYV5qVc16fM17veOP24qVttCiSKwmv 3brg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1740509308; x=1741114108; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=/oN23gi/Cv1G/Zd8ve35VhHoD43rI+bSAAj2I1PX/Ag=; b=OI3PBCyCs9ocrYsYTeBxC27cygzH4JxI1Pg8AnY45YH7VNbkYy4kx0Lyt5ULl4gzNS W/VP/TXcDg+DAKcYjRYjd5nth2CrwUMMJ5oqsEGHJ0CQa7mdHtsop3KYJxoRf5aSDOtK /5JT3pTb+36BKZOSfbzYvvxxnfRQs9gS+i0JH6SqcRBKW0N9pWmG0AGeJFTZ/khE1QtH OO0CxoZCCTXBm4PAEeX3cpmsRuMmgWyQJ3MMhBDOAbJ5Ox/mSOKwTV5ZQQCUBdRy5keE SumszU1m4A4WrdfR1KNZ0d2dwokZsmfF+0kR5/AfTvjTMag4AWseJQOE9KKRa2Kl9dcq 98ew== X-Forwarded-Encrypted: i=1; AJvYcCWkeV7RQJskeGd6zjVXzLI2+c+crf8ZKv200Rw/kdDCqwweB0GghgG6iR0gr7fmpTLHJYnR86VL84+za7j9ftx7@lists.infradead.org X-Gm-Message-State: AOJu0YzlretvhdDZSsLBaATuyB8MsEae6E6ST6u8Y5K+lZ+mWHv7UBso OqqJEtQZVxM599Un4TeWe9eG9bIHlykNP3DpK8uHOB/oW0CXmyHXKCX6mrhcgrE= X-Gm-Gg: ASbGncs2XIxoJBSQWHMHysFXodwtpO3R9OySTx9pbE38urIbovMG6fCUuESzDLPvkK7 2Ino81ryqG9+KI0pBTH0wAQiLQ72TDN1uuj/aO/8sY8DSVFEGx1rUPBZ+t59TzSDkabEuE4yOR8 uw0D8avtFRgI/okm/7mhfS9yCxuPc+i7WGrQH/W0ktUH6KCO4JE+vFGmVFDNZZec5YzR5BGt/zj cItKYuzFGpgNhD6j0wXw6OMwF6v74UxkDEZI/jGUxD1vkradf0RNnjfUglwUqO+hzHwmfazBCSb OOeSjhRTiFQg9dn7osvM/Ls5tNvdxMIykGH7qP6dXLAJETbpJakzZFaZHySrL/TfMBacz5mQSWY = X-Google-Smtp-Source: AGHT+IFKOZhRHtEHCCTN5BHVYVCPkFDsRkb7jCM4fs2ECH+kp2S3Jaa2qQ2qzrnOEoKDVZtT2N63uw== X-Received: by 2002:a17:907:2d20:b0:ab6:b8e0:4f25 with SMTP id a640c23a62f3a-abc09a14d0dmr792407366b.4.1740509307824; Tue, 25 Feb 2025 10:48:27 -0800 (PST) Received: from krzk-bin.. (78-11-220-99.static.ip.netia.com.pl. [78.11.220.99]) by smtp.gmail.com with ESMTPSA id a640c23a62f3a-abed1cdbf55sm185359466b.15.2025.02.25.10.48.26 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 25 Feb 2025 10:48:27 -0800 (PST) From: Krzysztof Kozlowski To: Rob Herring , Krzysztof Kozlowski , Conor Dooley , Arnd Bergmann , Jonathan Corbet , devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, soc@lists.linux.dev, workflows@vger.kernel.org, linux-doc@vger.kernel.org Cc: Krzysztof Kozlowski Subject: [PATCH 2/2] docs: process: maintainer-soc-clean-dts: linux-next is decisive Date: Tue, 25 Feb 2025 19:48:22 +0100 Message-ID: <20250225184822.213296-2-krzysztof.kozlowski@linaro.org> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20250225184822.213296-1-krzysztof.kozlowski@linaro.org> References: <20250225184822.213296-1-krzysztof.kozlowski@linaro.org> MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20250225_104829_540277_2C3C543E X-CRM114-Status: GOOD ( 12.94 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Devicetree bindings patches go usually via driver subsystem tree, so obviously testing only SoC branches would result in new dtbs_check warnings. Mention that linux-next branch is decisice for zero-warnings rule. Signed-off-by: Krzysztof Kozlowski --- Documentation/process/maintainer-soc-clean-dts.rst | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/Documentation/process/maintainer-soc-clean-dts.rst b/Documentation/process/maintainer-soc-clean-dts.rst index 1b32430d0cfc..5423fb7d6047 100644 --- a/Documentation/process/maintainer-soc-clean-dts.rst +++ b/Documentation/process/maintainer-soc-clean-dts.rst @@ -17,8 +17,9 @@ Strict DTS DT Schema and dtc Compliance No changes to the SoC platform Devicetree sources (DTS files) should introduce new ``make dtbs_check W=1`` warnings. Warnings in a new board DTS, which are results of issues in an included DTSI file, are considered existing, not new -warnings. The platform maintainers have automation in place which should point -out any new warnings. +warnings. For series split between different trees (DT bindings go via driver +subsystem tree), warnings on linux-next are decisive. The platform maintainers +have automation in place which should point out any new warnings. If a commit introducing new warnings gets accepted somehow, the resulting issues shall be fixed in reasonable time (e.g. within one release) or the