From patchwork Fri Feb 3 16:52:37 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Alexey Brodkin X-Patchwork-Id: 723821 X-Patchwork-Delegate: davem@davemloft.net Return-Path: X-Original-To: patchwork-incoming@ozlabs.org Delivered-To: patchwork-incoming@ozlabs.org Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by ozlabs.org (Postfix) with ESMTP id 3vFNGX3fddz9s7F for ; Sat, 4 Feb 2017 03:53:08 +1100 (AEDT) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751347AbdBCQw5 (ORCPT ); Fri, 3 Feb 2017 11:52:57 -0500 Received: from smtprelay4.synopsys.com ([198.182.47.9]:34140 "EHLO smtprelay.synopsys.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750944AbdBCQw4 (ORCPT ); Fri, 3 Feb 2017 11:52:56 -0500 Received: from mailhost.synopsys.com (mailhost2.synopsys.com [10.13.184.66]) by smtprelay.synopsys.com (Postfix) with ESMTP id 8BB4824E1232; Fri, 3 Feb 2017 08:52:44 -0800 (PST) Received: from mailhost.synopsys.com (localhost [127.0.0.1]) by mailhost.synopsys.com (Postfix) with ESMTP id 3FA131B1; Fri, 3 Feb 2017 08:52:44 -0800 (PST) Received: from ru20arcgnu1.internal.synopsys.com (ru20arcgnu1.internal.synopsys.com [10.121.9.48]) by mailhost.synopsys.com (Postfix) with ESMTP id 5E547152; Fri, 3 Feb 2017 08:52:41 -0800 (PST) From: Alexey Brodkin To: netdev@vger.kernel.org Cc: linux-snps-arc@lists.infradead.org, linux-kernel@vger.kernel.org, Alexey Brodkin , Murali Karicheri , Sekhar Nori , "David S . Miller" , Grygorii Strashko , Florian Fainelli , Mugunthan V N , Andrew Lunn Subject: [PATCH] net: phy: dp83867: Fix for automatically detected PHYs Date: Fri, 3 Feb 2017 19:52:37 +0300 Message-Id: <20170203165237.22452-1-abrodkin@synopsys.com> X-Mailer: git-send-email 2.10.2 Sender: netdev-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org Current implemntation returns ENODEV if device tree node for phy is absent. But in reality there're many boards with the one and only PHY on board and MACs that may find a PHY by querying MDIO bus. One good example is STMMAC. So that's what I saw: --- Booing --------------->8------------------------- libphy: Fixed MDIO Bus: probed stmmaceth f0008000.ethernet: no reset control found stmmac - user ID: 0x10, Synopsys ID: 0x37 stmmaceth f0008000.ethernet: Ring mode enabled stmmaceth f0008000.ethernet: DMA HW capability register supported stmmaceth f0008000.ethernet: Normal descriptors stmmaceth f0008000.ethernet: RX Checksum Offload Engine supported stmmaceth f0008000.ethernet: COE Type 2 stmmaceth f0008000.ethernet: TX Checksum insertion supported stmmaceth f0008000.ethernet: Enable RX Mitigation via HW Watchdog Timer libphy: stmmac: probed stmmaceth f0008000.ethernet (unnamed net_device) (uninitialized): PHY ID 2000a231 at 0 IRQ POLL (stmmac-0:00) active --- Bringing up eth0 ----->8------------------------- Starting network: stmmaceth f0008000.ethernet eth0: device MAC address b6:43:6b:1d:50:cf stmmaceth f0008000.ethernet eth0: Could not attach to PHY stmmaceth f0008000.ethernet eth0: stmmac_open: Cannot attach to PHY (error: -19) ip: SIOCSIFFLAGS: No such device -------------------------->8------------------------- Note PHY is detected properly but right after that we're getting -ENODEV. That makes no sense at all. So let's silently exit with 0 if there's no DT node for the PHY. That's what I'm seeing now: -------------------------->8------------------------- Starting network: stmmaceth f0008000.ethernet eth0: device MAC address de:1f:a9:f7:8d:88 stmmaceth f0008000.ethernet eth0: fail to init PTP. IPv6: ADDRCONF(NETDEV_UP): eth0: link is not ready udhcpc: started, v1.25.1 udhcpc: sending discover udhcpc: sending discover stmmaceth f0008000.ethernet eth0: Link is Up - 100Mbps/Full - flow control off IPv6: ADDRCONF(NETDEV_CHANGE): eth0: link becomes ready udhcpc: sending discover Link is Up - 100/Full udhcpc: sending select for x.x.x.x udhcpc: lease of x.x.x.x obtained, lease time 3600 deleting routers adding dns x.x.x.x -------------------------->8------------------------- Signed-off-by: Alexey Brodkin Cc: Murali Karicheri Cc: Sekhar Nori Cc: David S. Miller Cc: Grygorii Strashko Cc: Florian Fainelli Cc: Mugunthan V N Cc: Andrew Lunn Reviewed-by: Andrew Lunn --- drivers/net/phy/dp83867.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/phy/dp83867.c b/drivers/net/phy/dp83867.c index ca1b462..c44259b 100644 --- a/drivers/net/phy/dp83867.c +++ b/drivers/net/phy/dp83867.c @@ -120,7 +120,7 @@ static int dp83867_of_init(struct phy_device *phydev) int ret; if (!of_node) - return -ENODEV; + return 0; dp83867->io_impedance = -EINVAL;