Re: [PATCH v2 3/7] arm64: dts: freescale: Add Lino iMX93

[Francesco Dolcini <francesco@dolcini.it>]

Hello Frank,

On Fri, Sep 11, 2026 at 09:30:36AM -0500, Frank Li wrote:
> On Fri, Sep 11, 2026 at 11:19:46AM +0200, Francesco Dolcini wrote:
> > From: Francesco Dolcini <francesco.dolcini@toradex.com>
> >
> > Add support for the Lino iMX93 SoM, its Verdin adapter and development
> > carrier board.
> >
> > The SoM is available in two variants, with i.MX91 or i.MX93 SoC, using
> > the same PCB (the two chips are pin to pin compatible), to avoid
> > duplication create a common dtsi to share commonalities.
> >
> > Link: https://www.toradex.com/computer-on-modules/lino-arm-family/nxp-imx93
> > Link: https://www.toradex.com/accessories/lino-verdin-adapter
> > Link: https://www.toradex.com/products/carrier-board/verdin-development-board-kit
> > Co-developed-by: Emanuele Ghidoli <emanuele.ghidoli@toradex.com>
> > Signed-off-by: Emanuele Ghidoli <emanuele.ghidoli@toradex.com>
> > Co-developed-by: Ernest Van Hoecke <ernest.vanhoecke@toradex.com>
> > Signed-off-by: Ernest Van Hoecke <ernest.vanhoecke@toradex.com>
> > Co-developed-by: Franz Schnyder <franz.schnyder@toradex.com>
> > Signed-off-by: Franz Schnyder <franz.schnyder@toradex.com>
> > Signed-off-by: Francesco Dolcini <francesco.dolcini@toradex.com>

...

> > +&gpio1 {
> > +	gpio-line-names =
> > +		"LINO_P18",
> > +		"LINO_P16",
> > +		"PMIC_I2C_SCL",
> > +		"PMIC_I2C_SDA",
> > +		"",
> > +		"",
> > +		"",
> > +		"",
> > +		"",
> > +		"",
> > +		"LINO_P65", /* 10 */
> > +		"LINO_S18",
> > +		"LINO_S24",
> > +		"LINO_S20",
> > +		"LINO_S22",
> > +		"";
> 
> Can you wrap to 4 names pre line
> 
> 	gpio-line-names = "LINO_P18", "LINO_P16", "PMIC_I2C_SCL", "PMIC_I2C_SDA",
> 			  "", "", "", "",
> 
> Reduce LOC, and easy to locate number, such gpio 13,  second of 3rd line

I already replied on the v1 that I disagree on your request. To me it is
just more readable the way it is now and I do not see minimizing the
lines of code something that is bringing any kind of benefit on its own.

Is this blocking this from being picked up? 
Is there any other concern on this series?

Francesco