From mboxrd@z Thu Jan 1 00:00:00 1970 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 E66DD109B46D for ; Tue, 31 Mar 2026 14:09:08 +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: Content-Type:Subject:References:In-Reply-To:Message-Id:Cc:To:From:Date: MIME-Version:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=+wB5qVtq/fKVMAqMqoSITrLfao0bHO9kdmdL6ob1esQ=; b=yjD+eZDyMHCPCryYZw1ikCYkLP bSd1nIYKdg6liPofgAi6wsBvjVeNB1tVpE03TWi7+p8nRRbFYLbs5Iv5yYpruj68F6bWmR7OVXKwr flNzK/jDu6r3jt6uICfIz+No14yBMsye7JgAn+LGk+Zcc4JaJuoru1Q0Hd4SkpZfTqRfntKIP2O9V /b5C7LqfRXTHmtEYp7dGEjD7fPQmeirndJHaMv8OKRZ8oyCmKXS2Ia1VHM8gWgarzh9mmqQe7eMQ2 rkZ5bd7p0uRn5cKuTYPoEz/AqIH6GGCL0CViKFo8mleE6l+/8cm7H2xcXkBsT6Oev2kUjIkk3/DHo HbFr/YQw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98.2 #2 (Red Hat Linux)) id 1w7Zm9-0000000D2tu-1rwa; Tue, 31 Mar 2026 14:09:01 +0000 Received: from fout-b8-smtp.messagingengine.com ([202.12.124.151]) by bombadil.infradead.org with esmtps (Exim 4.98.2 #2 (Red Hat Linux)) id 1w7Zm5-0000000D2tM-32yC for linux-arm-kernel@lists.infradead.org; Tue, 31 Mar 2026 14:08:59 +0000 Received: from phl-compute-04.internal (phl-compute-04.internal [10.202.2.44]) by mailfout.stl.internal (Postfix) with ESMTP id 120971D0018F; Tue, 31 Mar 2026 10:08:53 -0400 (EDT) Received: from phl-imap-02 ([10.202.2.81]) by phl-compute-04.internal (MEProxy); Tue, 31 Mar 2026 10:08:54 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=arndb.de; h=cc :cc:content-transfer-encoding:content-type:content-type:date :date:from:from:in-reply-to:in-reply-to:message-id:mime-version :references:reply-to:subject:subject:to:to; s=fm1; t=1774966132; x=1775052532; bh=+wB5qVtq/fKVMAqMqoSITrLfao0bHO9kdmdL6ob1esQ=; b= B1LCww5SHprChvjdDhQI4LHZjDChEB2tZO8cjxh8WpbQZRShsreuUnnSTnlL3Ahn QCrNsUp35Pbx5nswQcBH0oZhDglNNa32RyMScKs8kvUZY0NiWEIoza0NNxuNW0Ld d4vNO5k9+oSPNTeUj1BfIIypgD1x0Shljb/8jBYd+qUE4KBzUiUVC6UHHOk1ioBj 25q1L9bl+kdWJa00VZeUGVC5hiQw+7lA8K/aOc7s2tR6wNweOpvcbHPQPRC0z2sl FTGPKJAohRBpJQu99nfNqaTWRzhKWLqz6JMiA0Sv5wo0ChMf/hVp2VvR8W6cSCUU 8C/g2dF9xSKEkjFusZhxwA== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:cc:content-transfer-encoding :content-type:content-type:date:date:feedback-id:feedback-id :from:from:in-reply-to:in-reply-to:message-id:mime-version :references:reply-to:subject:subject:to:to:x-me-proxy :x-me-sender:x-me-sender:x-sasl-enc; s=fm2; t=1774966132; x= 1775052532; bh=+wB5qVtq/fKVMAqMqoSITrLfao0bHO9kdmdL6ob1esQ=; b=C WbrwQ7WOlPJbXrTJ39oQjfRWzqdsAAO2jn+orvIp2vh9bj7yOlLP9iN29/e+QcL/ S3Wb5lvEgQP9NasSAud8hCBJxB1lGiMfqrteZG/cGMdTXS68WEw9skfIjPrKRzR3 MLrEVx7ZtkDjNj2rn4fQJqi6gdg9vKbBALveUnvvZWpn/gSxVNB4acU++9Zj0Gz4 1fvlr+1Ny37Z5DwbVCp7IDRHRDWtFT5VvDjavE633c34iluwLsta379T6CHNXfQ8 xTZvclmKmt8y9JJyDyvsSrW/wwiO4bOPY9cjloZ17BvhNJNpfI4sD6DJompSOucg 0NVQIObVTrRP17AVwka/A== X-ME-Sender: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgeefhedrtddtgdegtdcutefuodetggdotefrodftvf curfhrohhfihhlvgemucfhrghsthforghilhdpuffrtefokffrpgfnqfghnecuuegrihhl ohhuthemuceftddtnecusecvtfgvtghiphhivghnthhsucdlqddutddtmdenucfjughrpe foggffhffvvefkjghfufgtgfesthejredtredttdenucfhrhhomhepfdetrhhnugcuuegv rhhgmhgrnhhnfdcuoegrrhhnugesrghrnhgusgdruggvqeenucggtffrrghtthgvrhhnpe efhfehteffuddvgfeigefhjeetvdekteekjeefkeekleffjeetvedvgefhhfeihfenucff ohhmrghinhepkhgvrhhnvghlrdhorhhgnecuvehluhhsthgvrhfuihiivgeptdenucfrrg hrrghmpehmrghilhhfrhhomheprghrnhgusegrrhhnuggsrdguvgdpnhgspghrtghpthht ohepfedvpdhmohguvgepshhmthhpohhuthdprhgtphhtthhopegsrhhglhessghguggvvh drphhlpdhrtghpthhtoheptghhvghsthgvrheivdehudehsehgmhgrihhlrdgtohhmpdhr tghpthhtohepfhgvshhtvghvrghmsehgmhgrihhlrdgtohhmpdhrtghpthhtoheptghonh horhdoughtsehkvghrnhgvlhdrohhrghdprhgtphhtthhopehkrhiikhdoughtsehkvghr nhgvlhdrohhrghdprhgtphhtthhopehkrhiikheskhgvrhhnvghlrdhorhhgpdhrtghpth htoheplhgvvgeskhgvrhhnvghlrdhorhhgpdhrtghpthhtoheprhgrfhgrvghlsehkvghr nhgvlhdrohhrghdprhgtphhtthhopehrohgshheskhgvrhhnvghlrdhorhhg X-ME-Proxy: Feedback-ID: i56a14606:Fastmail Received: by mailuser.phl.internal (Postfix, from userid 501) id DE24A700065; Tue, 31 Mar 2026 10:08:51 -0400 (EDT) X-Mailer: MessagingEngine.com Webmail Interface MIME-Version: 1.0 X-ThreadId: ABMmOHXEXk43 Date: Tue, 31 Mar 2026 16:08:31 +0200 From: "Arnd Bergmann" To: "Khristine Andreea Barbulescu" , "Krzysztof Kozlowski" , "Ghennadi Procopciuc" Cc: "Linus Walleij" , "Bartosz Golaszewski" , "Krzysztof Kozlowski" , "Conor Dooley" , "Chester Lin" , "Matthias Brugger" , "Ghennadi Procopciuc" , "Larisa Grigore" , "Lee Jones" , "Shawn Guo" , "Sascha Hauer" , "Fabio Estevam" , "Aisheng Dong" , "Jacky Bai" , "Greg Kroah-Hartman" , "Rafael J . Wysocki" , "Alberto Ruiz" , "Christophe Lizzi" , devicetree@vger.kernel.org, "Enric Balletbo" , "Eric Chanudet" , imx@lists.linux.dev, linux-arm-kernel@lists.infradead.org, "open list:GPIO SUBSYSTEM" , linux-kernel@vger.kernel.org, "NXP S32 Linux Team" , "Pengutronix Kernel Team" , "Vincent Guittot" , "Rob Herring" Message-Id: <0666ab85-60ef-4eb9-81fb-636c604653fe@app.fastmail.com> In-Reply-To: References: <20260120115923.3463866-1-khristineandreea.barbulescu@oss.nxp.com> <20260120115923.3463866-2-khristineandreea.barbulescu@oss.nxp.com> <20260121021913.GA1704619-robh@kernel.org> <7d200097-51bc-4404-be8b-f536d0ecfc25@oss.nxp.com> <21531cdd-5ab9-493e-a722-61b98117e2c4@kernel.org> <22a5a072-847e-4cfd-8abd-e37163f73265@oss.nxp.com> <3c454da1-d949-4258-87ce-8b545000bf01@app.fastmail.com> <5f1b651b-1064-4280-a7e0-b7d66c396cde@oss.nxp.com> <4c46909d-641b-4389-bc4a-29394cb1d46d@oss.nxp.com> Subject: Re: [PATCH v8 01/10] dt-bindings: mfd: add support for the NXP SIUL2 module Content-Type: text/plain Content-Transfer-Encoding: 7bit X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20260331_070858_195529_AE410379 X-CRM114-Status: GOOD ( 21.07 ) 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 On Tue, Mar 31, 2026, at 15:43, Khristine Andreea Barbulescu wrote: > On 3/31/2026 1:11 PM, Arnd Bergmann wrote: > > Our initial intention had been to expose that SoC-information as > discussed in the earlier revisions of this series. However, > taking the review feedback into account, the current direction is > to stop handling those SoC information registers in the Linux driver > altogether and instead rely on a boot firmware to pass that > information forward, as you suggested. > > With this approach, the SIUL2 driver would no longer be responsible > for any separate SoC-information functionality. In that case, > I understand your point that a monolithic pinctrl/GPIO/irqchip > driver is a better fit than keeping the MFD structure. Ok > However, as you mentioned, this is still weird because it means > listing individual register areas of the larger device inside. > > For this reason, I was wondering whether it would still be > acceptable to move forward with the new binding introduced > in this series, but simplify it so that it describes a single > monolithic SIUL2 pinctrl/GPIO device instead of an MFD, > following the example node I included in my previous reply [1]. > > [1] > https://lore.kernel.org/linux-gpio/20260120115923.3463866-4-khristineandreea.barbulescu@oss.nxp.com/T/#m778088251774a15bde7463350d6e75d5e9b9b57d I can't think of a justification for making this an incompatible binding change, if the new "nxp,s32g-siul2-pinctrl" binding is almost the same as the old "nxp,s32g2-siul2-pinctrl" one, and you still plan to support both versions in the same driver indefinitely. It would seem much easier to me to make sure that nxp,s32g-siul2-pinctrl remains backwards compatible with the existing driver and only adds the properties for gpio support on top, so a single driver can handle both old and new dts files. Arnd