Re: [PATCH 1/2] net: phy: at803x: Fix RGMII RX and TX clock delays setup

2017-07-19 Thread Florian Fainelli
On 07/19/2017 02:29 PM, Mason wrote: > On 19/07/2017 21:30, Florian Fainelli wrote: >> On 07/19/2017 12:24 PM, Grygorii Strashko wrote: >>> Hi >>> >>> On 07/19/2017 10:31 AM, Marc Gonzalez wrote: The current code supports enabling RGMII RX and TX clock delays. The unstated assumption is

Re: [PATCH 1/2] net: phy: at803x: Fix RGMII RX and TX clock delays setup

2017-07-19 Thread Mason
On 19/07/2017 21:30, Florian Fainelli wrote: > On 07/19/2017 12:24 PM, Grygorii Strashko wrote: >> Hi >> >> On 07/19/2017 10:31 AM, Marc Gonzalez wrote: >>> The current code supports enabling RGMII RX and TX clock delays. >>> The unstated assumption is that these settings are disabled by >>>

Re: [PATCH 1/2] net: phy: at803x: Fix RGMII RX and TX clock delays setup

2017-07-19 Thread Grygorii Strashko
On 07/19/2017 02:30 PM, Florian Fainelli wrote: On 07/19/2017 12:24 PM, Grygorii Strashko wrote: Hi On 07/19/2017 10:31 AM, Marc Gonzalez wrote: The current code supports enabling RGMII RX and TX clock delays. The unstated assumption is that these settings are disabled by default at reset,

Re: [PATCH 1/2] net: phy: at803x: Fix RGMII RX and TX clock delays setup

2017-07-19 Thread Florian Fainelli
On 07/19/2017 12:24 PM, Grygorii Strashko wrote: > Hi > > On 07/19/2017 10:31 AM, Marc Gonzalez wrote: >> The current code supports enabling RGMII RX and TX clock delays. >> The unstated assumption is that these settings are disabled by >> default at reset, which is not the case. >> >> RX clock

Re: [PATCH 1/2] net: phy: at803x: Fix RGMII RX and TX clock delays setup

2017-07-19 Thread Grygorii Strashko
Hi On 07/19/2017 10:31 AM, Marc Gonzalez wrote: > The current code supports enabling RGMII RX and TX clock delays. > The unstated assumption is that these settings are disabled by > default at reset, which is not the case. > > RX clock delay is enabled at reset. And TX clock delay "survives" >

Re: [PATCH 1/2] net: phy: at803x: Fix RGMII RX and TX clock delays setup

2017-07-19 Thread Timur Tabi
On 07/19/2017 10:31 AM, Marc Gonzalez wrote: The current code supports enabling RGMII RX and TX clock delays. The unstated assumption is that these settings are disabled by default at reset, which is not the case. RX clock delay is enabled at reset. And TX clock delay "survives" across SW