Commit ae52c3e8 authored by Geert Uytterhoeven's avatar Geert Uytterhoeven Committed by Jakub Kicinski
Browse files

sh_eth: Use async pm_runtime_put() 



There is no stringent need to power down the device immediately after a
register read, or after a failed open.  Relax power down handling by
replacing calls to synchronous pm_runtime_put_sync() by calls to
asynchronous pm_runtime_put().

Signed-off-by: default avatarGeert Uytterhoeven <geert+renesas@glider.be>
Reviewed-by: default avatarAndrew Lunn <andrew@lunn.ch>
Reviewed-by: default avatarNiklas Söderlund <niklas.soderlund+renesas@ragnatech.se>
Tested-by: default avatarNiklas Söderlund <niklas.soderlund+renesas@ragnatech.se>
Link: https://patch.msgid.link/77562617360e30a47746e53e392905ea312a2f97.1756998732.git.geert+renesas@glider.be


Signed-off-by: default avatarJakub Kicinski <kuba@kernel.org>
parent 3406114a

drivers/net/ethernet/renesas/sh_eth.c

+2 −2
Original line number Diff line number Diff line
@@ -2233,7 +2233,7 @@ static void sh_eth_get_regs(struct net_device *ndev, struct ethtool_regs *regs, 


	pm_runtime_get_sync(&mdp->pdev->dev);

	__sh_eth_get_regs(ndev, buf);

	pm_runtime_put_sync(&mdp->pdev->dev);

	pm_runtime_put(&mdp->pdev->dev);

}



static u32 sh_eth_get_msglevel(struct net_device *ndev)
@@ -2447,7 +2447,7 @@ static int sh_eth_open(struct net_device *ndev) 
	free_irq(ndev->irq, ndev);

out_napi_off:

	napi_disable(&mdp->napi);

	pm_runtime_put_sync(&mdp->pdev->dev);

	pm_runtime_put(&mdp->pdev->dev);

	return ret;

}