diff options
| author | Guenter Roeck <linux@roeck-us.net> | 2014-04-21 10:38:08 -0700 |
|---|---|---|
| committer | Guenter Roeck <linux@roeck-us.net> | 2014-04-24 06:09:43 -0700 |
| commit | f75d72309192e52ef9a3efb390b1c4f408c142df (patch) | |
| tree | 7ba34fb0c598fca268d0db9e9f3c3585e62c565f | |
| parent | b2e5411ee26bff1deefd0b2c7beec9c133632e65 (diff) | |
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>
| -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 c104cc32989d..c9cddf5f056b 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); |