stmmac: intel: Simplify intel_eth_pci_remove()
authorChristophe JAILLET <christophe.jaillet@wanadoo.fr>
Fri, 2 Sep 2022 16:26:56 +0000 (18:26 +0200)
committerDavid S. Miller <davem@davemloft.net>
Mon, 5 Sep 2022 13:19:56 +0000 (14:19 +0100)
There is no point to call pcim_iounmap_regions() in the remove function,
this frees a managed resource that would be release by the framework
anyway.

Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr>
Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/stmicro/stmmac/dwmac-intel.c

index 4f2b82a..9af25be 100644 (file)
@@ -1136,8 +1136,6 @@ static void intel_eth_pci_remove(struct pci_dev *pdev)
 
        clk_disable_unprepare(priv->plat->stmmac_clk);
        clk_unregister_fixed_rate(priv->plat->stmmac_clk);
-
-       pcim_iounmap_regions(pdev, BIT(0));
 }
 
 static int __maybe_unused intel_eth_pci_suspend(struct device *dev)