Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752575AbcD1PoI (ORCPT ); Thu, 28 Apr 2016 11:44:08 -0400 Received: from eusmtp01.atmel.com ([212.144.249.242]:39271 "EHLO eusmtp01.atmel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751513AbcD1PoF (ORCPT ); Thu, 28 Apr 2016 11:44:05 -0400 Subject: Re: [PATCH v2] net: macb: do not scan PHYs manually To: Nathan Sullivan , References: <1461854802-8142-1-git-send-email-nathan.sullivan@ni.com> CC: , Florian Fainelli , Alexandre Belloni From: Nicolas Ferre X-Enigmail-Draft-Status: N1110 Organization: atmel Message-ID: <57222FCE.8050407@atmel.com> Date: Thu, 28 Apr 2016 17:44:14 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.7.2 MIME-Version: 1.0 In-Reply-To: <1461854802-8142-1-git-send-email-nathan.sullivan@ni.com> Content-Type: text/plain; charset="windows-1252" Content-Transfer-Encoding: 8bit X-Originating-IP: [10.161.30.18] Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2017 Lines: 64 Le 28/04/2016 16:46, Nathan Sullivan a ?crit : > Since of_mdiobus_register and mdiobus_register will scan automatically, > do not manually scan for PHY devices in the macb ethernet driver. Doing > so will result in many nonexistent PHYs on the MDIO bus if the MDIO > lines are floating or grounded, such as when they are not used. > > Signed-off-by: Nathan Sullivan Well, as explained in the commit message that added this feature and in the comment, if no phy is specified in the DT we end up without phy... There are AT91 platforms which lack specification for the phy node in the DT. So, I don't know if there is a better way to deal with this case but I see this removal as risky. Bye, > --- > drivers/net/ethernet/cadence/macb.c | 19 +------------------ > 1 file changed, 1 insertion(+), 18 deletions(-) > > diff --git a/drivers/net/ethernet/cadence/macb.c b/drivers/net/ethernet/cadence/macb.c > index 48a7d7d..6506b4e 100644 > --- a/drivers/net/ethernet/cadence/macb.c > +++ b/drivers/net/ethernet/cadence/macb.c > @@ -424,7 +424,7 @@ static int macb_mii_init(struct macb *bp) > { > struct macb_platform_data *pdata; > struct device_node *np; > - int err = -ENXIO, i; > + int err = -ENXIO; > > /* Enable management port */ > macb_writel(bp, NCR, MACB_BIT(MPE)); > @@ -450,23 +450,6 @@ static int macb_mii_init(struct macb *bp) > if (np) { > /* try dt phy registration */ > err = of_mdiobus_register(bp->mii_bus, np); > - > - /* fallback to standard phy registration if no phy were > - found during dt phy registration */ > - if (!err && !phy_find_first(bp->mii_bus)) { > - for (i = 0; i < PHY_MAX_ADDR; i++) { > - struct phy_device *phydev; > - > - phydev = mdiobus_scan(bp->mii_bus, i); > - if (IS_ERR(phydev)) { > - err = PTR_ERR(phydev); > - break; > - } > - } > - > - if (err) > - goto err_out_unregister_bus; > - } > } else { > if (pdata) > bp->mii_bus->phy_mask = pdata->phy_mask; > -- Nicolas Ferre