diff options
author | roel kluin <roel.kluin@gmail.com> | 2009-01-17 21:03:47 -0500 |
---|---|---|
committer | Benjamin Herrenschmidt <benh@kernel.crashing.org> | 2009-02-11 00:00:07 -0500 |
commit | 2e74778c76521f180516a593fb2b2786d6fffa4e (patch) | |
tree | 4379758bac4a8dc904ecc6ade8339f16c3b034c8 | |
parent | c3bd517de67d33c44059656194e316facef181a5 (diff) |
therm_adt746x: Fix signed/unsigned confusion
As suggested, this is used for signed rather than unsigned
Signed-off-by: Roel Kluin <roel.kluin@gmail.com>
Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
-rw-r--r-- | drivers/macintosh/therm_adt746x.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/macintosh/therm_adt746x.c b/drivers/macintosh/therm_adt746x.c index 82607add69a9..c0621d50c8a0 100644 --- a/drivers/macintosh/therm_adt746x.c +++ b/drivers/macintosh/therm_adt746x.c | |||
@@ -498,8 +498,8 @@ static ssize_t store_##name(struct device *dev, struct device_attribute *attr, c | |||
498 | #define BUILD_STORE_FUNC_INT(name, data) \ | 498 | #define BUILD_STORE_FUNC_INT(name, data) \ |
499 | static ssize_t store_##name(struct device *dev, struct device_attribute *attr, const char *buf, size_t n) \ | 499 | static ssize_t store_##name(struct device *dev, struct device_attribute *attr, const char *buf, size_t n) \ |
500 | { \ | 500 | { \ |
501 | u32 val; \ | 501 | int val; \ |
502 | val = simple_strtoul(buf, NULL, 10); \ | 502 | val = simple_strtol(buf, NULL, 10); \ |
503 | if (val < 0 || val > 255) \ | 503 | if (val < 0 || val > 255) \ |
504 | return -EINVAL; \ | 504 | return -EINVAL; \ |
505 | printk(KERN_INFO "Setting specified fan speed to %d\n", val); \ | 505 | printk(KERN_INFO "Setting specified fan speed to %d\n", val); \ |