On Fri, 23 Jul 2010 17:59:02 +0200 florian@xxxxxxxxxxx wrote: > Move it under the same ifdef as its sole caller is. > > Signed-off-by: Florian Mickler <florian@xxxxxxxxxxx> > --- > drivers/acpi/processor_idle.c | 10 ++++++---- > 1 files changed, 6 insertions(+), 4 deletions(-) > > diff --git a/drivers/acpi/processor_idle.c b/drivers/acpi/processor_idle.c > index b1b3856..8525070 100644 > --- a/drivers/acpi/processor_idle.c > +++ b/drivers/acpi/processor_idle.c > @@ -80,10 +80,6 @@ module_param(nocst, uint, 0000); > static unsigned int latency_factor __read_mostly = 2; > module_param(latency_factor, uint, 0644); > > -static u64 us_to_pm_timer_ticks(s64 t) > -{ > - return div64_u64(t * PM_TIMER_FREQUENCY, 1000000); > -} > /* > * IBM ThinkPad R40e crashes mysteriously when going into C2 or C3. > * For now disable this. Probably a bug somewhere else. > @@ -685,6 +681,12 @@ static int acpi_processor_get_power_info(struct acpi_processor *pr) > } > > #ifdef CONFIG_ACPI_PROCFS > + > +static u64 us_to_pm_timer_ticks(s64 t) > +{ > + return div64_u64(t * PM_TIMER_FREQUENCY, 1000000); > +} > + > static int acpi_processor_power_seq_show(struct seq_file *seq, void *offset) > { > struct acpi_processor *pr = seq->private; Ah, I just saw the Patch from KOSAKI Motohiro ([patch 1/2] acpi: fix unused function warning) to the same effect. I should have checked the mailinglist before sending this :) Sorry for the noise, Flo -- 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