[PATCH] sh_eth: Use dev_err_probe() helper
Subsystems:
networking drivers , renesas superh ethernet driver , the rest
STALE1760d
LANDED
Landed in mainline as e5d75fc20b92 on 2021-12-10.
4 messages,
4 authors,
2021-12-10 · open the first message on its own page
Use the dev_err_probe() helper, instead of open-coding the same
operation.
Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
drivers/net/ethernet/renesas/sh_eth.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/net/ethernet/renesas/sh_eth.c b/drivers/net/ethernet/renesas/sh_eth.c
index 223626290ce0e278..d947a628e1663009 100644
--- a/drivers/net/ethernet/renesas/sh_eth.c
+++ b/drivers/net/ethernet/renesas/sh_eth.c @@ -3368,8 +3368,7 @@ static int sh_eth_drv_probe(struct platform_device *pdev)
/* MDIO bus init */
ret = sh_mdio_init ( mdp , pd );
if ( ret ) {
- if ( ret != - EPROBE_DEFER )
- dev_err ( & pdev -> dev , "MDIO init failed: %d \n " , ret );
+ dev_err_probe ( & pdev -> dev , ret , "MDIO init failed \n " );
goto out_release ;
}
--
2.25.1
Hi Geert,
Thanks for your work, I learnt something new.
On 2021-12-08 11:32:07 +0100, Geert Uytterhoeven wrote:
Reviewed-by: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se>
quoted hunk Use the dev_err_probe() helper, instead of open-coding the same
operation.
Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
drivers/net/ethernet/renesas/sh_eth.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/net/ethernet/renesas/sh_eth.c b/drivers/net/ethernet/renesas/sh_eth.c
index 223626290ce0e278..d947a628e1663009 100644
--- a/drivers/net/ethernet/renesas/sh_eth.c
+++ b/drivers/net/ethernet/renesas/sh_eth.c @@ -3368,8 +3368,7 @@ static int sh_eth_drv_probe(struct platform_device *pdev)
/* MDIO bus init */
ret = sh_mdio_init ( mdp , pd );
if ( ret ) {
- if ( ret != - EPROBE_DEFER )
- dev_err ( & pdev -> dev , "MDIO init failed: %d \n " , ret );
+ dev_err_probe ( & pdev -> dev , ret , "MDIO init failed \n " );
goto out_release ;
}
--
2.25.1
--
Kind Regards,
Niklas Söderlund
Hello!
On 12/8/21 1:32 PM, Geert Uytterhoeven wrote:
Use the dev_err_probe() helper, instead of open-coding the same
operation.
Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
Reviewed-by: Sergey Shtylyov <redacted>
[...]
MBR, Sergey
Hello:
This patch was applied to netdev/net-next.git (master)
by Jakub Kicinski [off-list ref]:
On Wed, 8 Dec 2021 11:32:07 +0100 you wrote: Use the dev_err_probe() helper, instead of open-coding the same
operation.
Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
drivers/net/ethernet/renesas/sh_eth.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
Here is the summary with links:
- sh_eth: Use dev_err_probe() helper
https://git.kernel.org/netdev/net-next/c/e5d75fc20b92
You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html