diff options
| author | Waqar Hameed <waqar.hameed@axis.com> | 2026-01-23 11:55:43 +0300 |
|---|---|---|
| committer | Sebastian Reichel <sebastian.reichel@collabora.com> | 2026-01-30 22:57:32 +0300 |
| commit | 3db37475d78ed0c2857f289a535dd44b045a904a (patch) | |
| tree | a5c828302aa385430f3b6c82638a923377f93c0c | |
| parent | 72db889394d89ffde61e7438f0cdfc0135c9da48 (diff) | |
| download | linux-3db37475d78ed0c2857f289a535dd44b045a904a.tar.xz | |
power: supply: wm97xx: Use devm_power_supply_register()
Instead of handling the registration manually, use the automatic
`devres` variant `devm_power_supply_register()`. This is less error
prone and cleaner.
Signed-off-by: Waqar Hameed <waqar.hameed@axis.com>
Link: https://patch.msgid.link/b0b366d302f0605c8555dd68ed32973959f133bb.1769158280.git.waqar.hameed@axis.com
Signed-off-by: Sebastian Reichel <sebastian.reichel@collabora.com>
| -rw-r--r-- | drivers/power/supply/wm97xx_battery.c | 16 |
1 files changed, 4 insertions, 12 deletions
diff --git a/drivers/power/supply/wm97xx_battery.c b/drivers/power/supply/wm97xx_battery.c index c30c347b48f9..a6c55b1e0286 100644 --- a/drivers/power/supply/wm97xx_battery.c +++ b/drivers/power/supply/wm97xx_battery.c @@ -223,7 +223,7 @@ static int wm97xx_bat_probe(struct platform_device *dev) bat_psy_desc.properties = prop; bat_psy_desc.num_properties = props; - bat_psy = power_supply_register(&dev->dev, &bat_psy_desc, &cfg); + bat_psy = devm_power_supply_register(&dev->dev, &bat_psy_desc, &cfg); if (IS_ERR(bat_psy)) return PTR_ERR(bat_psy); @@ -232,19 +232,12 @@ static int wm97xx_bat_probe(struct platform_device *dev) if (charge_gpiod) { ret = request_irq(gpiod_to_irq(charge_gpiod), wm97xx_chrg_irq, 0, "AC Detect", dev); - if (ret) { - dev_err_probe(&dev->dev, ret, - "failed to request GPIO irq\n"); - goto unregister; - } + if (ret) + return dev_err_probe(&dev->dev, ret, + "failed to request GPIO irq\n"); } return 0; - -unregister: - power_supply_unregister(bat_psy); - - return ret; } static void wm97xx_bat_remove(struct platform_device *dev) @@ -252,7 +245,6 @@ static void wm97xx_bat_remove(struct platform_device *dev) if (charge_gpiod) free_irq(gpiod_to_irq(charge_gpiod), dev); cancel_work_sync(&bat_work); - power_supply_unregister(bat_psy); } static struct platform_driver wm97xx_bat_driver = { |