Thread (15 messages) flat view 15 messages, 4 authors, 2018-11-09

Re: [PATCH net-next 0/5] net: phy: improve and simplify phylib state machine

From: Andrew Lunn <andrew@lunn.ch>
Date: 2018-11-09 08:39:20

On Thu, Nov 08, 2018 at 03:00:01PM -0800, Florian Fainelli wrote:
On 11/8/18 2:58 PM, David Miller wrote:
quoted
From: Heiner Kallweit <hkallweit1@gmail.com>
Date: Wed, 7 Nov 2018 20:41:52 +0100
quoted
This patch series is based on two axioms:

- During autoneg a PHY always reports the link being down

- Info in clause 22/45 registers doesn't allow to differentiate between
  these two states:
  1. Link is physically down
  2. A link partner is connected and PHY is autonegotiating
  In both cases "link up" and "aneg finished" bits aren't set.
  One consequence is that having separate states PHY_NOLINK and PHY_AN
  isn't needed.

By using these two axioms the state machine can be significantly
simplified.
So how are we going to move forward on this?

Maybe we can apply this series and just watch carefully for any
problems that get reported or are found?
Given Heiner is always responsive and taking care of fixing what might
be/have broken, no objections with me on that.
Yes, lets try it.

     Andrew
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help