diff options
author | Guenter Roeck <linux@roeck-us.net> | 2014-04-21 17:38:08 (GMT) |
---|---|---|
committer | Guenter Roeck <linux@roeck-us.net> | 2014-04-24 13:09:43 (GMT) |
commit | f75d72309192e52ef9a3efb390b1c4f408c142df (patch) | |
tree | 7ba34fb0c598fca268d0db9e9f3c3585e62c565f /drivers/hwmon | |
parent | b2e5411ee26bff1deefd0b2c7beec9c133632e65 (diff) | |
download | linux-f75d72309192e52ef9a3efb390b1c4f408c142df.tar.xz |
hwmon: (ltc2945) Don't crash the kernel unnecessarily
An implementation error should not crash the kernel if it is avoidable.
Replace BUG() with WARN_ONCE().
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
Diffstat (limited to 'drivers/hwmon')
-rw-r--r-- | drivers/hwmon/ltc2945.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/drivers/hwmon/ltc2945.c b/drivers/hwmon/ltc2945.c index c104cc3..c9cddf5 100644 --- a/drivers/hwmon/ltc2945.c +++ b/drivers/hwmon/ltc2945.c @@ -1,4 +1,4 @@ -/* + /* * Driver for Linear Technology LTC2945 I2C Power Monitor * * Copyright (c) 2014 Guenter Roeck @@ -314,8 +314,8 @@ static ssize_t ltc2945_reset_history(struct device *dev, reg = LTC2945_MAX_ADIN_H; break; default: - BUG(); - break; + WARN_ONCE(1, "Bad register: 0x%x\n", reg); + return -EINVAL; } /* Reset maximum */ ret = regmap_bulk_write(regmap, reg, buf_max, num_regs); |