On Wed, Jan 30, 2019 at 10:16:39AM +0000, Carlo Caione wrote: > On 28/01/2019 19:04, Russell King - ARM Linux admin wrote: > > Hi Russell, > > > There is no "advertisement of SmartEEE" - it's just EEE. That is > > because as far as the link partner is concerned, SmartEEE is just > > EEE. > > > > Carlio posted a link to one of the datasheets for the family. In > > there, it describes the EEE capability register, which describes > > what is supported, and the EEE wake error counter register. > > > > It also describes the EEE advertisement and link parter advertisement > > registers. > > > > All these registers correspond to the 802.3 section 45 defined MMD > > and address offsets found in Clause 45 compliant PHYs, and these > > registers control not only EEE but also SmartEEE. > > > > Please stop thinking that SmartEEE is different on the link partner > > side from EEE - as far as the link partner is concerned, there is > > no difference. The difference is all to do with the MAC side of > > the local PHY. > > Thank you for clarifying how the SmartEEE is really working. > > Now, the problem is that the MMD registers controlling the EEE (7.3c and > 7.3d, touched by the "eee-broken-*" property) are not the same as the ones > for the SmartEEE (3.805b, 3.805c, 3.805d). So, is it worth to add a new DT > property to deal also with the cases where we want to selectively disable > the SmartEEE? That's because you're confusing what the registers are. Please take the time to read the data sheet for the AR803x devices, and the 802.3 specifications, and you will save yourself from asking a lot of questions by email. 7.3c (or 7.60 in decimal) is the EEE advertisement register. This is what gets sent _to_ the link partner. 7.3d (or 7.61 in decimal) is the EEE link partner ability register. This is what the link partner sent to us _from_ its own EEE advertisement register. The eee-broken-* properties allow the EEE advertisement to be altered, thereby preventing EEE being negotiated for the various link modes. Disabling EEE for a particular link mode means that EEE (in any form) will not operate while the link is operating at that speed. 3.805b, 3.805c, 3.805d in the AR803x family are specifically to do with the SmartEEE implementation. 3.805b controls how much time between the link waking up and buffered data already received from the MAC being sent. 3.805c controls the time from the last activity to entering low power mode, additional bits in 3.805d. 3.805d contains the SmartEEE enable bit, as well as the additional bits from 3.805c. Most of these registers are only useful when you have a MAC that has no EEE functionality - that is where SmartEEE can be enabled to provide the power savings, and in order to implement EEE, there are various timeouts required by the protocol. SmartEEE allows these timeouts to be programmed via the above register. When using a MAC that has EEE functionality, SmartEEE should be disabled via 3.805d to allow _full_ 802.3az EEE (from local MAC to link partner) to operate, rather than SmartEEE (from local PHY to link partner.) Otherwise, using the existing "eee-broken-*" properties to disable the link modes where EEE fails would be the correct way forward, and should be used in preference to disabling SmartEEE. However, no one has mentioned what the problem that is trying to be addressed. Is it data corruption? Is it that the link fails? Is it lost packets? Is it that the MAC supports EEE? I think there needs to be some better understanding of the problem at hand before trying to address it. -- RMK's Patch system: https://www.armlinux.org.uk/developer/patches/ FTTC broadband for 0.8mile line in suburbia: sync at 12.1Mbps down 622kbps up According to speedtest.net: 11.9Mbps down 500kbps up
