]> git.baikalelectronics.ru Git - kernel.git/commit
hwmon: (lm73} Detect and report i2c bus errors
authorChris Verges <kg4ysn@gmail.com>
Fri, 21 Dec 2012 09:58:34 +0000 (01:58 -0800)
committerGuenter Roeck <linux@roeck-us.net>
Sat, 22 Dec 2012 01:59:30 +0000 (17:59 -0800)
commit52ed796113c5c9bf4e84c244c7144bb1a6fe398f
treeb1a1522519696e7924c925478787e58a51b9ce53
parenta2e69c3c0f994da1cd5fbfdb8a9cfdb38b9f1ed5
hwmon: (lm73} Detect and report i2c bus errors

If an LM73 device does not exist on an I2C bus, attempts to communicate
with the device result in an error code returned from the i2c read/write
functions.  The current lm73 driver casts that return value from a s32
type to a s16 type, then converts it to a temperature in celsius.
Because negative temperatures are valid, it is difficult to distinguish
between an error code printed to the response buffer and a negative
temperature recorded by the sensor.

The solution is to evaluate the return value from the i2c functions
before performing any temperature calculations.  If the i2c function did
not succeed, the error code should be passed back through the virtual
file system layer instead of being printed into the response buffer.

Before:

   $ cat /sys/class/hwmon/hwmon0/device/temp1_input
   -46

After:

   $ cat /sys/class/hwmon/hwmon0/device/temp1_input
   cat: read error: No such device or address

Signed-off-by: Chris Verges <kg4ysn@gmail.com>
Cc: stable@vger.kernel.org
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
drivers/hwmon/lm73.c