diff options
| author | Rafael J. Wysocki <[email protected]> | 2016-11-18 12:57:54 +0000 |
|---|---|---|
| committer | Rafael J. Wysocki <[email protected]> | 2016-11-21 13:35:42 +0000 |
| commit | bca5f557dcea50a32ba789c4b4a438af7f49a61f (patch) | |
| tree | ea677e69429bf284dcf59cfe65cb602d0cd5bd01 /drivers/acpi/processor_perflib.c | |
| parent | cpufreq: Avoid using inactive policies (diff) | |
| download | kernel-bca5f557dcea50a32ba789c4b4a438af7f49a61f.tar.gz kernel-bca5f557dcea50a32ba789c4b4a438af7f49a61f.zip | |
ACPI / processor: Make acpi_processor_ppc_has_changed() void
The return value of acpi_processor_ppc_has_changed() is never used,
so make it void.
Signed-off-by: Rafael J. Wysocki <[email protected]>
Acked-by: Viresh Kumar <[email protected]>
Diffstat (limited to 'drivers/acpi/processor_perflib.c')
| -rw-r--r-- | drivers/acpi/processor_perflib.c | 10 |
1 files changed, 4 insertions, 6 deletions
diff --git a/drivers/acpi/processor_perflib.c b/drivers/acpi/processor_perflib.c index 9c67faa034b5..f0b4a981b8d3 100644 --- a/drivers/acpi/processor_perflib.c +++ b/drivers/acpi/processor_perflib.c @@ -157,7 +157,7 @@ static void acpi_processor_ppc_ost(acpi_handle handle, int status) status, NULL); } -int acpi_processor_ppc_has_changed(struct acpi_processor *pr, int event_flag) +void acpi_processor_ppc_has_changed(struct acpi_processor *pr, int event_flag) { int ret; @@ -168,7 +168,7 @@ int acpi_processor_ppc_has_changed(struct acpi_processor *pr, int event_flag) */ if (event_flag) acpi_processor_ppc_ost(pr->handle, 1); - return 0; + return; } ret = acpi_processor_get_platform_limit(pr); @@ -182,10 +182,8 @@ int acpi_processor_ppc_has_changed(struct acpi_processor *pr, int event_flag) else acpi_processor_ppc_ost(pr->handle, 0); } - if (ret < 0) - return (ret); - else - return cpufreq_update_policy(pr->id); + if (ret >= 0) + cpufreq_update_policy(pr->id); } int acpi_processor_get_bios_limit(int cpu, unsigned int *limit) |