power: supply: ln8411: Changes ln8411_remove() type to void

rriveramcrus · rriveramcrus · commit a054e38b1b38 · 2022-12-16T20:52:06.000-06:00
Updates ln8411_remove to be of type void to match upstream i2c_driver model interface. Change-Id: I90c78d36c22fdb8020d6702cebd4c611936a3116 Link: https://lore.kernel.org/linux-i2c/20220609091018.q52fhowlsdbdkct5@pengutronix.de/ Signed-off-by: Ricardo Rivera-Matos <rriveram@opensource.cirrus.com>
diff --git a/drivers/power/supply/ln8411_charger.c b/drivers/power/supply/ln8411_charger.c
@@ -2369,18 +2369,16 @@ static int ln8411_probe(struct i2c_client *client, const struct i2c_device_id *i
 	return ret;
 }
 
-static int ln8411_remove(struct i2c_client *client)
+static void ln8411_remove(struct i2c_client *client)
 {
 	struct ln8411_device *ln8411 = i2c_get_clientdata(client);
 	int ret;
 
 	ret = ln8411_soft_reset(ln8411);
 	if (ret)
-		return ret;
+		dev_err(ln8411->dev, "Failed to reset: %d\n", ret);
 
 	mutex_destroy(&ln8411->lock);
-
-	return ret;
 }
 
 static const struct i2c_device_id ln8411_i2c_ids[] = {

Original file line number	Diff line number	Diff line change
`@@ -2369,18 +2369,16 @@ static int ln8411_probe(struct i2c_client client, const struct i2c_device_id i`
`2369`	`2369`	`return ret;`
`2370`	`2370`	`}`
`2371`	`2371`
`2372`		`-static int ln8411_remove(struct i2c_client *client)`
	`2372`	`+static void ln8411_remove(struct i2c_client *client)`
`2373`	`2373`	`{`
`2374`	`2374`	`struct ln8411_device *ln8411 = i2c_get_clientdata(client);`
`2375`	`2375`	`int ret;`
`2376`	`2376`
`2377`	`2377`	`ret = ln8411_soft_reset(ln8411);`
`2378`	`2378`	`if (ret)`
`2379`		`- return ret;`
	`2379`	`+ dev_err(ln8411->dev, "Failed to reset: %d\n", ret);`
`2380`	`2380`
`2381`	`2381`	`mutex_destroy(&ln8411->lock);`
`2382`		`-`
`2383`		`- return ret;`
`2384`	`2382`	`}`
`2385`	`2383`
`2386`	`2384`	`static const struct i2c_device_id ln8411_i2c_ids[] = {`