Why not simply check acpi_disabled, like other drivers do?
thanks,
-Len
On Monday 05 February 2007 19:09, akpm@xxxxxxxxxxxxxxxxxxxx wrote:
> From: Thomas Renninger <trenn@xxxxxxx>
>
> Signed-off-by: Thomas Renninger <trenn@xxxxxxx>
> Cc: Len Brown <lenb@xxxxxxxxxx>
> Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>
> ---
>
> drivers/acpi/processor_core.c | 13 +++++++++----
> 1 file changed, 9 insertions(+), 4 deletions(-)
>
> diff -puN drivers/acpi/processor_core.c~exit-acpi-processor-module-gracefully-if-acpi-is-disabled drivers/acpi/processor_core.c
> --- a/drivers/acpi/processor_core.c~exit-acpi-processor-module-gracefully-if-acpi-is-disabled
> +++ a/drivers/acpi/processor_core.c
> @@ -994,6 +994,8 @@ void acpi_processor_uninstall_hotplug_no
> * ACPI, but needs symbols from this driver
> */
>
> +static int processor_driver_registered;
> +
> static int __init acpi_processor_init(void)
> {
> int result = 0;
> @@ -1019,6 +1021,8 @@ static int __init acpi_processor_init(vo
> return result;
> }
>
> + processor_driver_registered = 1;
> +
> acpi_processor_install_hotplug_notify();
>
> acpi_thermal_cpufreq_init();
> @@ -1035,12 +1039,13 @@ static void __exit acpi_processor_exit(v
>
> acpi_thermal_cpufreq_exit();
>
> - acpi_processor_uninstall_hotplug_notify();
> -
> - acpi_bus_unregister_driver(&acpi_processor_driver);
> + if (processor_driver_registered) {
> + acpi_processor_uninstall_hotplug_notify();
>
> - remove_proc_entry(ACPI_PROCESSOR_CLASS, acpi_root_dir);
> + acpi_bus_unregister_driver(&acpi_processor_driver);
>
> + remove_proc_entry(ACPI_PROCESSOR_CLASS, acpi_root_dir);
> + }
> return;
> }
>
> _
> -
> To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
-
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
