From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Miller Subject: Re: [PATCH net] net: phy: add workaround for issue where PHY driver doesn't bind to the device Date: Tue, 27 Nov 2018 15:01:09 -0800 (PST) Message-ID: <20181127.150109.131418314942116531.davem@davemloft.net> References: Mime-Version: 1.0 Content-Type: Text/Plain; charset=us-ascii Content-Transfer-Encoding: 7bit Cc: andrew@lunn.ch, f.fainelli@gmail.com, netdev@vger.kernel.org To: hkallweit1@gmail.com Return-path: Received: from shards.monkeyblade.net ([23.128.96.9]:42560 "EHLO shards.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726068AbeK1KAg (ORCPT ); Wed, 28 Nov 2018 05:00:36 -0500 In-Reply-To: Sender: netdev-owner@vger.kernel.org List-ID: From: Heiner Kallweit Date: Fri, 23 Nov 2018 19:41:29 +0100 > After switching the r8169 driver to use phylib some user reported that > their network is broken. This was caused by the genphy PHY driver being > used instead of the dedicated PHY driver for the RTL8211B. Users > reported that loading the Realtek PHY driver module upfront fixes the > issue. See also this mail thread: > https://marc.info/?t=154279781800003&r=1&w=2 > The issue is quite weird and the root cause seems to be somewhere in > the base driver core. The patch works around the issue and may be > removed once the actual issue is fixed. > > The Fixes tag refers to the first reported occurrence of the issue. > The issue itself may have been existing much longer and it may affect > users of other network chips as well. Users typically will recognize > this issue only if their PHY stops working when being used with the > genphy driver. > > Fixes: f1e911d5d0df ("r8169: add basic phylib support") > Signed-off-by: Heiner Kallweit Applied and queued up for -stable.