Received-SPF: pass (google.com: domain of linux-kernel+bounces-142899-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) client-ip=2604:1380:45d1:ec00::1;
Date: Fri, 12 Apr 2024 11:10:54 -0400
From: Frank Li <Frank.li@nxp.com>
To: "Peng Fan (OSS)" <peng.fan@oss.nxp.com>
Cc: Linus Walleij <linus.walleij@linaro.org>, 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>, Peng Fan <peng.fan@arm.com>,
	Sudeep Holla <sudeep.holla@arm.com>,
	Cristian Marussi <cristian.marussi@arm.com>,
	linux-gpio@vger.kernel.org, devicetree@vger.kernel.org,
	imx@lists.linux.dev, linux-arm-kernel@lists.infradead.org,
	linux-kernel@vger.kernel.org, Peng Fan <peng.fan@nxp.com>
Subject: Re: [PATCH 1/3] dt-bindings: pinctrl: support i.MX95 SCMI pinctrl
Message-ID: <ZhlO/oT8BXupXyKP@lizhi-Precision-Tower-5810>
References: <20240412-pinctrl-scmi-oem-v1-v1-0-704f242544c1@nxp.com>
 <20240412-pinctrl-scmi-oem-v1-v1-1-704f242544c1@nxp.com>
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <20240412-pinctrl-scmi-oem-v1-v1-1-704f242544c1@nxp.com>
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?utf-8?B?SDBqRWdiOWRzZENvcGhWQ0VJUnQ4cm1lZ0lnRmlBZXU2MlIvMkFYNXBQUFRJ?=
 =?utf-8?B?cmJSRWVFWmtXc3ZXVThkUXg4OS9IY0xyQ2VCOEZ0dXZtS1FYUGE2YXdXSE5C?=
 =?utf-8?B?SDhUeTB2QzJhUFE0MTFxSCtMMVk4QngzRUZwUGZNc2ZkNmFYdnVqQVNHRWVk?=
 =?utf-8?B?WXBZYkxGdXl5YkkrYUZmMEViU0U1NUhSM0oxcCtHcy9lTFVOMnVCaFBjT2dB?=
 =?utf-8?B?ZC9PVkxHbWRKZzNZdHMrYitiUWZHcE1FTkJZOE12TEJoZ2pxT1BIMWdpSWRO?=
 =?utf-8?B?WG44ZCtsMzFUY1J2bWFabE1BRW82SmZVSE1rTkQ5aUg1TDNpbTBGRmpSTmho?=
 =?utf-8?B?KzlpYzIrVDkvai96b2ZXckFicUdFNVl0aElMWlB3M0NkQU0vOFd0TkNhRDRo?=
 =?utf-8?B?U0hKZmpadUVEMDRMREpZNC9aYVIza3dxb0lCOWk3S3BvUDNSemZvOGw2dkZx?=
 =?utf-8?B?ZHVMYm5zeS8rQXhxaVFXd1hlZHROWG11Sk0vMEh1ZlJkK0RKUHd1RDVjWk1H?=
 =?utf-8?B?SlVraFpsN0tpT0xaTVYvVFZ6UlpsdnJrYnF0VlpNZGNrUGhhZ0J1Uk52bDFE?=
 =?utf-8?B?MUNCV3AyZEF4aUNndEloalQrVmlKR2F4c1M1VVl4ZUtSSlMzOU5kZEc0V3lI?=
 =?utf-8?B?VkJKSmVzcENvazVpeWhOSDQ2Y1BicEI0Y2tJLzR0MHNUMldvRTVvaTZFYngw?=
 =?utf-8?B?VnY5UDY3NnNoSElUSS9PQWxmVWxySUdVOWFBRXpnaXZrKzdSbTJjYkFyT0M5?=
 =?utf-8?B?T3ZxeGF6MkNSODdqR0dQY1JrRmdSOG5FbzB5L3FNcEt2U05PZ1k5Ynh4T0lV?=
 =?utf-8?B?ZlcwanNCR3V5aDkwZm83U1loTkFSV1Y1Kzl3TXpvSTJ4L2sweERjY3JhcmVv?=
 =?utf-8?B?UmZoOXFSRzZNRFhHM1lvL3RxRlJ0SjRUaVUveXJxTmUxNm5CVlFNOUNFVEpP?=
 =?utf-8?B?TVFTSGNpdWVjdjA1TWR5alVhMS9Ya3dRUXJVMzZZZDNhZjV3a1d2d0JsS3lK?=
 =?utf-8?B?bXNNazhrSjNKRFBaVkFGWjVhbEVYd2N0SFg5blFqSFFWb0Z5L0Q4N25hMTUy?=
 =?utf-8?B?QjdOQlZFcTlLckIxRnVzQUFJaUs4OGdVMDVoZzdadWd3cXd1MU91b1ZvZVkz?=
 =?utf-8?B?QU9vRkRnUXZJNGxJdlFoemM3UDkxQzRDTURxMng0YjkrWkZYZWVrV29tbFNJ?=
 =?utf-8?B?VUZXdDFkaWVoVWVpS2QwRmY2R1drdGRNUlBwcjFZVDhRcXFLRTdXeUJiUXRp?=
 =?utf-8?B?ZVA0eUFTSlk4TkxKbDVMU3Z3ZHNTSVV1VjRDTVlNZEEwZDRLNUs2Sm4wUVc0?=
 =?utf-8?B?K2RnWHovbW1pMy81Zmh2Ly90aU9PZm16L3hXZlBqb2Jtd2UyUU8rcHFwejh3?=
 =?utf-8?B?OGJid3VkWnRBOW92L2taWlBSazB1UXlPbTJZMlNLQ0VqUFkzODJBSnRCenZt?=
 =?utf-8?B?cGJYYWhPTDZWY3cwK0k1aUVUc0U4NWM5em9VUmJjZWcyc2RBcTI5U0trTFUx?=
 =?utf-8?B?YVIvT0REVldFbDZkK0hSL1ZON1lWenk2b3lvUExGY2lrSHRaUlB6dTRVdlp1?=
 =?utf-8?B?aWFodHdTTTUrUkRaODNRbXlBeDFkVFQyaFFRQlNvVGpidXhETzBTbU1wbWF0?=
 =?utf-8?B?YmRXYzdQRGgrSWo4WUhzYlBGcDB5d1kzR3YzUVZUNzlNcGtYRlY3YTFDRlIy?=
 =?utf-8?B?eTlrN09yUnpjaDdpL0dzTDFqRnVtRHB2ait5NVJIVE1ROXExQjhUVERkS0sv?=
 =?utf-8?B?T29DOGd1dStjZ3lzaVg5dndmejBJbUp6WnpFSTBsbFdFUHQyNmNVbXpscEZE?=
 =?utf-8?B?LzNtYnFnRklTUzJHWW5iTlJ3T3dzdUVvQm1NazErUWlsakRSbGZHYTJuUGNM?=
 =?utf-8?B?dEJRYlE1QVJSaVcrQU9LUU1QS3JHTDE2dlZUclFrVjFlbUdhT2ZZdUYwVmRE?=
 =?utf-8?B?VHdXWjdRL1VsM0dRekdYcEpEdzFySVZaVHkxRkJqUWVGZ1hYTnRJUzNHWita?=
 =?utf-8?B?OGFqVk9IRXJ4eDBYWGl4VDkwcStLOU1kWnJxZ2dkM3VtbzRqUjQvWWgzbGVz?=
 =?utf-8?B?ZGdldlBnOEJRQzIyRTQ1UzBSajg4VEtsR1lnZ0laVjFYaXEwRWNCSnRabkk0?=
 =?utf-8?Q?d23g=3D?=
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 18e2cd68-2f73-4dd2-f67b-08dc5b02c570
X-MS-Exchange-CrossTenant-AuthSource: PAXPR04MB9642.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 12 Apr 2024 15:11:04.2535
 (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: 6iNZgvMM8/Pgokeq8dafIpad5Lz8KTwjT0zFYrbGano/6WalonlNV45mOrVzJgXnMtr2l4TP18SOPk0IayLauw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM8PR04MB7282

On Fri, Apr 12, 2024 at 08:29:25AM +0800, Peng Fan (OSS) wrote:
> From: Peng Fan <peng.fan@nxp.com>
> 
> i.MX95 Pinctrl is managed by SCMI firmware using OEM extensions. This
> patch is to add i.MX95 Pinctrl OEM extensions properties.


Describe your changes in imperative mood, e.g. “make xyzzy do frotz” ...

"Add properties for i.MX95 Pinctrl OEM extensions."

> 
> Signed-off-by: Peng Fan <peng.fan@nxp.com>
> ---
>  .../bindings/pinctrl/nxp,imx95-pinctrl.yaml        | 44 ++++++++++++++++++++++
>  1 file changed, 44 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/pinctrl/nxp,imx95-pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/nxp,imx95-pinctrl.yaml
> new file mode 100644
> index 000000000000..4bf7a6192813
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/pinctrl/nxp,imx95-pinctrl.yaml
> @@ -0,0 +1,44 @@
> +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
> +# Copyright 2024 NXP
> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/pinctrl/nxp,imx95-pinctrl.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: i.MX System Control and Management Interface (SCMI) Pinctrl Protocol
> +
> +maintainers:
> +  - Peng Fan <peng.fan@arm.com>
> +
> +patternProperties:
> +  '-pins$': false
> +  'grp$':
> +    type: object
> +
> +    properties:
> +      $nodename:
> +        pattern: "^[0-9a-f]+$"
> +
> +      pins:
> +        $ref: /schemas/types.yaml#/definitions/string
> +        description: name of the pin
> +
> +      nxp,func-id:
> +        $ref: /schemas/types.yaml#/definitions/uint32
> +        description: high impedance mode ("third-state", "floating")
> +
> +      nxp,pin-conf:
> +        $ref: /schemas/types.yaml#/definitions/uint32
> +        description: latch weakly
> +
> +      nxp,daisy-id:
> +        $ref: /schemas/types.yaml#/definitions/uint32
> +        description: pull up the pin. Takes as optional argument on hardware
> +          supporting it the pull strength in Ohm.
> +
> +      nxp,daisy-conf:
> +        $ref: /schemas/types.yaml#/definitions/uint32
> +        description: pull down the pin. Takes as optional argument on hardware
> +          supporting it the pull strength in Ohm.
> +
> +additionalProperties: true
> 
> -- 
> 2.37.1
>