lkml.org 
[lkml]   [2021]   [Oct]   [4]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 5.10 38/93] hwmon: (tmp421) fix rounding for negative values
    Date
    From: Paul Fertser <fercerpav@gmail.com>

    [ Upstream commit 724e8af85854c4d3401313b6dd7d79cf792d8990 ]

    Old code produces -24999 for 0b1110011100000000 input in standard format due to
    always rounding up rather than "away from zero".

    Use the common macro for division, unify and simplify the conversion code along
    the way.

    Fixes: 9410700b881f ("hwmon: Add driver for Texas Instruments TMP421/422/423 sensor chips")
    Signed-off-by: Paul Fertser <fercerpav@gmail.com>
    Link: https://lore.kernel.org/r/20210924093011.26083-3-fercerpav@gmail.com
    Signed-off-by: Guenter Roeck <linux@roeck-us.net>
    Signed-off-by: Sasha Levin <sashal@kernel.org>
    ---
    drivers/hwmon/tmp421.c | 24 ++++++++----------------
    1 file changed, 8 insertions(+), 16 deletions(-)

    diff --git a/drivers/hwmon/tmp421.c b/drivers/hwmon/tmp421.c
    index c9ef83627bb7..b963a369c5ab 100644
    --- a/drivers/hwmon/tmp421.c
    +++ b/drivers/hwmon/tmp421.c
    @@ -100,23 +100,17 @@ struct tmp421_data {
    s16 temp[4];
    };

    -static int temp_from_s16(s16 reg)
    +static int temp_from_raw(u16 reg, bool extended)
    {
    /* Mask out status bits */
    int temp = reg & ~0xf;

    - return (temp * 1000 + 128) / 256;
    -}
    -
    -static int temp_from_u16(u16 reg)
    -{
    - /* Mask out status bits */
    - int temp = reg & ~0xf;
    -
    - /* Add offset for extended temperature range. */
    - temp -= 64 * 256;
    + if (extended)
    + temp = temp - 64 * 256;
    + else
    + temp = (s16)temp;

    - return (temp * 1000 + 128) / 256;
    + return DIV_ROUND_CLOSEST(temp * 1000, 256);
    }

    static int tmp421_update_device(struct tmp421_data *data)
    @@ -172,10 +166,8 @@ static int tmp421_read(struct device *dev, enum hwmon_sensor_types type,

    switch (attr) {
    case hwmon_temp_input:
    - if (tmp421->config & TMP421_CONFIG_RANGE)
    - *val = temp_from_u16(tmp421->temp[channel]);
    - else
    - *val = temp_from_s16(tmp421->temp[channel]);
    + *val = temp_from_raw(tmp421->temp[channel],
    + tmp421->config & TMP421_CONFIG_RANGE);
    return 0;
    case hwmon_temp_fault:
    /*
    --
    2.33.0


    \
     
     \ /
      Last update: 2021-10-04 15:23    [W:5.803 / U:0.360 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site