iv/linux
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

cpufreq: acpi-cpufreq: Drop pointless label from acpi_cpufreq_target()

Browse Source 
The "out" label at the final return statement in acpi_cpufreq_target()
is totally pointless, so drop them and modify the code to return the
right values immediately instead of jumping to it.

No functional changes.

Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Reviewed-by: Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com>
Acked-by: Viresh Kumar <viresh.kumar@linaro.org>
This commit is contained in:
Rafael J. Wysocki 2016-02-26 00:03:58 +01:00
parent 6019d23a73
commit 9a909a142f
1 changed files with 2 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
drivers/cpufreq/acpi-cpufreq.c
View File

@ -434,7 +434,7 @@ static int acpi_cpufreq_target(struct cpufreq_policy *policy,
} else { } else {
pr_debug("Already at target state (P%d)\n", pr_debug("Already at target state (P%d)\n",
next_perf_state); next_perf_state);
goto out; return 0;
} }
} }
@ -456,8 +456,7 @@ static int acpi_cpufreq_target(struct cpufreq_policy *policy,
cmd.val = (u32) perf->states[next_perf_state].control; cmd.val = (u32) perf->states[next_perf_state].control;
break; break;
default: default:
result = -ENODEV; return -ENODEV;
goto out;
} }
/* cpufreq holds the hotplug lock, so we are safe from here on */ /* cpufreq holds the hotplug lock, so we are safe from here on */
@ -480,7 +479,6 @@ static int acpi_cpufreq_target(struct cpufreq_policy *policy,
if (!result) if (!result)
perf->state = next_perf_state; perf->state = next_perf_state;
out:
return result; return result;
} }