power: supply: rk817: check correct variable

Linux kernel mirror (for testing) git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git

kernel os linux

This code checks "->chg_ps" twice when it was supposed to check
"->bat_ps".

Fixes: 11cb8da0189b ("power: supply: Add charger driver for Rockchip RK817")
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Sebastian Reichel <sebastian.reichel@collabora.com>

authored by

Dan Carpenter and committed by

Sebastian Reichel 3 years ago 172c65e6 9abf2313

+3 -5

1 changed file

expand all

drivers

power

supply

rk817_charger.c

+3 -5

drivers/power/supply/rk817_charger.c

··· 1116 1116 1117 1117 charger->bat_ps = devm_power_supply_register(&pdev->dev, 1118 1118 &rk817_bat_desc, &pscfg); 1119 - 1120 - charger->chg_ps = devm_power_supply_register(&pdev->dev, 1121 - &rk817_chg_desc, &pscfg); 1122 - 1123 - if (IS_ERR(charger->chg_ps)) 1119 + if (IS_ERR(charger->bat_ps)) 1124 1120 return dev_err_probe(dev, -EINVAL, 1125 1121 "Battery failed to probe\n"); 1126 1122 1123 + charger->chg_ps = devm_power_supply_register(&pdev->dev, 1124 + &rk817_chg_desc, &pscfg); 1127 1125 if (IS_ERR(charger->chg_ps)) 1128 1126 return dev_err_probe(dev, -EINVAL, 1129 1127 "Charger failed to probe\n");