From: Dan Carpenter <dan.carpenter@oracle.com>
Date: Sat, 19 Oct 2013 09:12:22 +0000 (+0300)
Subject: hwmon: (w83793) Clean up a signedness issue
X-Git-Tag: firefly_0821_release~176^2~5013^2
X-Git-Url: http://demsky.eecs.uci.edu/git/?a=commitdiff_plain;h=26336c8a36c0a6a28b9ecf6f1bb8c8f5605d6a21;p=firefly-linux-kernel-4.4.55.git

hwmon: (w83793) Clean up a signedness issue

We cap the upper bound of "mtimeout" but since it's signed we should
check for negative values as well.  The mistake is harmless.  But I have
changed it to unsigned as a cleanup.

Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
---

diff --git a/drivers/hwmon/w83793.c b/drivers/hwmon/w83793.c
index b6470ecda8fc..9d63d71214ca 100644
--- a/drivers/hwmon/w83793.c
+++ b/drivers/hwmon/w83793.c
@@ -1199,7 +1199,8 @@ static void w83793_init_client(struct i2c_client *client)
 
 static int watchdog_set_timeout(struct w83793_data *data, int timeout)
 {
-	int ret, mtimeout;
+	unsigned int mtimeout;
+	int ret;
 
 	mtimeout = DIV_ROUND_UP(timeout, 60);