From patchwork Wed Apr 5 09:27:02 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Marco Felsch X-Patchwork-Id: 670421 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id D263CC76188 for ; Wed, 5 Apr 2023 09:28:44 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S237346AbjDEJ2n (ORCPT ); Wed, 5 Apr 2023 05:28:43 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:54426 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S237470AbjDEJ21 (ORCPT ); Wed, 5 Apr 2023 05:28:27 -0400 Received: from metis.ext.pengutronix.de (metis.ext.pengutronix.de [IPv6:2001:67c:670:201:290:27ff:fe1d:cc33]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 5C9E35B87 for ; Wed, 5 Apr 2023 02:28:02 -0700 (PDT) Received: from dude02.red.stw.pengutronix.de ([2a0a:edc0:0:1101:1d::28]) by metis.ext.pengutronix.de with esmtp (Exim 4.92) (envelope-from ) id 1pjzQA-0004pA-0y; Wed, 05 Apr 2023 11:27:14 +0200 From: Marco Felsch Date: Wed, 05 Apr 2023 11:27:02 +0200 Subject: [PATCH 11/12] net: mdiobus: remove now unused fwnode helpers MIME-Version: 1.0 Message-Id: <20230405-net-next-topic-net-phy-reset-v1-11-7e5329f08002@pengutronix.de> References: <20230405-net-next-topic-net-phy-reset-v1-0-7e5329f08002@pengutronix.de> In-Reply-To: <20230405-net-next-topic-net-phy-reset-v1-0-7e5329f08002@pengutronix.de> To: Andrew Lunn , Heiner Kallweit , Russell King , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Florian Fainelli , Broadcom internal kernel review list , Richard Cochran , Radu Pirea , Shyam Sundar S K , Yisen Zhuang , Salil Mehta , Jassi Brar , Ilias Apalodimas , Iyappan Subramanian , Keyur Chudgar , Quan Nguyen , "Rafael J. Wysocki" , Len Brown , Rob Herring , Frank Rowand Cc: netdev@vger.kernel.org, linux-kernel@vger.kernel.org, linux-acpi@vger.kernel.org, devicetree@vger.kernel.org, kernel@pengutronix.de X-Mailer: b4 0.12.1 X-SA-Exim-Connect-IP: 2a0a:edc0:0:1101:1d::28 X-SA-Exim-Mail-From: m.felsch@pengutronix.de X-SA-Exim-Scanned: No (on metis.ext.pengutronix.de); SAEximRunCond expanded to false X-PTX-Original-Recipient: linux-acpi@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-acpi@vger.kernel.org These APIs are broken since the very first day because they assume that the phy is accessible to read the PHYID registers. If this requirement is not meet the code fails to add the required phys. The newly added phy_device_atomic_register() API have fixed this and supports firmware parsing as well. After we switched all in kernel users of the fwnode API we now can remove this part from the kernel. Signed-off-by: Marco Felsch --- MAINTAINERS | 1 - drivers/net/mdio/Kconfig | 7 -- drivers/net/mdio/Makefile | 1 - drivers/net/mdio/acpi_mdio.c | 2 +- drivers/net/mdio/fwnode_mdio.c | 186 ----------------------------------------- drivers/net/mdio/of_mdio.c | 1 - include/linux/fwnode_mdio.h | 35 -------- 7 files changed, 1 insertion(+), 232 deletions(-) diff --git a/MAINTAINERS b/MAINTAINERS index 7812f0e251ad..2894b456c0a3 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -7666,7 +7666,6 @@ F: Documentation/devicetree/bindings/net/qca,ar803x.yaml F: Documentation/networking/phy.rst F: drivers/net/mdio/ F: drivers/net/mdio/acpi_mdio.c -F: drivers/net/mdio/fwnode_mdio.c F: drivers/net/mdio/of_mdio.c F: drivers/net/pcs/ F: drivers/net/phy/ diff --git a/drivers/net/mdio/Kconfig b/drivers/net/mdio/Kconfig index 90309980686e..d0d19666f099 100644 --- a/drivers/net/mdio/Kconfig +++ b/drivers/net/mdio/Kconfig @@ -19,13 +19,6 @@ config MDIO_BUS reflects whether the mdio_bus/mdio_device code is built as a loadable module or built-in. -config FWNODE_MDIO - def_tristate PHYLIB - depends on (ACPI || OF) || COMPILE_TEST - select FIXED_PHY - help - FWNODE MDIO bus (Ethernet PHY) accessors - config OF_MDIO def_tristate PHYLIB depends on OF diff --git a/drivers/net/mdio/Makefile b/drivers/net/mdio/Makefile index 7d4cb4c11e4e..798ede184766 100644 --- a/drivers/net/mdio/Makefile +++ b/drivers/net/mdio/Makefile @@ -2,7 +2,6 @@ # Makefile for Linux MDIO bus drivers obj-$(CONFIG_ACPI_MDIO) += acpi_mdio.o -obj-$(CONFIG_FWNODE_MDIO) += fwnode_mdio.o obj-$(CONFIG_OF_MDIO) += of_mdio.o obj-$(CONFIG_MDIO_ASPEED) += mdio-aspeed.o diff --git a/drivers/net/mdio/acpi_mdio.c b/drivers/net/mdio/acpi_mdio.c index 25feb571bd1f..727b83bf3a15 100644 --- a/drivers/net/mdio/acpi_mdio.c +++ b/drivers/net/mdio/acpi_mdio.c @@ -10,8 +10,8 @@ #include #include #include -#include #include +#include #include MODULE_AUTHOR("Calvin Johnson "); diff --git a/drivers/net/mdio/fwnode_mdio.c b/drivers/net/mdio/fwnode_mdio.c deleted file mode 100644 index 47ef702d4ffd..000000000000 --- a/drivers/net/mdio/fwnode_mdio.c +++ /dev/null @@ -1,186 +0,0 @@ -// SPDX-License-Identifier: GPL-2.0-only -/* - * fwnode helpers for the MDIO (Ethernet PHY) API - * - * This file provides helper functions for extracting PHY device information - * out of the fwnode and using it to populate an mii_bus. - */ - -#include -#include -#include -#include -#include - -MODULE_AUTHOR("Calvin Johnson "); -MODULE_LICENSE("GPL"); - -static struct pse_control * -fwnode_find_pse_control(struct fwnode_handle *fwnode) -{ - struct pse_control *psec; - struct device_node *np; - - if (!IS_ENABLED(CONFIG_PSE_CONTROLLER)) - return NULL; - - np = to_of_node(fwnode); - if (!np) - return NULL; - - psec = of_pse_control_get(np); - if (PTR_ERR(psec) == -ENOENT) - return NULL; - - return psec; -} - -static struct mii_timestamper * -fwnode_find_mii_timestamper(struct fwnode_handle *fwnode) -{ - struct of_phandle_args arg; - int err; - - if (is_acpi_node(fwnode)) - return NULL; - - err = of_parse_phandle_with_fixed_args(to_of_node(fwnode), - "timestamper", 1, 0, &arg); - if (err == -ENOENT) - return NULL; - else if (err) - return ERR_PTR(err); - - if (arg.args_count != 1) - return ERR_PTR(-EINVAL); - - return register_mii_timestamper(arg.np, arg.args[0]); -} - -int fwnode_mdiobus_phy_device_register(struct mii_bus *mdio, - struct phy_device *phy, - struct fwnode_handle *child, u32 addr) -{ - int rc; - - rc = fwnode_irq_get(child, 0); - /* Don't wait forever if the IRQ provider doesn't become available, - * just fall back to poll mode - */ - if (rc == -EPROBE_DEFER) - rc = driver_deferred_probe_check_state(&phy->mdio.dev); - if (rc == -EPROBE_DEFER) - return rc; - - if (rc > 0) { - phy->irq = rc; - mdio->irq[addr] = rc; - } else { - phy->irq = mdio->irq[addr]; - } - - if (fwnode_property_read_bool(child, "broken-turn-around")) - mdio->phy_ignore_ta_mask |= 1 << addr; - - fwnode_property_read_u32(child, "reset-assert-us", - &phy->mdio.reset_assert_delay); - fwnode_property_read_u32(child, "reset-deassert-us", - &phy->mdio.reset_deassert_delay); - - /* Associate the fwnode with the device structure so it - * can be looked up later - */ - fwnode_handle_get(child); - device_set_node(&phy->mdio.dev, child); - - /* All data is now stored in the phy struct; - * register it - */ - rc = phy_device_register(phy); - if (rc) { - device_set_node(&phy->mdio.dev, NULL); - fwnode_handle_put(child); - return rc; - } - - dev_dbg(&mdio->dev, "registered phy %p fwnode at address %i\n", - child, addr); - return 0; -} -EXPORT_SYMBOL(fwnode_mdiobus_phy_device_register); - -int fwnode_mdiobus_register_phy(struct mii_bus *bus, - struct fwnode_handle *child, u32 addr) -{ - struct phy_device_config config = { - .mii_bus = bus, - .phy_addr = addr, - }; - struct mii_timestamper *mii_ts = NULL; - struct pse_control *psec = NULL; - struct phy_device *phy; - u32 phy_id; - int rc; - - psec = fwnode_find_pse_control(child); - if (IS_ERR(psec)) - return PTR_ERR(psec); - - mii_ts = fwnode_find_mii_timestamper(child); - if (IS_ERR(mii_ts)) { - rc = PTR_ERR(mii_ts); - goto clean_pse; - } - - config.is_c45 = fwnode_device_is_compatible(child, "ethernet-phy-ieee802.3-c45"); - if (config.is_c45 || fwnode_get_phy_id(child, &config.phy_id)) - phy = get_phy_device(&config); - else - phy = phy_device_create(&config); - if (IS_ERR(phy)) { - rc = PTR_ERR(phy); - goto clean_mii_ts; - } - - if (is_acpi_node(child)) { - phy->irq = bus->irq[addr]; - - /* Associate the fwnode with the device structure so it - * can be looked up later. - */ - phy->mdio.dev.fwnode = fwnode_handle_get(child); - - /* All data is now stored in the phy struct, so register it */ - rc = phy_device_register(phy); - if (rc) { - phy->mdio.dev.fwnode = NULL; - fwnode_handle_put(child); - goto clean_phy; - } - } else if (is_of_node(child)) { - rc = fwnode_mdiobus_phy_device_register(bus, phy, child, addr); - if (rc) - goto clean_phy; - } - - phy->psec = psec; - - /* phy->mii_ts may already be defined by the PHY driver. A - * mii_timestamper probed via the device tree will still have - * precedence. - */ - if (mii_ts) - phy->mii_ts = mii_ts; - - return 0; - -clean_phy: - phy_device_free(phy); -clean_mii_ts: - unregister_mii_timestamper(mii_ts); -clean_pse: - pse_control_put(psec); - - return rc; -} -EXPORT_SYMBOL(fwnode_mdiobus_register_phy); diff --git a/drivers/net/mdio/of_mdio.c b/drivers/net/mdio/of_mdio.c index e85be8a72978..15ae968ef186 100644 --- a/drivers/net/mdio/of_mdio.c +++ b/drivers/net/mdio/of_mdio.c @@ -10,7 +10,6 @@ #include #include -#include #include #include #include diff --git a/include/linux/fwnode_mdio.h b/include/linux/fwnode_mdio.h deleted file mode 100644 index faf603c48c86..000000000000 --- a/include/linux/fwnode_mdio.h +++ /dev/null @@ -1,35 +0,0 @@ -/* SPDX-License-Identifier: GPL-2.0-only */ -/* - * FWNODE helper for the MDIO (Ethernet PHY) API - */ - -#ifndef __LINUX_FWNODE_MDIO_H -#define __LINUX_FWNODE_MDIO_H - -#include - -#if IS_ENABLED(CONFIG_FWNODE_MDIO) -int fwnode_mdiobus_phy_device_register(struct mii_bus *mdio, - struct phy_device *phy, - struct fwnode_handle *child, u32 addr); - -int fwnode_mdiobus_register_phy(struct mii_bus *bus, - struct fwnode_handle *child, u32 addr); - -#else /* CONFIG_FWNODE_MDIO */ -int fwnode_mdiobus_phy_device_register(struct mii_bus *mdio, - struct phy_device *phy, - struct fwnode_handle *child, u32 addr) -{ - return -EINVAL; -} - -static inline int fwnode_mdiobus_register_phy(struct mii_bus *bus, - struct fwnode_handle *child, - u32 addr) -{ - return -EINVAL; -} -#endif - -#endif /* __LINUX_FWNODE_MDIO_H */