mirror of
https://source.denx.de/u-boot/u-boot.git
synced 2026-06-13 15:03:58 +03:00
net: rswitch: Fix error detection
In rswitch_probe the error detection after the call to devm_clk_get is very wrong. It checks the value of ret which is uninitialised at that point. Instead it should be using the macros for including errors into pointers. This issue was found by Smatch. Signed-off-by: Andrew Goodbody <andrew.goodbody@linaro.org> Reviewed-by: Marek Vasut <marek.vasut+renesas@mailbox.org>
This commit is contained in:
committed by
Marek Vasut
parent
393ace55be
commit
6c49b082f0
@@ -1110,8 +1110,10 @@ static int rswitch_probe(struct udevice *dev)
|
||||
return -EINVAL;
|
||||
|
||||
priv->rsw_clk = devm_clk_get(dev, NULL);
|
||||
if (ret)
|
||||
if (IS_ERR(priv->rsw_clk)) {
|
||||
ret = PTR_ERR(priv->rsw_clk);
|
||||
goto err_map;
|
||||
}
|
||||
|
||||
ret = clk_prepare_enable(priv->rsw_clk);
|
||||
if (ret)
|
||||
|
||||
Reference in New Issue
Block a user