diff options
| author | Guenter Roeck <linux@roeck-us.net> | 2012-09-11 13:34:29 -0700 |
|---|---|---|
| committer | Guenter Roeck <linux@roeck-us.net> | 2012-09-23 21:08:35 -0700 |
| commit | 860f37319e9a1141b041ca2e77ca121dccde4b93 (patch) | |
| tree | 55c1d127794cd5d6a3466b904c27cabab5af6c82 /drivers/hwmon | |
| parent | cc00e4ddbf00d5ee6d8b7df6d4b04d98f3d99af5 (diff) | |
| download | talos-op-linux-860f37319e9a1141b041ca2e77ca121dccde4b93.tar.gz talos-op-linux-860f37319e9a1141b041ca2e77ca121dccde4b93.zip | |
hwmon: (lm70) Simplify show_name function
Instead of using a switch statement to determine the device name, use
to_spi_device(dev)->modalias to simplify the code and reduce module size.
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
Diffstat (limited to 'drivers/hwmon')
| -rw-r--r-- | drivers/hwmon/lm70.c | 21 |
1 files changed, 1 insertions, 20 deletions
diff --git a/drivers/hwmon/lm70.c b/drivers/hwmon/lm70.c index 789753d0df79..2d1777a03edb 100644 --- a/drivers/hwmon/lm70.c +++ b/drivers/hwmon/lm70.c @@ -124,26 +124,7 @@ static DEVICE_ATTR(temp1_input, S_IRUGO, lm70_sense_temp, NULL); static ssize_t lm70_show_name(struct device *dev, struct device_attribute *devattr, char *buf) { - struct lm70 *p_lm70 = dev_get_drvdata(dev); - int ret; - - switch (p_lm70->chip) { - case LM70_CHIP_LM70: - ret = sprintf(buf, "lm70\n"); - break; - case LM70_CHIP_TMP121: - ret = sprintf(buf, "tmp121\n"); - break; - case LM70_CHIP_LM71: - ret = sprintf(buf, "lm71\n"); - break; - case LM70_CHIP_LM74: - ret = sprintf(buf, "lm74\n"); - break; - default: - ret = -EINVAL; - } - return ret; + return sprintf(buf, "%s\n", to_spi_device(dev)->modalias); } static DEVICE_ATTR(name, S_IRUGO, lm70_show_name, NULL); |
