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 C8624C4345F for ; Wed, 1 May 2024 12:41:24 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=MCgKLGg9Stim+yfX5eslFkCxyQllIpL6/z35le7VlME=; b=1xzpyp63jl2CLt tIG2ciHZgqJmn8YF7b6FkO7NAFtF4PtPDZb0IiQD0NmfqKEcVtrmjtJGPxmdTFRdckcP++y3i6Qnl 7w5GyUNjvU6Z2yUHyfDjlLuVe7g7QUA4WnmIwh0w5U4JUF2XnTVKna9ei9cbX2OUeRFZh8fccOek8 cGY3Q5uotXOyjIzPdBcpM39xJbLEmvTZeuwbai0VhqXvK5R6fZt05cRvaQRg1mY1HA/qlYzYHuAYp FUCa7QIbcBVXyn12RXrorwQd3fcE6E7BeAy8MOiPYIX8AKGXVrv6dJXKCG+B4+/ucXJ5wZ0MqPLGa ZMh1uiyklQSFQlYMqhGg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1s29Gt-00000009WlU-3ncj; Wed, 01 May 2024 12:41:15 +0000 Received: from foss.arm.com ([217.140.110.172]) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1s29Cs-00000009UM4-0qCY for linux-arm-kernel@lists.infradead.org; Wed, 01 May 2024 12:37:08 +0000 Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 2693B2F4; Wed, 1 May 2024 05:37:28 -0700 (PDT) Received: from pluto (usa-sjc-mx-foss1.foss.arm.com [172.31.20.19]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 98C753F73F; Wed, 1 May 2024 05:36:59 -0700 (PDT) Date: Wed, 1 May 2024 13:36:57 +0100 From: Cristian Marussi To: "Peng Fan (OSS)" Cc: Sudeep Holla , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Shawn Guo , Sascha Hauer , Pengutronix Kernel Team , Fabio Estevam , Linus Walleij , Dong Aisheng , Jacky Bai , linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, imx@lists.linux.dev, linux-gpio@vger.kernel.org, Peng Fan Subject: Re: [PATCH v3 4/6] pinctrl: scmi: export pinctrl_scmi_get_pins Message-ID: References: <20240428-pinctrl-scmi-oem-v3-v3-0-eda341eb47ed@nxp.com> <20240428-pinctrl-scmi-oem-v3-v3-4-eda341eb47ed@nxp.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20240428-pinctrl-scmi-oem-v3-v3-4-eda341eb47ed@nxp.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240501_053706_353570_689EDDF4 X-CRM114-Status: GOOD ( 17.27 ) 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: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Sun, Apr 28, 2024 at 01:07:50PM +0800, Peng Fan (OSS) wrote: > From: Peng Fan > > Add pinctrl-scmi.h to include the function prototype and 'struct > scmi_pinctrl' to export pinctrl_scmi_get_pins, so other drivers > could use it. > Hi Peng, so you wrote a new alternative SCMI driver using Pinctrl protocol@0x19 so that you can just parse you custom DT bindings and then use the SCMI pinctrl_ops to set the OEM extensions to configure your platform... ...since your firmware cannot cope with the all SCMI stack footprint.... ... you seemed to have solved the issue of having 2 Pinctrl drivers coexisting under the Linux Pinctrl subsystem while attached to the same protocol@19 node with patch 5/6 blocklist (if I get that right..) I think this approach of a standalone SCMI alternative Pinctrl driver that handles distinctly NXP OEM extensions and DT-parsing is certainly more preferable than the original series you posted months ago where custom NXP stuff were simply stuck on top of the Generic SCMI Pinctrl driver... ...what I still dont understand is why you exported data and structure from pincttl-scmi.c to use it here; when NXP pinctrl is active the standard Linux generic Pinctrl driver wont be alive, so not probed, so no data can be shared, the only thing I can imagine is that you are just trying to avoid duplicating a dozen lines from the logic of scmi_pinctrl_get_pins() into your new NXP driver. In this way, though, you are creating a dependency between 2 drivers, that are not even allowed to cohexist at runtime really (due to the blocklist trick). Am I missing something ? If not, I think it will be much better to just rewrite that few lines of scmi_pincrtrl_pins_get trivial logic into your NXP driver and keep the 2 drivers fully distinct at all times. Thanks, Cristian _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel