On Fri, 2007-02-09 at 00:47 -0500, Len Brown wrote:
> Why not simply check acpi_disabled, like other drivers do?
> 
See comment from Dominik when I sent something similar some time ago:
http://marc.theaimsgroup.com/?l=linux-acpi&m=114563882012355&w=2

     Thomas

> thanks,
> -Len
> 
> On Monday 05 February 2007 19:09, [EMAIL PROTECTED] wrote:
> > From: Thomas Renninger <[EMAIL PROTECTED]>
> > 
> > Signed-off-by: Thomas Renninger <[EMAIL PROTECTED]>
> > Cc: Len Brown <[EMAIL PROTECTED]>
> > Signed-off-by: Andrew Morton <[EMAIL PROTECTED]>
> > ---
> > 
> >  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 [EMAIL PROTECTED]
> > 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 [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to