linux-kernel - Re: [PATCH v3 2/3] hwmon: tmp421: report /PVLD condition as fault

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20210924115958.GA2695448@roeck-us.net>
Date:   Fri, 24 Sep 2021 04:59:58 -0700
From:   Guenter Roeck <linux@...ck-us.net>
To:     Paul Fertser <fercerpav@...il.com>
Cc:     linux-hwmon@...r.kernel.org, Jean Delvare <jdelvare@...e.com>,
        linux-kernel@...r.kernel.org
Subject: Re: [PATCH v3 2/3] hwmon: tmp421: report /PVLD condition as fault

On Fri, Sep 24, 2021 at 12:30:10PM +0300, Paul Fertser wrote:
> For both local and remote sensors all the supported ICs can report an
> "undervoltage lockout" condition which means the conversion wasn't
> properly performed due to insufficient power supply voltage and so the
> measurement results can't be trusted.
> 
> Fixes: 9410700b881f ("hwmon: Add driver for Texas Instruments TMP421/422/423 sensor chips")
> Signed-off-by: Paul Fertser <fercerpav@...il.com>

Applied.

Thanks,
Guenter

> ---
> 
> Changes from v2:
>  - Add Fixes: tag
> 
> Changes from v1:
>  - Trivial rebase
> 
>  drivers/hwmon/tmp421.c | 9 +++------
>  1 file changed, 3 insertions(+), 6 deletions(-)
> 
> diff --git a/drivers/hwmon/tmp421.c b/drivers/hwmon/tmp421.c
> index e6b2b31d17c8..3a789f21188c 100644
> --- a/drivers/hwmon/tmp421.c
> +++ b/drivers/hwmon/tmp421.c
> @@ -182,10 +182,10 @@ static int tmp421_read(struct device *dev, enum hwmon_sensor_types type,
>  		return 0;
>  	case hwmon_temp_fault:
>  		/*
> -		 * The OPEN bit signals a fault. This is bit 0 of the temperature
> -		 * register (low byte).
> +		 * Any of OPEN or /PVLD bits indicate a hardware mulfunction
> +		 * and the conversion result may be incorrect
>  		 */
> -		*val = tmp421->temp[channel] & 0x01;
> +		*val = !!(tmp421->temp[channel] & 0x03);
>  		return 0;
>  	default:
>  		return -EOPNOTSUPP;
> @@ -198,9 +198,6 @@ static umode_t tmp421_is_visible(const void *data, enum hwmon_sensor_types type,
>  {
>  	switch (attr) {
>  	case hwmon_temp_fault:
> -		if (channel == 0)
> -			return 0;
> -		return 0444;
>  	case hwmon_temp_input:
>  		return 0444;
>  	default: