Re: stmmac: GMAC_RGSMIIIS reports bogus values

2017-01-31 Thread Giuseppe CAVALLARO
Hello Alexey On 1/31/2017 2:24 PM, Alexey Brodkin wrote: Hi Giuseppe, On Tue, 2017-01-31 at 10:55 +0100, Giuseppe CAVALLARO wrote: On 1/27/2017 11:23 AM, Alexey Brodkin wrote: That's why my initial proposal was to ignore whatever we read from this register if we have MDIO bus instantiated al

Re: stmmac: GMAC_RGSMIIIS reports bogus values

2017-01-31 Thread Alexey Brodkin
HiĀ Giuseppe, On Tue, 2017-01-31 at 10:55 +0100, Giuseppe CAVALLARO wrote: > On 1/27/2017 11:23 AM, Alexey Brodkin wrote: > > > > That's why my initial proposal was to ignore whatever we read from this > > register > > if we have MDIO bus instantiated already. > > sorry for my late reply, I agre

Re: stmmac: GMAC_RGSMIIIS reports bogus values

2017-01-31 Thread Giuseppe CAVALLARO
On 1/27/2017 11:23 AM, Alexey Brodkin wrote: That's why my initial proposal was to ignore whatever we read from this register if we have MDIO bus instantiated already. sorry for my late reply, I agree with this approach, according to the HW and platform configuration the driver has to understan

Re: stmmac: GMAC_RGSMIIIS reports bogus values

2017-01-27 Thread Alexey Brodkin
HiĀ Giuseppe, On Wed, 2017-01-25 at 21:39 +0300, Alexey Brodkin wrote: > Hi Giuseppe, > > On Mon, 2016-11-14 at 09:14 +0100, Giuseppe CAVALLARO wrote: > > > > Hello Alexey > > > > Sorry for my late reply. In that case, I think that the stmmac > > is using own RGMII/SGMII support (PCS) to dialog

Re: stmmac: GMAC_RGSMIIIS reports bogus values

2017-01-25 Thread Vineet Gupta
On 01/25/2017 10:39 AM, Alexey Brodkin wrote: >> Also I wonder if, other version of the stmmac worked on this platform >> before. > It did work and still works. The only problem is we're getting > a lot of noise now about bogus link status change. That's because > this info is now in pr_info() comp

Re: stmmac: GMAC_RGSMIIIS reports bogus values

2017-01-25 Thread Alexey Brodkin
Hi Giuseppe, On Mon, 2016-11-14 at 09:14 +0100, Giuseppe CAVALLARO wrote: > Hello Alexey > > Sorry for my late reply. In that case, I think that the stmmac > is using own RGMII/SGMII support (PCS) to dialog with the > transceiver. I think, from the HW capability register > this feature is present

Re: stmmac: GMAC_RGSMIIIS reports bogus values

2016-11-14 Thread Giuseppe CAVALLARO
Hello Alexey Sorry for my late reply. In that case, I think that the stmmac is using own RGMII/SGMII support (PCS) to dialog with the transceiver. I think, from the HW capability register this feature is present and the driver is using it as default. I kindly ask you to verify if this is your se

stmmac: GMAC_RGSMIIIS reports bogus values

2016-11-03 Thread Alexey Brodkin
Hello, I'm seeing pretty strange issue with GMAC reporting a lot of link state changes based on bits in GMAC_RGSMIIIS. It looks like that: -->8--- Link is Down Link is Up - 10/Full Link is Down Link is Up - 10/Half Link is Down Link is Down Link is Up -