linux-kernel - Re: [PATCH 3/4] arm64: dts: imx8mp-phyboard-pollux: Enable i2c3

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: <8269481ecde6757a18f08ab76e35a8689f48a58e.camel@phytec.de>
Date: Wed, 22 Oct 2025 06:20:21 +0000
From: Yannic Moog <y.moog@...tec.de>
To: "Frank.li@....com" <Frank.li@....com>
CC: Teresa Remmet <t.remmet@...tec.de>, "kernel@...gutronix.de"
	<kernel@...gutronix.de>, Stefan Riedmüller
	<s.riedmueller@...tec.de>, "festevam@...il.com" <festevam@...il.com>,
	"s.hauer@...gutronix.de" <s.hauer@...gutronix.de>, "robh@...nel.org"
	<robh@...nel.org>, "shawnguo@...nel.org" <shawnguo@...nel.org>,
	"krzk+dt@...nel.org" <krzk+dt@...nel.org>, "upstream@...ts.phytec.de"
	<upstream@...ts.phytec.de>, "linux-arm-kernel@...ts.infradead.org"
	<linux-arm-kernel@...ts.infradead.org>, "conor+dt@...nel.org"
	<conor+dt@...nel.org>, "imx@...ts.linux.dev" <imx@...ts.linux.dev>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	"devicetree@...r.kernel.org" <devicetree@...r.kernel.org>
Subject: Re: [PATCH 3/4] arm64: dts: imx8mp-phyboard-pollux: Enable i2c3

On Mon, 2025-10-20 at 11:38 -0400, Frank Li wrote:
> On Mon, Oct 20, 2025 at 03:11:24PM +0200, Yannic Moog wrote:
> > From: Stefan Riedmueller <s.riedmueller@...tec.de>
> > 
> > On the phyBOARD-Pollux the i2c3 node is used on the CSI1 interface to
> > connect to imaging sensors. Thus define it so it can be easily enabled if
> > required.
> Nit:
> 
> The i2c3 of the phyBOARD-Pollux is used ...

Sorry, I can't follow. Would you like me to rephrase the commit message?

Yannic

> 
> Reviewed-by: Frank Li <Frank.Li@....com>
> 
> > 
> > Signed-off-by: Stefan Riedmueller <s.riedmueller@...tec.de>
> > Signed-off-by: Teresa Remmet <t.remmet@...tec.de>
> > Signed-off-by: Yannic Moog <y.moog@...tec.de>
> > ---
> >  .../dts/freescale/imx8mp-phyboard-pollux-rdk.dts   | 23 ++++++++++++++++++++++
> >  1 file changed, 23 insertions(+)
> > 
> > diff --git a/arch/arm64/boot/dts/freescale/imx8mp-phyboard-pollux-rdk.dts
> > b/arch/arm64/boot/dts/freescale/imx8mp-phyboard-pollux-rdk.dts
> > index 6203e39bc01be476f16f5ac80b6365bce150ae37..7d34b820e3213a3832c5be47444d4e9c636a6202 100644
> > --- a/arch/arm64/boot/dts/freescale/imx8mp-phyboard-pollux-rdk.dts
> > +++ b/arch/arm64/boot/dts/freescale/imx8mp-phyboard-pollux-rdk.dts
> > @@ -228,6 +228,15 @@ led-3 {
> >  	};
> >  };
> > 
> > +&i2c3 {
> > +	clock-frequency = <400000>;
> > +	pinctrl-names = "default", "gpio";
> > +	pinctrl-0 = <&pinctrl_i2c3>;
> > +	pinctrl-1 = <&pinctrl_i2c3_gpio>;
> > +	sda-gpios = <&gpio5 18 (GPIO_ACTIVE_HIGH | GPIO_OPEN_DRAIN)>;
> > +	scl-gpios = <&gpio5 19 (GPIO_ACTIVE_HIGH | GPIO_OPEN_DRAIN)>;
> > +};
> > +
> >  &ldb_lvds_ch1 {
> >  	remote-endpoint = <&panel1_in>;
> >  };
> > @@ -442,6 +451,20 @@ MX8MP_IOMUXC_I2C2_SDA__GPIO5_IO17	0x1e2
> >  		>;
> >  	};
> > 
> > +	pinctrl_i2c3: i2c3grp {
> > +		fsl,pins = <
> > +			MX8MP_IOMUXC_I2C3_SCL__I2C3_SCL		0x400001c2
> > +			MX8MP_IOMUXC_I2C3_SDA__I2C3_SDA		0x400001c2
> > +		>;
> > +	};
> > +
> > +	pinctrl_i2c3_gpio: i2c3gpiogrp {
> > +		fsl,pins = <
> > +			MX8MP_IOMUXC_I2C3_SCL__GPIO5_IO18	0x1e2
> > +			MX8MP_IOMUXC_I2C3_SDA__GPIO5_IO19	0x1e2
> > +		>;
> > +	};
> > +
> >  	pinctrl_lvds1: lvds1grp {
> >  		fsl,pins = <
> >  			MX8MP_IOMUXC_SD2_WP__GPIO2_IO20		0x12
> > 
> > --
> > 2.51.0
> >