Received-SPF: pass (google.com: domain of linux-kernel+bounces-172651-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) client-ip=147.75.80.249;
From: "Peng Fan (OSS)" <peng.fan@oss.nxp.com>
Subject: [PATCH v5 0/3] pinctrl: scmi: support i.MX95 OEM extensions
Date: Wed, 08 May 2024 11:32:01 +0800
Message-Id: <20240508-pinctrl-scmi-oem-v3-v5-0-6f2b167f71bc@nxp.com>
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
To: Sudeep Holla <sudeep.holla@arm.com>, 
 Cristian Marussi <cristian.marussi@arm.com>, Rob Herring <robh@kernel.org>, 
 Krzysztof Kozlowski <krzk+dt@kernel.org>, 
 Conor Dooley <conor+dt@kernel.org>, Shawn Guo <shawnguo@kernel.org>, 
 Sascha Hauer <s.hauer@pengutronix.de>, 
 Pengutronix Kernel Team <kernel@pengutronix.de>, 
 Fabio Estevam <festevam@gmail.com>, 
 Linus Walleij <linus.walleij@linaro.org>, 
 Dong Aisheng <aisheng.dong@nxp.com>, Jacky Bai <ping.bai@nxp.com>
Cc: 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 <peng.fan@nxp.com>
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-MessageSentRepresentingType: 1
X-MS-PublicTrafficType: Email
X-MS-TrafficTypeDiagnostic: DU0PR04MB9417:EE_|AS4PR04MB9243:EE_
X-MS-Office365-Filtering-Correlation-Id: 0845db41-428b-4424-029d-08dc6f0e6103
X-MS-Exchange-SharedMailbox-RoutingAgent-Processed: True
X-MS-Exchange-SenderADCheck: 1
X-MS-Exchange-AntiSpam-Relay: 0
X-Microsoft-Antispam:
	BCL:0;ARA:13230031|1800799015|376005|366007|52116005|7416005|38350700005|921011;
X-Microsoft-Antispam-Message-Info:
	=?utf-8?B?QUJrc0tEc1BZWEk0QzVnUDI0dEhRakRNYS9FNlBma0g2MnRNamliclpyZUYx?=
 =?utf-8?B?RmtFV0lHQW13NjVmUUJBOXJ3SjBPWmFqRTVsUXJSYkhQUHZZUjVPUDY5aWhj?=
 =?utf-8?B?eDM4RFhPV2REZVNLdTdJU1BRMVNxQk9sNkx5Vlc4MUd3OXVvYzNlZnkxUTJJ?=
 =?utf-8?B?dUlGZnhxQm9zbklOY3VqSVNqelcxeDkyK1RTOUhLVG9uNC9BUjN0ODhoYjl3?=
 =?utf-8?B?cUtVWmk0NWhwVWl2TzNSTnRmNXFtK3ZaWWRiMG5UUjhHUVVrT09uTHJXclJa?=
 =?utf-8?B?MmJsZTN0V01BOUw5em93d2U3dHpLanRoaGIrZmdHWjlFRGJBYU0vSWMxSDRt?=
 =?utf-8?B?b0FBL2NvaHVTc0hvQlJJMEY1RVB1UGhFaHRhc2dkQng5bFpyYkVFVjdwU1Ev?=
 =?utf-8?B?WEh0R0tjNTZEUXUrb0g0cXlsV0lNNFkzS3Vvb3pZbWdSOHhrSzJhc3JLRTVR?=
 =?utf-8?B?bU5mbGdkNW9weDg4N2Z5QzJnR2NvVTZkL3FBOVlKZS9EMkthVG85STRBT0J5?=
 =?utf-8?B?ZHhLdERaMWZpRXJkamtOWDlJcjc1UURXT3JXejB6Y293U01wc2oyRGZFYjFG?=
 =?utf-8?B?c3N6M1JDNWZ5M0VMWlFVcnJHL1V0MnNIY1AyTUdiZ2EyNzhnUjZ6MDVmZ3pp?=
 =?utf-8?B?UjNVRGVZQXRocGVhdWhiR04vRklxa2NRNmRaaFR6ZkhXOTB5NFpFNWp1aDR0?=
 =?utf-8?B?V3hRem8ybkwxQ0x6cU5RUFN2Y0ZtQ1Vvb0YzcnJhN1UxbUtGNzhGSEJCM1Z4?=
 =?utf-8?B?YkJjaWdxNHJQMGdFdVJxZjVBTktwS1VLUEttWVVhMmNEVGxRUlZZT0p2c0NZ?=
 =?utf-8?B?YkV1U0hnSTN3d2oyb2cvVkdCSUlpcG41YWp2RnYxYnRDWDZzWmxYV29rWEFa?=
 =?utf-8?B?OVlUQkRzc21VM2Z4c01MTHo1N3Q5dmVMTWY4SmRydDkzUUF1OXJwUW4rVzZY?=
 =?utf-8?B?ZElsZ2FhZnQ4cmMxaXNBT2N2STJaOGJVVnNURlM3YzlZZ2FEa1JEcU9VTVZX?=
 =?utf-8?B?NkJ5NnpHSFBlemFwaUw5RmNIMHI0YUxyOEN0cmRmMWtIMitIdURrT3E4Mk9k?=
 =?utf-8?B?cWZ6RVZad0xlY1pTak90Uy8xRXdYM2hYQkZrdmNPRUFmZjlQcGNlM0ZjZmhY?=
 =?utf-8?B?M05GNFdqQWs4THhiajJwakp1N0QxZnNJZENpWTd3NlU4a0JVclVwVURkdDhi?=
 =?utf-8?B?QUVsbUJ6dGNCZjB6RkM2MHY1OVV2LzhkenFjM2FUZEFuUHkyQ3NrTmRrcHRy?=
 =?utf-8?B?UHU2Q3ozbVpUZW53Zjg4N0VNSHM2ajQzUisrUDdBamRnb1ROZDhlc2FlNnB2?=
 =?utf-8?B?SVdJdmFQdGpIU0J2dXFmVmNzSXRFWVE1N00zaVI2OFhTNlhQdjhDd3owZlAv?=
 =?utf-8?B?aHhOaXZqK1NSbzlhSHVKZlVOTEZ0OGhQVUdIZUdmSUJRTFBRNEs2L0NlYmh0?=
 =?utf-8?B?M1BSRDRpaGZqZ3VkYTVKMVVzQzg5SVNyUGN0eUtnUlV4emttQzI1L3JrR2pm?=
 =?utf-8?B?bWRnaFFJTW52ZTIySlN1emlxSHNZZktmNWRPd2lZa0kyMGhTN0prU3hkcFFR?=
 =?utf-8?B?UG84aEE1UnF6VU1wYlJjdHBPZlMyUXJqNUloSzlNaGhIa29FV2VsRUZyN01J?=
 =?utf-8?B?d2N3OU00ZHlYbVdRc1dwZkZNVzI0K3ZQVDZuUGoxam9qYnc0RUZnZnJJWngr?=
 =?utf-8?B?UW9JSExxSkxuWmlhUUVFbmlkQTYya2hFUnRSeHEwWlJBTDdLS3Y3TWV5cHV0?=
 =?utf-8?B?V2VrSTVIbFJJOTUrWk1vZDJFN09QTDZCa0txU2ErS3BCeUxHVUE2MGNPMXpP?=
 =?utf-8?Q?t5a7ha79waotWG+89IFoXa0t+fMx6RHCOF070=3D?=
X-Forefront-Antispam-Report:
	CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:DU0PR04MB9417.eurprd04.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230031)(1800799015)(376005)(366007)(52116005)(7416005)(38350700005)(921011);DIR:OUT;SFP:1101;
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?utf-8?B?Mk5LNDBDQnZ0V2FCamZDaUs2WVBSRnh2dDFlQkRzMjRZMVVxVG9MRXpELzg2?=
 =?utf-8?B?cW1WM0JIMTlRWGU1MXhzbmJvRXJ4ZG5zZzNYRENlb20zSllEN1hNVlNPYXdk?=
 =?utf-8?B?VCtiN05IN0tVcjg1RG1OWlRJdURyMTdZSFo4RFBNaDM0UFp0cFNWcXdjWENq?=
 =?utf-8?B?cFVnVWw2RzhnNkJzT2VuWmFtalpqZDZaOUt1dWQrMWpkZXk4L2JyWnJGRENT?=
 =?utf-8?B?bHQ5cmFnS3Q4L0wrWUZvVHkxM0c2TUVqRlpHejZnZ3FqNWJaQXQ4elZrbFpM?=
 =?utf-8?B?KzhVNkRicy9GTHk4SVlwSGhHeUxXZlZPUGV6NEQ2WGRvSmxKVWlxTG1rYjR2?=
 =?utf-8?B?eHNwQXVjdmprcFB4UDhxQkJmS21mbTM4OVlocXFzQldrakIyZTdPbitlTS9v?=
 =?utf-8?B?cCtoUmZIZnNPVW1UZnVVa21ZR3AycGNJcTdJbzk3OEY0NUF4UzZmN24rOGZl?=
 =?utf-8?B?L0VRWU9EWWkyRnU2RUw5UDVDT216dVJFQVV5VDc1dHg4QkMxbDhGakNWS2dT?=
 =?utf-8?B?dzRvaUtEVmROK2lsclhWdVA3YVhEYWxyb0xJbDkvV3IyMVVoOHRJZkJGOFZV?=
 =?utf-8?B?NkQrM09TdDlxNHY5QlVmc0ZaL0hTakVBNndYT2hsQ1BlRzI3eEhXOXpIQjRV?=
 =?utf-8?B?WWlRZE9CVFNwR2tEMEp6RkFqdStaaEQ5eHVBcVFlenFmWm9oZkdHa1RoOFdv?=
 =?utf-8?B?L3JHVkVsWm5VVjlNMlQ2WFMydWlHKzdlM25MY28vSHc0aC9JRy9jL042c2Jm?=
 =?utf-8?B?S2Z2R3YrZGxTTnRUUVJSdm5LMWpKTlZzSzZRajNraWJFeHRYUG1ocmhoSUNN?=
 =?utf-8?B?eUNackJOdElSVGZYSXdUb0NkT1ZwVWhCUzlFV0VMa0tQNmx1dmU3SXpGd0xv?=
 =?utf-8?B?cWxDUDRkSkdBRkgrdWZhZmxpYm5sejQ2d01qTTFFNGRmbE51VU5VMHR6MGpQ?=
 =?utf-8?B?bWp5MnlFSEpGMVRPeTJ6bzBvTUNhbHV2Z2ZQajAxdDRCZ3pYWUZCQ0dtdjJD?=
 =?utf-8?B?T3hIZ2NVZ2FHY2VrMkRWb1FkNjV2eXk5M3g5OFhZUTBPWlVxVlpGbFFHd1I1?=
 =?utf-8?B?bVNYMU12NW43YnM1cnNGNmY1Q1pDT0xZWVE3MlNsS1hnN0taeDJucW9BUUwv?=
 =?utf-8?B?UTYwS00xcm9rUHprcEZmL0c5cHp2a2FubldJbkpOWHNlYjNKU3BlNDVCOFZk?=
 =?utf-8?B?Z2tkZ1pBWDkyWjBpc212b3phRXF3VkhiSjMwVzlTWE1NQnZpeUdhMnV5ZCty?=
 =?utf-8?B?eE1mS2RlVldjbWFRc09GZmprREZsMkNvTU1MQ2hzNUNoWjBuMTVuL2tBNzZq?=
 =?utf-8?B?U0N0eGcxZVJyZFpuSVF1THZvWHhXYzBaa1NKbStYYjZ2WU5aMmpzSFpkOVd2?=
 =?utf-8?B?c0d1eDR6dmZjdkhkYnFMd0xUU21nZ0dYVnlnV2VkVVNCN2pJaDdCTjdodi93?=
 =?utf-8?B?M2FWb2F6Mi8zdVFHODBkVjdKNFZUOWgwZ3puZnVsL21KRjBvZDhqVUNvZTQw?=
 =?utf-8?B?TWVpUEp1cVhMaXdNSVBwc3FXa2cvQXBXVDZ6VmZKWm1OYkluRWRaTCtnZWd0?=
 =?utf-8?B?emdnN2ZWSjZhb0lYNy9RSnBjcVh5dC84eFQrMmZqWHFFUlg2L3JuenFHUVFE?=
 =?utf-8?B?NGJrWVhiN1ZyRURUT2M1ek5ZMlVmU29rSVRVQkRWaStpa2JuRlEwQmExb05h?=
 =?utf-8?B?L1JEbmVRdkdBS1dQNGd4Ky9udlpaWFpMWVp5Y1hFRTUxWHBIckRwZHJOQUJ6?=
 =?utf-8?B?OGE5ekc3b3VmWmtUeXVOb3c1d3BkSzBGUzlCWlVNeCtPeU8zZ3dieURYaTYz?=
 =?utf-8?B?MDQ5bkdoRENKY2NTdFVWYWtROWxmQjQ2UGNqRFBFZVc0K1N5OTBsMnBQSVRV?=
 =?utf-8?B?R0ZzOGJLNm1Qb1pyNzh3a2IzN05kSHpZOHBEeXdSaVFBQ0xEN1h1amlvaTZt?=
 =?utf-8?B?QUxCT08rZmhrdzBVS205cWswL2cvM2VVRjBwc2J5elIycUJKdzRtRDV5enRB?=
 =?utf-8?B?Q05FSU56QWV5UnRRZENBMnZrQjMzZlllUjVDdG1mWHBIR29Za3hEeUw3a3F6?=
 =?utf-8?B?SE5wNW5raDJFOEhuTVNIbXVxSXFXMVROZDFFNFhHbzZhTVNkTDExZmlxSjlD?=
 =?utf-8?Q?qcKWTvirUiFoCybUotlft8Hfl?=
X-OriginatorOrg: oss.nxp.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 0845db41-428b-4424-029d-08dc6f0e6103
X-MS-Exchange-CrossTenant-AuthSource: DU0PR04MB9417.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 08 May 2024 03:24:32.8951
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: 1Bt88ySPht1wN9FYQ3noniuw2q4TV6D1wtRgmmY7X45xxP3cfIdpABHSTf9Tpa6m1p4uSMMML5fQgnps2o+wuQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AS4PR04MB9243

ARM SCMI v3.2 Table 24 Pin Configuration Type and Enumerations:
'192 -255 OEM specific units'.

i.MX95 System Manager FW supports SCMI PINCTRL protocol, but it has zero
functions, groups. So pinctrl-scmi.c could not be reused for i.MX95.
Because nxp,pin-func, nxp,pin-conf properties are rejected by dt
maintainers, so use generic property 'pinmux' which requires a new driver
pinctrl-imx-scmi.c

The node will be as below:
 pinctrl_usdhc1: usdhc1-pins {
         sd1-grp0 {
                 pinmux = <IMX95_PAD_SD1_CLK__USDHC1_CLK
                           IMX95_PAD_SD1_STROBE__USDHC1_STROBE>;
                 drive-strength = <0xe>;
                 input-schmitt-enable;
                 bias-pull-down;
                 slew-rate = <0x3>;
         };
         sd1-grp1 {
                 pinmux = <IMX95_PAD_SD1_CMD__USDHC1_CMD
                           IMX95_PAD_SD1_DATA0__USDHC1_DATA0
                           IMX95_PAD_SD1_DATA1__USDHC1_DATA1
                           IMX95_PAD_SD1_DATA2__USDHC1_DATA2
                           IMX95_PAD_SD1_DATA3__USDHC1_DATA3
                           IMX95_PAD_SD1_DATA4__USDHC1_DATA4
                           IMX95_PAD_SD1_DATA5__USDHC1_DATA5
                           IMX95_PAD_SD1_DATA6__USDHC1_DATA6
                           IMX95_PAD_SD1_DATA7__USDHC1_DATA7>;
                 drive-strength = <0xe>;
                 input-schmitt-enable;
                 bias-pull-up;
                 slew-rate = <0x3>;
         };
 };

Signed-off-by: Peng Fan <peng.fan@nxp.com>
---
Changes in v5:
- Added Rob'r R-b in patch 1 after addressed the comments
- Per Rob, use of_machine_compatible_match for patch 2
- Use of_machine_compatible_match for patch 3, and enable COMPILE_TEST
- Link to v4: https://lore.kernel.org/r/20240505-pinctrl-scmi-oem-v3-v4-0-7c99f989e9ba@nxp.com

Changes in v4:
- Rebase to next-20240503 
- Add pinctrl-scmi-imx.c itself get pins and scmi pinctrl structure to decouple
  pinctrl-scmi.c and pinctrl-scmi-imx.c, so drop patch 3,4,5.
- Link to v3: https://lore.kernel.org/r/20240428-pinctrl-scmi-oem-v3-v3-0-eda341eb47ed@nxp.com

Changes in v3:
- patch 2,3,4,5 are new.
- Rewrite the binding, drop nxp,pin-x properties, use generic properties
  as Rob commented.
- Switch to using pinmux means pinctrl-scmi.c could not be reused, so
  add a new driver in patch 6 for i.MX95. But pinctrl_scmi_get_pins and
  scmi_pinctrl are exported for i.MX95 usage.
- Link to v2: https://lore.kernel.org/r/20240418-pinctrl-scmi-oem-v1-v2-0-3a555a3c58c3@nxp.com

Changes in v2:
- Rename nxp,imx95-pinctrl.yaml  to nxp,imx95-scmi-pinctrl.yaml and move
  to firmware
- Merged patch [1,2]/3 v1 into patch 1/2 v2.
- nxp,imx95-scmi-pinctrl.yaml only has patterProperties for subnode
  The pinctrl will be as below for i.MX95.
        pinctrl_usdhc1: usdhc1-pins {
                sd1cmd {
                        pins = "sd1cmd";
                        nxp,func-id = <0>;
                        nxp,pin-conf = <0x138e>;
                };
                sd1data {
                        pins = "sd1data";
                        nxp,func-id = <0>;
                        nxp,pin-conf = <0x138e>;
                };
        };
- Add pins enum, correct description.
- Link to v1: https://lore.kernel.org/r/20240412-pinctrl-scmi-oem-v1-v1-0-704f242544c1@nxp.com

---
Peng Fan (3):
      dt-bindings: firmware: arm,scmi: Add properties for i.MX95 Pinctrl OEM extensions
      pinctrl: scmi: add blocklist
      pinctrl: imx: support SCMI pinctrl protocol for i.MX95

 .../devicetree/bindings/firmware/arm,scmi.yaml     |   9 +-
 .../bindings/firmware/nxp,imx95-scmi-pinctrl.yaml  |  41 ++
 drivers/pinctrl/freescale/Kconfig                  |   9 +
 drivers/pinctrl/freescale/Makefile                 |   1 +
 drivers/pinctrl/freescale/pinctrl-imx-scmi.c       | 585 +++++++++++++++++++++
 drivers/pinctrl/pinctrl-scmi.c                     |   9 +
 6 files changed, 651 insertions(+), 3 deletions(-)
---
base-commit: 4db57327adc359a3f9a3481d60104be67c42964f
change-id: 20240428-pinctrl-scmi-oem-v3-12130031a74d

Best regards,
-- 
Peng Fan <peng.fan@nxp.com>