From: Minsung Kim Date: Sun, 25 Aug 2013 10:23:34 +0000 (+0900) Subject: cpufreq: interactive: fix show_target_loads and show_above_hispeed_delay X-Git-Tag: firefly_0821_release~4090^2~503 X-Git-Url: http://demsky.eecs.uci.edu/git/?a=commitdiff_plain;h=cf0fad49d17cb8273ce555dd5b7afab67d7923bf;p=firefly-linux-kernel-4.4.55.git cpufreq: interactive: fix show_target_loads and show_above_hispeed_delay Remove a trailing whitespace from target_loads and above_hispeed_delay. Problem happens when user-space program tried to restore parameters that saved before changing parameters. In this case was returned error(EINVAL). Change-Id: I5a74e3824602cd6f2b74651adda5ec1b627e61e9 Signed-off-by: Minsung Kim --- diff --git a/drivers/cpufreq/cpufreq_interactive.c b/drivers/cpufreq/cpufreq_interactive.c index 90958fdd64dc..a66748e13b38 100644 --- a/drivers/cpufreq/cpufreq_interactive.c +++ b/drivers/cpufreq/cpufreq_interactive.c @@ -742,7 +742,7 @@ static ssize_t show_target_loads( ret += sprintf(buf + ret, "%u%s", target_loads[i], i & 0x1 ? ":" : " "); - ret += sprintf(buf + ret, "\n"); + ret += sprintf(buf + --ret, "\n"); spin_unlock_irqrestore(&target_loads_lock, flags); return ret; } @@ -785,7 +785,7 @@ static ssize_t show_above_hispeed_delay( ret += sprintf(buf + ret, "%u%s", above_hispeed_delay[i], i & 0x1 ? ":" : " "); - ret += sprintf(buf + ret, "\n"); + ret += sprintf(buf + --ret, "\n"); spin_unlock_irqrestore(&above_hispeed_delay_lock, flags); return ret; }