From 9c9b504f066294fd18fdb68dec2e6bb44de57565 Mon Sep 17 00:00:00 2001 From: Rohit Gupta Date: Thu, 10 Dec 2015 14:53:34 -0800 Subject: [PATCH] PM / devfreq: Fix the return value check in memlat governor The return value of kstrtouint is erroneously checked while setting the tunables for mem_latency governor due to which the tunables cannot be changed from their default values. This change rectifies that behavior. Change-Id: Ief7dda4638ede2c97b26229f1188a1559b238920 Signed-off-by: Rohit Gupta --- drivers/devfreq/governor_memlat.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/devfreq/governor_memlat.c b/drivers/devfreq/governor_memlat.c index 90c92f38053c..2851e9935fab 100644 --- a/drivers/devfreq/governor_memlat.c +++ b/drivers/devfreq/governor_memlat.c @@ -70,8 +70,8 @@ static ssize_t store_##name(struct device *dev, \ int ret; \ unsigned int val; \ ret = kstrtouint(buf, 10, &val); \ - if (ret != 1) \ - return -EINVAL; \ + if (ret) \ + return ret; \ val = max(val, _min); \ val = min(val, _max); \ hw->name = val; \