Le 16/03/2018 à 20:27, David Miller a écrit :
From: Christophe JAILLET <christophe.jaillet@xxxxxxxxxx>
Date: Wed, 14 Mar 2018 22:09:34 +0100
diff --git a/drivers/net/ethernet/arc/emac_rockchip.c b/drivers/net/ethernet/arc/emac_rockchip.c
index 16f9bee992fe..8ee9dfd0e363 100644
--- a/drivers/net/ethernet/arc/emac_rockchip.c
+++ b/drivers/net/ethernet/arc/emac_rockchip.c
@@ -169,8 +169,10 @@ static int emac_rockchip_probe(struct platform_device *pdev)
/* Optional regulator for PHY */
priv->regulator = devm_regulator_get_optional(dev, "phy");
if (IS_ERR(priv->regulator)) {
- if (PTR_ERR(priv->regulator) == -EPROBE_DEFER)
- return -EPROBE_DEFER;
+ if (PTR_ERR(priv->regulator) == -EPROBE_DEFER) {
+ ret = -EPROBE_DEFER;
+ goto out_clk_disable;
+ }
Please build test your changes.
There is no 'ret' variable in this function, perhaps you meant 'err'.
Yes, obviously, this is 'err'.
I apologize. I usually build-test all my patches before submission.
This one seems to have gone out of my normal process.
Can you fix it yourself or do you want me to re-submit?
CJ
--
To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html