power: supply: ab8500: add missing destroy_workqueue in ab8500_charger_bind

zhengbin13 · sre · commit 38d45444e257 · 2022-07-17T01:00:25.000+02:00
ab8500_charger_bind misses destroy_workqueue in error path,
this patch fixes that.

Signed-off-by: Zheng Bin &lt;zhengbin13@huawei.com&gt;
Acked-by: Linus Walleij &lt;linus.walleij@linaro.org&gt;
Signed-off-by: Sebastian Reichel &lt;sebastian.reichel@collabora.com&gt;
diff --git a/drivers/power/supply/ab8500_charger.c b/drivers/power/supply/ab8500_charger.c
@@ -3351,6 +3351,7 @@ static int ab8500_charger_bind(struct device *dev)
 	ret = component_bind_all(dev, di);
 	if (ret) {
 		dev_err(dev, "can't bind component devices\n");
+		destroy_workqueue(di->charger_wq);
 		return ret;
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -3351,6 +3351,7 @@ static int ab8500_charger_bind(struct device *dev)`
`3351`	`3351`	`ret = component_bind_all(dev, di);`
`3352`	`3352`	`if (ret) {`
`3353`	`3353`	`dev_err(dev, "can't bind component devices\n");`
	`3354`	`+ destroy_workqueue(di->charger_wq);`
`3354`	`3355`	`return ret;`
`3355`	`3356`	`}`
`3356`	`3357`