diff options
| author | Rohit Gupta <rohgup@codeaurora.org> | 2015-12-10 14:53:34 -0800 |
|---|---|---|
| committer | David Keitel <dkeitel@codeaurora.org> | 2016-03-23 20:04:44 -0700 |
| commit | 9c9b504f066294fd18fdb68dec2e6bb44de57565 (patch) | |
| tree | a0386e9aa37918510baa5bcaeb006187705fb186 /drivers/devfreq | |
| parent | 44cac5da6ad74f73e81c9e0444bcb5b9a07c32db (diff) | |
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 <rohgup@codeaurora.org>
Diffstat (limited to 'drivers/devfreq')
| -rw-r--r-- | drivers/devfreq/governor_memlat.c | 4 |
1 files 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; \ |