Re: [PATCH 4/5] ARM: dts: imx6sx-sabreauto: add fec support

2018-05-04 Thread Shawn Guo
On Thu, Apr 26, 2018 at 06:57:07AM +, Anson Huang wrote: > > > Sorry, I made a mistake here, the MAX7320 IO0 is for adjusting FEC1's > > > voltage, > > > > In this case you need to pass the 'phy-supply' property inside the fec node > > and > > add a regulator that is controlled via MAX7320 IO

RE: [PATCH 4/5] ARM: dts: imx6sx-sabreauto: add fec support

2018-04-26 Thread Anson Huang
SCALE IMX / MXC > ARM ARCHITECTURE ; open list:OPEN > FIRMWARE AND FLATTENED DEVICE TREE BINDINGS > ; linux-kernel ; > Andy Duan > Subject: RE: [PATCH 4/5] ARM: dts: imx6sx-sabreauto: add fec support > > > > Anson Huang > Best Regards! > > > > -

RE: [PATCH 4/5] ARM: dts: imx6sx-sabreauto: add fec support

2018-04-26 Thread Anson Huang
d list:ARM/FREESCALE IMX / MXC > ARM ARCHITECTURE ; open list:OPEN > FIRMWARE AND FLATTENED DEVICE TREE BINDINGS > ; linux-kernel ; > Andy Duan > Subject: Re: [PATCH 4/5] ARM: dts: imx6sx-sabreauto: add fec support > > On Thu, Apr 26, 2018 at 3:57 AM, Anson Huang >

Re: [PATCH 4/5] ARM: dts: imx6sx-sabreauto: add fec support

2018-04-26 Thread Fabio Estevam
On Thu, Apr 26, 2018 at 3:57 AM, Anson Huang wrote: > The 'phy-supply' is for enabling/disabling phy regulator, but here the > MAX7322 IO0 is NOT for > enabling/disabling PHY regulator, it is for IO voltage switch between 1.5V > and 1.8V, our ENET > IO can work with both 1.5V and 1.8V, so any c

RE: [PATCH 4/5] ARM: dts: imx6sx-sabreauto: add fec support

2018-04-25 Thread Anson Huang
imx ; moderated list:ARM/FREESCALE IMX / MXC > ARM ARCHITECTURE ; open list:OPEN > FIRMWARE AND FLATTENED DEVICE TREE BINDINGS > ; linux-kernel > Subject: Re: [PATCH 4/5] ARM: dts: imx6sx-sabreauto: add fec support > > Hi Anson, > > On Wed, Apr 25, 2018 at 2:36 AM, Anson Hua

Re: [PATCH 4/5] ARM: dts: imx6sx-sabreauto: add fec support

2018-04-25 Thread Fabio Estevam
Hi Anson, On Wed, Apr 25, 2018 at 2:36 AM, Anson Huang wrote: > Sorry, I made a mistake here, the MAX7320 IO0 is for adjusting FEC1's voltage, In this case you need to pass the 'phy-supply' property inside the fec node and add a regulator that is controlled via MAX7320 IO0 pin.

RE: [PATCH 4/5] ARM: dts: imx6sx-sabreauto: add fec support

2018-04-24 Thread Anson Huang
list:ARM/FREESCALE IMX / MXC > ARM ARCHITECTURE ; open list:OPEN > FIRMWARE AND FLATTENED DEVICE TREE BINDINGS > ; linux-kernel > Subject: RE: [PATCH 4/5] ARM: dts: imx6sx-sabreauto: add fec support > > Hi, Fabio > > Anson Huang > Best Regards! > > > >

RE: [PATCH 4/5] ARM: dts: imx6sx-sabreauto: add fec support

2018-04-24 Thread Anson Huang
imx ; moderated list:ARM/FREESCALE IMX / MXC > ARM ARCHITECTURE ; open list:OPEN > FIRMWARE AND FLATTENED DEVICE TREE BINDINGS > ; linux-kernel > Subject: Re: [PATCH 4/5] ARM: dts: imx6sx-sabreauto: add fec support > > Hi Anson, > > On Mon, Apr 23, 2018 at 11:09 PM, Anson Hua

Re: [PATCH 4/5] ARM: dts: imx6sx-sabreauto: add fec support

2018-04-24 Thread Fabio Estevam
Hi Anson, On Mon, Apr 23, 2018 at 11:09 PM, Anson Huang wrote: > Ah, yes, thanks for pointing out this issue, I just removed it and the > function is still working, > already sent out V2 patch set, thanks. So now maybe it is working only because the bootloader activated this GPIO, which is not

RE: [PATCH 4/5] ARM: dts: imx6sx-sabreauto: add fec support

2018-04-23 Thread Anson Huang
imx ; moderated list:ARM/FREESCALE IMX / MXC > ARM ARCHITECTURE ; open list:OPEN > FIRMWARE AND FLATTENED DEVICE TREE BINDINGS > ; linux-kernel > Subject: Re: [PATCH 4/5] ARM: dts: imx6sx-sabreauto: add fec support > > Hi Anson, > > On Mon, Apr 23, 2018 at 12:46 AM, Anson Huan

Re: [PATCH 4/5] ARM: dts: imx6sx-sabreauto: add fec support

2018-04-23 Thread Fabio Estevam
Hi Anson, On Mon, Apr 23, 2018 at 12:46 AM, Anson Huang wrote: > Add FEC support on i.MX6SX Sabre Auto board. > > Signed-off-by: Anson Huang > --- > arch/arm/boot/dts/imx6sx-sabreauto.dts | 70 > ++ > 1 file changed, 70 insertions(+) > > diff --git a/arch/arm/bo