From nobody Tue Aug 25 00:19:15 2026 Received: from mail-oo1-f43.google.com (mail-oo1-f43.google.com [209.85.161.43]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id DF16A3A1696 for ; Fri, 7 Aug 2026 03:54:44 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.161.43 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1786074890; cv=none; b=eoCGmmmBMmaN6E3dnQNrFJSPYxWHJG+wAIDpPYWPyvbnnCsjfPe2/QTWzbnfnSBVRnNDMPuX/4wMCaQNjdveRlyMgfAmSn+E3sUXtipnqxmSBg1azBkbWxolpw/BivQ/tJ0OStKB7sYRFmUEUUmGFyi8Wi4mUbrGn9OIO6Tlzwo= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1786074890; c=relaxed/simple; bh=0+TjxDD3Jy9In/dYW3swTJjk2WwZ3m7x7rp80MwetQc=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=htdlvA09NscqJar3EonKyILggOssV76RlAddZqWoxlhiAEIMurs/G4scCuJZ5yE6Hv1gDQVqSGpW+N2qeGLhflrGtv+seSg7zaBoZCO2Pfkz5g8JCOcP0e2eaF66P7qapRetk3Y1evRwpEyxTrfa3s8Wu0N0QmfydqhIiseNX5M= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=lpEX/7BI; arc=none smtp.client-ip=209.85.161.43 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="lpEX/7BI" Received: by mail-oo1-f43.google.com with SMTP id 006d021491bc7-6ae9b721927so1559601eaf.0 for ; Thu, 06 Aug 2026 20:54:44 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1786074883; x=1786679683; darn=vger.kernel.org; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :content-type:mime-version:subject:date:from:from:to:cc:subject:date :message-id:reply-to:content-type; bh=oG17fuSe5U2XTSseOTKToMBz/2cRmkeSjfgTvQP48IM=; b=lpEX/7BI85Fi/0G5+0D2xup9DX4yz60TfCAwoHRQUZ8vNfDwj0b1q5H7nz2xeO59b0 fyBTIsSZr3zl7i0m8aH+JmxbrgAFJUphZe+HHFvRWEtoU2ApRPKQF2levV6kdjtJifv4 2OIk1cbOGyT37h0z83/WywXHs50v06NqWDN8McDAyOawzI2B2XKsVWgXArqEmb0ca7oz zsMgAwWJW4EWWpI4nlB5IrCkANpuRtO2N6ztmMx9n9zZ9txMW/CvU9VcjxQ20kktyqaz Z8FQlF8MfAIDnfA8RPbjHGMLrBHX0GG1H20LsCCgVqKj2bOquVlTm0Uah6SICXjpHNA1 PvMA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1786074883; x=1786679683; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :content-type:mime-version:subject:date:from:x-gm-gg :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to :content-type; bh=oG17fuSe5U2XTSseOTKToMBz/2cRmkeSjfgTvQP48IM=; b=HGf9p3KxnjscHd21pC6Oy6Ivc8cKp1ZkxYG1zDsieSnJxKf38FxdhJe2RkSzWmVpNq S53SSp8pVGFUpiI8n9KRuToYOnHnuI91TyPvnGB8QpReEdi9PMwC43fiXtRylwzqxX0u FUyRgjDCljoBpXS+d6aSfXAYZ9gobche5SYmo2CVKOP3mXxwbBUY3ulDBXe6C86xfDVK 2KJSHv9cc7legw2sgPB0nfpKADalEa05hjk6KUXhLzTrWp57fP92Ic76UQCelGn6jQfn 10qvaSFaXrwaBTzi4pdl/CVN3vIGw2588MrGRZ79CvaKe/8IceRe9jqmlz1FPU7resTa dAiw== X-Forwarded-Encrypted: i=1; AHgh+RpawKRhbggIzgtE3zUpEVHbvEgoIozQ6+m0hWSridkUekLpY3sgKLvqDzwBT+oXZSog5/k/vUy/HjUSMVY=@vger.kernel.org X-Gm-Message-State: AOJu0YyPqG+YM/n6iFMaki5YzAR4FhSwnuCiAGsxybwT1VUSxEFBXelW CcuDbjbUWe7aqDYcV+zgJRlbACxYowSzL60WOSTfRBhPx+tK3c2YTM7j X-Gm-Gg: AR+sD12CfmxGZqy47xCLvZ6hyjvX2N+mUTNDEmqBq0a+BTl0mkxmVh2eyEseF9RBggk Ofd+b8wkQwQVm0+XkVF4BFzz3J80cowgSftHoC4dgoUqO9hY3+V+f1hCgTYhUPIeO8iO6ctcYyW Zhuuc/KvpgBrpGl6A1AcXxOtrZ8e9Oi1aAlSi1JqwuU8GQY+JQUjc8TOj6o9XxLZ1mhF1t0bQVz aTMK/uoOaTqpF76a/LiMJDql4yi/CzIM5VHqnaQTbd+0nDpDJwoET9YBaUg164jRi1gdwWDjstU ENzIei+lVRTOJilhnua+89XE5yntyQWQGMZ4Nlm1IFGJKl0P47bsmVeDth6zRSIl5K0dEJ/kH7H P+A8alrfIuA5owqrDwKgS6u2ZqVqUCgNbq329Iux+B0J6GW0XG/0tn8sw5Pkh7YDUUnsICxk3RI oVNRwr/ndxHti6uNElUcFjKjGqeOa3SaIyocdV1sQ86huf7CpyAMjnVk7IM+23y78MyHFxPRwG1 E1gEYm13JY0doN5jY0CLskD/YSNF/8ZjFyLlIx6OFMwr5lDg9uXXTJZXaEZjkSYjq2NqTpi2ABN GXQfLUfLMiR/M7uLwLFG7nAoMCR0tqMMm63xZXsSmS86ZhJqMnETM1T3cZWG+pt4geFlcv7JZzq aHeBYCVvm9jYUS2MisA== X-Received: by 2002:a05:6820:987:b0:6aa:e573:9ce0 with SMTP id 006d021491bc7-6ae96d0b246mr11060312eaf.16.1786074883485; Thu, 06 Aug 2026 20:54:43 -0700 (PDT) Received: from [127.0.1.1] (184-96-154-59.hlrn.qwest.net. [184.96.154.59]) by smtp.gmail.com with ESMTPSA id 006d021491bc7-6b02bbc41bbsm1329339eaf.3.2026.08.06.20.54.41 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 06 Aug 2026 20:54:43 -0700 (PDT) From: James Hilliard Date: Thu, 06 Aug 2026 21:54:20 -0600 Subject: [PATCH net-next v4 1/4] net: phy: cache MDIO bus owner before dropping PHY reference Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Message-Id: <20260806-submit-mdio-of-dynamic-v2-v4-1-4beb0c50d5ba@gmail.com> References: <20260806-submit-mdio-of-dynamic-v2-v4-0-4beb0c50d5ba@gmail.com> In-Reply-To: <20260806-submit-mdio-of-dynamic-v2-v4-0-4beb0c50d5ba@gmail.com> To: Andrew Lunn , Heiner Kallweit , Russell King , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Florian Fainelli , Richard Cochran , Rob Herring , Saravana Kannan , Florian Fainelli , Jonas Gorski , Vladimir Oltean , Ciprian Regus , Andreas Larsson , Mark Einon , Iyappan Subramanian , Keyur Chudgar , Michael Chan , =?utf-8?q?Rafa=C5=82_Mi=C5=82ecki?= , Broadcom internal kernel review list , Doug Berger , Pavan Chebbi , =?utf-8?q?Th=C3=A9o_Lebrun?= , Conor Dooley , Wei Fang , Frank Li , Shenwei Wang , Jijie Shao , Jian Shen , Bryan Whitehead , UNGLinuxDriver@microchip.com, Vladimir Zapolskiy , Piotr Wojtaszczyk , Parthiban Veerasooran , Timur Tabi , nic_swsd@realtek.com, Byungho An , Steve Glendinning , Maxime Chevallier , Maxime Coquelin , Alexandre Torgue , FUJITA Tomonori , Siddharth Vadapalli , Roger Quadros , Jiawen Wu , Mengyuan Lou , Thangaraj Samynathan , Sebastian Reichel , Simon Horman Cc: netdev@vger.kernel.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, James Hilliard , imx@lists.linux.dev, linux-arm-kernel@lists.infradead.org, linux-stm32@st-md-mailman.stormreply.com, linux-omap@vger.kernel.org, linux-usb@vger.kernel.org, linux-pm@vger.kernel.org X-Mailer: b4 0.15.2 phy_attach_direct() and phy_detach() cache the MDIO bus pointer, drop their PHY device reference, and then read bus->owner. If the put releases the PHY during concurrent teardown, the cached pointer does not itself keep the parent bus alive. Cache the module pointer before dropping the PHY reference. The existing module reference protects the cached pointer whenever the MDIO bus and network device have different owners. Fixes: ec988ad78ed6 ("phy: Don't increment MDIO bus refcount unless it's a = different owner") Signed-off-by: James Hilliard --- drivers/net/phy/phy_device.c | 21 +++++++++------------ 1 file changed, 9 insertions(+), 12 deletions(-) diff --git a/drivers/net/phy/phy_device.c b/drivers/net/phy/phy_device.c index 94b2e85e00a3..34d00f1ee483 100644 --- a/drivers/net/phy/phy_device.c +++ b/drivers/net/phy/phy_device.c @@ -1752,7 +1752,7 @@ static bool phy_drv_supports_irq(const struct phy_dri= ver *phydrv) int phy_attach_direct(struct net_device *dev, struct phy_device *phydev, u32 flags, phy_interface_t interface) { - struct mii_bus *bus =3D phydev->mdio.bus; + struct module *bus_owner =3D phydev->mdio.bus->owner; struct device *d =3D &phydev->mdio.dev; struct module *ndev_owner =3D NULL; int err; @@ -1764,7 +1764,7 @@ int phy_attach_direct(struct net_device *dev, struct = phy_device *phydev, */ if (dev) ndev_owner =3D dev->dev.parent->driver->owner; - if (ndev_owner !=3D bus->owner && !try_module_get(bus->owner)) { + if (ndev_owner !=3D bus_owner && !try_module_get(bus_owner)) { phydev_err(phydev, "failed to get the bus module\n"); return -EIO; } @@ -1900,8 +1900,8 @@ int phy_attach_direct(struct net_device *dev, struct = phy_device *phydev, d->driver =3D NULL; error_put_device: put_device(d); - if (ndev_owner !=3D bus->owner) - module_put(bus->owner); + if (ndev_owner !=3D bus_owner) + module_put(bus_owner); return err; } EXPORT_SYMBOL(phy_attach_direct); @@ -1917,7 +1917,7 @@ void phy_detach(struct phy_device *phydev) { struct net_device *dev =3D phydev->attached_dev; struct module *ndev_owner =3D NULL; - struct mii_bus *bus; + struct module *bus_owner; =20 if (phydev->devlink) { device_link_del(phydev->devlink); @@ -1972,17 +1972,14 @@ void phy_detach(struct phy_device *phydev) /* Assert the reset signal */ phy_device_reset(phydev, 1); =20 - /* - * The phydev might go away on the put_device() below, so avoid - * a use-after-free bug by reading the underlying bus first. - */ - bus =3D phydev->mdio.bus; + /* The PHY and its parent bus may be released by put_device() below. */ + bus_owner =3D phydev->mdio.bus->owner; =20 put_device(&phydev->mdio.dev); if (dev) ndev_owner =3D dev->dev.parent->driver->owner; - if (ndev_owner !=3D bus->owner) - module_put(bus->owner); + if (ndev_owner !=3D bus_owner) + module_put(bus_owner); } EXPORT_SYMBOL(phy_detach); =20 --=20 2.53.0 From nobody Tue Aug 25 00:19:15 2026 Received: from mail-oa1-f46.google.com (mail-oa1-f46.google.com [209.85.160.46]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 9E8FA399354 for ; Fri, 7 Aug 2026 03:54:46 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.160.46 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1786074888; cv=none; b=KVLOu1wGjidPSF/hmGheAvArZF5a4E1NIsP4knhKweEN14e43lwDUC6VX+ll0Y/hJWSMhpTF0Fo9G+IgLqDgSlQeRd9Sg4LP8sRBk9tsyWo67DJKVTcMDlssuBEC4IgRXG/sCxy+1IHXZN8obtx9whoCa3mqM5uB0VunQeRP0Bs= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1786074888; c=relaxed/simple; bh=PTx1xmaBWvPevkP6ZbmwzZ7YY+06PmbkiJMfuow2Mow=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=ZcmGwODDsanEQ5QCZzYcDbwi6KcIOg175R80CvQd3qplVSDVsgM5LaE4ATNuLq4vh8yZRBy69gVN7HpOc78cXVmIFMhm+czHt4iMqN1tgiLiAFCb7qKJX/Br3FDHonYuVhGRaed6e9GbBMDW3eXsg1eElhwkLRnRr3hj0ISw/EM= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=Lxs3lwMX; arc=none smtp.client-ip=209.85.160.46 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="Lxs3lwMX" Received: by mail-oa1-f46.google.com with SMTP id 586e51a60fabf-456f7012050so1355310fac.0 for ; Thu, 06 Aug 2026 20:54:46 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1786074885; x=1786679685; darn=vger.kernel.org; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :content-type:mime-version:subject:date:from:from:to:cc:subject:date :message-id:reply-to:content-type; bh=387qnwbkMAAVcWYZO25x2XelYHqPuaJgYNRgF/uAO2o=; b=Lxs3lwMXw2rSykmSvFCeMxtFLAgRWTuyorrT/njVaSBDN3G78bS/msNbhr16C8LWlp wexI0H+T+6LKslK4Lv6VDj5roOOB+wMgsVqeMj5kkJp3cVYuyUkygVEn4igTIsckZtNe RpjWr1AktZ29R6kZ8oL1CnTy+infJOfKaiidxGsTZ3G+UZm1he038+95RGYtClHXhgnh IJ6HcQ7i9LmFPKjfecK3i6g5l2qqa3NcsTQTTBNMjNonkhV7JVY8WEii+BjzROehNt3L I//bACXeik0z0Ukd0/H/OkmFdarom6dhGZhwAQGGOXUAvKI07fUx5iL8LHc7+rwc4GK/ iZDg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1786074885; x=1786679685; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :content-type:mime-version:subject:date:from:x-gm-gg :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to :content-type; bh=387qnwbkMAAVcWYZO25x2XelYHqPuaJgYNRgF/uAO2o=; b=mQMqIVoDP/9y5T8BAlfS5WX+H3fHBf7VY1UrFQvwQ+gON8jWAf3p86ezXUmzMY/sUh u+2baI5T2IyKOY91csGC+a2SElBxRSnKR/4dXUiLme33bRgY4i4ElfufJkn//Byu65lu rbY5fsIbBkdcANn/HFG7nObvrqQlcEMC6GRoppQ+DvFmRFtLtUZGtzyKtnNuZV5/A8T8 3SsZQgF5iU8Wemchs1hedSQ9jXCbAhoRkdVKVwT+DRWVlYiI3xUMxiNLm12+z0LiTxH5 69hK99YhPqW4LjfrgPAl2NArMUOSjYLVR2yj/VFZe1pXyKzCOGkRvXk8L6dqHlOYXIxV JO0Q== X-Forwarded-Encrypted: i=1; AHgh+RrplcBpltE2Y41q5E0O9bD3lWCHcVtbjT4JnWpon40MITzfZ2XcHPlhfqThEBV7SvCwljQ8bIXb+j8GuEE=@vger.kernel.org X-Gm-Message-State: AOJu0YwLK/y21NBfqnw+jqnaFLOmCV4ytjV9ZndvgRqRXc9ufzdYF764 qknH35IdxDZxXkXecSNdoBCHZRImczHUm/JzRfdmfLIfr6KZikAmwgKb X-Gm-Gg: AR+sD12+zHOR8aEcQSz5EeFGujNBGGA11B0w9PeAor7pBIcbsMQoOg8ZlTKsawn1dO0 NQzJKGUn0cTQu8swJIoO6oqKfqIJNQmffIHHYgCbO8nMTr4UZXW6tcFbKHHCUIWpzq5oGykPAP5 q5oz/PmjyfxXZw7lqyal1ONJ1lAQ+m+g7tG70yHORNaBj8XK6ijPa4alAbtHDGYQopZ6AhQY2/C aCxCNirCYt83MTDq58h0rum0XfWE1EM5TkVPg3eQ1cqNcKcBykwbWkzo9MZsZQgS+RGST9sFkED u15pB5KfIzsow1PnR1dLUt4xBOphDty1YPDPy89Pw9HLtRlvWQHSUUFs+hYQNAfJX7s3h+INDuJ p4wZKxWPr6UhnIMLRFdUEPyFInoloxWpPjuPfQ8nagDV43sUKVzp1QAw/ECmzPpy17l33ith/Nm 97Epxnon+EKkL8iPVTWMAizpE0aa/2V9hpMSbgsTEowm87AKGwjRKzM2DdZvpGXN55+DtOv/0T5 G/hO/zy3bsCeQxFO5O2xvgoohOY8/d0vtYIfydfvBmf8MsJXD9KHf/lfAWIyBfT1TaUTINoworb tlSI8B9MPAU92CiA+x43ZZP0sZ6bpvFZmXYJJlZd9BT+JLwRxgL3mPeR18Zk+v0+pKEKq0VMsbJ jgGRxOn7duz/+57+isRZM+Lb1bAc= X-Received: by 2002:a05:6820:c91:b0:6aa:e482:3ee3 with SMTP id 006d021491bc7-6b016c9eed1mr4663164eaf.8.1786074885511; Thu, 06 Aug 2026 20:54:45 -0700 (PDT) Received: from [127.0.1.1] (184-96-154-59.hlrn.qwest.net. [184.96.154.59]) by smtp.gmail.com with ESMTPSA id 006d021491bc7-6b02bbc41bbsm1329339eaf.3.2026.08.06.20.54.43 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 06 Aug 2026 20:54:45 -0700 (PDT) From: James Hilliard Date: Thu, 06 Aug 2026 21:54:21 -0600 Subject: [PATCH net-next v4 2/4] net: mdio: factor out OF child registration helpers Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Message-Id: <20260806-submit-mdio-of-dynamic-v2-v4-2-4beb0c50d5ba@gmail.com> References: <20260806-submit-mdio-of-dynamic-v2-v4-0-4beb0c50d5ba@gmail.com> In-Reply-To: <20260806-submit-mdio-of-dynamic-v2-v4-0-4beb0c50d5ba@gmail.com> To: Andrew Lunn , Heiner Kallweit , Russell King , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Florian Fainelli , Richard Cochran , Rob Herring , Saravana Kannan , Florian Fainelli , Jonas Gorski , Vladimir Oltean , Ciprian Regus , Andreas Larsson , Mark Einon , Iyappan Subramanian , Keyur Chudgar , Michael Chan , =?utf-8?q?Rafa=C5=82_Mi=C5=82ecki?= , Broadcom internal kernel review list , Doug Berger , Pavan Chebbi , =?utf-8?q?Th=C3=A9o_Lebrun?= , Conor Dooley , Wei Fang , Frank Li , Shenwei Wang , Jijie Shao , Jian Shen , Bryan Whitehead , UNGLinuxDriver@microchip.com, Vladimir Zapolskiy , Piotr Wojtaszczyk , Parthiban Veerasooran , Timur Tabi , nic_swsd@realtek.com, Byungho An , Steve Glendinning , Maxime Chevallier , Maxime Coquelin , Alexandre Torgue , FUJITA Tomonori , Siddharth Vadapalli , Roger Quadros , Jiawen Wu , Mengyuan Lou , Thangaraj Samynathan , Sebastian Reichel , Simon Horman Cc: netdev@vger.kernel.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, James Hilliard , imx@lists.linux.dev, linux-arm-kernel@lists.infradead.org, linux-stm32@st-md-mailman.stormreply.com, linux-omap@vger.kernel.org, linux-usb@vger.kernel.org, linux-pm@vger.kernel.org X-Mailer: b4 0.15.2 Live OF reconfiguration needs to create the same PHY and generic MDIO devices as initial bus population, including legacy PHY nodes that need address scanning. Factor fixed-address child registration and PHY address scanning into helpers. The old scan loop also visited every free address for a child without a reg property even when the child was not a PHY. Reject that case before scanning while preserving enumeration of valid PHY and generic MDIO children. Signed-off-by: James Hilliard --- Changes v1 -> v2: - split from the ACx00 series without code changes --- drivers/net/mdio/of_mdio.c | 66 ++++++++++++++++++++++++++++--------------= ---- 1 file changed, 40 insertions(+), 26 deletions(-) diff --git a/drivers/net/mdio/of_mdio.c b/drivers/net/mdio/of_mdio.c index b8d298c04d3f..051e449bbe7c 100644 --- a/drivers/net/mdio/of_mdio.c +++ b/drivers/net/mdio/of_mdio.c @@ -77,6 +77,15 @@ static int of_mdiobus_register_device(struct mii_bus *md= io, return 0; } =20 +static int of_mdiobus_register_child(struct mii_bus *mdio, + struct device_node *child, u32 addr) +{ + if (of_mdiobus_child_is_phy(child)) + return of_mdiobus_register_phy(mdio, child, addr); + + return of_mdiobus_register_device(mdio, child, addr); +} + /* The following is a list of PHY compatible strings which appear in * some DTBs. The compatible string is never matched against a PHY * driver, so is pointless. We only expect devices which are not PHYs @@ -136,6 +145,32 @@ bool of_mdiobus_child_is_phy(struct device_node *child) } EXPORT_SYMBOL(of_mdiobus_child_is_phy); =20 +static int of_mdiobus_scan_phy(struct mii_bus *mdio, + struct device_node *child) +{ + int addr, rc; + + if (!of_mdiobus_child_is_phy(child)) + return -ENODEV; + + for (addr =3D 0; addr < PHY_MAX_ADDR; addr++) { + if (mdiobus_is_registered_device(mdio, addr)) + continue; + + dev_info(&mdio->dev, "scan phy %pOFn at address %i\n", + child, addr); + + /* -ENODEV means that scanning should continue. */ + rc =3D of_mdiobus_register_child(mdio, child, addr); + if (!rc) + return 0; + if (rc !=3D -ENODEV) + return rc; + } + + return -ENODEV; +} + static int __of_mdiobus_parse_phys(struct mii_bus *mdio, struct device_nod= e *np, bool *scanphys) { @@ -164,10 +199,7 @@ static int __of_mdiobus_parse_phys(struct mii_bus *mdi= o, struct device_node *np, continue; } =20 - if (of_mdiobus_child_is_phy(child)) - rc =3D of_mdiobus_register_phy(mdio, child, addr); - else - rc =3D of_mdiobus_register_device(mdio, child, addr); + rc =3D of_mdiobus_register_child(mdio, child, addr); =20 if (rc =3D=3D -ENODEV) dev_err(&mdio->dev, @@ -197,7 +229,7 @@ int __of_mdiobus_register(struct mii_bus *mdio, struct = device_node *np, { struct device_node *child; bool scanphys =3D false; - int addr, rc; + int rc; =20 if (!np) return __mdiobus_register(mdio, owner); @@ -238,27 +270,9 @@ int __of_mdiobus_register(struct mii_bus *mdio, struct= device_node *np, of_node_name_eq(child, "ethernet-phy-package")) continue; =20 - for (addr =3D 0; addr < PHY_MAX_ADDR; addr++) { - /* skip already registered PHYs */ - if (mdiobus_is_registered_device(mdio, addr)) - continue; - - /* be noisy to encourage people to set reg property */ - dev_info(&mdio->dev, "scan phy %pOFn at address %i\n", - child, addr); - - if (of_mdiobus_child_is_phy(child)) { - /* -ENODEV is the return code that PHYLIB has - * standardized on to indicate that bus - * scanning should continue. - */ - rc =3D of_mdiobus_register_phy(mdio, child, addr); - if (!rc) - break; - if (rc !=3D -ENODEV) - goto put_unregister; - } - } + rc =3D of_mdiobus_scan_phy(mdio, child); + if (rc && rc !=3D -ENODEV) + goto put_unregister; } =20 return 0; --=20 2.53.0 From nobody Tue Aug 25 00:19:15 2026 Received: from mail-ot1-f52.google.com (mail-ot1-f52.google.com [209.85.210.52]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id B86283A8FE1 for ; Fri, 7 Aug 2026 03:54:49 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.210.52 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1786074899; cv=none; b=o3xr9XndUdQU73GlvGtgWJKxHQ/P99kLkNj75GbctjmNp6M55ZqzvTkZddTkks4gO0roLX7QSWlfdh30YHY44/lItnshiGhHBBGPxrmfwL6lA04r+L6qA6lpT8PLfc9aJUyAU0j/91Z8bjmM61HR92RlklgDjvNmWYkHP8wGYSU= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1786074899; c=relaxed/simple; bh=urJKRS4nkm2bfOmYvvDKgabL9LEo8/9eE7IfW9T6/a0=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=Oqo9HFmLiZPQG4GepP50xeEymD1zP/udVPfA3p+f4uu1/Peju+5S+uNOCvfyukATOv95nq0LjOdz0Gw9HDtjWIO/2vlazCjsXBCefCGzNPSPseza01CeZ0EQMpj+6M5w68p9b9dsLC3iDsAb3jaqZ+ya9WFBTdFWkUj0BV3rPUA= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=LpJINB2l; arc=none smtp.client-ip=209.85.210.52 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="LpJINB2l" Received: by mail-ot1-f52.google.com with SMTP id 46e09a7af769-7eb4d532e65so1407100a34.0 for ; Thu, 06 Aug 2026 20:54:49 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1786074888; x=1786679688; darn=vger.kernel.org; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :content-type:mime-version:subject:date:from:from:to:cc:subject:date :message-id:reply-to:content-type; bh=OXJkPo/CoVJ+XVdgEWXyycv7lfOXrpB/Ske012W9nQM=; b=LpJINB2lcmt0BuaOWIZ0yx2lQYXa0pN8Q6tkOrq4fyzY/bht2TG9xmpcwET7vFlVzz 0iF0OfLi8YFB9MlAHEi3yGLcbhO7CcfIrpImMF/fC0zXDLyR35mwoqpPZk7TnNjKKLjX OSIfQgM0MvIbl3wrztm4WQUff1npoBVwRiv9d/YJAyxSHnQ7JF38X9dd2Uyk8khIx2Cr RnCUe+WbTqA4A6VFSvHhkWxNAV4H08UHGMHKJEz9iG9UpGDEGb70MzDUriFj41MbACrq s0iOW/9hPaYYuo8GlClXM8P9PoY1/o3+FbtAP++KoW/7ncR0I8JqGukrVYGOsaIJcxRj eT5Q== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1786074888; x=1786679688; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :content-type:mime-version:subject:date:from:x-gm-gg :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to :content-type; bh=OXJkPo/CoVJ+XVdgEWXyycv7lfOXrpB/Ske012W9nQM=; b=Q2og+3CpbqPARLTklqIpFdYBlKPJ6itNUUi+wz7Hg5vJmUA9o/SFfmUHAHoDSOhmV6 hI9ewX6oRSpJSmVWHWMBagvtfoKfw+f570d9cKL74Fl9qbb7jFcRG/IegCCeMaWKYN/O 0u7q7sgjOfGDOM9os93HDPD8b9YTJ86drhTMnw9UzBQUfRsNKlP3bHPggYVgZfUkch9r 8n/eoycM6KWFixsbZMlLzkA1v5rYprKksAGxy6paLntSZQ6sKjsX/eyh26LrxIjYuv+s jNh15sa6Cf5hr2WHuI9BRR28UKOqkMiW0zGEYD9U2PtYhr4JULc6LMRnIutGCpi1B4Bq GaCQ== X-Forwarded-Encrypted: i=1; AHgh+RrkMzj+JneITHuqrLYSnLNPMFg+razW17yP3NEI32dqoWAWwPFmdyC2TLs5gxeWbuIbXQfDS9pon99BA78=@vger.kernel.org X-Gm-Message-State: AOJu0YxXlGWAAtK0YFj9RnIZph1Dj5FIfdj8huDOr1qw381xqQbqrI+S osz78ZYKIVIY7iFi1hRGm36C/Kvx2bx52p7KSC1cvWBqr9yOMNsebLx/ X-Gm-Gg: AR+sD103C9sDynZC7N0V22gFbFyk0tlrhh13fyIkgGODfDhmYbCxQQnH/UXwGW2R07d PftJDX6Sura9lUKjwpGGC+vuvNJRVUkjVuPJMHZc4oI0TbH4tpFgiGUkAXqibea28A+z7pMQIil 6q5jBWr6kunrC4a1tb0oAS8ip7tjQy+xcv5j3giz0iGy2WmS0WKNG9QKu9vLlw+VGyfdD5ATDJC RUeLOi+pGSWFJ8YMgs7XXWaceL1fNvyeN/n+bGqq5hHCq1TnhxcOtEkn8QIvr0oFDhPHrpXhDDP yW2TOGfkmSqlJIxs2lhzYqqAp8qW8avPk/tU1MpRaVksiu8kQkbPv5MNKZihCudgN603+I2+LAz XX2JhrIXpZxKF+4pShc0b5lM0dpjp/YCNhDKx7s95Br0CQKKayaFUXyEbtVDlIK8dQ3SNld4t8E Ryu9OXINYSUldq3shj9PqAUzu+2oPUGgYgGSAF5YCZO6kiDZ2wu13ihLKzznglIUdPLfnBBtRO6 oVxoy2pKQh0/DK5gHty2MdQyI+RUc6xap8srVDfxh1b0FsyPj1P7NxEzLbdsos/3QySMFNU5+O/ EmPmP1zC88hx5hayAIIlv75YSPqS3etXkCE9eO8rCp9DFN1rOCZ3WdW/55m+5wBSYOJ9cg8qJ1K v9OtvcacKj3/arAk1 X-Received: by 2002:a05:6820:4414:b0:6b0:1ba4:dcd0 with SMTP id 006d021491bc7-6b01ba50a1dmr3849042eaf.27.1786074887783; Thu, 06 Aug 2026 20:54:47 -0700 (PDT) Received: from [127.0.1.1] (184-96-154-59.hlrn.qwest.net. [184.96.154.59]) by smtp.gmail.com with ESMTPSA id 006d021491bc7-6b02bbc41bbsm1329339eaf.3.2026.08.06.20.54.45 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 06 Aug 2026 20:54:47 -0700 (PDT) From: James Hilliard Date: Thu, 06 Aug 2026 21:54:22 -0600 Subject: [PATCH net-next v4 3/4] net: mdio: make device map changes hotplug-safe Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Message-Id: <20260806-submit-mdio-of-dynamic-v2-v4-3-4beb0c50d5ba@gmail.com> References: <20260806-submit-mdio-of-dynamic-v2-v4-0-4beb0c50d5ba@gmail.com> In-Reply-To: <20260806-submit-mdio-of-dynamic-v2-v4-0-4beb0c50d5ba@gmail.com> To: Andrew Lunn , Heiner Kallweit , Russell King , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Florian Fainelli , Richard Cochran , Rob Herring , Saravana Kannan , Florian Fainelli , Jonas Gorski , Vladimir Oltean , Ciprian Regus , Andreas Larsson , Mark Einon , Iyappan Subramanian , Keyur Chudgar , Michael Chan , =?utf-8?q?Rafa=C5=82_Mi=C5=82ecki?= , Broadcom internal kernel review list , Doug Berger , Pavan Chebbi , =?utf-8?q?Th=C3=A9o_Lebrun?= , Conor Dooley , Wei Fang , Frank Li , Shenwei Wang , Jijie Shao , Jian Shen , Bryan Whitehead , UNGLinuxDriver@microchip.com, Vladimir Zapolskiy , Piotr Wojtaszczyk , Parthiban Veerasooran , Timur Tabi , nic_swsd@realtek.com, Byungho An , Steve Glendinning , Maxime Chevallier , Maxime Coquelin , Alexandre Torgue , FUJITA Tomonori , Siddharth Vadapalli , Roger Quadros , Jiawen Wu , Mengyuan Lou , Thangaraj Samynathan , Sebastian Reichel , Simon Horman Cc: netdev@vger.kernel.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, James Hilliard , imx@lists.linux.dev, linux-arm-kernel@lists.infradead.org, linux-stm32@st-md-mailman.stormreply.com, linux-omap@vger.kernel.org, linux-usb@vger.kernel.org, linux-pm@vger.kernel.org X-Mailer: b4 0.15.2 MDIO device registration currently publishes directly into mdio_map without serializing address ownership, and removal frees the object immediately. That is sufficient while bus population is static, but dynamic firmware changes can race registration, PHY attachment and bus teardown. Serialize device-map state and reserve addresses while registration is in progress. Track active scans, registrations and firmware changes so bus teardown first blocks new work and waits for existing work before consuming the final map. Make firmware removal transactions exclusive with registration, scanning and other firmware changes: start one only when no tracked map operation is active, then block new firmware changes, registrations and PHY attachment until it ends. Keep an address reserved while device_add() and driver probing run, then publish the fully registered device. A device becomes visible through the driver core before map publication completes, so make PHY attachment wait for another task's pending registration. Return -EPROBE_DEFER instead of waiting when registration and attachment recurse in the same task. Protect published map entries with RCU and make mdiobus_get_phy() return a reference to its PHY. Give phy_find_first() and phy_find_next() the same ownership rule. Make the bus iterator release each reference and update every in-tree caller. Unpublish a removed device before device_del(), then wait for an RCU grace period before the registration reference can be dropped. Readers can therefore acquire their own reference safely, while repeated dynamic remove/re-add cycles release old PHY generations once their real users are done instead of retaining them until bus teardown. Make PHY attachment atomic with map removal, including standalone PHY users. Have address scans atomically claim a matching OF node as well, so explicit OF population cannot instantiate the same device concurrently. Check the Microsemi timestamping driver's base-PHY lookup before using its private data. Make the internal removal callback report whether it unpublished the device. Bus teardown holds a temporary reference while dropping the map lock and releases the device's registration reference only when its removal attempt succeeded. When a concurrent normal removal has already unpublished the device, teardown therefore does not release the same registration reference again. Drop a dynamically removed device's firmware-node reference after device_del() so it does not keep an overlay node alive. Clear an error-valued optional reset GPIO before the common registration rollback uses the reset helpers. Signed-off-by: James Hilliard --- Changes v3 -> v4: - make PHY map lookups reference-counted and RCU-protected - update all in-tree mdiobus_get_phy() and PHY iterator callers - reclaim dynamically removed PHY generations after their users release them instead of retaining every generation until bus teardown - wait for concurrent map publication before attaching a PHY and defer same-task recursive attachment - handle a missing VSC8584 base PHY without dereferencing NULL Changes v2 -> v3: - hold a device reference across unlocked teardown removal - free only devices successfully claimed by the teardown path - consolidate normal and dynamic internal removal callbacks - publish map entries only after device registration completes - route the remaining direct map reader through mdiobus_get_phy() - retire only PHY devices which have a borrowed-pointer lookup API - make firmware removal exclusive with registration, scanning and other firmware changes - propagate only population conflicts from recursive package lookup Changes v1 -> v2: - split from the ACx00 series - rebase onto net-next after stopped including - move touched declarations to function scope for netdev style --- drivers/net/dsa/b53/b53_common.c | 9 +- drivers/net/ethernet/8390/ax88796.c | 2 + drivers/net/ethernet/adi/adin1140.c | 6 +- drivers/net/ethernet/aeroflex/greth.c | 2 + drivers/net/ethernet/agere/et131x.c | 10 +- drivers/net/ethernet/amd/au1000_eth.c | 27 ++- drivers/net/ethernet/apm/xgene-v2/mdio.c | 8 +- drivers/net/ethernet/broadcom/b44.c | 6 + drivers/net/ethernet/broadcom/bgmac-bcma.c | 1 + drivers/net/ethernet/broadcom/genet/bcmmii.c | 4 + drivers/net/ethernet/broadcom/sb1250-mac.c | 8 +- drivers/net/ethernet/broadcom/tg3.c | 32 +-- drivers/net/ethernet/broadcom/tg3.h | 1 + drivers/net/ethernet/cadence/macb_main.c | 1 + drivers/net/ethernet/engleder/tsnep_main.c | 4 + drivers/net/ethernet/ethoc.c | 14 +- drivers/net/ethernet/faraday/ftgmac100.c | 8 +- drivers/net/ethernet/freescale/fec_main.c | 14 +- drivers/net/ethernet/hisilicon/hibmcge/hbg_mdio.c | 8 + .../ethernet/hisilicon/hns3/hns3pf/hclge_main.c | 10 +- drivers/net/ethernet/lantiq_etop.c | 8 +- .../ethernet/mellanox/mlxbf_gige/mlxbf_gige_main.c | 2 + drivers/net/ethernet/microchip/lan743x_main.c | 1 + drivers/net/ethernet/nxp/lpc_eth.c | 10 +- drivers/net/ethernet/oa_tc6.c | 3 + drivers/net/ethernet/qualcomm/emac/emac-phy.c | 7 - drivers/net/ethernet/rdc/r6040.c | 10 +- drivers/net/ethernet/realtek/r8169_main.c | 15 +- drivers/net/ethernet/samsung/sxgbe/sxgbe_mdio.c | 1 + drivers/net/ethernet/smsc/smsc911x.c | 6 +- drivers/net/ethernet/smsc/smsc9420.c | 10 +- drivers/net/ethernet/stmicro/stmmac/stmmac_main.c | 1 + drivers/net/ethernet/stmicro/stmmac/stmmac_mdio.c | 2 + drivers/net/ethernet/tehuti/tn40_phy.c | 5 +- drivers/net/ethernet/ti/davinci_emac.c | 20 +- drivers/net/ethernet/toshiba/tc35815.c | 8 +- drivers/net/ethernet/wangxun/ngbe/ngbe_mdio.c | 8 + drivers/net/ethernet/wangxun/txgbe/txgbe_phy.c | 8 + drivers/net/phy/mdio_bus.c | 46 ++-- drivers/net/phy/mdio_bus_provider.c | 141 ++++++++---- drivers/net/phy/mdio_device.c | 238 +++++++++++++++++= +--- drivers/net/phy/mscc/mscc_ptp.c | 50 ++--- drivers/net/phy/phy_device.c | 130 ++++++++--- drivers/net/phy/phylib-internal.h | 4 +- drivers/net/usb/asix_devices.c | 4 + drivers/net/usb/lan78xx.c | 7 +- drivers/net/usb/smsc95xx.c | 2 + drivers/power/reset/linkstation-poweroff.c | 1 + include/linux/mdio.h | 4 +- include/linux/phy.h | 34 ++- net/dsa/user.c | 12 +- 51 files changed, 748 insertions(+), 225 deletions(-) diff --git a/drivers/net/dsa/b53/b53_common.c b/drivers/net/dsa/b53/b53_com= mon.c index 3f5b9592794d..2c79b3d03bf7 100644 --- a/drivers/net/dsa/b53/b53_common.c +++ b/drivers/net/dsa/b53/b53_common.c @@ -1156,6 +1156,7 @@ void b53_get_strings(struct dsa_switch *ds, int port,= u32 stringset, return; =20 phy_ethtool_get_strings(phydev, data); + phy_device_put(phydev); } } EXPORT_SYMBOL(b53_get_strings); @@ -1202,6 +1203,7 @@ void b53_get_ethtool_phy_stats(struct dsa_switch *ds,= int port, uint64_t *data) return; =20 phy_ethtool_get_stats(phydev, NULL, data); + phy_device_put(phydev); } EXPORT_SYMBOL(b53_get_ethtool_phy_stats); =20 @@ -1213,11 +1215,16 @@ int b53_get_sset_count(struct dsa_switch *ds, int p= ort, int sset) if (sset =3D=3D ETH_SS_STATS) { return b53_get_mib_size(dev); } else if (sset =3D=3D ETH_SS_PHY_STATS) { + int count; + phydev =3D b53_get_phy_device(ds, port); if (!phydev) return 0; =20 - return phy_ethtool_get_sset_count(phydev); + count =3D phy_ethtool_get_sset_count(phydev); + phy_device_put(phydev); + + return count; } =20 return 0; diff --git a/drivers/net/ethernet/8390/ax88796.c b/drivers/net/ethernet/839= 0/ax88796.c index e1695d0fbd8b..f36f742e4c02 100644 --- a/drivers/net/ethernet/8390/ax88796.c +++ b/drivers/net/ethernet/8390/ax88796.c @@ -377,6 +377,7 @@ static int ax_mii_probe(struct net_device *dev) ret =3D phy_connect_direct(dev, phy_dev, ax_handle_link_change, PHY_INTERFACE_MODE_MII); if (ret) { + phy_device_put(phy_dev); netdev_err(dev, "Could not attach to PHY\n"); return ret; } @@ -385,6 +386,7 @@ static int ax_mii_probe(struct net_device *dev) =20 netdev_info(dev, "PHY driver [%s] (mii_bus:phy_addr=3D%s, irq=3D%d)\n", phy_dev->drv->name, phydev_name(phy_dev), phy_dev->irq); + phy_device_put(phy_dev); =20 return 0; } diff --git a/drivers/net/ethernet/adi/adin1140.c b/drivers/net/ethernet/adi= /adin1140.c index 93710baca151..5dc4c4d70dfc 100644 --- a/drivers/net/ethernet/adi/adin1140.c +++ b/drivers/net/ethernet/adi/adin1140.c @@ -664,13 +664,17 @@ static int adin1140_phy_init(struct adin1140_priv *pr= iv, ret =3D phy_connect_direct(priv->netdev, priv->phydev, &adin1140_handle_link_change, PHY_INTERFACE_MODE_INTERNAL); - if (ret) + if (ret) { + phy_device_put(priv->phydev); + priv->phydev =3D NULL; return dev_err_probe(&spidev->dev, ret, "Can't attach PHY to %s\n", priv->mdiobus->id); + } =20 ret =3D devm_add_action_or_reset(&spidev->dev, adin1140_phy_remove, priv->phydev); + phy_device_put(priv->phydev); if (ret) return ret; =20 diff --git a/drivers/net/ethernet/aeroflex/greth.c b/drivers/net/ethernet/a= eroflex/greth.c index a593adc16c78..7a9528195a0a 100644 --- a/drivers/net/ethernet/aeroflex/greth.c +++ b/drivers/net/ethernet/aeroflex/greth.c @@ -1268,6 +1268,7 @@ static int greth_mdio_probe(struct net_device *dev) ret =3D phy_connect_direct(dev, phy, &greth_link_change, greth->gbit_mac ? PHY_INTERFACE_MODE_GMII : PHY_INTERFACE_MODE_MII); if (ret) { + phy_device_put(phy); if (netif_msg_ifup(greth)) dev_err(&dev->dev, "could not attach to PHY\n"); return ret; @@ -1279,6 +1280,7 @@ static int greth_mdio_probe(struct net_device *dev) phy_set_max_speed(phy, SPEED_100); =20 linkmode_copy(phy->advertising, phy->supported); + phy_device_put(phy); =20 greth->link =3D 0; greth->speed =3D 0; diff --git a/drivers/net/ethernet/agere/et131x.c b/drivers/net/ethernet/age= re/et131x.c index 1b465a167672..bfefb367a6b5 100644 --- a/drivers/net/ethernet/agere/et131x.c +++ b/drivers/net/ethernet/agere/et131x.c @@ -3273,16 +3273,18 @@ static void et131x_adjust_link(struct net_device *n= etdev) static int et131x_mii_probe(struct net_device *netdev) { struct et131x_adapter *adapter =3D netdev_priv(netdev); - struct phy_device *phydev =3D NULL; + struct phy_device *found; + struct phy_device *phydev; =20 - phydev =3D phy_find_first(adapter->mii_bus); - if (!phydev) { + found =3D phy_find_first(adapter->mii_bus); + if (!found) { dev_err(&adapter->pdev->dev, "no PHY found\n"); return -ENODEV; } =20 - phydev =3D phy_connect(netdev, phydev_name(phydev), + phydev =3D phy_connect(netdev, phydev_name(found), &et131x_adjust_link, PHY_INTERFACE_MODE_MII); + phy_device_put(found); =20 if (IS_ERR(phydev)) { dev_err(&adapter->pdev->dev, "Could not attach to PHY\n"); diff --git a/drivers/net/ethernet/amd/au1000_eth.c b/drivers/net/ethernet/a= md/au1000_eth.c index 5a04056e38fa..e9a9667e19ba 100644 --- a/drivers/net/ethernet/amd/au1000_eth.c +++ b/drivers/net/ethernet/amd/au1000_eth.c @@ -473,6 +473,7 @@ static int au1000_mii_probe(struct net_device *dev) { struct au1000_private *const aup =3D netdev_priv(dev); struct phy_device *phydev =3D NULL; + struct phy_device *found; int phy_addr; =20 if (aup->phy_static_config) { @@ -482,19 +483,25 @@ static int au1000_mii_probe(struct net_device *dev) phydev =3D mdiobus_get_phy(aup->mii_bus, aup->phy_addr); else netdev_info(dev, "using PHY-less setup\n"); + phy_device_put(phydev); return 0; } =20 /* find the first (lowest address) PHY * on the current MAC's MII bus */ - for (phy_addr =3D 0; phy_addr < PHY_MAX_ADDR; phy_addr++) - if (mdiobus_get_phy(aup->mii_bus, phy_addr)) { - phydev =3D mdiobus_get_phy(aup->mii_bus, phy_addr); + for (phy_addr =3D 0; phy_addr < PHY_MAX_ADDR; phy_addr++) { + struct phy_device *candidate; + + candidate =3D mdiobus_get_phy(aup->mii_bus, phy_addr); + if (candidate) { + phy_device_put(phydev); + phydev =3D candidate; if (!aup->phy_search_highest_addr) /* break out with first one found */ break; } + } =20 if (aup->phy1_search_mac0) { /* try harder to find a PHY */ @@ -511,16 +518,20 @@ static int au1000_mii_probe(struct net_device *dev) mdiobus_get_phy(aup->mii_bus, phy_addr); =20 - if (aup->mac_id =3D=3D 1) + if (aup->mac_id =3D=3D 1) { + phy_device_put(tmp_phydev); break; + } =20 /* no PHY here... */ if (!tmp_phydev) continue; =20 /* already claimed by MAC0 */ - if (tmp_phydev->attached_dev) + if (tmp_phydev->attached_dev) { + phy_device_put(tmp_phydev); continue; + } =20 phydev =3D tmp_phydev; break; /* found it */ @@ -536,8 +547,10 @@ static int au1000_mii_probe(struct net_device *dev) /* now we are supposed to have a proper phydev, to attach to... */ BUG_ON(phydev->attached_dev); =20 - phydev =3D phy_connect(dev, phydev_name(phydev), - &au1000_adjust_link, PHY_INTERFACE_MODE_MII); + found =3D phydev; + phydev =3D phy_connect(dev, phydev_name(found), &au1000_adjust_link, + PHY_INTERFACE_MODE_MII); + phy_device_put(found); =20 if (IS_ERR(phydev)) { netdev_err(dev, "Could not attach to PHY\n"); diff --git a/drivers/net/ethernet/apm/xgene-v2/mdio.c b/drivers/net/etherne= t/apm/xgene-v2/mdio.c index 6a17045a5f62..cca25136f6f7 100644 --- a/drivers/net/ethernet/apm/xgene-v2/mdio.c +++ b/drivers/net/ethernet/apm/xgene-v2/mdio.c @@ -100,6 +100,7 @@ int xge_mdio_config(struct net_device *ndev) struct xge_pdata *pdata =3D netdev_priv(ndev); struct device *dev =3D &pdata->pdev->dev; struct mii_bus *mdio_bus; + struct phy_device *found; struct phy_device *phydev; int ret; =20 @@ -120,15 +121,16 @@ int xge_mdio_config(struct net_device *ndev) if (ret) goto err; =20 - phydev =3D phy_find_first(mdio_bus); - if (!phydev) { + found =3D phy_find_first(mdio_bus); + if (!found) { dev_err(dev, "no PHY found\n"); ret =3D -ENODEV; goto err; } - phydev =3D phy_connect(ndev, phydev_name(phydev), + phydev =3D phy_connect(ndev, phydev_name(found), &xge_adjust_link, pdata->resources.phy_mode); + phy_device_put(found); =20 if (IS_ERR(phydev)) { netdev_err(ndev, "Could not attach to PHY\n"); diff --git a/drivers/net/ethernet/broadcom/b44.c b/drivers/net/ethernet/bro= adcom/b44.c index f994636fbd5f..be2c5786afca 100644 --- a/drivers/net/ethernet/broadcom/b44.c +++ b/drivers/net/ethernet/broadcom/b44.c @@ -2238,6 +2238,7 @@ static int b44_register_phy_one(struct b44 *bp) struct ssb_device *sdev =3D bp->sdev; struct phy_device *phydev; struct ssb_sprom *sprom =3D &sdev->bus->sprom; + bool phydev_ref =3D false; int err; =20 mii_bus =3D mdiobus_alloc(); @@ -2264,6 +2265,7 @@ static int b44_register_phy_one(struct b44 *bp) } =20 phydev =3D mdiobus_get_phy(bp->mii_bus, bp->phy_addr); + phydev_ref =3D !!phydev; if (!phydev && sprom->boardflags_lo & (B44_BOARDFLAG_ROBO | B44_BOARDFLAG_ADM)) { dev_info(sdev->dev, @@ -2281,6 +2283,8 @@ static int b44_register_phy_one(struct b44 *bp) err =3D phy_connect_direct(bp->dev, phydev, &b44_adjust_link, PHY_INTERFACE_MODE_MII); if (err) { + if (phydev_ref) + phy_device_put(phydev); dev_err(sdev->dev, "could not attach PHY at %i\n", bp->phy_addr); goto err_out_mdiobus_unregister; @@ -2297,6 +2301,8 @@ static int b44_register_phy_one(struct b44 *bp) bp->old_link =3D 0; =20 phy_attached_info(phydev); + if (phydev_ref) + phy_device_put(phydev); =20 return 0; =20 diff --git a/drivers/net/ethernet/broadcom/bgmac-bcma.c b/drivers/net/ether= net/broadcom/bgmac-bcma.c index 36f9bad28e6a..9d562952c377 100644 --- a/drivers/net/ethernet/broadcom/bgmac-bcma.c +++ b/drivers/net/ethernet/broadcom/bgmac-bcma.c @@ -209,6 +209,7 @@ static int bgmac_probe(struct bcma_device *core) if (ci->id =3D=3D BCMA_CHIP_ID_BCM53573 && phydev && (phydev->drv->phy_id & phydev->drv->phy_id_mask) =3D=3D PHY_ID_BCM54= 210E) phydev->dev_flags |=3D PHY_BRCM_EN_MASTER_MODE; + phy_device_put(phydev); } =20 if (core->bus->hosttype =3D=3D BCMA_HOSTTYPE_PCI) { diff --git a/drivers/net/ethernet/broadcom/genet/bcmmii.c b/drivers/net/eth= ernet/broadcom/genet/bcmmii.c index a4e0d5a68268..c8cbdcfb223b 100644 --- a/drivers/net/ethernet/broadcom/genet/bcmmii.c +++ b/drivers/net/ethernet/broadcom/genet/bcmmii.c @@ -305,6 +305,7 @@ int bcmgenet_mii_probe(struct net_device *dev) struct device_node *dn =3D kdev->of_node; phy_interface_t phy_iface =3D priv->phy_interface; struct phy_device *phydev; + bool put_phydev =3D false; u32 phy_flags =3D PHY_BRCM_AUTO_PWRDWN_ENABLE | PHY_BRCM_DIS_TXCRXC_NOENRGY | PHY_BRCM_IDDQ_SUSPEND; @@ -373,6 +374,7 @@ int bcmgenet_mii_probe(struct net_device *dev) pr_err("Unable to find PHY\n"); return -ENODEV; } + put_phydev =3D true; } else { phydev =3D dev->phydev; } @@ -380,6 +382,8 @@ int bcmgenet_mii_probe(struct net_device *dev) =20 ret =3D phy_connect_direct(dev, phydev, bcmgenet_mii_setup, phy_iface); + if (put_phydev) + phy_device_put(phydev); if (ret) { pr_err("could not attach to PHY\n"); return -ENODEV; diff --git a/drivers/net/ethernet/broadcom/sb1250-mac.c b/drivers/net/ether= net/broadcom/sb1250-mac.c index c839dd9804d3..a46e2d0f2480 100644 --- a/drivers/net/ethernet/broadcom/sb1250-mac.c +++ b/drivers/net/ethernet/broadcom/sb1250-mac.c @@ -2324,16 +2324,18 @@ static int sbmac_open(struct net_device *dev) static int sbmac_mii_probe(struct net_device *dev) { struct sbmac_softc *sc =3D netdev_priv(dev); + struct phy_device *found; struct phy_device *phy_dev; =20 - phy_dev =3D phy_find_first(sc->mii_bus); - if (!phy_dev) { + found =3D phy_find_first(sc->mii_bus); + if (!found) { printk(KERN_ERR "%s: no PHY found\n", dev->name); return -ENXIO; } =20 - phy_dev =3D phy_connect(dev, dev_name(&phy_dev->mdio.dev), + phy_dev =3D phy_connect(dev, dev_name(&found->mdio.dev), &sbmac_mii_poll, PHY_INTERFACE_MODE_GMII); + phy_device_put(found); if (IS_ERR(phy_dev)) { printk(KERN_ERR "%s: could not attach to PHY\n", dev->name); return PTR_ERR(phy_dev); diff --git a/drivers/net/ethernet/broadcom/tg3.c b/drivers/net/ethernet/bro= adcom/tg3.c index 73a4b569b03e..d55b979e01fe 100644 --- a/drivers/net/ethernet/broadcom/tg3.c +++ b/drivers/net/ethernet/broadcom/tg3.c @@ -1415,7 +1415,7 @@ static void tg3_mdio_config_5785(struct tg3 *tp) u32 val; struct phy_device *phydev; =20 - phydev =3D mdiobus_get_phy(tp->mdio_bus, tp->phy_addr); + phydev =3D tp->phydev; switch (phydev->drv->phy_id & phydev->drv->phy_id_mask) { case PHY_ID_BCM50610: case PHY_ID_BCM50610M: @@ -1566,6 +1566,7 @@ static int tg3_mdio_init(struct tg3 *tp) =20 if (!phydev || !phydev->drv) { dev_warn(&tp->pdev->dev, "No PHY devices\n"); + phy_device_put(phydev); mdiobus_unregister(tp->mdio_bus); mdiobus_free(tp->mdio_bus); return -ENODEV; @@ -1593,6 +1594,7 @@ static int tg3_mdio_init(struct tg3 *tp) tp->phy_flags |=3D TG3_PHYFLG_IS_FET; break; } + tp->phydev =3D phydev; =20 tg3_flag_set(tp, MDIOBUS_INITED); =20 @@ -1606,6 +1608,8 @@ static void tg3_mdio_fini(struct tg3 *tp) { if (tg3_flag(tp, MDIOBUS_INITED)) { tg3_flag_clear(tp, MDIOBUS_INITED); + phy_device_put(tp->phydev); + tp->phydev =3D NULL; mdiobus_unregister(tp->mdio_bus); mdiobus_free(tp->mdio_bus); } @@ -1966,7 +1970,7 @@ static void tg3_setup_flow_control(struct tg3 *tp, u3= 2 lcladv, u32 rmtadv) u32 old_tx_mode =3D tp->tx_mode; =20 if (tg3_flag(tp, USE_PHYLIB)) - autoneg =3D mdiobus_get_phy(tp->mdio_bus, tp->phy_addr)->autoneg; + autoneg =3D tp->phydev->autoneg; else autoneg =3D tp->link_config.autoneg; =20 @@ -2002,7 +2006,7 @@ static void tg3_adjust_link(struct net_device *dev) u8 oldflowctrl, linkmesg =3D 0; u32 mac_mode, lcl_adv, rmt_adv; struct tg3 *tp =3D netdev_priv(dev); - struct phy_device *phydev =3D mdiobus_get_phy(tp->mdio_bus, tp->phy_addr); + struct phy_device *phydev =3D tp->phydev; =20 spin_lock_bh(&tp->lock); =20 @@ -2091,7 +2095,7 @@ static int tg3_phy_init(struct tg3 *tp) /* Bring the PHY back to a known state. */ tg3_bmcr_reset(tp); =20 - phydev =3D mdiobus_get_phy(tp->mdio_bus, tp->phy_addr); + phydev =3D tp->phydev; =20 /* Attach the MAC to the PHY. */ phydev =3D phy_connect(tp->dev, phydev_name(phydev), @@ -2116,7 +2120,7 @@ static int tg3_phy_init(struct tg3 *tp) phy_support_asym_pause(phydev); break; default: - phy_disconnect(mdiobus_get_phy(tp->mdio_bus, tp->phy_addr)); + phy_disconnect(tp->phydev); return -EINVAL; } =20 @@ -2134,7 +2138,7 @@ static void tg3_phy_start(struct tg3 *tp) if (!(tp->phy_flags & TG3_PHYFLG_IS_CONNECTED)) return; =20 - phydev =3D mdiobus_get_phy(tp->mdio_bus, tp->phy_addr); + phydev =3D tp->phydev; =20 if (tp->phy_flags & TG3_PHYFLG_IS_LOW_POWER) { tp->phy_flags &=3D ~TG3_PHYFLG_IS_LOW_POWER; @@ -2155,13 +2159,13 @@ static void tg3_phy_stop(struct tg3 *tp) if (!(tp->phy_flags & TG3_PHYFLG_IS_CONNECTED)) return; =20 - phy_stop(mdiobus_get_phy(tp->mdio_bus, tp->phy_addr)); + phy_stop(tp->phydev); } =20 static void tg3_phy_fini(struct tg3 *tp) { if (tp->phy_flags & TG3_PHYFLG_IS_CONNECTED) { - phy_disconnect(mdiobus_get_phy(tp->mdio_bus, tp->phy_addr)); + phy_disconnect(tp->phydev); tp->phy_flags &=3D ~TG3_PHYFLG_IS_CONNECTED; } } @@ -4047,7 +4051,7 @@ static void tg3_power_down_prepare(struct tg3 *tp) struct phy_device *phydev; u32 phyid; =20 - phydev =3D mdiobus_get_phy(tp->mdio_bus, tp->phy_addr); + phydev =3D tp->phydev; =20 tp->phy_flags |=3D TG3_PHYFLG_IS_LOW_POWER; =20 @@ -12257,7 +12261,7 @@ static int tg3_get_link_ksettings(struct net_device= *dev, struct phy_device *phydev; if (!(tp->phy_flags & TG3_PHYFLG_IS_CONNECTED)) return -EAGAIN; - phydev =3D mdiobus_get_phy(tp->mdio_bus, tp->phy_addr); + phydev =3D tp->phydev; phy_ethtool_ksettings_get(phydev, cmd); =20 return 0; @@ -12333,7 +12337,7 @@ static int tg3_set_link_ksettings(struct net_device= *dev, struct phy_device *phydev; if (!(tp->phy_flags & TG3_PHYFLG_IS_CONNECTED)) return -EAGAIN; - phydev =3D mdiobus_get_phy(tp->mdio_bus, tp->phy_addr); + phydev =3D tp->phydev; return phy_ethtool_ksettings_set(phydev, cmd); } =20 @@ -12490,7 +12494,7 @@ static int tg3_nway_reset(struct net_device *dev) if (tg3_flag(tp, USE_PHYLIB)) { if (!(tp->phy_flags & TG3_PHYFLG_IS_CONNECTED)) return -EAGAIN; - r =3D phy_start_aneg(mdiobus_get_phy(tp->mdio_bus, tp->phy_addr)); + r =3D phy_start_aneg(tp->phydev); } else { u32 bmcr; =20 @@ -12623,7 +12627,7 @@ static int tg3_set_pauseparam(struct net_device *de= v, struct ethtool_pauseparam if (tg3_flag(tp, USE_PHYLIB)) { struct phy_device *phydev; =20 - phydev =3D mdiobus_get_phy(tp->mdio_bus, tp->phy_addr); + phydev =3D tp->phydev; =20 if (!phy_validate_pause(phydev, epause)) return -EINVAL; @@ -14064,7 +14068,7 @@ static int tg3_ioctl(struct net_device *dev, struct= ifreq *ifr, int cmd) struct phy_device *phydev; if (!(tp->phy_flags & TG3_PHYFLG_IS_CONNECTED)) return -EAGAIN; - phydev =3D mdiobus_get_phy(tp->mdio_bus, tp->phy_addr); + phydev =3D tp->phydev; return phy_mii_ioctl(phydev, ifr, cmd); } =20 diff --git a/drivers/net/ethernet/broadcom/tg3.h b/drivers/net/ethernet/bro= adcom/tg3.h index a9e7f88fa26d..b7cf1f0513cb 100644 --- a/drivers/net/ethernet/broadcom/tg3.h +++ b/drivers/net/ethernet/broadcom/tg3.h @@ -3273,6 +3273,7 @@ struct tg3 { int pcie_readrq; =20 struct mii_bus *mdio_bus; + struct phy_device *phydev; int old_link; =20 u8 phy_addr; diff --git a/drivers/net/ethernet/cadence/macb_main.c b/drivers/net/etherne= t/cadence/macb_main.c index d394f1f43b68..d2c835ad9e00 100644 --- a/drivers/net/ethernet/cadence/macb_main.c +++ b/drivers/net/ethernet/cadence/macb_main.c @@ -998,6 +998,7 @@ static int macb_phylink_connect(struct macb *bp) =20 /* attach the mac to the phy */ ret =3D phylink_connect_phy(bp->phylink, phydev); + phy_device_put(phydev); } =20 if (ret) { diff --git a/drivers/net/ethernet/engleder/tsnep_main.c b/drivers/net/ether= net/engleder/tsnep_main.c index 5db04e209dc6..dfae85dd7968 100644 --- a/drivers/net/ethernet/engleder/tsnep_main.c +++ b/drivers/net/ethernet/engleder/tsnep_main.c @@ -2668,6 +2668,8 @@ static int tsnep_probe(struct platform_device *pdev) tsnep_ptp_cleanup(adapter); ptp_init_failed: phy_init_failed: + phy_device_put(adapter->phydev); + adapter->phydev =3D NULL; if (adapter->mdiobus) mdiobus_unregister(adapter->mdiobus); mdio_init_failed: @@ -2686,6 +2688,8 @@ static void tsnep_remove(struct platform_device *pdev) =20 tsnep_ptp_cleanup(adapter); =20 + phy_device_put(adapter->phydev); + adapter->phydev =3D NULL; if (adapter->mdiobus) mdiobus_unregister(adapter->mdiobus); =20 diff --git a/drivers/net/ethernet/ethoc.c b/drivers/net/ethernet/ethoc.c index 0c418557264c..021c1719131c 100644 --- a/drivers/net/ethernet/ethoc.c +++ b/drivers/net/ethernet/ethoc.c @@ -715,10 +715,13 @@ static int ethoc_mdio_probe(struct net_device *dev) =20 err =3D phy_connect_direct(dev, phy, ethoc_mdio_poll, PHY_INTERFACE_MODE_GMII); - if (err) + if (err) { + phy_device_put(phy); return dev_err_probe(&dev->dev, err, "could not attach to PHY\n"); + } =20 phy_set_max_speed(phy, SPEED_100); + phy_device_put(phy); =20 return 0; } @@ -782,6 +785,8 @@ static int ethoc_ioctl(struct net_device *dev, struct i= freq *ifr, int cmd) struct ethoc *priv =3D netdev_priv(dev); struct mii_ioctl_data *mdio =3D if_mii(ifr); struct phy_device *phy =3D NULL; + bool put_phy =3D false; + int ret; =20 if (!netif_running(dev)) return -EINVAL; @@ -793,11 +798,16 @@ static int ethoc_ioctl(struct net_device *dev, struct= ifreq *ifr, int cmd) phy =3D mdiobus_get_phy(priv->mdio, mdio->phy_id); if (!phy) return -ENODEV; + put_phy =3D true; } else { phy =3D dev->phydev; } =20 - return phy_mii_ioctl(phy, ifr, cmd); + ret =3D phy_mii_ioctl(phy, ifr, cmd); + if (put_phy) + phy_device_put(phy); + + return ret; } =20 static void ethoc_do_set_mac_address(struct net_device *dev) diff --git a/drivers/net/ethernet/faraday/ftgmac100.c b/drivers/net/etherne= t/faraday/ftgmac100.c index 6d2fe5c2f390..69228f09796f 100644 --- a/drivers/net/ethernet/faraday/ftgmac100.c +++ b/drivers/net/ethernet/faraday/ftgmac100.c @@ -1500,6 +1500,7 @@ static int ftgmac100_mii_probe(struct net_device *net= dev) struct platform_device *pdev =3D to_platform_device(priv->dev); struct device_node *np =3D pdev->dev.of_node; struct phy_device *phydev; + struct phy_device *found; phy_interface_t phy_intf; int err; =20 @@ -1533,14 +1534,15 @@ static int ftgmac100_mii_probe(struct net_device *n= etdev) phy_modes(phy_intf)); } =20 - phydev =3D phy_find_first(priv->mii_bus); - if (!phydev) { + found =3D phy_find_first(priv->mii_bus); + if (!found) { netdev_info(netdev, "%s: no PHY found\n", netdev->name); return -ENODEV; } =20 - phydev =3D phy_connect(netdev, phydev_name(phydev), + phydev =3D phy_connect(netdev, phydev_name(found), &ftgmac100_adjust_link, phy_intf); + phy_device_put(found); =20 if (IS_ERR(phydev)) { netdev_err(netdev, "%s: Could not attach to PHY\n", netdev->name); diff --git a/drivers/net/ethernet/freescale/fec_main.c b/drivers/net/ethern= et/freescale/fec_main.c index ced4dbf8cd90..71d42e061789 100644 --- a/drivers/net/ethernet/freescale/fec_main.c +++ b/drivers/net/ethernet/freescale/fec_main.c @@ -2982,6 +2982,7 @@ static int fec_enet_mii_probe(struct net_device *ndev) { struct fec_enet_private *fep =3D netdev_priv(ndev); struct phy_device *phy_dev; + bool phydev_ref =3D false; int ret; =20 if (fep->phy_node) { @@ -2995,8 +2996,13 @@ static int fec_enet_mii_probe(struct net_device *nde= v) } else { /* check for attached phy */ phy_dev =3D phy_find_first(fep->mii_bus); - if (fep->dev_id && phy_dev) - phy_dev =3D phy_find_next(fep->mii_bus, phy_dev); + if (fep->dev_id && phy_dev) { + struct phy_device *first =3D phy_dev; + + phy_dev =3D phy_find_next(fep->mii_bus, first); + phy_device_put(first); + } + phydev_ref =3D !!phy_dev; =20 if (!phy_dev) { netdev_info(ndev, "no PHY, assuming direct connection to switch\n"); @@ -3012,6 +3018,8 @@ static int fec_enet_mii_probe(struct net_device *ndev) if (ret) { if (phy_is_pseudo_fixed_link(phy_dev)) fixed_phy_unregister(phy_dev); + if (phydev_ref) + phy_device_put(phy_dev); netdev_err(ndev, "could not attach to PHY\n"); return ret; } @@ -3035,6 +3043,8 @@ static int fec_enet_mii_probe(struct net_device *ndev) fep->full_duplex =3D 0; =20 phy_attached_info(phy_dev); + if (phydev_ref) + phy_device_put(phy_dev); =20 return 0; } diff --git a/drivers/net/ethernet/hisilicon/hibmcge/hbg_mdio.c b/drivers/ne= t/ethernet/hisilicon/hibmcge/hbg_mdio.c index b6f0a2780ea8..bf5cf9a6c927 100644 --- a/drivers/net/ethernet/hisilicon/hibmcge/hbg_mdio.c +++ b/drivers/net/ethernet/hisilicon/hibmcge/hbg_mdio.c @@ -263,6 +263,11 @@ static int hbg_fixed_phy_init(struct hbg_priv *priv) return hbg_phy_connect(priv); } =20 +static void hbg_phy_put(void *data) +{ + phy_device_put(data); +} + int hbg_mdio_init(struct hbg_priv *priv) { struct device *dev =3D &priv->pdev->dev; @@ -297,6 +302,9 @@ int hbg_mdio_init(struct hbg_priv *priv) if (!phydev) return dev_err_probe(dev, -ENODEV, "failed to get phy device\n"); + ret =3D devm_add_action_or_reset(dev, hbg_phy_put, phydev); + if (ret) + return ret; =20 mac->phydev =3D phydev; hbg_mdio_init_hw(priv); diff --git a/drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_main.c b/driv= ers/net/ethernet/hisilicon/hns3/hns3pf/hclge_main.c index a08d8a35aef9..25926bb77c5b 100644 --- a/drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_main.c +++ b/drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_main.c @@ -9731,8 +9731,11 @@ static int hclge_init_ae_dev(struct hnae3_ae_dev *ae= _dev) err_ptp_uninit: hclge_ptp_uninit(hdev); err_mdiobus_unreg: - if (hdev->hw.mac.phydev) + if (hdev->hw.mac.phydev) { + phy_device_put(hdev->hw.mac.phydev); + hdev->hw.mac.phydev =3D NULL; mdiobus_unregister(hdev->hw.mac.mdio_bus); + } err_msi_irq_uninit: hclge_misc_irq_uninit(hdev); err_msi_uninit: @@ -10131,8 +10134,11 @@ static void hclge_uninit_ae_dev(struct hnae3_ae_de= v *ae_dev) hclge_uninit_mac_table(hdev); hclge_del_all_fd_entries(hdev); =20 - if (mac->phydev) + if (mac->phydev) { + phy_device_put(mac->phydev); + mac->phydev =3D NULL; mdiobus_unregister(mac->mdio_bus); + } =20 /* Disable MISC vector(vector0) */ hclge_enable_vector(&hdev->misc_vector, false); diff --git a/drivers/net/ethernet/lantiq_etop.c b/drivers/net/ethernet/lant= iq_etop.c index 83ce3bfefa5c..0ace9a7f52ac 100644 --- a/drivers/net/ethernet/lantiq_etop.c +++ b/drivers/net/ethernet/lantiq_etop.c @@ -358,16 +358,18 @@ ltq_etop_mdio_probe(struct net_device *dev) { struct ltq_etop_priv *priv =3D netdev_priv(dev); struct phy_device *phydev; + struct phy_device *found; =20 - phydev =3D phy_find_first(priv->mii_bus); + found =3D phy_find_first(priv->mii_bus); =20 - if (!phydev) { + if (!found) { netdev_err(dev, "no PHY found\n"); return -ENODEV; } =20 - phydev =3D phy_connect(dev, phydev_name(phydev), + phydev =3D phy_connect(dev, phydev_name(found), <q_etop_mdio_link, priv->pldata->mii_mode); + phy_device_put(found); =20 if (IS_ERR(phydev)) { netdev_err(dev, "Could not attach to PHY\n"); diff --git a/drivers/net/ethernet/mellanox/mlxbf_gige/mlxbf_gige_main.c b/d= rivers/net/ethernet/mellanox/mlxbf_gige/mlxbf_gige_main.c index d1f8a72cae53..7c61b5a2e35e 100644 --- a/drivers/net/ethernet/mellanox/mlxbf_gige/mlxbf_gige_main.c +++ b/drivers/net/ethernet/mellanox/mlxbf_gige/mlxbf_gige_main.c @@ -478,6 +478,7 @@ static int mlxbf_gige_probe(struct platform_device *pde= v) mlxbf_gige_link_cfgs[priv->hw_version].adjust_link, mlxbf_gige_link_cfgs[priv->hw_version].phy_mode); if (err) { + phy_device_put(phydev); dev_err(&pdev->dev, "Could not attach to PHY: %pe\n", ERR_PTR(err)); goto out; } @@ -486,6 +487,7 @@ static int mlxbf_gige_probe(struct platform_device *pde= v) =20 /* Display information about attached PHY device */ phy_attached_info(phydev); + phy_device_put(phydev); =20 err =3D register_netdev(netdev); if (err) { diff --git a/drivers/net/ethernet/microchip/lan743x_main.c b/drivers/net/et= hernet/microchip/lan743x_main.c index 24ae56a3c9ed..133422f96cd5 100644 --- a/drivers/net/ethernet/microchip/lan743x_main.c +++ b/drivers/net/ethernet/microchip/lan743x_main.c @@ -3245,6 +3245,7 @@ static int lan743x_phylink_connect(struct lan743x_ada= pter *adapter) if (phydev) { /* attach the mac to the phy */ ret =3D phylink_connect_phy(adapter->phylink, phydev); + phy_device_put(phydev); } else if (((adapter->csr.id_rev & ID_REV_ID_MASK_) =3D=3D ID_REV_ID_LAN7431_) || adapter->is_pci11x1x) { struct phylink_link_state state; diff --git a/drivers/net/ethernet/nxp/lpc_eth.c b/drivers/net/ethernet/nxp/= lpc_eth.c index 8b9a3e3bba30..281a2c15669a 100644 --- a/drivers/net/ethernet/nxp/lpc_eth.c +++ b/drivers/net/ethernet/nxp/lpc_eth.c @@ -752,6 +752,7 @@ static int lpc_mii_probe(struct net_device *ndev) { struct netdata_local *pldat =3D netdev_priv(ndev); struct phy_device *phydev; + struct phy_device *found; =20 /* Attach to the PHY */ if (lpc_phy_interface_mode(&pldat->pdev->dev) =3D=3D PHY_INTERFACE_MODE_M= II) @@ -760,17 +761,18 @@ static int lpc_mii_probe(struct net_device *ndev) netdev_info(ndev, "using RMII interface\n"); =20 if (pldat->phy_node) - phydev =3D of_phy_find_device(pldat->phy_node); + found =3D of_phy_find_device(pldat->phy_node); else - phydev =3D phy_find_first(pldat->mii_bus); - if (!phydev) { + found =3D phy_find_first(pldat->mii_bus); + if (!found) { netdev_err(ndev, "no PHY found\n"); return -ENODEV; } =20 - phydev =3D phy_connect(ndev, phydev_name(phydev), + phydev =3D phy_connect(ndev, phydev_name(found), &lpc_handle_link_change, lpc_phy_interface_mode(&pldat->pdev->dev)); + phy_device_put(found); if (IS_ERR(phydev)) { netdev_err(ndev, "Could not attach to PHY\n"); return PTR_ERR(phydev); diff --git a/drivers/net/ethernet/oa_tc6.c b/drivers/net/ethernet/oa_tc6.c index 417c15d1ff42..9dd6c91f7a70 100644 --- a/drivers/net/ethernet/oa_tc6.c +++ b/drivers/net/ethernet/oa_tc6.c @@ -605,11 +605,14 @@ static int oa_tc6_phy_init(struct oa_tc6 *tc6) &oa_tc6_handle_link_change, PHY_INTERFACE_MODE_INTERNAL); if (ret) { + phy_device_put(tc6->phydev); + tc6->phydev =3D NULL; netdev_err(tc6->netdev, "Can't attach PHY to %s\n", tc6->mdiobus->id); oa_tc6_mdiobus_unregister(tc6); return ret; } + phy_device_put(tc6->phydev); =20 phy_attached_info(tc6->netdev->phydev); =20 diff --git a/drivers/net/ethernet/qualcomm/emac/emac-phy.c b/drivers/net/et= hernet/qualcomm/emac/emac-phy.c index 5c94af7bb6b6..43e968d1b430 100644 --- a/drivers/net/ethernet/qualcomm/emac/emac-phy.c +++ b/drivers/net/ethernet/qualcomm/emac/emac-phy.c @@ -125,13 +125,6 @@ int emac_phy_config(struct platform_device *pdev, stru= ct emac_adapter *adpt) else adpt->phydev =3D mdiobus_get_phy(mii_bus, phy_addr); =20 - /* of_phy_find_device() claims a reference to the phydev, - * so we do that here manually as well. When the driver - * later unloads, it can unilaterally drop the reference - * without worrying about ACPI vs DT. - */ - if (adpt->phydev) - get_device(&adpt->phydev->mdio.dev); } else { struct device_node *phy_np; =20 diff --git a/drivers/net/ethernet/rdc/r6040.c b/drivers/net/ethernet/rdc/r6= 040.c index f4d434c379e7..e80aa4cd43a6 100644 --- a/drivers/net/ethernet/rdc/r6040.c +++ b/drivers/net/ethernet/rdc/r6040.c @@ -998,16 +998,18 @@ static void r6040_adjust_link(struct net_device *dev) static int r6040_mii_probe(struct net_device *dev) { struct r6040_private *lp =3D netdev_priv(dev); - struct phy_device *phydev =3D NULL; + struct phy_device *phydev; + struct phy_device *found; =20 - phydev =3D phy_find_first(lp->mii_bus); - if (!phydev) { + found =3D phy_find_first(lp->mii_bus); + if (!found) { dev_err(&lp->pdev->dev, "no PHY found\n"); return -ENODEV; } =20 - phydev =3D phy_connect(dev, phydev_name(phydev), &r6040_adjust_link, + phydev =3D phy_connect(dev, phydev_name(found), &r6040_adjust_link, PHY_INTERFACE_MODE_MII); + phy_device_put(found); =20 if (IS_ERR(phydev)) { dev_err(&lp->pdev->dev, "could not attach to PHY\n"); diff --git a/drivers/net/ethernet/realtek/r8169_main.c b/drivers/net/ethern= et/realtek/r8169_main.c index ec4fc21fa21f..bbab465bf90d 100644 --- a/drivers/net/ethernet/realtek/r8169_main.c +++ b/drivers/net/ethernet/realtek/r8169_main.c @@ -5423,6 +5423,11 @@ static int r8169_mdio_write_reg_c45(struct mii_bus *= mii_bus, int addr, return 0; } =20 +static void r8169_phy_put(void *data) +{ + phy_device_put(data); +} + static int r8169_mdio_register(struct rtl8169_private *tp) { struct pci_dev *pdev =3D tp->pci_dev; @@ -5463,9 +5468,15 @@ static int r8169_mdio_register(struct rtl8169_privat= e *tp) return ret; =20 tp->phydev =3D mdiobus_get_phy(new_bus, 0); - if (!tp->phydev) { + if (!tp->phydev) return -ENODEV; - } else if (!tp->phydev->drv) { + + ret =3D devm_add_action_or_reset(&pdev->dev, r8169_phy_put, + tp->phydev); + if (ret) + return ret; + + if (!tp->phydev->drv) { /* Most chip versions fail with the genphy driver. * Therefore ensure that the dedicated PHY driver is loaded. */ diff --git a/drivers/net/ethernet/samsung/sxgbe/sxgbe_mdio.c b/drivers/net/= ethernet/samsung/sxgbe/sxgbe_mdio.c index 0227223c06fa..6598978d8a92 100644 --- a/drivers/net/ethernet/samsung/sxgbe/sxgbe_mdio.c +++ b/drivers/net/ethernet/samsung/sxgbe/sxgbe_mdio.c @@ -273,6 +273,7 @@ int sxgbe_mdio_register(struct net_device *ndev) phydev_name(phy), act ? " active" : ""); phy_found =3D true; } + phy_device_put(phy); } =20 if (!phy_found) { diff --git a/drivers/net/ethernet/smsc/smsc911x.c b/drivers/net/ethernet/sm= sc/smsc911x.c index 3ebd0664c697..5429c78b64e5 100644 --- a/drivers/net/ethernet/smsc/smsc911x.c +++ b/drivers/net/ethernet/smsc/smsc911x.c @@ -1032,6 +1032,7 @@ static int smsc911x_mii_probe(struct net_device *dev) pdata->config.phy_interface); =20 if (ret) { + phy_device_put(phydev); netdev_err(dev, "Could not attach to PHY\n"); return ret; } @@ -1042,6 +1043,7 @@ static int smsc911x_mii_probe(struct net_device *dev) =20 /* mask with MAC supported features */ phy_support_asym_pause(phydev); + phy_device_put(phydev); =20 pdata->last_duplex =3D -1; pdata->last_carrier =3D -1; @@ -1107,8 +1109,10 @@ static int smsc911x_mii_init(struct platform_device = *pdev, } =20 phydev =3D phy_find_first(pdata->mii_bus); - if (phydev) + if (phydev) { phydev->mac_managed_pm =3D true; + phy_device_put(phydev); + } =20 return 0; =20 diff --git a/drivers/net/ethernet/smsc/smsc9420.c b/drivers/net/ethernet/sm= sc/smsc9420.c index 0c97211b6254..19a6727676d4 100644 --- a/drivers/net/ethernet/smsc/smsc9420.c +++ b/drivers/net/ethernet/smsc/smsc9420.c @@ -1102,19 +1102,21 @@ static void smsc9420_phy_adjust_link(struct net_dev= ice *dev) static int smsc9420_mii_probe(struct net_device *dev) { struct smsc9420_pdata *pd =3D netdev_priv(dev); - struct phy_device *phydev =3D NULL; + struct phy_device *phydev; + struct phy_device *found; =20 BUG_ON(dev->phydev); =20 /* Device only supports internal PHY at address 1 */ - phydev =3D mdiobus_get_phy(pd->mii_bus, 1); - if (!phydev) { + found =3D mdiobus_get_phy(pd->mii_bus, 1); + if (!found) { netdev_err(dev, "no PHY found at address 1\n"); return -ENODEV; } =20 - phydev =3D phy_connect(dev, phydev_name(phydev), + phydev =3D phy_connect(dev, phydev_name(found), smsc9420_phy_adjust_link, PHY_INTERFACE_MODE_MII); + phy_device_put(found); =20 if (IS_ERR(phydev)) { netdev_err(dev, "Could not attach to PHY\n"); diff --git a/drivers/net/ethernet/stmicro/stmmac/stmmac_main.c b/drivers/ne= t/ethernet/stmicro/stmmac/stmmac_main.c index 79b71466d5b0..dd74e2e2d0ac 100644 --- a/drivers/net/ethernet/stmicro/stmmac/stmmac_main.c +++ b/drivers/net/ethernet/stmicro/stmmac/stmmac_main.c @@ -1347,6 +1347,7 @@ static int stmmac_init_phy(struct net_device *dev) phydev->dev_flags |=3D dev_flags; =20 ret =3D phylink_connect_phy(priv->phylink, phydev); + phy_device_put(phydev); } else { fwnode_handle_put(phy_fwnode); ret =3D phylink_fwnode_phy_connect(priv->phylink, fwnode, dev_flags); diff --git a/drivers/net/ethernet/stmicro/stmmac/stmmac_mdio.c b/drivers/ne= t/ethernet/stmicro/stmmac/stmmac_mdio.c index afe98ff5bdcb..ce7e52c638b4 100644 --- a/drivers/net/ethernet/stmicro/stmmac/stmmac_mdio.c +++ b/drivers/net/ethernet/stmicro/stmmac/stmmac_mdio.c @@ -681,6 +681,7 @@ int stmmac_mdio_register(struct net_device *ndev) =20 phydev =3D phy_find_first(new_bus); if (!phydev || phydev->mdio.addr > max_addr) { + phy_device_put(phydev); dev_warn(dev, "No PHY found\n"); err =3D -ENODEV; goto no_phy_found; @@ -703,6 +704,7 @@ int stmmac_mdio_register(struct net_device *ndev) priv->plat->phy_addr =3D phydev->mdio.addr; =20 phy_attached_info(phydev); + phy_device_put(phydev); =20 bus_register_done: priv->mii =3D new_bus; diff --git a/drivers/net/ethernet/tehuti/tn40_phy.c b/drivers/net/ethernet/= tehuti/tn40_phy.c index 39eef7ca7958..694967f7ccca 100644 --- a/drivers/net/ethernet/tehuti/tn40_phy.c +++ b/drivers/net/ethernet/tehuti/tn40_phy.c @@ -62,8 +62,10 @@ int tn40_phy_register(struct tn40_priv *priv) =20 phylink =3D phylink_create(config, NULL, PHY_INTERFACE_MODE_XAUI, &tn40_mac_ops); - if (IS_ERR(phylink)) + if (IS_ERR(phylink)) { + phy_device_put(phydev); return PTR_ERR(phylink); + } =20 priv->phydev =3D phydev; priv->phylink =3D phylink; @@ -73,4 +75,5 @@ int tn40_phy_register(struct tn40_priv *priv) void tn40_phy_unregister(struct tn40_priv *priv) { phylink_destroy(priv->phylink); + phy_device_put(priv->phydev); } diff --git a/drivers/net/ethernet/ti/davinci_emac.c b/drivers/net/ethernet/= ti/davinci_emac.c index 3802122d0f1b..dee71d2858ec 100644 --- a/drivers/net/ethernet/ti/davinci_emac.c +++ b/drivers/net/ethernet/ti/davinci_emac.c @@ -1401,7 +1401,9 @@ static int emac_devioctl(struct net_device *ndev, str= uct ifreq *ifrq, int cmd) */ static int emac_dev_open(struct net_device *ndev) { + char phy_id_buf[MII_BUS_ID_SIZE + 3]; struct device *emac_dev =3D &ndev->dev; + const char *phy_id; struct resource *res; int q, m, ret; int res_num =3D 0, irq_num =3D 0; @@ -1409,6 +1411,8 @@ static int emac_dev_open(struct net_device *ndev) struct emac_priv *priv =3D netdev_priv(ndev); struct phy_device *phydev =3D NULL; =20 + phy_id =3D priv->phy_id; + ret =3D pm_runtime_resume_and_get(&priv->pdev->dev); if (ret < 0) { dev_err(&priv->pdev->dev, "%s: failed to get_sync(%d)\n", @@ -1493,7 +1497,7 @@ static int emac_dev_open(struct net_device *ndev) } =20 /* if no phy-handle and no fixed link, use the first phy on the bus */ - if (!phydev && !priv->phy_id) { + if (!phydev && !phy_id) { struct device_node *np; =20 np =3D of_find_compatible_node(NULL, NULL, "ti,davinci_mdio"); @@ -1503,21 +1507,25 @@ static int emac_dev_open(struct net_device *ndev) if (bus) { struct phy_device *phy =3D phy_find_first(bus); =20 - if (phy) - priv->phy_id =3D phydev_name(phy); + if (phy) { + strscpy(phy_id_buf, phydev_name(phy), + sizeof(phy_id_buf)); + phy_id =3D phy_id_buf; + phy_device_put(phy); + } put_device(&bus->dev); /* of_mdio_find_bus */ } of_node_put(np); /* of_find_compatible_node */ } } =20 - if (!phydev && priv->phy_id && *priv->phy_id) { - phydev =3D phy_connect(ndev, priv->phy_id, + if (!phydev && phy_id && *phy_id) { + phydev =3D phy_connect(ndev, phy_id, &emac_adjust_link, PHY_INTERFACE_MODE_MII); if (IS_ERR(phydev)) { dev_err(emac_dev, "could not connect to phy %s\n", - priv->phy_id); + phy_id); ret =3D PTR_ERR(phydev); goto err; } diff --git a/drivers/net/ethernet/toshiba/tc35815.c b/drivers/net/ethernet/= toshiba/tc35815.c index 6e3758dfbdbd..89a8e53f0342 100644 --- a/drivers/net/ethernet/toshiba/tc35815.c +++ b/drivers/net/ethernet/toshiba/tc35815.c @@ -609,17 +609,19 @@ static int tc_mii_probe(struct net_device *dev) __ETHTOOL_DECLARE_LINK_MODE_MASK(mask) =3D { 0, }; struct tc35815_local *lp =3D netdev_priv(dev); struct phy_device *phydev; + struct phy_device *found; =20 - phydev =3D phy_find_first(lp->mii_bus); - if (!phydev) { + found =3D phy_find_first(lp->mii_bus); + if (!found) { printk(KERN_ERR "%s: no PHY found\n", dev->name); return -ENODEV; } =20 /* attach the mac to the phy */ - phydev =3D phy_connect(dev, phydev_name(phydev), + phydev =3D phy_connect(dev, phydev_name(found), &tc_handle_link_change, lp->chiptype =3D=3D TC35815_TX4939 ? PHY_INTERFACE_MODE_RMII : PHY= _INTERFACE_MODE_MII); + phy_device_put(found); if (IS_ERR(phydev)) { printk(KERN_ERR "%s: Could not attach to PHY\n", dev->name); return PTR_ERR(phydev); diff --git a/drivers/net/ethernet/wangxun/ngbe/ngbe_mdio.c b/drivers/net/et= hernet/wangxun/ngbe/ngbe_mdio.c index c63bb6e6f405..a201569e6b72 100644 --- a/drivers/net/ethernet/wangxun/ngbe/ngbe_mdio.c +++ b/drivers/net/ethernet/wangxun/ngbe/ngbe_mdio.c @@ -156,6 +156,11 @@ static int ngbe_phylink_init(struct wx *wx) return 0; } =20 +static void ngbe_phy_put(void *data) +{ + phy_device_put(data); +} + int ngbe_mdio_init(struct wx *wx) { struct pci_dev *pdev =3D wx->pdev; @@ -186,6 +191,9 @@ int ngbe_mdio_init(struct wx *wx) wx->phydev =3D phy_find_first(mii_bus); if (!wx->phydev) return -ENODEV; + ret =3D devm_add_action_or_reset(&pdev->dev, ngbe_phy_put, wx->phydev); + if (ret) + return ret; =20 phy_attached_info(wx->phydev); =20 diff --git a/drivers/net/ethernet/wangxun/txgbe/txgbe_phy.c b/drivers/net/e= thernet/wangxun/txgbe/txgbe_phy.c index dc9f24314658..466218fd831c 100644 --- a/drivers/net/ethernet/wangxun/txgbe/txgbe_phy.c +++ b/drivers/net/ethernet/wangxun/txgbe/txgbe_phy.c @@ -521,6 +521,11 @@ static int txgbe_sfp_register(struct txgbe *txgbe) return 0; } =20 +static void txgbe_phy_put(void *data) +{ + phy_device_put(data); +} + static int txgbe_ext_phy_init(struct txgbe *txgbe) { struct phy_device *phydev; @@ -555,6 +560,9 @@ static int txgbe_ext_phy_init(struct txgbe *txgbe) wx_err(wx, "no PHY found\n"); return -ENODEV; } + ret =3D devm_add_action_or_reset(&pdev->dev, txgbe_phy_put, phydev); + if (ret) + return ret; =20 phy_attached_info(phydev); =20 diff --git a/drivers/net/phy/mdio_bus.c b/drivers/net/phy/mdio_bus.c index 00d0e4159e9b..8c6d0e6ab472 100644 --- a/drivers/net/phy/mdio_bus.c +++ b/drivers/net/phy/mdio_bus.c @@ -18,6 +18,7 @@ #include #include #include +#include #include #include #include @@ -27,34 +28,47 @@ #define CREATE_TRACE_POINTS #include =20 -static struct mdio_device *mdiobus_find_device(struct mii_bus *bus, int ad= dr) -{ - bool addr_valid =3D addr >=3D 0 && addr < ARRAY_SIZE(bus->mdio_map); - - if (WARN_ONCE(!addr_valid, "addr %d out of range\n", addr)) - return NULL; - - return bus->mdio_map[addr]; -} - +/** + * mdiobus_get_phy - get a PHY from an MDIO bus by address + * @bus: MDIO bus containing the PHY + * @addr: PHY address on the bus + * + * Return: A referenced PHY device, or %NULL when the address is invalid or + * does not contain a registered PHY. The caller must call phy_device_put(= ). + */ struct phy_device *mdiobus_get_phy(struct mii_bus *bus, int addr) { struct mdio_device *mdiodev; + struct phy_device *phydev =3D NULL; + bool addr_valid =3D addr >=3D 0 && addr < ARRAY_SIZE(bus->mdio_map); =20 - mdiodev =3D mdiobus_find_device(bus, addr); - if (!mdiodev) + if (WARN_ONCE(!addr_valid, "addr %d out of range\n", addr)) return NULL; =20 - if (!(mdiodev->flags & MDIO_DEVICE_FLAG_PHY)) - return NULL; + rcu_read_lock(); + mdiodev =3D rcu_dereference(bus->mdio_map[addr]); + if (mdiodev && mdiodev->flags & MDIO_DEVICE_FLAG_PHY) { + get_device(&mdiodev->dev); + phydev =3D container_of(mdiodev, struct phy_device, mdio); + } + rcu_read_unlock(); =20 - return container_of(mdiodev, struct phy_device, mdio); + return phydev; } EXPORT_SYMBOL(mdiobus_get_phy); =20 bool mdiobus_is_registered_device(struct mii_bus *bus, int addr) { - return mdiobus_find_device(bus, addr) !=3D NULL; + bool addr_valid =3D addr >=3D 0 && addr < ARRAY_SIZE(bus->mdio_map); + bool registered; + + if (WARN_ONCE(!addr_valid, "addr %d out of range\n", addr)) + return false; + + registered =3D rcu_access_pointer(bus->mdio_map[addr]) || + (READ_ONCE(bus->mdio_map_pending) & BIT(addr)); + + return registered; } EXPORT_SYMBOL(mdiobus_is_registered_device); =20 diff --git a/drivers/net/phy/mdio_bus_provider.c b/drivers/net/phy/mdio_bus= _provider.c index ce3a607a40cb..6ff647b28e8f 100644 --- a/drivers/net/phy/mdio_bus_provider.c +++ b/drivers/net/phy/mdio_bus_provider.c @@ -330,6 +330,8 @@ struct mii_bus *mdiobus_alloc_size(size_t size) return NULL; =20 bus->state =3D MDIOBUS_ALLOCATED; + mutex_init(&bus->mdio_map_lock); + init_waitqueue_head(&bus->mdio_map_wait); if (size) bus->priv =3D (void *)bus + aligned_size; =20 @@ -355,10 +357,9 @@ static int of_mdiobus_find_phy(struct device *dev, str= uct mdio_device *mdiodev, struct device_node *np) { struct device_node *child; + int addr, ret; =20 for_each_available_child_of_node(np, child) { - int addr; - if (of_node_name_eq(child, "ethernet-phy-package")) { /* Validate PHY package reg presence */ if (!of_property_present(child, "reg")) { @@ -366,12 +367,13 @@ static int of_mdiobus_find_phy(struct device *dev, st= ruct mdio_device *mdiodev, return -EINVAL; } =20 - if (!of_mdiobus_find_phy(dev, mdiodev, child)) { + ret =3D of_mdiobus_find_phy(dev, mdiodev, child); + if (!ret || ret =3D=3D -EBUSY) { /* The refcount for the PHY package will be * incremented later when PHY join the Package. */ of_node_put(child); - return 0; + return ret; } =20 continue; @@ -382,6 +384,11 @@ static int of_mdiobus_find_phy(struct device *dev, str= uct mdio_device *mdiodev, continue; =20 if (addr =3D=3D mdiodev->addr) { + if (of_node_test_and_set_flag(child, OF_POPULATED)) { + of_node_put(child); + return -EBUSY; + } + device_set_node(dev, of_fwnode_handle(child)); /* The refcount on "child" is passed to the mdio * device. Do _not_ use of_node_put(child) here. @@ -393,22 +400,26 @@ static int of_mdiobus_find_phy(struct device *dev, st= ruct mdio_device *mdiodev, return -ENODEV; } =20 -static void of_mdiobus_link_mdiodev(struct mii_bus *bus, - struct mdio_device *mdiodev) +static int of_mdiobus_link_mdiodev(struct mii_bus *bus, + struct mdio_device *mdiodev) { struct device *dev =3D &mdiodev->dev; =20 if (dev->of_node || !bus->dev.of_node) - return; + return 0; =20 - of_mdiobus_find_phy(dev, mdiodev, bus->dev.of_node); + return of_mdiobus_find_phy(dev, mdiodev, bus->dev.of_node); } #endif =20 -static struct phy_device *mdiobus_scan(struct mii_bus *bus, int addr, bool= c45) +static struct phy_device *__mdiobus_scan(struct mii_bus *bus, int addr, + bool c45) { struct phy_device *phydev =3D ERR_PTR(-ENODEV); struct fwnode_handle *fwnode; +#if IS_ENABLED(CONFIG_OF_MDIO) + bool of_node_populated =3D false; +#endif char node_name[16]; int err; =20 @@ -420,7 +431,12 @@ static struct phy_device *mdiobus_scan(struct mii_bus = *bus, int addr, bool c45) /* For DT, see if the auto-probed phy has a corresponding child * in the bus node, and set the of_node pointer in this case. */ - of_mdiobus_link_mdiodev(bus, &phydev->mdio); + err =3D of_mdiobus_link_mdiodev(bus, &phydev->mdio); + if (err =3D=3D -EBUSY) { + phy_device_free(phydev); + return ERR_PTR(-ENODEV); + } + of_node_populated =3D !!phydev->mdio.dev.of_node; #endif =20 /* Search for a swnode for the phy in the swnode hierarchy of the bus. @@ -437,6 +453,11 @@ static struct phy_device *mdiobus_scan(struct mii_bus = *bus, int addr, bool c45) =20 err =3D phy_device_register(phydev); if (err) { +#if IS_ENABLED(CONFIG_OF_MDIO) + if (of_node_populated) + of_node_clear_flag(phydev->mdio.dev.of_node, + OF_POPULATED); +#endif phy_device_free(phydev); return ERR_PTR(-ENODEV); } @@ -458,7 +479,17 @@ static struct phy_device *mdiobus_scan(struct mii_bus = *bus, int addr, bool c45) */ struct phy_device *mdiobus_scan_c22(struct mii_bus *bus, int addr) { - return mdiobus_scan(bus, addr, false); + struct phy_device *phydev; + int err; + + err =3D mdiobus_device_change_begin(bus, false); + if (err) + return ERR_PTR(err); + + phydev =3D __mdiobus_scan(bus, addr, false); + mdiobus_device_change_end(bus, false); + + return phydev; } EXPORT_SYMBOL(mdiobus_scan_c22); =20 @@ -476,7 +507,7 @@ EXPORT_SYMBOL(mdiobus_scan_c22); */ static struct phy_device *mdiobus_scan_c45(struct mii_bus *bus, int addr) { - return mdiobus_scan(bus, addr, true); + return __mdiobus_scan(bus, addr, true); } =20 static int mdiobus_scan_bus_c22(struct mii_bus *bus) @@ -487,7 +518,7 @@ static int mdiobus_scan_bus_c22(struct mii_bus *bus) if ((bus->phy_mask & BIT(i)) =3D=3D 0) { struct phy_device *phydev; =20 - phydev =3D mdiobus_scan_c22(bus, i); + phydev =3D __mdiobus_scan(bus, i, false); if (IS_ERR(phydev) && (PTR_ERR(phydev) !=3D -ENODEV)) return PTR_ERR(phydev); } @@ -504,7 +535,7 @@ static int mdiobus_scan_bus_c45(struct mii_bus *bus) struct phy_device *phydev; =20 /* Don't scan C45 if we already have a C22 device */ - if (bus->mdio_map[i]) + if (mdiobus_is_registered_device(bus, i)) continue; =20 phydev =3D mdiobus_scan_c45(bus, i); @@ -529,13 +560,50 @@ static bool mdiobus_prevent_c45_scan(struct mii_bus *= bus) mdiobus_for_each_phy(bus, phydev) { u32 oui =3D phydev->phy_id >> 10; =20 - if (oui =3D=3D MICREL_OUI) + if (oui =3D=3D MICREL_OUI) { + phy_device_put(phydev); return true; + } } =20 return false; } =20 +static void mdiobus_stop_device_changes(struct mii_bus *bus) +{ + mutex_lock(&bus->mdio_map_lock); + bus->state =3D MDIOBUS_UNREGISTERING; + wake_up_all(&bus->mdio_map_wait); + mutex_unlock(&bus->mdio_map_lock); + + wait_event(bus->mdio_map_wait, !READ_ONCE(bus->mdio_map_ops)); +} + +static void mdiobus_remove_devices(struct mii_bus *bus) +{ + struct mdio_device *mdiodev; + int i; + + for (i =3D 0; i < PHY_MAX_ADDR; i++) { + mutex_lock(&bus->mdio_map_lock); + mdiodev =3D rcu_dereference_protected(bus->mdio_map[i], + lockdep_is_held(&bus->mdio_map_lock)); + if (mdiodev) + mdio_device_get(mdiodev); + mutex_unlock(&bus->mdio_map_lock); + if (!mdiodev) + continue; + + if (!mdiodev->device_remove(mdiodev, false)) + mdiodev->device_free(mdiodev); + mdio_device_put(mdiodev); + } + + mutex_lock(&bus->mdio_map_lock); + bus->state =3D MDIOBUS_UNREGISTERED; + mutex_unlock(&bus->mdio_map_lock); +} + /** * __mdiobus_register - bring up all the PHYs on a given bus and attach th= em to bus * @bus: target mii_bus @@ -552,10 +620,9 @@ static bool mdiobus_prevent_c45_scan(struct mii_bus *b= us) */ int __mdiobus_register(struct mii_bus *bus, struct module *owner) { - struct mdio_device *mdiodev; struct gpio_desc *gpiod; bool prevent_c45_scan; - int i, err; + int err; =20 if (!bus || !bus->name) return -EINVAL; @@ -596,7 +663,9 @@ int __mdiobus_register(struct mii_bus *bus, struct modu= le *owner) * * State will be updated later in this function in case of success */ + mutex_lock(&bus->mdio_map_lock); bus->state =3D MDIOBUS_UNREGISTERED; + mutex_unlock(&bus->mdio_map_lock); =20 err =3D device_register(&bus->dev); if (err) { @@ -613,8 +682,7 @@ int __mdiobus_register(struct mii_bus *bus, struct modu= le *owner) err =3D dev_err_probe(&bus->dev, PTR_ERR(gpiod), "mii_bus %s couldn't get reset GPIO\n", bus->id); - device_del(&bus->dev); - return err; + goto error_reset_gpiod; } else if (gpiod) { bus->reset_gpiod =3D gpiod; fsleep(bus->reset_delay_us); @@ -629,6 +697,10 @@ int __mdiobus_register(struct mii_bus *bus, struct mod= ule *owner) goto error_reset_gpiod; } =20 + mutex_lock(&bus->mdio_map_lock); + bus->state =3D MDIOBUS_REGISTERING; + mutex_unlock(&bus->mdio_map_lock); + if (bus->read) { err =3D mdiobus_scan_bus_c22(bus); if (err) @@ -643,20 +715,17 @@ int __mdiobus_register(struct mii_bus *bus, struct mo= dule *owner) goto error; } =20 + mutex_lock(&bus->mdio_map_lock); bus->state =3D MDIOBUS_REGISTERED; + mutex_unlock(&bus->mdio_map_lock); dev_dbg(&bus->dev, "probed\n"); return 0; =20 error: - for (i =3D 0; i < PHY_MAX_ADDR; i++) { - mdiodev =3D bus->mdio_map[i]; - if (!mdiodev) - continue; - - mdiodev->device_remove(mdiodev); - mdiodev->device_free(mdiodev); - } error_reset_gpiod: + mdiobus_stop_device_changes(bus); + mdiobus_remove_devices(bus); + /* Put PHYs in RESET to save power */ if (bus->reset_gpiod) gpiod_set_value_cansleep(bus->reset_gpiod, 1); @@ -668,21 +737,11 @@ EXPORT_SYMBOL(__mdiobus_register); =20 void mdiobus_unregister(struct mii_bus *bus) { - struct mdio_device *mdiodev; - int i; - if (WARN_ON_ONCE(bus->state !=3D MDIOBUS_REGISTERED)) return; - bus->state =3D MDIOBUS_UNREGISTERED; =20 - for (i =3D 0; i < PHY_MAX_ADDR; i++) { - mdiodev =3D bus->mdio_map[i]; - if (!mdiodev) - continue; - - mdiodev->device_remove(mdiodev); - mdiodev->device_free(mdiodev); - } + mdiobus_stop_device_changes(bus); + mdiobus_remove_devices(bus); =20 /* Put PHYs in RESET to save power */ if (bus->reset_gpiod) @@ -702,8 +761,11 @@ EXPORT_SYMBOL(mdiobus_unregister); */ void mdiobus_free(struct mii_bus *bus) { + mutex_lock(&bus->mdio_map_lock); + /* For compatibility with error handling in drivers. */ if (bus->state =3D=3D MDIOBUS_ALLOCATED) { + mutex_unlock(&bus->mdio_map_lock); kfree(bus); return; } @@ -711,6 +773,7 @@ void mdiobus_free(struct mii_bus *bus) WARN(bus->state !=3D MDIOBUS_UNREGISTERED, "%s: not in UNREGISTERED state\n", bus->id); bus->state =3D MDIOBUS_RELEASED; + mutex_unlock(&bus->mdio_map_lock); =20 put_device(&bus->dev); } diff --git a/drivers/net/phy/mdio_device.c b/drivers/net/phy/mdio_device.c index 06151f207134..8166e779f177 100644 --- a/drivers/net/phy/mdio_device.c +++ b/drivers/net/phy/mdio_device.c @@ -15,8 +15,11 @@ #include #include #include +#include #include +#include #include +#include #include #include #include @@ -33,12 +36,16 @@ static int mdio_device_register_reset(struct mdio_device *mdiodev) { struct reset_control *reset; + int err; =20 /* Deassert the optional reset signal */ mdiodev->reset_gpio =3D gpiod_get_optional(&mdiodev->dev, "reset", GPIOD_OUT_LOW); - if (IS_ERR(mdiodev->reset_gpio)) - return PTR_ERR(mdiodev->reset_gpio); + if (IS_ERR(mdiodev->reset_gpio)) { + err =3D PTR_ERR(mdiodev->reset_gpio); + mdiodev->reset_gpio =3D NULL; + return err; + } =20 if (mdiodev->reset_gpio) gpiod_set_consumer_name(mdiodev->reset_gpio, "PHY reset"); @@ -116,6 +123,8 @@ static void mdio_device_release(struct device *dev) kfree(to_mdio_device(dev)); } =20 +static int __mdio_device_remove(struct mdio_device *mdiodev, bool dynamic); + struct mdio_device *mdio_device_create(struct mii_bus *bus, int addr) { struct mdio_device *mdiodev; @@ -129,7 +138,7 @@ struct mdio_device *mdio_device_create(struct mii_bus *= bus, int addr) mdiodev->dev.parent =3D &bus->dev; mdiodev->dev.bus =3D &mdio_bus_type; mdiodev->device_free =3D mdio_device_free; - mdiodev->device_remove =3D mdio_device_remove; + mdiodev->device_remove =3D __mdio_device_remove; mdiodev->bus =3D bus; mdiodev->addr =3D addr; mdiodev->reset_state =3D -1; @@ -159,19 +168,27 @@ int mdio_device_register(struct mdio_device *mdiodev) return err; =20 err =3D device_add(&mdiodev->dev); - if (err) { + if (err) pr_err("MDIO %d failed to add\n", mdiodev->addr); - goto out; - } - - return 0; =20 - out: - mdiobus_unregister_device(mdiodev); - return err; + return mdiobus_registration_done(mdiodev, err); } EXPORT_SYMBOL(mdio_device_register); =20 +static int __mdio_device_remove(struct mdio_device *mdiodev, bool dynamic) +{ + int err; + + err =3D mdiobus_begin_remove(mdiodev, dynamic); + if (err) + return err; + + device_del(&mdiodev->dev); + mdiobus_finish_remove(mdiodev, dynamic); + + return 0; +} + /** * mdio_device_remove - Remove a previously registered mdio device from the * MDIO bus @@ -183,42 +200,215 @@ EXPORT_SYMBOL(mdio_device_register); */ void mdio_device_remove(struct mdio_device *mdiodev) { - device_del(&mdiodev->dev); - mdiobus_unregister_device(mdiodev); + __mdio_device_remove(mdiodev, false); } EXPORT_SYMBOL(mdio_device_remove); =20 int mdiobus_register_device(struct mdio_device *mdiodev) { + struct mii_bus *bus =3D mdiodev->bus; int err; =20 - if (mdiodev->bus->mdio_map[mdiodev->addr]) - return -EBUSY; + mutex_lock(&bus->mdio_map_lock); + if (bus->state !=3D MDIOBUS_REGISTERING && + bus->state !=3D MDIOBUS_REGISTERED) { + err =3D -ENODEV; + goto out_unlock; + } + if (bus->mdio_map_removing) { + err =3D -EBUSY; + goto out_unlock; + } + + if (rcu_access_pointer(bus->mdio_map[mdiodev->addr]) || + bus->mdio_map_pending & BIT(mdiodev->addr)) { + err =3D -EBUSY; + goto out_unlock; + } + + bus->mdio_map_pending |=3D BIT(mdiodev->addr); + bus->mdio_map_pending_owner[mdiodev->addr] =3D current; + bus->mdio_map_ops++; + mutex_unlock(&bus->mdio_map_lock); =20 if (mdiodev->flags & MDIO_DEVICE_FLAG_PHY) { err =3D mdio_device_register_reset(mdiodev); - if (err) + if (err) { + mdiobus_registration_done(mdiodev, err); return err; + } =20 /* Assert the reset signal */ mdio_device_reset(mdiodev, 1); } =20 - mdiodev->bus->mdio_map[mdiodev->addr] =3D mdiodev; - return 0; + +out_unlock: + mutex_unlock(&bus->mdio_map_lock); + return err; } =20 -int mdiobus_unregister_device(struct mdio_device *mdiodev) +/** + * mdiobus_device_change_begin - start changing devices on a registered bus + * @bus: MDIO bus that will be scanned or changed + * @removing: whether to start an exclusive removal transaction + * + * Return: zero on success or a negative error code when the bus is unavai= lable + */ +int mdiobus_device_change_begin(struct mii_bus *bus, bool removing) { - if (mdiodev->bus->mdio_map[mdiodev->addr] !=3D mdiodev) - return -EINVAL; + int err =3D 0; =20 - mdio_device_unregister_reset(mdiodev); + mutex_lock(&bus->mdio_map_lock); + if (bus->state !=3D MDIOBUS_REGISTERED) { + err =3D -ENODEV; + } else if (bus->mdio_map_removing || + (removing && bus->mdio_map_ops)) { + err =3D -EBUSY; + } else { + bus->mdio_map_ops++; + if (removing) + bus->mdio_map_removing =3D true; + } + mutex_unlock(&bus->mdio_map_lock); =20 - mdiodev->bus->mdio_map[mdiodev->addr] =3D NULL; + return err; +} +EXPORT_SYMBOL_GPL(mdiobus_device_change_begin); =20 - return 0; +static void mdiobus_operation_done_locked(struct mii_bus *bus) +{ + lockdep_assert_held(&bus->mdio_map_lock); + + if (WARN_ON_ONCE(!bus->mdio_map_ops)) + return; + bus->mdio_map_ops--; + if (!bus->mdio_map_ops) + wake_up_all(&bus->mdio_map_wait); +} + +/** + * mdiobus_device_change_end - finish changing devices on an MDIO bus + * @bus: MDIO bus previously passed to mdiobus_device_change_begin() + * @removing: value passed to mdiobus_device_change_begin() + */ +void mdiobus_device_change_end(struct mii_bus *bus, bool removing) +{ + mutex_lock(&bus->mdio_map_lock); + if (removing) { + WARN_ON_ONCE(!bus->mdio_map_removing); + bus->mdio_map_removing =3D false; + } + mdiobus_operation_done_locked(bus); + mutex_unlock(&bus->mdio_map_lock); +} +EXPORT_SYMBOL_GPL(mdiobus_device_change_end); + +static void mdiobus_operation_done(struct mii_bus *bus) +{ + mutex_lock(&bus->mdio_map_lock); + mdiobus_operation_done_locked(bus); + mutex_unlock(&bus->mdio_map_lock); +} + +static void mdiobus_unpublish_device(struct mdio_device *mdiodev) +{ + struct mii_bus *bus =3D mdiodev->bus; + + lockdep_assert_held(&bus->mdio_map_lock); + + if (rcu_access_pointer(bus->mdio_map[mdiodev->addr]) =3D=3D mdiodev) + rcu_assign_pointer(bus->mdio_map[mdiodev->addr], NULL); + if (mdiodev->dev.of_node) + of_node_clear_flag(mdiodev->dev.of_node, OF_POPULATED); +} + +int mdiobus_registration_done(struct mdio_device *mdiodev, int err) +{ + struct mii_bus *bus =3D mdiodev->bus; + + mutex_lock(&bus->mdio_map_lock); + if (WARN_ON_ONCE(!(bus->mdio_map_pending & BIT(mdiodev->addr)))) + goto out_unlock; + + if (err) { + mdiobus_unpublish_device(mdiodev); + } else { + WARN_ON_ONCE(rcu_access_pointer(bus->mdio_map[mdiodev->addr])); + /* Teardown waits for this operation before consuming the map. */ + rcu_assign_pointer(bus->mdio_map[mdiodev->addr], mdiodev); + } + + bus->mdio_map_pending &=3D ~BIT(mdiodev->addr); + bus->mdio_map_pending_owner[mdiodev->addr] =3D NULL; + wake_up_all(&bus->mdio_map_wait); + +out_unlock: + mutex_unlock(&bus->mdio_map_lock); + if (err) { + if (mdiodev->flags & MDIO_DEVICE_FLAG_PHY) { + mdio_device_reset(mdiodev, 1); + mdio_device_unregister_reset(mdiodev); + } + } + mdiobus_operation_done(bus); + + return err; +} + +int mdiobus_begin_remove(struct mdio_device *mdiodev, bool dynamic) +{ + struct mii_bus *bus =3D mdiodev->bus; + struct phy_device *phydev; + int err =3D 0; + + mutex_lock(&bus->mdio_map_lock); + if (dynamic && bus->state !=3D MDIOBUS_REGISTERED) { + err =3D -ENODEV; + goto out_unlock; + } + if (bus->mdio_map_pending & BIT(mdiodev->addr)) { + err =3D -EBUSY; + goto out_unlock; + } + + if (rcu_access_pointer(bus->mdio_map[mdiodev->addr]) !=3D mdiodev) { + err =3D -ENODEV; + goto out_unlock; + } + + if (dynamic && mdiodev->flags & MDIO_DEVICE_FLAG_PHY) { + phydev =3D to_phy_device(&mdiodev->dev); + if (phydev->attached) { + err =3D -EBUSY; + goto out_unlock; + } + } + + mdiobus_unpublish_device(mdiodev); + +out_unlock: + mutex_unlock(&bus->mdio_map_lock); + return err; +} + +void mdiobus_finish_remove(struct mdio_device *mdiodev, bool dynamic) +{ + struct fwnode_handle *fwnode; + + /* Let map readers acquire their device reference before it is dropped. */ + synchronize_rcu(); + + if (mdiodev->flags & MDIO_DEVICE_FLAG_PHY) + mdio_device_unregister_reset(mdiodev); + + /* Do not keep an overlay node alive with the removed device. */ + if (dynamic) { + fwnode =3D dev_fwnode(&mdiodev->dev); + device_set_node(&mdiodev->dev, NULL); + fwnode_handle_put(fwnode); + } } =20 /** diff --git a/drivers/net/phy/mscc/mscc_ptp.c b/drivers/net/phy/mscc/mscc_pt= p.c index 4865eac74b0e..7ca9d14dc02c 100644 --- a/drivers/net/phy/mscc/mscc_ptp.c +++ b/drivers/net/phy/mscc/mscc_ptp.c @@ -1274,43 +1274,27 @@ static const struct ptp_clock_info vsc85xx_clk_caps= =3D { .do_aux_work =3D &vsc85xx_do_aux_work, }; =20 -static struct vsc8531_private *vsc8584_base_priv(struct phy_device *phydev) -{ - struct vsc8531_private *vsc8531 =3D phydev->priv; - - if (vsc8531->ts_base_addr !=3D phydev->mdio.addr) { - struct mdio_device *dev; - - dev =3D phydev->mdio.bus->mdio_map[vsc8531->ts_base_addr]; - phydev =3D container_of(dev, struct phy_device, mdio); - - return phydev->priv; - } - - return vsc8531; -} - -static bool vsc8584_is_1588_input_clk_configured(struct phy_device *phydev) -{ - struct vsc8531_private *vsc8531 =3D vsc8584_base_priv(phydev); - - return vsc8531->input_clk_init; -} - -static void vsc8584_set_input_clk_configured(struct phy_device *phydev) -{ - struct vsc8531_private *vsc8531 =3D vsc8584_base_priv(phydev); - - vsc8531->input_clk_init =3D true; -} - static int __vsc8584_init_ptp(struct phy_device *phydev) { static const u32 ltc_seq_e[] =3D { 0, 400000, 0, 0, 0 }; static const u8 ltc_seq_a[] =3D { 8, 6, 5, 4, 2 }; + struct vsc8531_private *base_vsc8531 =3D phydev->priv; + struct phy_device *base_phydev =3D NULL; u32 val; =20 - if (!vsc8584_is_1588_input_clk_configured(phydev)) { + if (base_vsc8531->ts_base_addr !=3D phydev->mdio.addr) { + base_phydev =3D mdiobus_get_phy(phydev->mdio.bus, + base_vsc8531->ts_base_addr); + if (!base_phydev) + return -ENODEV; + if (!base_phydev->priv) { + phy_device_put(base_phydev); + return -ENODEV; + } + base_vsc8531 =3D base_phydev->priv; + } + + if (!base_vsc8531->input_clk_init) { phy_lock_mdio_bus(phydev); =20 /* 1588_DIFF_INPUT_CLK configuration: Use an external clock for @@ -1325,8 +1309,10 @@ static int __vsc8584_init_ptp(struct phy_device *phy= dev) =20 phy_unlock_mdio_bus(phydev); =20 - vsc8584_set_input_clk_configured(phydev); + base_vsc8531->input_clk_init =3D true; } + if (base_phydev) + phy_device_put(base_phydev); =20 /* Disable predictor before configuring the 1588 block */ val =3D vsc85xx_ts_read_csr(phydev, PROCESSOR, diff --git a/drivers/net/phy/phy_device.c b/drivers/net/phy/phy_device.c index 34d00f1ee483..e767aa174f2e 100644 --- a/drivers/net/phy/phy_device.c +++ b/drivers/net/phy/phy_device.c @@ -35,6 +35,7 @@ #include #include #include +#include #include #include #include @@ -209,10 +210,21 @@ static void __init features_init(void) =20 void phy_device_free(struct phy_device *phydev) { - put_device(&phydev->mdio.dev); + phy_device_put(phydev); } EXPORT_SYMBOL(phy_device_free); =20 +/** + * phy_device_put - drop a reference to a PHY device + * @phydev: PHY device, or %NULL + */ +void phy_device_put(struct phy_device *phydev) +{ + if (phydev) + put_device(&phydev->mdio.dev); +} +EXPORT_SYMBOL(phy_device_put); + static void phy_mdio_device_free(struct mdio_device *mdiodev) { struct phy_device *phydev; @@ -227,12 +239,14 @@ static void phy_device_release(struct device *dev) kfree(to_phy_device(dev)); } =20 -static void phy_mdio_device_remove(struct mdio_device *mdiodev) +static int __phy_device_remove(struct phy_device *phydev, bool dynamic); + +static int phy_mdio_device_remove(struct mdio_device *mdiodev, bool dynami= c) { struct phy_device *phydev; =20 phydev =3D container_of(mdiodev, struct phy_device, mdio); - phy_device_remove(phydev); + return __phy_device_remove(phydev, dynamic); } =20 static struct phy_driver genphy_driver; @@ -1121,25 +1135,40 @@ int phy_device_register(struct phy_device *phydev) err =3D phy_scan_fixups(phydev); if (err) { phydev_err(phydev, "failed to initialize\n"); - goto out; + return mdiobus_registration_done(&phydev->mdio, err); } =20 err =3D device_add(&phydev->mdio.dev); - if (err) { + if (err) phydev_err(phydev, "failed to add\n"); - goto out; - } =20 - return 0; + return mdiobus_registration_done(&phydev->mdio, err); +} +EXPORT_SYMBOL(phy_device_register); + +static int __phy_device_remove(struct phy_device *phydev, bool dynamic) +{ + int err; + + err =3D mdiobus_begin_remove(&phydev->mdio, dynamic); + if (dynamic && err =3D=3D -EBUSY) + dev_warn(&phydev->mdio.dev, + "cannot remove a PHY while it is attached or being registered\n"); + if (err) + return err; + + unregister_mii_timestamper(phydev->mii_ts); + pse_control_put(phydev->psec); + + device_del(&phydev->mdio.dev); =20 - out: /* Assert the reset signal */ phy_device_reset(phydev, 1); =20 - mdiobus_unregister_device(&phydev->mdio); - return err; + mdiobus_finish_remove(&phydev->mdio, dynamic); + + return 0; } -EXPORT_SYMBOL(phy_device_register); =20 /** * phy_device_remove - Remove a previously registered phy device from the = MDIO bus @@ -1151,15 +1180,7 @@ EXPORT_SYMBOL(phy_device_register); */ void phy_device_remove(struct phy_device *phydev) { - unregister_mii_timestamper(phydev->mii_ts); - pse_control_put(phydev->psec); - - device_del(&phydev->mdio.dev); - - /* Assert the reset signal */ - phy_device_reset(phydev, 1); - - mdiobus_unregister_device(&phydev->mdio); + __phy_device_remove(phydev, false); } EXPORT_SYMBOL(phy_device_remove); =20 @@ -1182,7 +1203,8 @@ EXPORT_SYMBOL(phy_get_c45_ids); * @bus: the target MII bus * @pos: cursor * - * Return: next phy_device on the bus, or NULL + * Return: next referenced phy_device on the bus, or %NULL. The caller must + * call phy_device_put() on the returned PHY. */ struct phy_device *phy_find_next(struct mii_bus *bus, struct phy_device *p= os) { @@ -1734,6 +1756,53 @@ static bool phy_drv_supports_irq(const struct phy_dr= iver *phydrv) return phydrv->config_intr && phydrv->handle_interrupt; } =20 +static int phy_claim(struct phy_device *phydev) +{ + struct mdio_device *mdiodev =3D &phydev->mdio; + struct mii_bus *bus =3D mdiodev->bus; + unsigned int addr =3D mdiodev->addr; + int err =3D 0; + +retry: + mutex_lock(&bus->mdio_map_lock); + if (bus->state !=3D MDIOBUS_REGISTERING && + bus->state !=3D MDIOBUS_REGISTERED) { + err =3D -ENODEV; + } else if (bus->mdio_map_removing) { + err =3D -ENODEV; + } else if (bus->mdio_map_pending & BIT(addr)) { + if (bus->mdio_map_pending_owner[addr] =3D=3D current) { + err =3D -EPROBE_DEFER; + } else { + mutex_unlock(&bus->mdio_map_lock); + wait_event(bus->mdio_map_wait, + !(READ_ONCE(bus->mdio_map_pending) & BIT(addr)) || + READ_ONCE(bus->state) =3D=3D MDIOBUS_UNREGISTERING || + READ_ONCE(bus->state) =3D=3D MDIOBUS_UNREGISTERED || + READ_ONCE(bus->state) =3D=3D MDIOBUS_RELEASED); + goto retry; + } + } else if (rcu_access_pointer(bus->mdio_map[addr]) !=3D mdiodev) { + err =3D -ENODEV; + } else if (phydev->attached) { + err =3D -EBUSY; + } else { + phydev->attached =3D true; + } + mutex_unlock(&bus->mdio_map_lock); + + return err; +} + +static void phy_release(struct phy_device *phydev) +{ + struct mii_bus *bus =3D phydev->mdio.bus; + + mutex_lock(&bus->mdio_map_lock); + phydev->attached =3D false; + mutex_unlock(&bus->mdio_map_lock); +} + /** * phy_attach_direct - attach a network device to a given PHY device point= er * @dev: network device to attach @@ -1755,6 +1824,7 @@ int phy_attach_direct(struct net_device *dev, struct = phy_device *phydev, struct module *bus_owner =3D phydev->mdio.bus->owner; struct device *d =3D &phydev->mdio.dev; struct module *ndev_owner =3D NULL; + bool claimed =3D false; int err; =20 /* For Ethernet device drivers that register their own MDIO bus, we @@ -1770,6 +1840,13 @@ int phy_attach_direct(struct net_device *dev, struct= phy_device *phydev, } =20 get_device(d); + err =3D phy_claim(phydev); + if (err =3D=3D -EBUSY) + phydev_err(phydev, "PHY already attached\n"); + if (!err) + claimed =3D true; + if (err) + goto error_put_device; =20 /* Assume that if there is no driver, that it doesn't * exist, and we should use the genphy driver. @@ -1798,12 +1875,6 @@ int phy_attach_direct(struct net_device *dev, struct= phy_device *phydev, goto error_module_put; } =20 - if (phydev->attached_dev) { - dev_err(&dev->dev, "PHY already attached\n"); - err =3D -EBUSY; - goto error; - } - phydev->phy_link_change =3D phy_link_change; if (dev) { phydev->attached_dev =3D dev; @@ -1899,6 +1970,8 @@ int phy_attach_direct(struct net_device *dev, struct = phy_device *phydev, phydev->is_genphy_driven =3D 0; d->driver =3D NULL; error_put_device: + if (claimed) + phy_release(phydev); put_device(d); if (ndev_owner !=3D bus_owner) module_put(bus_owner); @@ -1974,6 +2047,7 @@ void phy_detach(struct phy_device *phydev) =20 /* The PHY and its parent bus may be released by put_device() below. */ bus_owner =3D phydev->mdio.bus->owner; + phy_release(phydev); =20 put_device(&phydev->mdio.dev); if (dev) diff --git a/drivers/net/phy/phylib-internal.h b/drivers/net/phy/phylib-int= ernal.h index 664ed7faa518..7e9161fcdd90 100644 --- a/drivers/net/phy/phylib-internal.h +++ b/drivers/net/phy/phylib-internal.h @@ -25,7 +25,9 @@ int phy_speed_down_core(struct phy_device *phydev); void phy_check_downshift(struct phy_device *phydev); =20 int mdiobus_register_device(struct mdio_device *mdiodev); -int mdiobus_unregister_device(struct mdio_device *mdiodev); +int mdiobus_registration_done(struct mdio_device *mdiodev, int err); +int mdiobus_begin_remove(struct mdio_device *mdiodev, bool dynamic); +void mdiobus_finish_remove(struct mdio_device *mdiodev, bool dynamic); =20 int genphy_c45_read_eee_adv(struct phy_device *phydev, unsigned long *adv); =20 diff --git a/drivers/net/usb/asix_devices.c b/drivers/net/usb/asix_devices.c index 293ef80c4e30..21249777062e 100644 --- a/drivers/net/usb/asix_devices.c +++ b/drivers/net/usb/asix_devices.c @@ -696,6 +696,10 @@ static int ax88772_init_mdio(struct usbnet *dev) =20 static void ax88772_mdio_unregister(struct asix_common_private *priv) { + phy_device_put(priv->phydev_int); + priv->phydev_int =3D NULL; + phy_device_put(priv->phydev); + priv->phydev =3D NULL; mdiobus_unregister(priv->mdio); mdiobus_free(priv->mdio); } diff --git a/drivers/net/usb/lan78xx.c b/drivers/net/usb/lan78xx.c index cb782d81d84f..69fb2ec9252f 100644 --- a/drivers/net/usb/lan78xx.c +++ b/drivers/net/usb/lan78xx.c @@ -2683,6 +2683,7 @@ static struct phy_device *lan78xx_get_phy(struct lan7= 8xx_net *dev) =20 default: netdev_err(dev->net, "Unknown CHIP ID: 0x%08x\n", dev->chipid); + phy_device_put(phydev); return ERR_PTR(-ENODEV); } } @@ -2869,8 +2870,10 @@ static int lan78xx_phy_init(struct lan78xx_net *dev) return PTR_ERR(phydev); =20 ret =3D lan78xx_phylink_setup(dev); - if (ret < 0) + if (ret < 0) { + phy_device_put(phydev); return ret; + } =20 ret =3D lan78xx_mac_prepare_for_phy(dev); if (ret < 0) @@ -2909,10 +2912,12 @@ static int lan78xx_phy_init(struct lan78xx_net *dev) ret =3D lan78xx_configure_leds_from_dt(dev, phydev); if (ret < 0) goto phylink_uninit; + phy_device_put(phydev); =20 return 0; =20 phylink_uninit: + phy_device_put(phydev); lan78xx_phy_uninit(dev); =20 return ret; diff --git a/drivers/net/usb/smsc95xx.c b/drivers/net/usb/smsc95xx.c index 42e4048b574b..a41f0dd7676a 100644 --- a/drivers/net/usb/smsc95xx.c +++ b/drivers/net/usb/smsc95xx.c @@ -1289,12 +1289,14 @@ static int smsc95xx_bind(struct usbnet *dev, struct= usb_interface *intf) netdev_err(dev->net, "can't attach PHY to %s\n", pdata->mdiobus->id); goto unregister_mdio; } + phy_device_put(pdata->phydev); =20 phy_attached_info(dev->net->phydev); =20 return 0; =20 unregister_mdio: + phy_device_put(pdata->phydev); mdiobus_unregister(pdata->mdiobus); =20 free_mdio: diff --git a/drivers/power/reset/linkstation-poweroff.c b/drivers/power/res= et/linkstation-poweroff.c index e56d75bfcc43..bb93dd14f450 100644 --- a/drivers/power/reset/linkstation-poweroff.c +++ b/drivers/power/reset/linkstation-poweroff.c @@ -192,6 +192,7 @@ static void __exit linkstation_poweroff_exit(void) { pm_power_off =3D NULL; unregister_reboot_notifier(&linkstation_reboot_nb); + phy_device_put(phydev); } =20 module_init(linkstation_poweroff_init); diff --git a/include/linux/mdio.h b/include/linux/mdio.h index a7d9e3ae362a..7ce784af0c66 100644 --- a/include/linux/mdio.h +++ b/include/linux/mdio.h @@ -31,7 +31,7 @@ struct mdio_device { =20 int (*bus_match)(struct device *dev, const struct device_driver *drv); void (*device_free)(struct mdio_device *mdiodev); - void (*device_remove)(struct mdio_device *mdiodev); + int (*device_remove)(struct mdio_device *mdiodev, bool dynamic); =20 /* Bus address of the MDIO device (0-31) */ int addr; @@ -694,6 +694,8 @@ static inline int mdiodev_c45_write(struct mdio_device = *mdiodev, u32 devad, =20 bool mdiobus_is_registered_device(struct mii_bus *bus, int addr); struct phy_device *mdiobus_get_phy(struct mii_bus *bus, int addr); +int mdiobus_device_change_begin(struct mii_bus *bus, bool removing); +void mdiobus_device_change_end(struct mii_bus *bus, bool removing); =20 /** * mdio_module_driver() - Helper macro for registering mdio drivers diff --git a/include/linux/phy.h b/include/linux/phy.h index 5f8d65868e0f..6a64314dd67a 100644 --- a/include/linux/phy.h +++ b/include/linux/phy.h @@ -22,6 +22,7 @@ #include #include #include +#include #include #include #include @@ -331,6 +332,7 @@ struct phy_port; struct sfp_bus; struct sfp_upstream_ops; struct sk_buff; +struct task_struct; =20 /** * struct mdio_bus_stats - Statistics counters for MDIO busses @@ -391,7 +393,9 @@ struct mii_bus { /** @state: State of bus structure */ enum { MDIOBUS_ALLOCATED =3D 1, + MDIOBUS_REGISTERING, MDIOBUS_REGISTERED, + MDIOBUS_UNREGISTERING, MDIOBUS_UNREGISTERED, MDIOBUS_RELEASED, } state; @@ -400,7 +404,19 @@ struct mii_bus { struct device dev; =20 /** @mdio_map: list of all MDIO devices on bus */ - struct mdio_device *mdio_map[PHY_MAX_ADDR]; + struct mdio_device __rcu *mdio_map[PHY_MAX_ADDR]; + /** @mdio_map_pending: addresses with registration in progress */ + u32 mdio_map_pending; + /** @mdio_map_pending_owner: task registering each pending address */ + struct task_struct *mdio_map_pending_owner[PHY_MAX_ADDR]; + /** @mdio_map_lock: protects the MDIO device map and bus state */ + struct mutex mdio_map_lock; + /** @mdio_map_wait: wait for active map operations during teardown */ + wait_queue_head_t mdio_map_wait; + /** @mdio_map_ops: active registrations, scans and firmware changes */ + unsigned int mdio_map_ops; + /** @mdio_map_removing: firmware removal blocking map changes and attachm= ent */ + bool mdio_map_removing; =20 /** @phy_mask: PHY addresses to be ignored when probing */ u32 phy_mask; @@ -652,6 +668,8 @@ struct phy_oatc14_sqi_capability { * @n_ports: Number of ports currently attached to the PHY * @max_n_ports: Max number of ports this PHY can expose * @lock: Mutex for serialization access to PHY + * @attached: Whether a network device or standalone user attached the PHY; + * protected by the MDIO bus map lock * @state_queue: Work queue for state machine * @link_down_events: Number of times link was lost * @shared: Pointer to private data shared by phys in one package @@ -781,6 +799,7 @@ struct phy_device { struct delayed_work state_queue; =20 struct mutex lock; + bool attached; =20 /* This may be modified under the rtnl lock */ bool sfp_bus_attached; @@ -2159,6 +2178,7 @@ struct fwnode_handle *fwnode_get_phy_node(const struc= t fwnode_handle *fwnode); struct phy_device *get_phy_device(struct mii_bus *bus, int addr, bool is_c= 45); int phy_device_register(struct phy_device *phy); void phy_device_free(struct phy_device *phydev); +void phy_device_put(struct phy_device *phydev); void phy_device_remove(struct phy_device *phydev); int phy_get_c45_ids(struct phy_device *phydev); int phy_init_hw(struct phy_device *phydev); @@ -2198,9 +2218,19 @@ static inline struct phy_device *phy_find_first(stru= ct mii_bus *bus) return phy_find_next(bus, NULL); } =20 +static inline struct phy_device * +phy_find_next_and_put(struct mii_bus *bus, struct phy_device *phydev) +{ + struct phy_device *next =3D phy_find_next(bus, phydev); + + phy_device_put(phydev); + + return next; +} + #define mdiobus_for_each_phy(_bus, _phydev) \ for (_phydev =3D phy_find_first(_bus); _phydev; \ - _phydev =3D phy_find_next(_bus, _phydev)) + _phydev =3D phy_find_next_and_put(_bus, _phydev)) =20 #if IS_ENABLED(CONFIG_PHYLIB) int phy_start_cable_test(struct phy_device *phydev, diff --git a/net/dsa/user.c b/net/dsa/user.c index 03c7af6abe18..1ad5e5a398f5 100644 --- a/net/dsa/user.c +++ b/net/dsa/user.c @@ -2640,16 +2640,20 @@ static int dsa_user_phy_connect(struct net_device *= user_dev, int addr, { struct dsa_port *dp =3D dsa_user_to_port(user_dev); struct dsa_switch *ds =3D dp->ds; + struct phy_device *phydev; + int ret; =20 - user_dev->phydev =3D mdiobus_get_phy(ds->user_mii_bus, addr); - if (!user_dev->phydev) { + phydev =3D mdiobus_get_phy(ds->user_mii_bus, addr); + if (!phydev) { netdev_err(user_dev, "no phy at %d\n", addr); return -ENODEV; } =20 - user_dev->phydev->dev_flags |=3D flags; + phydev->dev_flags |=3D flags; + ret =3D phylink_connect_phy(dp->pl, phydev); + phy_device_put(phydev); =20 - return phylink_connect_phy(dp->pl, user_dev->phydev); + return ret; } =20 static int dsa_user_phy_setup(struct net_device *user_dev) --=20 2.53.0 From nobody Tue Aug 25 00:19:15 2026 Received: from mail-oo1-f52.google.com (mail-oo1-f52.google.com [209.85.161.52]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 9C9FF3ABD82 for ; Fri, 7 Aug 2026 03:54:51 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.161.52 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1786074896; cv=none; b=ABN/6ZrqR8mJXSsSOudx/Dfp0cXgha5+X+CXc1EJrc5kH7y0yw1Bs1ZQDXWX8aY0Bh6nu2aoFeZssjNm5cWU3EL0WkSplgMm9SAa0O+6OBrielZTW6uyzhl3B1PUMexlpJd4LuEOTdtZiQJTcF2tJCuvwHWE4DBgY09+XL5aI9w= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1786074896; c=relaxed/simple; bh=YMYpQ6XFhM2cLP/3ZcZBUZKBv+hhpqEzcr8uR8DX0hI=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=S4Xi5kmgUKm/Yne8Frgnw39VZOpruGmFD9Iltds8Sn7HAS4hvsd5qDW8p8ZMbY/l1bAY4lLkoZrctovd7McN8j6AUvsfB9VMpH+4pA2BW2VhFiSdeye3DdjWJCyz/uMtIIbXYt3hScZZ5Hs6s4uHqPaj2trBgXyif3qtloG7iwc= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=JClCp8dA; arc=none smtp.client-ip=209.85.161.52 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="JClCp8dA" Received: by mail-oo1-f52.google.com with SMTP id 006d021491bc7-6acc88b9c5fso1487731eaf.2 for ; Thu, 06 Aug 2026 20:54:51 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1786074890; x=1786679690; darn=vger.kernel.org; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :content-type:mime-version:subject:date:from:from:to:cc:subject:date :message-id:reply-to:content-type; bh=mzaXaAGZuzZqDVtohWJRcwtoKOyfow4JWerY6rOg8/Q=; b=JClCp8dA3GMoLxfqXEgDTxOOLEB1fcGC5VsYOlbSSgq0pCT+E5nntU10yP94PddfOK gix8KsX77L+mIWvnt6LroW5W1hrI56HDEg7zxjnkOir+rnsd6MXoihdAR5h6MMVYQEG6 Gvjmok9j8k2hgd/j0ENfs3XaNuc3Uhn604rlPsfls+AQaRPkdAJwPEuSpYQ6Od1HJ7ff dXVCgYw9wXs4BMhUKaVaHvJIMK2nKgl9puZvxPbyI1ZTmpmdtoOJPA+RNjTxfviYPsh5 yEzPcyIJIhPNwel1xeeNjKCwO4a0oZor7C5B57WorawPibAS4UBoA4rHGxCTqluMCXZZ FlVA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1786074890; x=1786679690; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :content-type:mime-version:subject:date:from:x-gm-gg :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to :content-type; bh=mzaXaAGZuzZqDVtohWJRcwtoKOyfow4JWerY6rOg8/Q=; b=Tj9S08bveoS6G5OEjO72dtjqruiPtR3guolZL4FVLlkVVIuF1YJ5kh2WgK45m95Hg2 ZrymrnscR5y2uQJ4cLh6XLzsDlTwjX/CjWx8wPBK2CSMDB1hH99OWOq4L+TxCHrwoQIx VsDGwFHwyMW6R+ll+1l2BLeouTk0xLi3VJgDZFuydcw7vPBWNF8gaATy161wi8/mkPY/ /slob8c4xjd5WTp2vKrgJj5Ni9HRcEJ9QkHFdB7c5fUxeZYuVqM4KFTgwjVF8IcStGMz tgnzgh3wJE/COMDmJxvfvIVemLBsytfUOQTFvbY/lhfVyfvgu2Lbt7HTTJA4p4X344lW BmRg== X-Forwarded-Encrypted: i=1; AHgh+RriHKJgwZqzYqiVRvimNvRXtMbOTvYKAdLErjSAxmcolhZyQg7fpBM+ULrvGMOpl/pjNHdtBhJW9Lcjg/4=@vger.kernel.org X-Gm-Message-State: AOJu0YzTgpTHrYn5DfQu2q9z+IUI6Qk2O1LT2jqYgQ/UImw1hMuu+dmK g/B+1uegk9vt+1YfkeDq3EE1Sjio8SbY2/pDv3BcOnvNB/SvegPjc6Zz X-Gm-Gg: AR+sD13pYaoU3f1BsMSVNk6ZkPd6R137VLJ/hG/l1kydLnBG6W+6TeXSzc7PQXxynf6 9iCLx0UDrz0QDRCSyK57zpVf+XO0S1sM/BHGAk2QoHCY6Pvv93Mysol8SqDLQMT8V2u6gMYaqOW rtGpn6/SUespeWa29eMpLnnFZ0VSuUGvh1pBp8SQRDmp+z/cWctenlWMQGAGbN8EQwVDMpbrVv0 S7vHLwP228DZhxbs2XH/zBgjluBh+cvNloj29Uq3DWGlH/Ta8TfPQV1S3W/U/F8/8hkOs/7qRsC VNs8EDkoR8UIbYFoYZ7hpEKiqvzAxIcflr4KJ+cRURkH3IDalFuxps0AX2RQEVm0CHmpJVcCmaB jQViZIlq8Iqv+aMiZtMhhE2QkjdlCZudujxJnfybSY3S+iJDxL7oU1RMUNoet0fQMgW7WdS6A2s 7vp+ADBNH79Tf4rJEFKN+3dESlIWiAZVYregSAin0fc+kPviJgJc7QI+ZD0y5NnsAAa1uI/Jmcx 1Tp9WjQvxFDzSotEt5v6CuNowy56Bin2nJR/LEEi0tURwn5+wths2U9ZLA8rsevkR7pYZZI9pHs UPHi/OS7PxGwFP6zDkOo4hW9IchYzE04WEP5aynk8tjfAbSPdjJ1cuJn/pqHrXxr8on/3aaWT/H G6FNF7xRNz8hLKeDN X-Received: by 2002:a05:6820:81e:b0:6ae:871c:e3f2 with SMTP id 006d021491bc7-6ae970269a4mr9643139eaf.29.1786074889885; Thu, 06 Aug 2026 20:54:49 -0700 (PDT) Received: from [127.0.1.1] (184-96-154-59.hlrn.qwest.net. [184.96.154.59]) by smtp.gmail.com with ESMTPSA id 006d021491bc7-6b02bbc41bbsm1329339eaf.3.2026.08.06.20.54.47 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 06 Aug 2026 20:54:49 -0700 (PDT) From: James Hilliard Date: Thu, 06 Aug 2026 21:54:23 -0600 Subject: [PATCH net-next v4 4/4] net: mdio: support dynamic OF device changes Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Message-Id: <20260806-submit-mdio-of-dynamic-v2-v4-4-4beb0c50d5ba@gmail.com> References: <20260806-submit-mdio-of-dynamic-v2-v4-0-4beb0c50d5ba@gmail.com> In-Reply-To: <20260806-submit-mdio-of-dynamic-v2-v4-0-4beb0c50d5ba@gmail.com> To: Andrew Lunn , Heiner Kallweit , Russell King , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Florian Fainelli , Richard Cochran , Rob Herring , Saravana Kannan , Florian Fainelli , Jonas Gorski , Vladimir Oltean , Ciprian Regus , Andreas Larsson , Mark Einon , Iyappan Subramanian , Keyur Chudgar , Michael Chan , =?utf-8?q?Rafa=C5=82_Mi=C5=82ecki?= , Broadcom internal kernel review list , Doug Berger , Pavan Chebbi , =?utf-8?q?Th=C3=A9o_Lebrun?= , Conor Dooley , Wei Fang , Frank Li , Shenwei Wang , Jijie Shao , Jian Shen , Bryan Whitehead , UNGLinuxDriver@microchip.com, Vladimir Zapolskiy , Piotr Wojtaszczyk , Parthiban Veerasooran , Timur Tabi , nic_swsd@realtek.com, Byungho An , Steve Glendinning , Maxime Chevallier , Maxime Coquelin , Alexandre Torgue , FUJITA Tomonori , Siddharth Vadapalli , Roger Quadros , Jiawen Wu , Mengyuan Lou , Thangaraj Samynathan , Sebastian Reichel , Simon Horman Cc: netdev@vger.kernel.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, James Hilliard , imx@lists.linux.dev, linux-arm-kernel@lists.infradead.org, linux-stm32@st-md-mailman.stormreply.com, linux-omap@vger.kernel.org, linux-usb@vger.kernel.org, linux-pm@vger.kernel.org X-Mailer: b4 0.15.2 MDIO buses enumerate firmware children only when the bus is registered. A later DT overlay or status transition is ignored, unlike I2C and SPI. Add an OF reconfiguration notifier. On addition, create fixed-address PHYs and generic MDIO devices, scan free addresses for legacy PHY nodes without reg, and recurse into Ethernet PHY packages. Track populated nodes so static enumeration, address scanning and notifier-driven enumeration cannot create duplicates. Walk every package child and report the first population error without undoing devices which may already have acquired consumers. Only scan nodes which omit reg; reject malformed fixed addresses instead of treating them as legacy addressless PHYs. Serialize static and notifier-driven population while allowing a PHY probe to enable another node on the same bus. Check the node's current state after taking that lock so an older notification cannot override a newer status change. On removal, refuse to unregister attached or in-flight PHYs and find the object by its OF node. This also covers PHYs found by address scanning. Treat all PHYs in a package as one removal scope, so removing one member cannot invalidate state used by an attached peer. Preflight overlay removal while it can still be refused; normal changeset errors are propagated so the caller can restore the previous state. Ignore unrelated nodes and MDIO buses which have already disappeared when the global overlay preflight walks a changeset. Keep overlay preflight outside the global MDIO reconfiguration lock. The OF core invokes PRE_REMOVE while holding its own mutex, and an MDIO device probe running under the reconfiguration lock may apply an OF changeset. Use a per-bus removal transaction to block new registration and attachment while checking the map. Starting that exclusive transaction fails when another map operation is active, so refuse the overlay removal rather than waiting under the OF mutex; the caller can retry after that operation finishes. Hold a core MDIO change transaction across discovery and registration so bus teardown cannot race the MDIO reads used to identify a PHY. Compile the notifier only with CONFIG_OF_DYNAMIC and the overlay preflight only with CONFIG_OF_OVERLAY. Signed-off-by: James Hilliard --- Changes v3 -> v4: - use the RCU-safe map access introduced by patch 3 while checking a removal scope Changes v2 -> v3: - do not block overlays containing unrelated non-MDIO nodes - do not block overlays whose MDIO bus is no longer present - use READ_ONCE() and WRITE_ONCE() for recursive lock-owner accesses - use the common checked MDIO device-removal callback - avoid an OF/MDIO lock inversion during overlay preflight - refuse overlay removal while another MDIO map change is active - propagate removal-transaction conflicts from the OF notifier - do not scan nodes with a malformed reg property - treat a populated node associated with another MDIO bus as busy Changes v1 -> v2: - split from the ACx00 series - move the PHY declaration to function scope for netdev style --- drivers/net/mdio/of_mdio.c | 454 +++++++++++++++++++++++++++++++++++++++++= +++- 1 file changed, 445 insertions(+), 9 deletions(-) diff --git a/drivers/net/mdio/of_mdio.c b/drivers/net/mdio/of_mdio.c index 051e449bbe7c..7bd994cab5a0 100644 --- a/drivers/net/mdio/of_mdio.c +++ b/drivers/net/mdio/of_mdio.c @@ -13,6 +13,7 @@ #include #include #include +#include #include #include #include @@ -20,6 +21,8 @@ #include #include #include +#include +#include =20 #define DEFAULT_GPIO_RESET_DELAY 10 /* in microseconds */ =20 @@ -27,6 +30,52 @@ MODULE_AUTHOR("Grant Likely "= ); MODULE_LICENSE("GPL"); MODULE_DESCRIPTION("OpenFirmware MDIO bus (Ethernet PHY) accessors"); =20 +#if IS_ENABLED(CONFIG_OF_DYNAMIC) +/* + * OF changes can nest when probing one MDIO device enables another node on + * the same bus. Serialize independent changes while allowing that nesting. + */ +static DEFINE_MUTEX(of_mdio_reconfig_mutex); +static struct task_struct *of_mdio_reconfig_owner; +static unsigned int of_mdio_reconfig_depth; + +static void of_mdio_reconfig_lock(void) +{ + if (!mutex_trylock(&of_mdio_reconfig_mutex)) { + if (READ_ONCE(of_mdio_reconfig_owner) =3D=3D current) { + of_mdio_reconfig_depth++; + return; + } + mutex_lock(&of_mdio_reconfig_mutex); + } + + WARN_ON_ONCE(READ_ONCE(of_mdio_reconfig_owner)); + WARN_ON_ONCE(of_mdio_reconfig_depth); + WRITE_ONCE(of_mdio_reconfig_owner, current); + of_mdio_reconfig_depth =3D 1; +} + +static void of_mdio_reconfig_unlock(void) +{ + WARN_ON_ONCE(READ_ONCE(of_mdio_reconfig_owner) !=3D current); + WARN_ON_ONCE(!of_mdio_reconfig_depth); + + if (--of_mdio_reconfig_depth) + return; + + WRITE_ONCE(of_mdio_reconfig_owner, NULL); + mutex_unlock(&of_mdio_reconfig_mutex); +} +#else +static inline void of_mdio_reconfig_lock(void) +{ +} + +static inline void of_mdio_reconfig_unlock(void) +{ +} +#endif + /* Extract the clause 22 phy ID from the compatible string of the form * ethernet-phy-idAAAA.BBBB */ static int of_get_phy_id(struct device_node *device, u32 *phy_id) @@ -80,10 +129,20 @@ static int of_mdiobus_register_device(struct mii_bus *= mdio, static int of_mdiobus_register_child(struct mii_bus *mdio, struct device_node *child, u32 addr) { + int rc; + + if (of_node_test_and_set_flag(child, OF_POPULATED)) + return 0; + if (of_mdiobus_child_is_phy(child)) - return of_mdiobus_register_phy(mdio, child, addr); + rc =3D of_mdiobus_register_phy(mdio, child, addr); + else + rc =3D of_mdiobus_register_device(mdio, child, addr); + + if (rc) + of_node_clear_flag(child, OF_POPULATED); =20 - return of_mdiobus_register_device(mdio, child, addr); + return rc; } =20 /* The following is a list of PHY compatible strings which appear in @@ -255,13 +314,21 @@ int __of_mdiobus_register(struct mii_bus *mdio, struc= t device_node *np, if (rc) return rc; =20 + of_mdio_reconfig_lock(); + rc =3D mdiobus_device_change_begin(mdio, false); + if (rc) { + of_mdio_reconfig_unlock(); + mdiobus_unregister(mdio); + return rc; + } + /* Loop over the child nodes and register a phy_device for each phy */ rc =3D __of_mdiobus_parse_phys(mdio, np, &scanphys); if (rc) - goto unregister; + goto out_change; =20 if (!scanphys) - return 0; + goto out_change; =20 /* auto scan for PHYs with empty reg property */ for_each_available_child_of_node(np, child) { @@ -272,19 +339,388 @@ int __of_mdiobus_register(struct mii_bus *mdio, stru= ct device_node *np, =20 rc =3D of_mdiobus_scan_phy(mdio, child); if (rc && rc !=3D -ENODEV) - goto put_unregister; + goto put_child; + rc =3D 0; } =20 - return 0; +out_change: + mdiobus_device_change_end(mdio, false); + of_mdio_reconfig_unlock(); + if (!rc) + return 0; =20 -put_unregister: - of_node_put(child); -unregister: mdiobus_unregister(mdio); return rc; + +put_child: + of_node_put(child); + goto out_change; } EXPORT_SYMBOL(__of_mdiobus_register); =20 +#if IS_ENABLED(CONFIG_OF_DYNAMIC) +static bool of_mdiobus_node_is_available(struct device_node *node) +{ + return !of_node_check_flag(node, OF_DETACHED) && + of_device_is_available(node); +} + +static int of_mdiobus_add_node(struct mii_bus *mdio, + struct device_node *node) +{ + struct device_node *child; + int addr, rc, ret =3D 0; + + if (!of_mdiobus_node_is_available(node)) + return 0; + + if (of_node_name_eq(node, "ethernet-phy-package")) { + if (!of_property_present(node, "reg")) + return 0; + + for_each_available_child_of_node(node, child) { + rc =3D of_mdiobus_add_node(mdio, child); + if (rc && rc !=3D -ENODEV && !ret) + ret =3D rc; + } + + return ret; + } + + addr =3D of_mdio_parse_addr(&mdio->dev, node); + if (addr < 0) { + if (of_property_present(node, "reg")) + return addr; + + rc =3D of_mdiobus_scan_phy(mdio, node); + } else { + rc =3D of_mdiobus_register_child(mdio, node, addr); + } + + if (rc =3D=3D -ENODEV && addr >=3D 0) + dev_err(&mdio->dev, + "MDIO device at address %d is missing.\n", addr); + + return rc; +} + +static bool of_mdiobus_node_is_busy(struct mii_bus *mdio, + struct device_node *node) +{ + struct mdio_device *mdiodev; + struct phy_device *phydev; + bool busy =3D false; + + if (of_node_name_eq(node, "ethernet-phy-package")) { + for_each_child_of_node_scoped(node, child) { + if (of_mdiobus_node_is_busy(mdio, child)) + return true; + } + + return false; + } + + if (!of_node_check_flag(node, OF_POPULATED)) + return false; + + mdiodev =3D of_mdio_find_device(node); + if (!mdiodev) + return true; + if (mdiodev->bus !=3D mdio) { + put_device(&mdiodev->dev); + return true; + } + + mutex_lock(&mdio->mdio_map_lock); + busy =3D rcu_access_pointer(mdio->mdio_map[mdiodev->addr]) !=3D mdiodev || + (mdio->mdio_map_pending & BIT(mdiodev->addr)); + if (mdiodev->flags & MDIO_DEVICE_FLAG_PHY) { + phydev =3D to_phy_device(&mdiodev->dev); + if (phydev->attached) { + busy =3D true; + dev_warn(&mdiodev->dev, + "cannot remove an attached PHY; remove its consumer first\n"); + } + } + mutex_unlock(&mdio->mdio_map_lock); + put_device(&mdiodev->dev); + + return busy; +} + +static struct device_node * +of_mdiobus_get_removal_scope(struct device_node *node) +{ + struct device_node *parent; + + if (of_node_name_eq(node, "ethernet-phy-package")) + return of_node_get(node); + + parent =3D of_get_parent(node); + if (of_node_name_eq(parent, "ethernet-phy-package")) + return parent; + + of_node_put(parent); + return of_node_get(node); +} + +static int of_mdiobus_remove_node(struct mii_bus *mdio, + struct device_node *node) +{ + struct mdio_device *mdiodev; + struct device_node *child; + int ret; + + if (of_node_name_eq(node, "ethernet-phy-package")) { + for_each_child_of_node(node, child) { + ret =3D of_mdiobus_remove_node(mdio, child); + if (ret) { + of_node_put(child); + return ret; + } + } + return 0; + } + + if (!of_node_check_flag(node, OF_POPULATED)) + return 0; + + /* The OF node lookup also covers PHYs found by address scanning. */ + mdiodev =3D of_mdio_find_device(node); + if (!mdiodev) + return -EBUSY; + if (mdiodev->bus !=3D mdio) { + put_device(&mdiodev->dev); + return -ENODEV; + } + + ret =3D mdiodev->device_remove(mdiodev, true); + if (!ret) + mdiodev->device_free(mdiodev); + put_device(&mdiodev->dev); + + return ret =3D=3D -ENODEV ? 0 : ret; +} + +static struct mii_bus *of_mdiobus_find_parent(struct device_node *node) +{ + struct device_node *parent, *bus_node; + struct mii_bus *mdio; + + parent =3D of_get_parent(node); + if (!parent) + return NULL; + + if (of_node_name_eq(parent, "ethernet-phy-package")) { + if (!of_device_is_available(parent) || + !of_property_present(parent, "reg")) { + of_node_put(parent); + return NULL; + } + + bus_node =3D of_get_parent(parent); + of_node_put(parent); + } else { + bus_node =3D parent; + } + + mdio =3D of_mdio_find_bus(bus_node); + of_node_put(bus_node); + + return mdio; +} + +#if IS_ENABLED(CONFIG_OF_OVERLAY) +/* Overlay entry notifier errors cannot stop removal after the tree change= d. */ +static bool of_mdiobus_live_node_is_busy(struct device_node *node) +{ + struct device_node *scope; + struct mii_bus *mdio; + bool busy; + + scope =3D of_mdiobus_get_removal_scope(node); + mdio =3D of_mdiobus_find_parent(scope); + /* Non-MDIO nodes and buses already removed cannot block an overlay. */ + if (!mdio) { + busy =3D false; + goto out_put_scope; + } + + busy =3D true; + if (!mdiobus_device_change_begin(mdio, true)) { + busy =3D of_mdiobus_node_is_busy(mdio, scope); + mdiobus_device_change_end(mdio, true); + } + put_device(&mdio->dev); + +out_put_scope: + of_node_put(scope); + return busy; +} + +static struct device_node * +of_mdiobus_overlay_target_child(struct device_node *target, + struct device_node *overlay_child) +{ + const char *name =3D kbasename(overlay_child->full_name); + struct device_node *child; + + for_each_child_of_node(target, child) { + if (!of_node_cmp(kbasename(child->full_name), name)) + return child; + } + + return NULL; +} + +static bool of_mdiobus_overlay_node_is_busy(struct device_node *overlay, + struct device_node *target, + bool added) +{ + struct device_node *overlay_child, *target_child; + bool busy, child_added; + + if ((added || of_property_present(overlay, "status")) && + of_mdiobus_live_node_is_busy(target)) + return true; + + for_each_child_of_node(overlay, overlay_child) { + target_child =3D of_mdiobus_overlay_target_child(target, + overlay_child); + if (!target_child) + continue; + + child_added =3D of_node_check_flag(target_child, OF_OVERLAY); + busy =3D of_mdiobus_overlay_node_is_busy(overlay_child, + target_child, child_added); + of_node_put(target_child); + if (busy) { + of_node_put(overlay_child); + return true; + } + } + + return false; +} + +static int of_mdiobus_overlay_notify(struct notifier_block *nb, + unsigned long action, void *arg) +{ + struct of_overlay_notify_data *nd =3D arg; + bool busy; + + if (action !=3D OF_OVERLAY_PRE_REMOVE) + return NOTIFY_OK; + + busy =3D of_mdiobus_overlay_node_is_busy(nd->overlay, nd->target, + false); + + return busy ? notifier_from_errno(-EBUSY) : NOTIFY_OK; +} + +static struct notifier_block of_mdio_overlay_notifier =3D { + .notifier_call =3D of_mdiobus_overlay_notify, +}; +#endif + +static int of_mdiobus_notify(struct notifier_block *nb, unsigned long acti= on, + void *arg) +{ + struct of_reconfig_data *rd =3D arg; + struct device_node *scope; + struct mii_bus *mdio; + enum of_reconfig_change change; + bool removing; + int rc, ret =3D NOTIFY_OK; + + of_mdio_reconfig_lock(); + change =3D of_reconfig_get_state_change(action, rd); + switch (change) { + case OF_RECONFIG_CHANGE_ADD: + /* A newer change may have made this notification stale. */ + if (!of_mdiobus_node_is_available(rd->dn)) + goto out_unlock; + removing =3D false; + break; + case OF_RECONFIG_CHANGE_REMOVE: + /* A newer change may have made this notification stale. */ + if (of_mdiobus_node_is_available(rd->dn)) + goto out_unlock; + removing =3D true; + break; + default: + goto out_unlock; + } + + mdio =3D of_mdiobus_find_parent(rd->dn); + if (!mdio) + goto out_unlock; + + rc =3D mdiobus_device_change_begin(mdio, removing); + if (rc) { + if (!removing) + ret =3D notifier_from_errno(-EPROBE_DEFER); + else if (rc !=3D -ENODEV) + ret =3D notifier_from_errno(rc); + goto out_put_mdio; + } + + if (!removing) { + rc =3D of_mdiobus_add_node(mdio, rd->dn); + } else { + /* The node may already be detached from its parent hierarchy. */ + scope =3D of_mdiobus_get_removal_scope(rd->dn); + if (of_mdiobus_node_is_busy(mdio, scope)) + rc =3D -EBUSY; + else + rc =3D of_mdiobus_remove_node(mdio, rd->dn); + of_node_put(scope); + } + + mdiobus_device_change_end(mdio, removing); + if (rc && (removing || rc !=3D -ENODEV)) + ret =3D notifier_from_errno(rc); + +out_put_mdio: + put_device(&mdio->dev); +out_unlock: + of_mdio_reconfig_unlock(); + + return ret; +} + +static struct notifier_block of_mdio_notifier =3D { + .notifier_call =3D of_mdiobus_notify, +}; + +static int __init of_mdio_init(void) +{ + int ret; + + ret =3D of_reconfig_notifier_register(&of_mdio_notifier); + if (ret) + return ret; + +#if IS_ENABLED(CONFIG_OF_OVERLAY) + ret =3D of_overlay_notifier_register(&of_mdio_overlay_notifier); + if (ret) + of_reconfig_notifier_unregister(&of_mdio_notifier); +#endif + + return ret; +} +module_init(of_mdio_init); + +static void __exit of_mdio_exit(void) +{ +#if IS_ENABLED(CONFIG_OF_OVERLAY) + of_overlay_notifier_unregister(&of_mdio_overlay_notifier); +#endif + of_reconfig_notifier_unregister(&of_mdio_notifier); +} +module_exit(of_mdio_exit); +#endif /* CONFIG_OF_DYNAMIC */ + /** * of_mdio_find_device - Given a device tree node, find the mdio_device * @np: pointer to the mdio_device's device tree node --=20 2.53.0