[lm-sensors] [PATCH 6/7] hwmon: (w83795) Clear intrusion alarm immediately

Guenter Roeck guenter.roeck at ericsson.com
Sun Nov 7 18:24:59 CET 2010


On Sun, Nov 07, 2010 at 10:47:19AM -0500, Jean Delvare wrote:
> When asked to clear the intrusion alarm, do so immediately. We have to
> invalidate the cache to make sure the new status will be read. But we
> also have to read from the status register once to clear the pending
> alarm, as writing to CLR_CHS surprising won't clear it automatically.
> 
> Signed-off-by: Jean Delvare <khali at linux-fr.org>
> ---
>  w83795.c |    4 ++++
>  1 file changed, 4 insertions(+)
> 
> --- a/drivers/hwmon/w83795.c	2010-11-06 22:37:55.000000000 +0100
> +++ b/drivers/hwmon/w83795.c	2010-11-06 22:40:03.000000000 +0100
> @@ -755,6 +755,10 @@ store_chassis_clear(struct device *dev,
>  	val = w83795_read(client, W83795_REG_CLR_CHASSIS);
>  	val |= 0x80;
>  	w83795_write(client, W83795_REG_CLR_CHASSIS, val);
> +
> +	/* Clear status and force cache refresh */
> +	w83795_read(client, W83795_REG_ALARM(5));

Won't that clear other (possibly unreported) alarms as well ?

If that is not a problem, 
	Acked-by: Guenter Roeck <guenter.roeck at ericsson.com>




More information about the lm-sensors mailing list