Received-SPF: pass (google.com: domain of linux-kernel+bounces-143593-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) client-ip=2604:1380:45d1:ec00::1;
From: Peng Fan <peng.fan@nxp.com>
To: Frank Li <frank.li@nxp.com>, "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" <linux-gpio@vger.kernel.org>,
	"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
	"imx@lists.linux.dev" <imx@lists.linux.dev>,
	"linux-arm-kernel@lists.infradead.org"
	<linux-arm-kernel@lists.infradead.org>, "linux-kernel@vger.kernel.org"
	<linux-kernel@vger.kernel.org>
Subject: RE: [PATCH 2/3] dt-bindings: firmware: arm,scmi: support i.MX95 SCMI
 Pinctrl
Thread-Topic: [PATCH 2/3] dt-bindings: firmware: arm,scmi: support i.MX95 SCMI
 Pinctrl
Thread-Index: AQHajG9fogbrSJSEIkC/zlbnV4Z6dbFkvpCAgAD3YxA=
Date: Sat, 13 Apr 2024 06:01:17 +0000
Message-ID:
 <DU0PR04MB9417C3A75792347AB3A12774880B2@DU0PR04MB9417.eurprd04.prod.outlook.com>
References: <20240412-pinctrl-scmi-oem-v1-v1-0-704f242544c1@nxp.com>
 <20240412-pinctrl-scmi-oem-v1-v1-2-704f242544c1@nxp.com>
 <ZhlPK4PmnYHj0K2d@lizhi-Precision-Tower-5810>
In-Reply-To: <ZhlPK4PmnYHj0K2d@lizhi-Precision-Tower-5810>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0:
 =?us-ascii?Q?n1QWokcENZ2ErWubg77GYRpufKmL1yvzvFCmK/gb9xEn/VhcfusAy5dx/7U5?=
 =?us-ascii?Q?/LYfT8N5w3HW9frWNGD1euE9n528T14EUYwdvxj4h9rQR05klUFxj9GbuCzv?=
 =?us-ascii?Q?j2NpFrNRPwQ3g8ZPfHldo0qqvK+UBWBfpg6Pn6TP9JJcJbUtdh9v3uMOfqTj?=
 =?us-ascii?Q?E0DSXA4e5ybpYMVAFYh98w78JsyQaX779YQkvtw3cHlcyCrZKEoY/RMUZ7x6?=
 =?us-ascii?Q?Tn42wl9RZCcxJzH6LtE6wyhjzucXjICAGFxpg/nQMIarMl24QquuJ1DSZsDT?=
 =?us-ascii?Q?7i6/wL5niqxh0cqqYL1gL7fw2wuJysDVECbDibgsMac3KG0QNfZuEtvSUYWq?=
 =?us-ascii?Q?SKaXz48eKy5clmEHcGtGN08Wt3MmJPCGrRxuXVYTWNZb+qBxzZ+XNL1N6/T1?=
 =?us-ascii?Q?2Is7zQX9sJFuU/aha7ku0l+z5sFEockoOCQOKqIwHGgplkYy3Dqzmrei1VVJ?=
 =?us-ascii?Q?QCtMnrnV68Elt8e3OFGHBW4QLTXSzSkb3kWKqAb8Zzx341oxnrItxCzBPV3c?=
 =?us-ascii?Q?zSTmFXwLHZzi9Y5Ed3Js8tOYSFVkBCvkOHyl+M1qYKd4QYiynmiZSu11qLbF?=
 =?us-ascii?Q?lnmAtgZToVDdm5fRTvMZL2yyF41SiSZ252xaYWasMjm4q27Z092xWimc4SUm?=
 =?us-ascii?Q?jwKyw1m0xik87YrqUzJvr4tNqwX/GqrJqj3TVkOLn5eQvrVbKJ8GR78Ge3cC?=
 =?us-ascii?Q?0mucvsOcs3KdSRXRgKywuLpXD/5XwlvCNZU8hmCHW3EiN4t4mlkRWguAljXM?=
 =?us-ascii?Q?h+bQIMN1jOAaJQMEfO4JPwdFggUM+rSLfitxxbdZv7xSzacbkc91ielLdM8Y?=
 =?us-ascii?Q?41LGUcLQoWI6VslpZdd7fwJ8pRKd2WRgK2p+4WqEqmJGlvHHy7xP15Hhq/fk?=
 =?us-ascii?Q?aTuZrLVqIg5ZFB5N4otTkYzbesjbWAvKhjQA7JgmcX8Lxme+4NHUCvHFLXTJ?=
 =?us-ascii?Q?9ovUO385obfrbqn4BxdaskvB383vFeUVby/pVWMB16XDv7Ag2l1HU38JO+2O?=
 =?us-ascii?Q?JwMTsMp/+0GNyOagY/H8PuRwkt4LebaphuRdVRf7k8d8wNWT38nOShD8Ay+4?=
 =?us-ascii?Q?X4gN0jMT9DSvw5CvHPaMSaXvFkjndIMpABa7wvOiz4g9Pz3hZE8bSj2PXXhN?=
 =?us-ascii?Q?i0vZ0p1a3KVb1sACIe+3g5VyD59X/J5MPSeYbEGBKxzW0WhPy59C5CcPKRIY?=
 =?us-ascii?Q?Q/DGrv2o9+IvWwXXnUHiPBUy67O/KIO5vQsmsAFvVE6M0fMcBZcwCOFQK1Rk?=
 =?us-ascii?Q?u0u9aZdm6dsUcWWwGlR91Ft12S7RNEkXY41U/mMN7mQpuPE5cLiN4tRAw9bb?=
 =?us-ascii?Q?o/Ji1huYPuzmMDW20qMEw5xZd9tFLbQRipn6V5Fa7sHdATSL6k4yaOzFQ1w8?=
 =?us-ascii?Q?l1JFCgiEiyaICx44iO09Ww9yPk36EwD0dyx3PqVp08tg0psQPwW8vG+vHrLk?=
 =?us-ascii?Q?WGDntIkFQgPsPyh6UdOeZOg8FCEhq2GPW1yrTjp0EW9a6Bo2RXZI1jxQci1M?=
 =?us-ascii?Q?h2RcEx/iVtzehkTBug6RnTf8PaIoAa/FOHrxaEKlJGdmcES/yZIfsWM/J3sw?=
 =?us-ascii?Q?yQ+Z2DmTCB8op9QFa1U=3D?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
Precedence: bulk
X-Mailing-List: linux-kernel@vger.kernel.org
List-Id: <linux-kernel.vger.kernel.org>
List-Subscribe: <mailto:linux-kernel+subscribe@vger.kernel.org>
List-Unsubscribe: <mailto:linux-kernel+unsubscribe@vger.kernel.org>
MIME-Version: 1.0
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: DU0PR04MB9417.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 888ee029-2cd1-4f33-a3e6-08dc5b7f224a
X-MS-Exchange-CrossTenant-originalarrivaltime: 13 Apr 2024 06:01:17.3677
 (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: COMyqcuC5ef+cmJQmAIVoUIZG5+0X9qS0u23/JnxwFANR0dURLnOVykxMS+9qKRDLpne87Z5auXpaKGpfdyRIA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AS8PR04MB9095

> Subject: Re: [PATCH 2/3] dt-bindings: firmware: arm,scmi: support i.MX95
> SCMI Pinctrl
>=20
> On Fri, Apr 12, 2024 at 08:29:26AM +0800, Peng Fan (OSS) wrote:
> > From: Peng Fan <peng.fan@nxp.com>
> >
> > i.MX95 SCMI Pinctrl uses OEM specific units, so add '$ref' to
> > '/schemas/pinctrl/nxp,imx95-pinctrl.yaml' and an example.
>=20
> where 'example'?

I was thinking to add the whole example including pinctrl/bbm/misc
using a separate patch after all the vendor stuff got accepted.

Thanks,
Peng.

>=20
> Frank
>=20
> >
> > Signed-off-by: Peng Fan <peng.fan@nxp.com>
> > ---
> >  Documentation/devicetree/bindings/firmware/arm,scmi.yaml | 8 +++++---
> >  1 file changed, 5 insertions(+), 3 deletions(-)
> >
> > diff --git a/Documentation/devicetree/bindings/firmware/arm,scmi.yaml
> > b/Documentation/devicetree/bindings/firmware/arm,scmi.yaml
> > index e9d3f043c4ed..ebc6c083b538 100644
> > --- a/Documentation/devicetree/bindings/firmware/arm,scmi.yaml
> > +++ b/Documentation/devicetree/bindings/firmware/arm,scmi.yaml
> > @@ -249,9 +249,11 @@ properties:
> >
> >    protocol@19:
> >      type: object
> > -    allOf:
> > -      - $ref: '#/$defs/protocol-node'
> > -      - $ref: /schemas/pinctrl/pinctrl.yaml
> > +    anyOf:
> > +      - $ref: /schemas/pinctrl/nxp,imx95-pinctrl.yaml
> > +      - allOf:
> > +          - $ref: '#/$defs/protocol-node'
> > +          - $ref: /schemas/pinctrl/pinctrl.yaml
> >
> >      unevaluatedProperties: false
> >
> >
> > --
> > 2.37.1
> >