power: supply: core: Break capacity loop
authorLinus Walleij <linus.walleij@linaro.org>
Sun, 14 Nov 2021 23:12:07 +0000 (00:12 +0100)
committerSebastian Reichel <sebastian.reichel@collabora.com>
Mon, 15 Nov 2021 13:20:11 +0000 (14:20 +0100)
We should not go on looking for more capacity tables after
we realize we have looked at the last one in
power_supply_find_ocv2cap_table().

Fixes: 3afb50d7125b ("power: supply: core: Add some helpers to use the battery OCV capacity table")
Cc: Chunyan Zhang <chunyan.zhang@unisoc.com>
Cc: Baolin Wang <baolin.wang@linux.alibaba.com>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
Reviewed-by: Baolin Wang <baolin.wang@linux.alibaba.com>
Signed-off-by: Sebastian Reichel <sebastian.reichel@collabora.com>
drivers/power/supply/power_supply_core.c

index fc12a4f..6093754 100644 (file)
@@ -853,6 +853,10 @@ power_supply_find_ocv2cap_table(struct power_supply_battery_info *info,
                return NULL;
 
        for (i = 0; i < POWER_SUPPLY_OCV_TEMP_MAX; i++) {
+               /* Out of capacity tables */
+               if (!info->ocv_table[i])
+                       break;
+
                temp_diff = abs(info->ocv_temp[i] - temp);
 
                if (temp_diff < best_temp_diff) {