diff options
| author | Marek Vasut <marex@denx.de> | 2023-03-06 15:53:44 +0100 |
|---|---|---|
| committer | Stefano Babic <sbabic@denx.de> | 2023-03-30 13:47:03 +0200 |
| commit | 2e0bade78542998134378ae33c50b054532b70a1 (patch) | |
| tree | e810b1c6b8703a97b22ddb71cd3c2c671c375a8d | |
| parent | 895b05ce02ece2482534351dbe618a12e3d223bb (diff) | |
| download | u-boot-2e0bade78542998134378ae33c50b054532b70a1.zip u-boot-2e0bade78542998134378ae33c50b054532b70a1.tar.gz u-boot-2e0bade78542998134378ae33c50b054532b70a1.tar.bz2 | |
net: dwc_eth_qos: Drop unused dm_gpio_free() on STM32
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>
| -rw-r--r-- | drivers/net/dwc_eth_qos.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/drivers/net/dwc_eth_qos.c b/drivers/net/dwc_eth_qos.c index 00690b2..b97b3ea 100644 --- a/drivers/net/dwc_eth_qos.c +++ b/drivers/net/dwc_eth_qos.c @@ -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; } |