diff options
| author | Linus Torvalds <torvalds@linux-foundation.org> | 2015-02-26 18:23:02 -0500 |
|---|---|---|
| committer | Linus Torvalds <torvalds@linux-foundation.org> | 2015-02-26 18:23:02 -0500 |
| commit | 4f671fe2f9523a1ea206f63fe60a7c7b3a56d5c7 (patch) | |
| tree | 0ee267a1b02f14040009fb337ff35542482626a5 | |
| parent | ed42e71accf63f47ac49e1d75760f5389c4b8529 (diff) | |
| parent | d9ef72cd1c15c2fc1f7131cd78304dd21bc15953 (diff) | |
Merge tag 'hwmon-for-linus-v4.0-rc2' of git://git.kernel.org/pub/scm/linux/kernel/git/groeck/linux-staging
Pull hwmon fix from Guenter Roeck:
"Add missing return value check to ads7828 driver"
* tag 'hwmon-for-linus-v4.0-rc2' of git://git.kernel.org/pub/scm/linux/kernel/git/groeck/linux-staging:
hwmon: (ads7828) Check return value of devm_regmap_init_i2c
| -rw-r--r-- | drivers/hwmon/ads7828.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/drivers/hwmon/ads7828.c b/drivers/hwmon/ads7828.c index bce4e9ff21bf..6c99ee7bafa3 100644 --- a/drivers/hwmon/ads7828.c +++ b/drivers/hwmon/ads7828.c | |||
| @@ -147,6 +147,9 @@ static int ads7828_probe(struct i2c_client *client, | |||
| 147 | &ads2830_regmap_config); | 147 | &ads2830_regmap_config); |
| 148 | } | 148 | } |
| 149 | 149 | ||
| 150 | if (IS_ERR(data->regmap)) | ||
| 151 | return PTR_ERR(data->regmap); | ||
| 152 | |||
| 150 | data->cmd_byte = ext_vref ? ADS7828_CMD_PD1 : ADS7828_CMD_PD3; | 153 | data->cmd_byte = ext_vref ? ADS7828_CMD_PD1 : ADS7828_CMD_PD3; |
| 151 | if (!diff_input) | 154 | if (!diff_input) |
| 152 | data->cmd_byte |= ADS7828_CMD_SD_SE; | 155 | data->cmd_byte |= ADS7828_CMD_SD_SE; |
