[PATCH 12/14] net: sh_eth: Drop reset GPIO handling in favor of common code
Ramon Fried
rfried.dev at gmail.com
Sat Jun 10 13:01:31 CEST 2023
On Wed, May 31, 2023 at 1:51 AM Marek Vasut
<marek.vasut+renesas at mailbox.org> wrote:
>
> The common code is now capable of handling reset GPIO associated
> with PHY. Drop the local ad-hoc code in favor of common code.
>
> Signed-off-by: Marek Vasut <marek.vasut+renesas at mailbox.org>
> ---
> Cc: Geert Uytterhoeven <geert+renesas at glider.be>
> Cc: Joe Hershberger <joe.hershberger at ni.com>
> Cc: Michal Simek <michal.simek at amd.com>
> Cc: Nishanth Menon <nm at ti.com>
> Cc: Ramon Fried <rfried.dev at gmail.com>
> ---
> drivers/net/Kconfig | 1 +
> drivers/net/sh_eth.c | 17 -----------------
> 2 files changed, 1 insertion(+), 17 deletions(-)
>
> diff --git a/drivers/net/Kconfig b/drivers/net/Kconfig
> index 48f9efcc94f..7d482cb7745 100644
> --- a/drivers/net/Kconfig
> +++ b/drivers/net/Kconfig
> @@ -695,6 +695,7 @@ config SUN8I_EMAC
> config SH_ETHER
> bool "Renesas SH Ethernet MAC"
> select PHYLIB
> + select PHY_ETHERNET_ID
> help
> This driver supports the Ethernet for Renesas SH and ARM SoCs.
>
> diff --git a/drivers/net/sh_eth.c b/drivers/net/sh_eth.c
> index 14f7fd1bcc0..95042daade0 100644
> --- a/drivers/net/sh_eth.c
> +++ b/drivers/net/sh_eth.c
> @@ -530,7 +530,6 @@ struct sh_ether_priv {
> struct mii_dev *bus;
> phys_addr_t iobase;
> struct clk clk;
> - struct gpio_desc reset_gpio;
> };
>
> static int sh_ether_send(struct udevice *dev, void *packet, int len)
> @@ -650,7 +649,6 @@ static int sh_ether_probe(struct udevice *udev)
> struct eth_pdata *pdata = dev_get_plat(udev);
> struct sh_ether_priv *priv = dev_get_priv(udev);
> struct sh_eth_dev *eth = &priv->shdev;
> - struct ofnode_phandle_args phandle_args;
> struct mii_dev *mdiodev;
> int ret;
>
> @@ -661,18 +659,6 @@ static int sh_ether_probe(struct udevice *udev)
> if (ret < 0)
> return ret;
> #endif
> -
> - ret = dev_read_phandle_with_args(udev, "phy-handle", NULL, 0, 0, &phandle_args);
> - if (!ret) {
> - gpio_request_by_name_nodev(phandle_args.node, "reset-gpios", 0,
> - &priv->reset_gpio, GPIOD_IS_OUT);
> - }
> -
> - if (!dm_gpio_is_valid(&priv->reset_gpio)) {
> - gpio_request_by_name(udev, "reset-gpios", 0, &priv->reset_gpio,
> - GPIOD_IS_OUT);
> - }
> -
> mdiodev = mdio_alloc();
> if (!mdiodev) {
> ret = -ENOMEM;
> @@ -735,9 +721,6 @@ static int sh_ether_remove(struct udevice *udev)
> mdio_unregister(priv->bus);
> mdio_free(priv->bus);
>
> - if (dm_gpio_is_valid(&priv->reset_gpio))
> - dm_gpio_free(udev, &priv->reset_gpio);
> -
> return 0;
> }
>
> --
> 2.39.2
>
Reviewed-by: Ramon Fried <rfried.dev at gmail.com>
More information about the U-Boot
mailing list