Re: [PATCH] ARM: dts: add phy-reset property for rk3066a-rayeager emac

2017-11-09 Thread Roger Quadros
On 09/11/17 10:28, Roger Quadros wrote: > On 08/11/17 11:49, Chris Zhong wrote: >> Hi Florian Fainelli >> >> >> On 2017年11月08日 02:26, Florian Fainelli wrote: >>> On 11/07/2017 01:51 AM, Chris Zhong wrote: On 2017年11月07日 15:54, Vladimir Zapolskiy wrote: > Hello Chris, > > On

Re: [PATCH] ARM: dts: add phy-reset property for rk3066a-rayeager emac

2017-11-09 Thread Roger Quadros
On 09/11/17 10:28, Roger Quadros wrote: > On 08/11/17 11:49, Chris Zhong wrote: >> Hi Florian Fainelli >> >> >> On 2017年11月08日 02:26, Florian Fainelli wrote: >>> On 11/07/2017 01:51 AM, Chris Zhong wrote: On 2017年11月07日 15:54, Vladimir Zapolskiy wrote: > Hello Chris, > > On

Re: [PATCH] ARM: dts: add phy-reset property for rk3066a-rayeager emac

2017-11-09 Thread Roger Quadros
On 08/11/17 11:49, Chris Zhong wrote: > Hi Florian Fainelli > > > On 2017年11月08日 02:26, Florian Fainelli wrote: >> On 11/07/2017 01:51 AM, Chris Zhong wrote: >>> >>> On 2017年11月07日 15:54, Vladimir Zapolskiy wrote: Hello Chris, On 11/07/2017 04:49 AM, Chris Zhong wrote: > The

Re: [PATCH] ARM: dts: add phy-reset property for rk3066a-rayeager emac

2017-11-09 Thread Roger Quadros
On 08/11/17 11:49, Chris Zhong wrote: > Hi Florian Fainelli > > > On 2017年11月08日 02:26, Florian Fainelli wrote: >> On 11/07/2017 01:51 AM, Chris Zhong wrote: >>> >>> On 2017年11月07日 15:54, Vladimir Zapolskiy wrote: Hello Chris, On 11/07/2017 04:49 AM, Chris Zhong wrote: > The

Re: [PATCH] ARM: dts: add phy-reset property for rk3066a-rayeager emac

2017-11-09 Thread Roger Quadros
Hi, On 07/11/17 09:54, Vladimir Zapolskiy wrote: > Hello Chris, > > On 11/07/2017 04:49 AM, Chris Zhong wrote: >> The ethernet phy of rk3066a-rayeager has a reset pin, it controlled by >> GPIO1_D6, this pin should be pull down then pull up to reset the phy. >> Add a phy-reset property in emac,

Re: [PATCH] ARM: dts: add phy-reset property for rk3066a-rayeager emac

2017-11-09 Thread Roger Quadros
Hi, On 07/11/17 09:54, Vladimir Zapolskiy wrote: > Hello Chris, > > On 11/07/2017 04:49 AM, Chris Zhong wrote: >> The ethernet phy of rk3066a-rayeager has a reset pin, it controlled by >> GPIO1_D6, this pin should be pull down then pull up to reset the phy. >> Add a phy-reset property in emac,

Re: [PATCH] ARM: dts: add phy-reset property for rk3066a-rayeager emac

2017-11-08 Thread Chris Zhong
Hi Florian Fainelli On 2017年11月08日 02:26, Florian Fainelli wrote: On 11/07/2017 01:51 AM, Chris Zhong wrote: On 2017年11月07日 15:54, Vladimir Zapolskiy wrote: Hello Chris, On 11/07/2017 04:49 AM, Chris Zhong wrote: The ethernet phy of rk3066a-rayeager has a reset pin, it controlled by

Re: [PATCH] ARM: dts: add phy-reset property for rk3066a-rayeager emac

2017-11-08 Thread Chris Zhong
Hi Florian Fainelli On 2017年11月08日 02:26, Florian Fainelli wrote: On 11/07/2017 01:51 AM, Chris Zhong wrote: On 2017年11月07日 15:54, Vladimir Zapolskiy wrote: Hello Chris, On 11/07/2017 04:49 AM, Chris Zhong wrote: The ethernet phy of rk3066a-rayeager has a reset pin, it controlled by

Re: [PATCH] ARM: dts: add phy-reset property for rk3066a-rayeager emac

2017-11-07 Thread Florian Fainelli
On 11/07/2017 01:51 AM, Chris Zhong wrote: > > > On 2017年11月07日 15:54, Vladimir Zapolskiy wrote: >> Hello Chris, >> >> On 11/07/2017 04:49 AM, Chris Zhong wrote: >>> The ethernet phy of rk3066a-rayeager has a reset pin, it controlled by >>> GPIO1_D6, this pin should be pull down then pull up to

Re: [PATCH] ARM: dts: add phy-reset property for rk3066a-rayeager emac

2017-11-07 Thread Florian Fainelli
On 11/07/2017 01:51 AM, Chris Zhong wrote: > > > On 2017年11月07日 15:54, Vladimir Zapolskiy wrote: >> Hello Chris, >> >> On 11/07/2017 04:49 AM, Chris Zhong wrote: >>> The ethernet phy of rk3066a-rayeager has a reset pin, it controlled by >>> GPIO1_D6, this pin should be pull down then pull up to

Re: [PATCH] ARM: dts: add phy-reset property for rk3066a-rayeager emac

2017-11-07 Thread Andrew Lunn
> As a side question, which is mainly addressed to Sergei and Roger, > I don't quite understand why PHY properties were initially added to > MAC/MDIO bus device tree nodes, in my opinion they must be moved under > PHY device tree nodes. If you look at the implementation, they are not actually PHY

Re: [PATCH] ARM: dts: add phy-reset property for rk3066a-rayeager emac

2017-11-07 Thread Andrew Lunn
> As a side question, which is mainly addressed to Sergei and Roger, > I don't quite understand why PHY properties were initially added to > MAC/MDIO bus device tree nodes, in my opinion they must be moved under > PHY device tree nodes. If you look at the implementation, they are not actually PHY

Re: [PATCH] ARM: dts: add phy-reset property for rk3066a-rayeager emac

2017-11-07 Thread Chris Zhong
On 2017年11月07日 15:54, Vladimir Zapolskiy wrote: Hello Chris, On 11/07/2017 04:49 AM, Chris Zhong wrote: The ethernet phy of rk3066a-rayeager has a reset pin, it controlled by GPIO1_D6, this pin should be pull down then pull up to reset the phy. Add a phy-reset property in emac, make the phy

Re: [PATCH] ARM: dts: add phy-reset property for rk3066a-rayeager emac

2017-11-07 Thread Chris Zhong
On 2017年11月07日 15:54, Vladimir Zapolskiy wrote: Hello Chris, On 11/07/2017 04:49 AM, Chris Zhong wrote: The ethernet phy of rk3066a-rayeager has a reset pin, it controlled by GPIO1_D6, this pin should be pull down then pull up to reset the phy. Add a phy-reset property in emac, make the phy

Re: [PATCH] ARM: dts: add phy-reset property for rk3066a-rayeager emac

2017-11-06 Thread Vladimir Zapolskiy
Hello Chris, On 11/07/2017 04:49 AM, Chris Zhong wrote: > The ethernet phy of rk3066a-rayeager has a reset pin, it controlled by > GPIO1_D6, this pin should be pull down then pull up to reset the phy. > Add a phy-reset property in emac, make the phy can be reset when emac > power on. for PHY

Re: [PATCH] ARM: dts: add phy-reset property for rk3066a-rayeager emac

2017-11-06 Thread Vladimir Zapolskiy
Hello Chris, On 11/07/2017 04:49 AM, Chris Zhong wrote: > The ethernet phy of rk3066a-rayeager has a reset pin, it controlled by > GPIO1_D6, this pin should be pull down then pull up to reset the phy. > Add a phy-reset property in emac, make the phy can be reset when emac > power on. for PHY