| Message ID | 20191015152654.26726-3-andrew.smirnov@gmail.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | [1/4] arm64: dts: zii-ultra: Fix regulator-vsd-3v3's vin-supply | expand |
On Tue, Oct 15, 2019 at 08:26:53AM -0700, Andrey Smirnov wrote: > Add I2C node for accelerometer present on both Zest and RMB3 boards. > > Signed-off-by: Andrey Smirnov <andrew.smirnov@gmail.com> > Cc: Fabio Estevam <festevam@gmail.com> > Cc: Chris Healy <cphealy@gmail.com> > Cc: Lucas Stach <l.stach@pengutronix.de> > Cc: Shawn Guo <shawnguo@kernel.org> > Cc: linux-arm-kernel@lists.infradead.org, > Cc: linux-kernel@vger.kernel.org > --- > .../boot/dts/freescale/imx8mq-zii-ultra.dtsi | 18 ++++++++++++++++++ > 1 file changed, 18 insertions(+) > > diff --git a/arch/arm64/boot/dts/freescale/imx8mq-zii-ultra.dtsi b/arch/arm64/boot/dts/freescale/imx8mq-zii-ultra.dtsi > index 21eb52341ba8..8395c5a73ba6 100644 > --- a/arch/arm64/boot/dts/freescale/imx8mq-zii-ultra.dtsi > +++ b/arch/arm64/boot/dts/freescale/imx8mq-zii-ultra.dtsi > @@ -262,6 +262,18 @@ > pinctrl-0 = <&pinctrl_i2c1>; > status = "okay"; > > + accel@1c { s/accel/accelerometer I fixed it up and applied the series. Shawn > + compatible = "fsl,mma8451"; > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_accel>; > + reg = <0x1c>; > + interrupt-parent = <&gpio3>; > + interrupts = <20 IRQ_TYPE_LEVEL_LOW>; > + interrupt-names = "INT2"; > + vdd-supply = <®_gen_3p3>; > + vddio-supply = <®_gen_3p3>; > + }; > + > ucs1002: charger@32 { > compatible = "microchip,ucs1002"; > pinctrl-names = "default"; > @@ -522,6 +534,12 @@ > }; > > &iomuxc { > + pinctrl_accel: accelgrp { > + fsl,pins = < > + MX8MQ_IOMUXC_SAI5_RXC_GPIO3_IO20 0x41 > + >; > + }; > + > pinctrl_fec1: fec1grp { > fsl,pins = < > MX8MQ_IOMUXC_ENET_MDC_ENET1_MDC 0x3 > -- > 2.21.0 >
On Sat, Oct 26, 2019 at 4:55 AM Shawn Guo <shawnguo@kernel.org> wrote: > > On Tue, Oct 15, 2019 at 08:26:53AM -0700, Andrey Smirnov wrote: > > Add I2C node for accelerometer present on both Zest and RMB3 boards. > > > > Signed-off-by: Andrey Smirnov <andrew.smirnov@gmail.com> > > Cc: Fabio Estevam <festevam@gmail.com> > > Cc: Chris Healy <cphealy@gmail.com> > > Cc: Lucas Stach <l.stach@pengutronix.de> > > Cc: Shawn Guo <shawnguo@kernel.org> > > Cc: linux-arm-kernel@lists.infradead.org, > > Cc: linux-kernel@vger.kernel.org > > --- > > .../boot/dts/freescale/imx8mq-zii-ultra.dtsi | 18 ++++++++++++++++++ > > 1 file changed, 18 insertions(+) > > > > diff --git a/arch/arm64/boot/dts/freescale/imx8mq-zii-ultra.dtsi b/arch/arm64/boot/dts/freescale/imx8mq-zii-ultra.dtsi > > index 21eb52341ba8..8395c5a73ba6 100644 > > --- a/arch/arm64/boot/dts/freescale/imx8mq-zii-ultra.dtsi > > +++ b/arch/arm64/boot/dts/freescale/imx8mq-zii-ultra.dtsi > > @@ -262,6 +262,18 @@ > > pinctrl-0 = <&pinctrl_i2c1>; > > status = "okay"; > > > > + accel@1c { > > s/accel/accelerometer > > I fixed it up and applied the series. > I'm fine with that change, but FYI, I originally had it as "accelerometer', but changed to "accel" to match the name in DT for RDU2. Thanks, Andrey Smirnov
On Sat, Oct 26, 2019 at 11:54:13AM -0700, Andrey Smirnov wrote: > On Sat, Oct 26, 2019 at 4:55 AM Shawn Guo <shawnguo@kernel.org> wrote: > > > > On Tue, Oct 15, 2019 at 08:26:53AM -0700, Andrey Smirnov wrote: > > > Add I2C node for accelerometer present on both Zest and RMB3 boards. > > > > > > Signed-off-by: Andrey Smirnov <andrew.smirnov@gmail.com> > > > Cc: Fabio Estevam <festevam@gmail.com> > > > Cc: Chris Healy <cphealy@gmail.com> > > > Cc: Lucas Stach <l.stach@pengutronix.de> > > > Cc: Shawn Guo <shawnguo@kernel.org> > > > Cc: linux-arm-kernel@lists.infradead.org, > > > Cc: linux-kernel@vger.kernel.org > > > --- > > > .../boot/dts/freescale/imx8mq-zii-ultra.dtsi | 18 ++++++++++++++++++ > > > 1 file changed, 18 insertions(+) > > > > > > diff --git a/arch/arm64/boot/dts/freescale/imx8mq-zii-ultra.dtsi b/arch/arm64/boot/dts/freescale/imx8mq-zii-ultra.dtsi > > > index 21eb52341ba8..8395c5a73ba6 100644 > > > --- a/arch/arm64/boot/dts/freescale/imx8mq-zii-ultra.dtsi > > > +++ b/arch/arm64/boot/dts/freescale/imx8mq-zii-ultra.dtsi > > > @@ -262,6 +262,18 @@ > > > pinctrl-0 = <&pinctrl_i2c1>; > > > status = "okay"; > > > > > > + accel@1c { > > > > s/accel/accelerometer > > > > I fixed it up and applied the series. > > > > I'm fine with that change, but FYI, I originally had it as > "accelerometer', but changed to "accel" to match the name in DT for > RDU2. We should change RDU2 instead, IMO. Shawn
diff --git a/arch/arm64/boot/dts/freescale/imx8mq-zii-ultra.dtsi b/arch/arm64/boot/dts/freescale/imx8mq-zii-ultra.dtsi index 21eb52341ba8..8395c5a73ba6 100644 --- a/arch/arm64/boot/dts/freescale/imx8mq-zii-ultra.dtsi +++ b/arch/arm64/boot/dts/freescale/imx8mq-zii-ultra.dtsi @@ -262,6 +262,18 @@ pinctrl-0 = <&pinctrl_i2c1>; status = "okay"; + accel@1c { + compatible = "fsl,mma8451"; + pinctrl-names = "default"; + pinctrl-0 = <&pinctrl_accel>; + reg = <0x1c>; + interrupt-parent = <&gpio3>; + interrupts = <20 IRQ_TYPE_LEVEL_LOW>; + interrupt-names = "INT2"; + vdd-supply = <®_gen_3p3>; + vddio-supply = <®_gen_3p3>; + }; + ucs1002: charger@32 { compatible = "microchip,ucs1002"; pinctrl-names = "default"; @@ -522,6 +534,12 @@ }; &iomuxc { + pinctrl_accel: accelgrp { + fsl,pins = < + MX8MQ_IOMUXC_SAI5_RXC_GPIO3_IO20 0x41 + >; + }; + pinctrl_fec1: fec1grp { fsl,pins = < MX8MQ_IOMUXC_ENET_MDC_ENET1_MDC 0x3
Add I2C node for accelerometer present on both Zest and RMB3 boards. Signed-off-by: Andrey Smirnov <andrew.smirnov@gmail.com> Cc: Fabio Estevam <festevam@gmail.com> Cc: Chris Healy <cphealy@gmail.com> Cc: Lucas Stach <l.stach@pengutronix.de> Cc: Shawn Guo <shawnguo@kernel.org> Cc: linux-arm-kernel@lists.infradead.org, Cc: linux-kernel@vger.kernel.org --- .../boot/dts/freescale/imx8mq-zii-ultra.dtsi | 18 ++++++++++++++++++ 1 file changed, 18 insertions(+)