diff options
| author | Paul Fertser <[email protected]> | 2021-09-24 12:30:09 +0300 |
|---|---|---|
| committer | Guenter Roeck <[email protected]> | 2021-09-24 04:59:10 -0700 |
| commit | 2938b2978a70d4cc10777ee71c9e512ffe4e0f4b (patch) | |
| tree | 018a9374d1c95e53b6c4ba1e53a877485f5eb9e9 /tools/perf/scripts/python/syscall-counts.py | |
| parent | e6fab7af6ba1bc77c78713a83876f60ca7a4a064 (diff) | |
hwmon: (tmp421) handle I2C errors
Function i2c_smbus_read_byte_data() can return a negative error number
instead of the data read if I2C transaction failed for whatever reason.
Lack of error checking can lead to serious issues on production
hardware, e.g. errors treated as temperatures produce spurious critical
temperature-crossed-threshold errors in BMC logs for OCP server
hardware. The patch was tested with Mellanox OCP Mezzanine card
emulating TMP421 protocol for temperature sensing which sometimes leads
to I2C protocol error during early boot up stage.
Fixes: 9410700b881f ("hwmon: Add driver for Texas Instruments TMP421/422/423 sensor chips")
Cc: [email protected]
Signed-off-by: Paul Fertser <[email protected]>
Link: https://lore.kernel.org/r/[email protected]
[groeck: dropped unnecessary line breaks]
Signed-off-by: Guenter Roeck <[email protected]>
Diffstat (limited to 'tools/perf/scripts/python/syscall-counts.py')
0 files changed, 0 insertions, 0 deletions