Re: [PATCH 1/5] ACPI processor: Do not export acpi_idle_driver in processor.h

[Thomas Renninger <trenn@suse.de>]

On Tuesday, January 17, 2012 12:25:08 PM Len Brown wrote:
> On 11/17/2011 05:36 PM, Thomas Renninger wrote:
> 
> > Instead move its declaration from processor_idle.c to processor_driver.c
> > and declare it static as it's the only file using this struct.
> > 
> > Cleanup only, no functional change.
> > 
> > Signed-off-by: Thomas Renninger <trenn@suse.de>
> > CC: Len Brown <lenb@kernel.org>
> > CC: linux-acpi@vger.kernel.org
> > ---
> >  drivers/acpi/processor_driver.c |    5 +++++
> >  drivers/acpi/processor_idle.c   |    5 -----
> >  include/acpi/processor.h        |    1 -
> >  3 files changed, 5 insertions(+), 6 deletions(-)
> > 
> 
> 
> drivers/acpi/processor_idle.c: In function ‘acpi_processor_setup_cpuidle_states’:
> drivers/acpi/processor_idle.c:1032:32: error: ‘acpi_idle_driver’ undeclared (first use in this function)
> drivers/acpi/processor_idle.c:1032:32: note: each undeclared identifier is reported only once for each function it appears in
> drivers/acpi/processor_idle.c: In function ‘acpi_processor_cst_has_changed’:
> drivers/acpi/processor_idle.c:1158:29: error: ‘acpi_idle_driver’ undeclared (first use in this function)
> drivers/acpi/processor_idle.c: In function ‘acpi_processor_power_init’:
> drivers/acpi/processor_idle.c:1239:38: error: ‘acpi_idle_driver’ undeclared (first use in this function)
> drivers/acpi/processor_idle.c: In function ‘acpi_processor_power_exit’:
> drivers/acpi/processor_idle.c:1270:31: error: ‘acpi_idle_driver’ undeclared (first use in this function)
> 
> eh?
Oh, this one does not work anymore since:
commit 46bcfad7a819bd17ac4e831b04405152d59784ab
Author: Deepthi Dharwar <deepthi@linux.vnet.ibm.com>
Date:   Fri Oct 28 16:20:42 2011 +0530

    cpuidle: Single/Global registration of idle states

got merged in recently.
Please ignore this patch.

   Thomas
--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html