From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Miller Subject: Re: [PATCH net-next 0/5] net: phy: improve and simplify phylib state machine Date: Thu, 08 Nov 2018 14:58:46 -0800 (PST) Message-ID: <20181108.145846.295661679780466934.davem@davemloft.net> References: <922c223b-7bc0-e0ec-345d-2034b796af91@gmail.com> Mime-Version: 1.0 Content-Type: Text/Plain; charset=us-ascii Content-Transfer-Encoding: 7bit Cc: f.fainelli@gmail.com, andrew@lunn.ch, netdev@vger.kernel.org To: hkallweit1@gmail.com Return-path: Received: from shards.monkeyblade.net ([23.128.96.9]:38642 "EHLO shards.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726831AbeKIIgg (ORCPT ); Fri, 9 Nov 2018 03:36:36 -0500 In-Reply-To: <922c223b-7bc0-e0ec-345d-2034b796af91@gmail.com> Sender: netdev-owner@vger.kernel.org List-ID: From: Heiner Kallweit Date: Wed, 7 Nov 2018 20:41:52 +0100 > 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?