From: Jean Delvare Date: Mon, 23 Jun 2008 08:14:26 +0000 (+0200) Subject: hwmon: (lm85) Fix function RANGE_TO_REG() X-Git-Tag: v2.6.25.9~3 X-Git-Url: https://git.fsl.cs.sunysb.edu/?a=commitdiff_plain;h=73377b7ebafbb69016a6cf4342b65c560a85342e;p=unionfs-2.6.39.y.git hwmon: (lm85) Fix function RANGE_TO_REG() Function RANGE_TO_REG() is broken. For a requested range of 2000 (2 degrees C), it will return an index value of 15, i.e. 80.0 degrees C, instead of the expected index value of 0. All other values are handled properly, just 2000 isn't. The bug was introduced back in November 2004 by this patch: http://git.kernel.org/?p=linux/kernel/git/tglx/history.git;a=commit;h=1c28d80f1992240373099d863e4996cdd5d646d0 In Linus' kernel I decided to rewrite the whole function in a way which was more obviously correct. But for -stable let's just do the minimal fix. Signed-off-by: Jean Delvare Signed-off-by: Greg Kroah-Hartman --- diff --git a/drivers/hwmon/lm85.c b/drivers/hwmon/lm85.c index 182fe6a5605..30bdd508ea9 100644 --- a/drivers/hwmon/lm85.c +++ b/drivers/hwmon/lm85.c @@ -192,7 +192,7 @@ static int RANGE_TO_REG( int range ) { int i; - if ( range < lm85_range_map[0] ) { + if (range <= lm85_range_map[0]) { return 0 ; } else if ( range > lm85_range_map[15] ) { return 15 ;