On Thu, Feb 01, 2024 at 07:14:17AM +0000, Peng Fan wrote:

quoted

Subject: Re: [PATCH v3 0/6] firmware: arm_scmi: Add SCMI v3.2 pincontrol
protocol basic support

Hi Peng,

quoted

On Mon, Jan 29, 2024 at 1:37 PM Peng Fan [off-list ref] wrote:

quoted

And for i.MX95 OEM extenstion, do you have any suggestions?
I have two points:
1. use vendor compatible. This would also benefit when supporting
vendor protocol.
2. Introduce a property saying supporting-generic-pinconf

How do you think?

While I don't know how OEM extensions to SCMI were designed, the pin
control subsystem has the philosophy that extensions are for minor fringe
stuff, such as a pin config option that no other silicon is using and thus have
no use for anyone else. Well that is actually all the custom extensions we
have.
(This notion is even carried over to SCMI pinctrl.)

The i.MX95 OEM extension is really odd to me, it looks like a
reimplementation of the core aspects of SCMI pin control, and looks much
more like the old i.MX drivers than like the SCMI driver.

i.MX SCMI pin protocol conf settings follows non-SCMI pin conf settings.

It is not just a matter of using custom SCMI OEM types, it is the whole
layout/definitions of the i.MX pin/groups/funcs DT bindings that deviates from
the generic DT bindings layout as handled and expected by the Linux Pinctrl
subsystem (AFAIU), while the SCMI Pinctrl driver as it stands in this series,
was conceived, designed and implemented originally by Oleksii to just use the
generic existing Pinctrl DT bindings; as a consequence, in your i.MX extensions,
you had to add a dedicated i.MX DT parser to interpret the protocol@19 DT snippet
in a completely different way, to try to stick your custom solution on top of
the generic one.

Thanks,
Cristian

quoted

But I sure cannot speak of what is allowed in SCMI OEM extensions or not.

+ SPEC owner, Souvik. Any comments?

Thanks,
Peng.

quoted

Yours,
Linus Walleij

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel