diff options
| author | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2013-10-25 10:22:47 -0400 |
|---|---|---|
| committer | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2013-10-25 10:22:47 -0400 |
| commit | 75c0758137c7ac647927b4b12bb5cfca96a0e4e6 (patch) | |
| tree | 9f40f1e9668e37d853e2ce29c930c39927b03607 | |
| parent | d3c345dbc7c083414ef74eb22ff26ba2bd100759 (diff) | |
acpi-cpufreq: Fail initialization if driver cannot be registered
Make acpi_cpufreq_init() return error codes when the driver cannot be
registered so that the module doesn't stay useless in memory and so
that acpi_cpufreq_exit() doesn't attempt to unregister things that
have never been registered when the module is unloaded.
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Acked-by: Viresh Kumar <viresh.kumar@linaro.org>
| -rw-r--r-- | drivers/cpufreq/acpi-cpufreq.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/drivers/cpufreq/acpi-cpufreq.c b/drivers/cpufreq/acpi-cpufreq.c index d2c3253e015e..506fd23c7550 100644 --- a/drivers/cpufreq/acpi-cpufreq.c +++ b/drivers/cpufreq/acpi-cpufreq.c | |||
| @@ -986,12 +986,12 @@ static int __init acpi_cpufreq_init(void) | |||
| 986 | { | 986 | { |
| 987 | int ret; | 987 | int ret; |
| 988 | 988 | ||
| 989 | if (acpi_disabled) | ||
| 990 | return -ENODEV; | ||
| 991 | |||
| 989 | /* don't keep reloading if cpufreq_driver exists */ | 992 | /* don't keep reloading if cpufreq_driver exists */ |
| 990 | if (cpufreq_get_current_driver()) | 993 | if (cpufreq_get_current_driver()) |
| 991 | return 0; | 994 | return -EEXIST; |
| 992 | |||
| 993 | if (acpi_disabled) | ||
| 994 | return 0; | ||
| 995 | 995 | ||
| 996 | pr_debug("acpi_cpufreq_init\n"); | 996 | pr_debug("acpi_cpufreq_init\n"); |
| 997 | 997 | ||
