Received-SPF: pass (google.com: domain of linux-kernel+bounces-172980-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 <peng.fan@nxp.com>
To: Sudeep Holla <sudeep.holla@arm.com>, "Peng Fan (OSS)"
	<peng.fan@oss.nxp.com>
CC: 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>,
	Aisheng Dong <aisheng.dong@nxp.com>, Jacky Bai <ping.bai@nxp.com>,
	"linux-arm-kernel@lists.infradead.org"
	<linux-arm-kernel@lists.infradead.org>, "devicetree@vger.kernel.org"
	<devicetree@vger.kernel.org>, "linux-kernel@vger.kernel.org"
	<linux-kernel@vger.kernel.org>, "imx@lists.linux.dev" <imx@lists.linux.dev>,
	"linux-gpio@vger.kernel.org" <linux-gpio@vger.kernel.org>
Subject: RE: [PATCH v5 0/3] pinctrl: scmi: support i.MX95 OEM extensions
Thread-Topic: [PATCH v5 0/3] pinctrl: scmi: support i.MX95 OEM extensions
Thread-Index: AQHaoPc+EplymBzv+UOcaM7zveyY+bGNDeaAgAAFlBA=
Date: Wed, 8 May 2024 09:41:24 +0000
Message-ID:
 <DU0PR04MB94170BB9A2C41DE40C8F575288E52@DU0PR04MB9417.eurprd04.prod.outlook.com>
References: <20240508-pinctrl-scmi-oem-v3-v5-0-6f2b167f71bc@nxp.com>
 <ZjtCGXgwgtZ4X49v@bogus>
In-Reply-To: <ZjtCGXgwgtZ4X49v@bogus>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0:
 =?us-ascii?Q?BwL/Wv2ccgyTjseo0ZaZK2+LjIOxc+ctPuCF13mu7ciThQqwQkvtHNag+yeg?=
 =?us-ascii?Q?/9fHNobMakXS3+LSq3enJkI9Qkf/zlbARDv5NLhdeP1KaZsf2CXLiDd5NlBo?=
 =?us-ascii?Q?AFmG3Yzkd5CNGjiNqHeD2VJpmXMu/dGHsBL2dfB/wFzLWRQQ2WV0+Pc2W3C1?=
 =?us-ascii?Q?oY6iTepRfEvdqZuC1ojmj6SVi4Qx1GPNDvD7TUFX94Zz/WfaEOHIuxgxABYE?=
 =?us-ascii?Q?RTy4DIzaTfIVyb+KKkAClUCOY9H45BfY3o7QGwrV15qlsiz7hhS6L+hlAJi3?=
 =?us-ascii?Q?hvde7x2CFa01c8j9P3YuWvFb7RujGXmU0e9ykd9PVsVALqjCipCNNcbm0n8R?=
 =?us-ascii?Q?M7K/UBfoxekWSp1kWdIaHS20ABkxNxvv6oOKo7sNOPmlzIkdk7p1/JkvlvEz?=
 =?us-ascii?Q?xkskHDDBfWzvC3H9wcjAuOcMqayV4LJcRD6Al8o0UgVVpMbtIx1IHisrkL1v?=
 =?us-ascii?Q?vt/K/gdD7SHvz+pxpqAWVBu2u8mIJS4aDcCWNVbdc5qRga5j/jr8joQFb7gr?=
 =?us-ascii?Q?tfWMZA0IkB3is0QvoMywtA6/dN+r5Voc9elwyqGZoHuQrxUBt/jh2VeBcKQL?=
 =?us-ascii?Q?6DnuCEeDYIYXV0RntqOA4LFz6w6hXv/5v5+QRL00ITEo4bH5Vp0HBmLRIk1O?=
 =?us-ascii?Q?XdW77IZXZ1gbLS6ya6ZVlCPwII1LGTKdL+SefuQ5PaaiZdvqA9Ma6m/taCl4?=
 =?us-ascii?Q?seA4ZqD2+NS7e4HPDCutOu0dmnphUn+MgZfpWXI1aWP69MqEZq6DHz1BUwhV?=
 =?us-ascii?Q?rauny9TQfiz0iyVjdElb0NJfyeVTsQteZYZkSUnthwAoqr1rsVQxBrxBDmBq?=
 =?us-ascii?Q?RJiRqp3hxvJcfAefvXHDgZyHEeE5NIsh9Outhc4EqkDSpu80TSVjEyJCsW3N?=
 =?us-ascii?Q?wL7TAxwtKONpKki1rXrAQDqa67dGVIzdBpcGsRHM3s/6o9ACH2oSe0MOvIvv?=
 =?us-ascii?Q?ktUuotrQN+ofWkxsrFA6JTcypWytaNv4VX/z9msEwElMfgQexJzaW6E9GHrv?=
 =?us-ascii?Q?AAADjNIKD3JZ0cmC4hUX2AB03Fm4tPaCo2BFRXqXk+1+mHyqrHKtBsCXmyBX?=
 =?us-ascii?Q?a5zBiRDv/YjXwwoRkkcQTa03p7Qexus6yZrdVq9ND9dorhdg1Dbbl0OGIQCt?=
 =?us-ascii?Q?QYrguM+lWeZYzcU+3N+mYD32gd3Evp1apZLfQ789+mcPfBxo8GvqVGz3ka3e?=
 =?us-ascii?Q?eNRlzM+Io4nNwJrSBqJ95WGcsct2LNjWgqkOGEn/33yeZxyeOrLz0EE6Jqz/?=
 =?us-ascii?Q?G79GHWGxnVFAxKQMpWjp+LBjzY19OUarEzXSUbBmdBaP78tmZswS9cuN5gT9?=
 =?us-ascii?Q?zMDxOsc0Y6CM3PtnbZsSYqmpM++pNk+GqPLNBhESNQcL5CKm76lMzp5cQNao?=
 =?us-ascii?Q?YDzpbWbRMxcK50lgRx4TABnZRhux8/iFfQm8YVmhstxGJopKtQmleM3kDJ/4?=
 =?us-ascii?Q?bvqGos1zu2kbg86R3N2WyeXWokMahWHeLON6V/6ByoXouK6WXpUQbVC+Yuaq?=
 =?us-ascii?Q?8m0MZq94T2rysOMViAwWhVaIfnELm6nSQ6MpD7DMY5bsslxjCZUm3BHWrmsF?=
 =?us-ascii?Q?NgBr1T2A7jwDXaEsdwA=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: c22d5a0c-c6f3-4285-151d-08dc6f43066a
X-MS-Exchange-CrossTenant-originalarrivaltime: 08 May 2024 09:41:24.0634
 (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: 8izIXGo0HJPrwaty7PKUjnSR0SBHz5mDF4YT/RUAFJf2iOE7oyjN7dTdNN2V+L80OC55XB/sPSSPYuTQYisVkg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DUZPR04MB9967

Hi Sudeep,

> Subject: Re: [PATCH v5 0/3] pinctrl: scmi: support i.MX95 OEM extensions
>=20
> On Wed, May 08, 2024 at 11:32:01AM +0800, Peng Fan (OSS) wrote:
> > 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.MX9=
5.
> > 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
> >
>=20
> Not a review in particular, but if we decide to merge this deviation, it =
must be
> under the condition that it can be deleted anytime in the future if this
> becomes annoyance(like other vendors using this as a way to deviate from
> the specification). If we can't agree with that, then we better not merge=
 this
> at all.

It is ok for me. I agree.
But actually this driver still follows Spec by using OEM extensions.

If in future, the spec adds something saying pinmux bit field, and pinctrl-=
scmi.c
supports it, we could drop the imx driver.

Thanks,
Peng.

>=20
> --
> Regards,
> Sudeep