[PATCH] thermal: Fix check of unsigned in thermal_cooling_device_cur_state_store()

From: Roel Kluin
Date: Fri Oct 16 2009 - 09:27:19 EST


state is unsigned long so the test does not work.

Signed-off-by: Roel Kluin <roel.kluin@xxxxxxxxx>
---
Is this the right fix? please review.

diff --git a/drivers/thermal/thermal_sys.c b/drivers/thermal/thermal_sys.c
index 4e83c29..04970bf 100644
--- a/drivers/thermal/thermal_sys.c
+++ b/drivers/thermal/thermal_sys.c
@@ -374,7 +374,7 @@ thermal_cooling_device_cur_state_store(struct device *dev,
if (!sscanf(buf, "%ld\n", &state))
return -EINVAL;

- if (state < 0)
+ if ((long)state < 0)
return -EINVAL;

result = cdev->ops->set_cur_state(cdev, state);
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/