power: supply: max17042_battery: Refactor max17042_external_power_changed()
authorHans de Goede <hdegoede@redhat.com>
Sat, 15 Apr 2023 16:07:33 +0000 (18:07 +0200)
committerSebastian Reichel <sre@kernel.org>
Mon, 8 May 2023 13:07:34 +0000 (15:07 +0200)
max17042_external_power_changed() is a wrapper around
"power_supply_changed(psy);" and it has the same prototype.

Remove it, replacing it with making the external_power_changed
callback directly point to power_supply_changed.

Cc: Marek Szyprowski <m.szyprowski@samsung.com>
Cc: Sebastian Krzyszkowiak <sebastian.krzyszkowiak@puri.sm>
Cc: Purism Kernel Team <kernel@puri.sm>
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Signed-off-by: Sebastian Reichel <sebastian.reichel@collabora.com>
drivers/power/supply/max17042_battery.c

index 89cabe8ed3b06edc9b3c755ec03f43ea12a5ee8c..6a2d00d95a39dea5ebaa0e700e6d72033ac4bd23 100644 (file)
@@ -499,11 +499,6 @@ static int max17042_property_is_writeable(struct power_supply *psy,
        return ret;
 }
 
-static void max17042_external_power_changed(struct power_supply *psy)
-{
-       power_supply_changed(psy);
-}
-
 static int max17042_write_verify_reg(struct regmap *map, u8 reg, u32 value)
 {
        int retries = 8;
@@ -1016,7 +1011,7 @@ static const struct power_supply_desc max17042_psy_desc = {
        .get_property   = max17042_get_property,
        .set_property   = max17042_set_property,
        .property_is_writeable  = max17042_property_is_writeable,
-       .external_power_changed = max17042_external_power_changed,
+       .external_power_changed = power_supply_changed,
        .properties     = max17042_battery_props,
        .num_properties = ARRAY_SIZE(max17042_battery_props),
 };