linux-kernel - Re: [PATCH 4/6] ARM: dts: vf: rename io-expander@20 to pinctrl@20

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] [day] [month] [year] [list]

Message-ID: <aDcuOFhYxWs9VXl6@lizhi-Precision-Tower-5810>
Date: Wed, 28 May 2025 11:39:36 -0400
From: Frank Li <Frank.li@....com>
To: Peng Fan <peng.fan@....com>
Cc: Shawn Guo <shawnguo@...nel.org>, Sascha Hauer <s.hauer@...gutronix.de>,
	Pengutronix Kernel Team <kernel@...gutronix.de>,
	Stefan Agner <stefan@...er.ch>, Rob Herring <robh@...nel.org>,
	Krzysztof Kozlowski <krzk+dt@...nel.org>,
	Conor Dooley <conor+dt@...nel.org>,
	"moderated list:ARM/FREESCALE VYBRID ARM ARCHITECTURE" <linux-arm-kernel@...ts.infradead.org>,
	"open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" <devicetree@...r.kernel.org>,
	open list <linux-kernel@...r.kernel.org>,
	"imx@...ts.linux.dev" <imx@...ts.linux.dev>
Subject: Re: [PATCH 4/6] ARM: dts: vf: rename io-expander@20 to pinctrl@20

On Tue, May 27, 2025 at 05:24:23AM +0000, Peng Fan wrote:
> > Subject: [PATCH 4/6] ARM: dts: vf: rename io-expander@20 to
> > pinctrl@20
> >
> > Rename io-expander@20 to pinctrl@20 to fix below CHECK_DTB
> > warning:
> >
> > arch/arm/boot/dts/nxp/vf/vf610-zii-dev-rev-c.dtb: sx1503@20
> > (semtech,sx1503q):
> >     $nodename:0: 'sx1503@20' does not match '^(pinctrl|pinmux)(@[0-
> > 9a-f]+)?$
> >
> > Signed-off-by: Frank Li <Frank.Li@....com>
> > ---
> >  arch/arm/boot/dts/nxp/vf/vf610-zii-dev-rev-c.dts | 2 +-
> > arch/arm/boot/dts/nxp/vf/vf610-zii-scu4-aib.dts  | 2 +-
> >  2 files changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/arch/arm/boot/dts/nxp/vf/vf610-zii-dev-rev-c.dts
> > b/arch/arm/boot/dts/nxp/vf/vf610-zii-dev-rev-c.dts
> > index 7a209c9c57a4d..79ea7cf57a4dc 100644
> > --- a/arch/arm/boot/dts/nxp/vf/vf610-zii-dev-rev-c.dts
> > +++ b/arch/arm/boot/dts/nxp/vf/vf610-zii-dev-rev-c.dts
> > @@ -311,7 +311,7 @@ gpio5: io-expander@18 {
> >  	 *    I/O14 - OPT1_TX_DIS
> >  	 *    I/O15 - OPT2_TX_DIS
> >  	 */
> > -	gpio6: sx1503@20 {
> > +	gpio6: pinctrl@20 {
>
> Use gpio-controller?

binding doc require pinctrl

Documentation/devicetree/bindings/pinctrl/semtech,sx1501q.yaml

Frank
>
> >  		compatible = "semtech,sx1503q";
> >
> >  		pinctrl-names = "default";
> > diff --git a/arch/arm/boot/dts/nxp/vf/vf610-zii-scu4-aib.dts
> > b/arch/arm/boot/dts/nxp/vf/vf610-zii-scu4-aib.dts
> > index b3d338312df4f..8020a644dd9d1 100644
> > --- a/arch/arm/boot/dts/nxp/vf/vf610-zii-scu4-aib.dts
> > +++ b/arch/arm/boot/dts/nxp/vf/vf610-zii-scu4-aib.dts
> > @@ -583,7 +583,7 @@ &i2c2 {
> >  	pinctrl-0 = <&pinctrl_i2c2>;
> >  	status = "okay";
> >
> > -	gpio9: io-expander@20 {
> > +	gpio9: pinctrl@20 {
>
> Ditto.
>
> Regards
> Peng.
>
> >  		compatible = "semtech,sx1503q";
> >  		pinctrl-names = "default";
> >  		pinctrl-0 = <&pinctrl_sx1503_20>;
> > --
> > 2.34.1
> >
>