[3.8.y.z extended stable] Patch "hwmon: (emc1403) fix inverted store_hyst()" has been added to staging queue

Kamal Mostafa kamal at canonical.com
Mon Jun 23 21:17:31 UTC 2014


This is a note to let you know that I have just added a patch titled

    hwmon: (emc1403) fix inverted store_hyst()

to the linux-3.8.y-queue branch of the 3.8.y.z extended stable tree 
which can be found at:

 http://kernel.ubuntu.com/git?p=ubuntu/linux.git;a=shortlog;h=refs/heads/linux-3.8.y-queue

This patch is scheduled to be released in version 3.8.13.25.

If you, or anyone else, feels it should not be added to this tree, please 
reply to this email.

For more information about the 3.8.y.z tree, see
https://wiki.ubuntu.com/Kernel/Dev/ExtendedStable

Thanks.
-Kamal

------

>From 5edef2fe30412ceeaa6dff3f23afeaecb2f9f186 Mon Sep 17 00:00:00 2001
From: Josef Gajdusek <atx at atx.name>
Date: Sun, 11 May 2014 14:40:44 +0200
Subject: [PATCH 14/66] hwmon: (emc1403) fix inverted store_hyst()

commit 17c048fc4bd95efea208a1920f169547d8588f1f upstream.

Attempts to set the hysteresis value to a temperature below the target
limit fails with "write error: Numerical result out of range" due to
an inverted comparison.

Signed-off-by: Josef Gajdusek <atx at atx.name>
Reviewed-by: Jean Delvare <jdelvare at suse.de>
[Guenter Roeck: Updated headline and description]
Signed-off-by: Guenter Roeck <linux at roeck-us.net>
Signed-off-by: Kamal Mostafa <kamal at canonical.com>
---
 drivers/hwmon/emc1403.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/hwmon/emc1403.c b/drivers/hwmon/emc1403.c
index 142e1cb..62fba71 100644
--- a/drivers/hwmon/emc1403.c
+++ b/drivers/hwmon/emc1403.c
@@ -162,7 +162,7 @@ static ssize_t store_hyst(struct device *dev,
 	if (retval < 0)
 		goto fail;

-	hyst = val - retval * 1000;
+	hyst = retval * 1000 - val;
 	hyst = DIV_ROUND_CLOSEST(hyst, 1000);
 	if (hyst < 0 || hyst > 255) {
 		retval = -ERANGE;
--
1.9.1





More information about the kernel-team mailing list