net: dwc_eth_qos: Drop unused dm_gpio_free() on STM32
authorMarek Vasut <marex@denx.de>
Mon, 6 Mar 2023 14:53:44 +0000 (15:53 +0100)
committerStefano Babic <sbabic@denx.de>
Thu, 30 Mar 2023 11:47:03 +0000 (13:47 +0200)
The dm_gpio_free() is never called, because for stm32, the phy_reset_gpio
pointer is never valid. This is because only tegra186 ever claims the
phy_reset_gpio, all other platforms use the PHY framework to reset the
PHY instead. Drop the dm_gpio_free() and dm_gpio_is_valid().

Reviewed-by: Ramon Fried <rfried.dev@gmail.com>
Signed-off-by: Marek Vasut <marex@denx.de>
drivers/net/dwc_eth_qos.c

index 00690b2..b97b3ea 100644 (file)
@@ -1493,7 +1493,7 @@ static int eqos_remove_resources_tegra186(struct udevice *dev)
 
 static int eqos_remove_resources_stm32(struct udevice *dev)
 {
-       struct eqos_priv *eqos = dev_get_priv(dev);
+       struct eqos_priv * __maybe_unused eqos = dev_get_priv(dev);
 
        debug("%s(dev=%p):\n", __func__, dev);
 
@@ -1505,9 +1505,6 @@ static int eqos_remove_resources_stm32(struct udevice *dev)
                clk_free(&eqos->clk_ck);
 #endif
 
-       if (dm_gpio_is_valid(&eqos->phy_reset_gpio))
-               dm_gpio_free(dev, &eqos->phy_reset_gpio);
-
        debug("%s: OK\n", __func__);
        return 0;
 }