Skip to content

Commit

Permalink
ARM: u300: fix return value check in __u300_init_boardpower()
Browse files Browse the repository at this point in the history
In case of error, the function syscon_node_to_regmap() returns ERR_PTR()
and never returns NULL. The NULL test in the return value check should
be replaced with IS_ERR().

Signed-off-by: Wei Yongjun <[email protected]>
Signed-off-by: Linus Walleij <[email protected]>
  • Loading branch information
Wei Yongjun authored and linusw committed Nov 26, 2013
1 parent 6ce4eac commit 1e74de3
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions arch/arm/mach-u300/regulator.c
Original file line number Diff line number Diff line change
Expand Up @@ -69,9 +69,9 @@ static int __init __u300_init_boardpower(struct platform_device *pdev)
return -ENODEV;
}
regmap = syscon_node_to_regmap(syscon_np);
if (!regmap) {
if (IS_ERR(regmap)) {
pr_crit("U300: could not locate syscon regmap\n");
return -ENODEV;
return PTR_ERR(regmap);
}

main_power_15 = regulator_get(&pdev->dev, "vana15");
Expand Down

0 comments on commit 1e74de3

Please sign in to comment.