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@xxxxxxxxxxxx> > --- > 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@xxxxxxxxxxxx> _______________________________________________ lm-sensors mailing list lm-sensors@xxxxxxxxxxxxxx http://lists.lm-sensors.org/mailman/listinfo/lm-sensors
