linux-kernel - Re: [PATCH] pinctrl: scmi: support i.MX OEM pin configuration type

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CACRpkdYX1CtmOkDhyfCAcbY2yHHfjjRhTg0r=sW3iA6dvxvmdw@mail.gmail.com>
Date: Thu, 29 Feb 2024 14:51:16 +0100
From: Linus Walleij <linus.walleij@...aro.org>
To: "Peng Fan (OSS)" <peng.fan@....nxp.com>, Sascha Hauer <kernel@...gutronix.de>
Cc: sudeep.holla@....com, cristian.marussi@....com, 
	linux-kernel@...r.kernel.org, linux-arm-kernel@...ts.infradead.org, 
	linux-gpio@...r.kernel.org, aisheng.dong@....com, nitin.garg_3@....com, 
	ranjani.vaidyanathan@....com, ye.li@....com, Peng Fan <peng.fan@....com>
Subject: Re: [PATCH] pinctrl: scmi: support i.MX OEM pin configuration type

On Fri, Feb 23, 2024 at 8:07 AM Peng Fan (OSS) <peng.fan@....nxp.com> wrote:

> From: Peng Fan <peng.fan@....com>
>
> i.MX95 System Manager FW supports SCMI PINCTRL protocol, but uses
> OEM Pin Configuration type, so extend the driver to support custom
> params.
>
> Signed-off-by: Peng Fan <peng.fan@....com>

I can't really say much about this as pinctrl maintainer other than that
it makes me a bit unhappy that i.MX95 is not using the "default"
SCMI pinctrl bindings.

If the spec allows for this, and NXP Freescale is using it, I will just
have to accept it.

It feels like that's the old NXP Freescale pin controller living on
just hidden behind SCMI, so potentially it should also share code
with the old i.MX pin controller driver. But I think you wrote part of
that driver so you would be the best to ask about that in any case
I think?

Yours,
Linus Walleij