power: supply: ab8500: Rename charging algorithm psy
authorLinus Walleij <linus.walleij@linaro.org>
Tue, 13 Jul 2021 15:27:08 +0000 (17:27 +0200)
committerSebastian Reichel <sebastian.reichel@collabora.com>
Fri, 16 Jul 2021 13:14:30 +0000 (15:14 +0200)
If we rename the "abx500_chargalg" supply to "ab8500_chargalg"
as it should be named, the existing supplies are supplying that
supply but that was obviously not working since it had the
wrong name.

Now that the dependency kicks in we get a bunch
of NULL references from ab8500_chargalg_external_power_changed()
so check that the workqueue is allocated before we try to
queue work on it.

Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
Signed-off-by: Sebastian Reichel <sebastian.reichel@collabora.com>
drivers/power/supply/ab8500_chargalg.c

index 8dd66eb18fd5e5d7235494a47b9dcadde6462d7b..4ef34f64e9c034b4b43da96555e7e1ee2d633910 100644 (file)
@@ -1275,7 +1275,8 @@ static void ab8500_chargalg_external_power_changed(struct power_supply *psy)
         * Trigger execution of the algorithm instantly and read
         * all power_supply properties there instead
         */
-       queue_work(di->chargalg_wq, &di->chargalg_work);
+       if (di->chargalg_wq)
+               queue_work(di->chargalg_wq, &di->chargalg_work);
 }
 
 /**
@@ -1949,7 +1950,7 @@ static char *supply_interface[] = {
 };
 
 static const struct power_supply_desc ab8500_chargalg_desc = {
-       .name                   = "abx500_chargalg",
+       .name                   = "ab8500_chargalg",
        .type                   = POWER_SUPPLY_TYPE_BATTERY,
        .properties             = ab8500_chargalg_props,
        .num_properties         = ARRAY_SIZE(ab8500_chargalg_props),