RE: [PATCH V4 3/6] ARM: dts: imx6sx-sabreauto: add IO expander max7310 support

2018-04-26 Thread Anson Huang
ARM Linux ; dl-linux-imx > ; moderated list:ARM/FREESCALE IMX / MXC ARM > ARCHITECTURE ; open list:OPEN > FIRMWARE AND FLATTENED DEVICE TREE BINDINGS > ; linux-kernel > Subject: Re: [PATCH V4 3/6] ARM: dts: imx6sx-sabreauto: add IO expander > max7310 support > > On Thu, Apr 26,

Re: [PATCH V4 3/6] ARM: dts: imx6sx-sabreauto: add IO expander max7310 support

2018-04-26 Thread Fabio Estevam
On Thu, Apr 26, 2018 at 3:31 AM, Anson Huang wrote: > I searched the mail I sent, I can NOT found where I said "MAX7310 controls > the ethernet phy" Initially you passed: pinctrl-assert-gpios = <&max7322 0 GPIO_ACTIVE_HIGH> inside the fec1 node. Then I explained that 'pinctrl-assert-gpios

RE: [PATCH V4 3/6] ARM: dts: imx6sx-sabreauto: add IO expander max7310 support

2018-04-25 Thread Anson Huang
ARM Linux ; dl-linux-imx > ; moderated list:ARM/FREESCALE IMX / MXC ARM > ARCHITECTURE ; open list:OPEN > FIRMWARE AND FLATTENED DEVICE TREE BINDINGS > ; linux-kernel > Subject: Re: [PATCH V4 3/6] ARM: dts: imx6sx-sabreauto: add IO expander > max7310 support > > On Thu

Re: [PATCH V4 3/6] ARM: dts: imx6sx-sabreauto: add IO expander max7310 support

2018-04-25 Thread Fabio Estevam
On Thu, Apr 26, 2018 at 12:06 AM, Anson Huang wrote: > MAX7310 pins are for PCIe, CAN, MLB. GPS etc., these modules are NOT enabled > yet, so > I can NOT test it, I think we should add MAX7310 users when these modules are > enabled, > ethernet PHY is NOT controlled by it. You said previously t

RE: [PATCH V4 3/6] ARM: dts: imx6sx-sabreauto: add IO expander max7310 support

2018-04-25 Thread Anson Huang
g - ARM Linux ; dl-linux-imx > ; moderated list:ARM/FREESCALE IMX / MXC ARM > ARCHITECTURE ; open list:OPEN > FIRMWARE AND FLATTENED DEVICE TREE BINDINGS > ; linux-kernel > Subject: Re: [PATCH V4 3/6] ARM: dts: imx6sx-sabreauto: add IO expander > max7310 support > > On We

Re: [PATCH V4 3/6] ARM: dts: imx6sx-sabreauto: add IO expander max7310 support

2018-04-25 Thread Fabio Estevam
On Wed, Apr 25, 2018 at 2:36 AM, Anson Huang wrote: > +&i2c3 { > + clock-frequency = <10>; > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_i2c3_2>; > + status = "okay"; > + > + max7310_a: gpio@30 { > + compatible = "maxim,max7310"; > +

[PATCH V4 3/6] ARM: dts: imx6sx-sabreauto: add IO expander max7310 support

2018-04-24 Thread Anson Huang
i.MX6SX Sabre Auto board has two max7310 IO expander on I2C3 bus, add support for them. Signed-off-by: Anson Huang --- no changes. arch/arm/boot/dts/imx6sx-sabreauto.dts | 28 1 file changed, 28 insertions(+) diff --git a/arch/arm/boot/dts/imx6sx-sabreauto.dts b/ar