]> git.baikalelectronics.ru Git - kernel.git/commit
stmmac: intel: Add a missing clk_disable_unprepare() call in intel_eth_pci_remove()
authorChristophe JAILLET <christophe.jaillet@wanadoo.fr>
Tue, 16 Aug 2022 14:23:57 +0000 (16:23 +0200)
committerJakub Kicinski <kuba@kernel.org>
Thu, 18 Aug 2022 17:17:46 +0000 (10:17 -0700)
commit6fcc8f036be91dd8a4a49649ce490641820c15a7
tree5be7f7d8cd9e1db5a597fdd38babafba6a052c6b
parent8ebb7a069ec585d39d219c93b04e3ee907f8ad9e
stmmac: intel: Add a missing clk_disable_unprepare() call in intel_eth_pci_remove()

Commit e57086dbdee4 ("stmmac: intel: Fix clock handling on error and remove
paths") removed this clk_disable_unprepare()

This was partly revert by commit c03cc9b5591f ("net: stmmac: Fix clock
handling on remove path") which removed this clk_disable_unprepare()
because:
"
   While unloading the dwmac-intel driver, clk_disable_unprepare() is
   being called twice in stmmac_dvr_remove() and
   intel_eth_pci_remove(). This causes kernel panic on the second call.
"

However later on, commit 9a84d76486da8 ("net: stmmac: add clocks management
for gmac driver") has updated stmmac_dvr_remove() which do not call
clk_disable_unprepare() anymore.

So this call should now be called from intel_eth_pci_remove().

Fixes: 9a84d76486da8 ("net: stmmac: add clocks management for gmac driver")
Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr>
Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Link: https://lore.kernel.org/r/d7c8c1dadf40df3a7c9e643f76ffadd0ccc1ad1b.1660659689.git.christophe.jaillet@wanadoo.fr
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
drivers/net/ethernet/stmicro/stmmac/dwmac-intel.c