From fbd17e58d472d7ec49810c6cd87900d73e75bb2b Mon Sep 17 00:00:00 2001 From: Sebastian Reichel Date: Sat, 29 Sep 2018 08:47:43 +0200 Subject: [PATCH] power: supply: ds2781: switch to devm_power_supply_register Simplify/Cleanup the driver by switching to devm_power_supply_register and dropping the driver's remove function. Signed-off-by: Sebastian Reichel --- drivers/power/supply/ds2781_battery.c | 15 +++------------ 1 file changed, 3 insertions(+), 12 deletions(-) diff --git a/drivers/power/supply/ds2781_battery.c b/drivers/power/supply/ds2781_battery.c index c81cc14..166a8bd 100644 --- a/drivers/power/supply/ds2781_battery.c +++ b/drivers/power/supply/ds2781_battery.c @@ -772,8 +772,9 @@ static int ds2781_battery_probe(struct platform_device *pdev) psy_cfg.drv_data = dev_info; psy_cfg.attr_grp = ds2781_sysfs_groups; - dev_info->bat = power_supply_register(&pdev->dev, &dev_info->bat_desc, - &psy_cfg); + dev_info->bat = devm_power_supply_register(&pdev->dev, + &dev_info->bat_desc, + &psy_cfg); if (IS_ERR(dev_info->bat)) { dev_err(dev_info->dev, "failed to register battery\n"); return PTR_ERR(dev_info->bat); @@ -782,21 +783,11 @@ static int ds2781_battery_probe(struct platform_device *pdev) return 0; } -static int ds2781_battery_remove(struct platform_device *pdev) -{ - struct ds2781_device_info *dev_info = platform_get_drvdata(pdev); - - power_supply_unregister(dev_info->bat); - - return 0; -} - static struct platform_driver ds2781_battery_driver = { .driver = { .name = "ds2781-battery", }, .probe = ds2781_battery_probe, - .remove = ds2781_battery_remove, }; module_platform_driver(ds2781_battery_driver); -- 2.7.4