Re: [PATCH v1 0/4] arm64: dts: Mark Variscite i.MX9 I2C recovery GPIOs as open drain

[Stefano Radaelli <stefano.radaelli21@gmail.com>]

On Tue, Oct 20, 2026 at 02:41:04PM +0800, Peng Fan (OSS) wrote:
> On Sun, 18 Oct 2026 14:32:23 +0200 Stefano Radaelli <stefano.radaelli21@gmail.com> wrote:
> 
> > The I2C recovery GPIOs on VAR-SOM-MX91 and VAR-SOM-MX93, including their
> > Symphony carrier boards, are missing the open-drain flag.
> > 
> > This series adds GPIO_OPEN_DRAIN to the SCL and SDA GPIO specifiers to
> > match the I2C bus requirements and avoid the warning emitted when the
> > I2C core enforces open-drain operation on SCL.
> > 
> > Stefano Radaelli (4):
> >   arm64: dts: imx91-var-som: Mark I2C recovery GPIOs as open drain
> >   arm64: dts: imx91-var-som-symphony: Mark I2C recovery GPIOs as open
> >     drain
> >   arm64: dts: imx93-var-som: Mark I2C recovery GPIOs as open drain
> >   arm64: dts: imx93-var-som-symphony: Mark I2C recovery GPIOs as open
> >     drain
> > 
> >  arch/arm64/boot/dts/freescale/imx91-var-som-symphony.dts | 4 ++--
> >  arch/arm64/boot/dts/freescale/imx91-var-som.dtsi         | 4 ++--
> >  arch/arm64/boot/dts/freescale/imx93-var-som-symphony.dts | 8 ++++----
> >  arch/arm64/boot/dts/freescale/imx93-var-som.dtsi         | 4 ++--
> >  4 files changed, 10 insertions(+), 10 deletions(-)
> > 
> 
> LGTM:
> 
> Reviewed-by: Peng Fan <peng.fan@nxp.com>
> 
> BTW: Do you need fixes tag?

Hi Peng,

Thanks for the review!
I treated this as a DT cleanup since the core already enforces
open-drain operation on SCL, so I think it is fine to keep the series
without Fixes tags.

Thanks,
Best Regards,
Stefano