lkml.org 
[lkml]   [2020]   [Dec]   [28]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 5.10 230/717] power: supply: max17042_battery: Fix current_{avg,now} hiding with no current sense
    Date
    From: Sebastian Krzyszkowiak <sebastian.krzyszkowiak@puri.sm>

    [ Upstream commit 4b0a56e692503692da6555337a697c17feabbb3e ]

    When current sense is disabled, max17042_no_current_sense_psy_desc gets
    used which ignores two last properties from the list.

    Fixes: 21b01cc879cc ("power: supply: max17042_battery: Add support for the TTE_NOW prop")
    Reported-by: Timon Baetz <timon.baetz@protonmail.com>
    Signed-off-by: Sebastian Krzyszkowiak <sebastian.krzyszkowiak@puri.sm>
    Signed-off-by: Sebastian Reichel <sebastian.reichel@collabora.com>
    Signed-off-by: Sasha Levin <sashal@kernel.org>
    ---
    drivers/power/supply/max17042_battery.c | 3 ++-
    1 file changed, 2 insertions(+), 1 deletion(-)

    diff --git a/drivers/power/supply/max17042_battery.c b/drivers/power/supply/max17042_battery.c
    index f284547913d6f..2e9672fe4df1f 100644
    --- a/drivers/power/supply/max17042_battery.c
    +++ b/drivers/power/supply/max17042_battery.c
    @@ -85,9 +85,10 @@ static enum power_supply_property max17042_battery_props[] = {
    POWER_SUPPLY_PROP_TEMP_MAX,
    POWER_SUPPLY_PROP_HEALTH,
    POWER_SUPPLY_PROP_SCOPE,
    + POWER_SUPPLY_PROP_TIME_TO_EMPTY_NOW,
    + // these two have to be at the end on the list
    POWER_SUPPLY_PROP_CURRENT_NOW,
    POWER_SUPPLY_PROP_CURRENT_AVG,
    - POWER_SUPPLY_PROP_TIME_TO_EMPTY_NOW,
    };

    static int max17042_get_temperature(struct max17042_chip *chip, int *temp)
    --
    2.27.0


    \
     
     \ /
      Last update: 2020-12-28 16:09    [W:2.336 / U:0.088 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site