public inbox for linux-acpi@vger.kernel.org
 help / color / mirror / Atom feed
* [patch 8/8] acpi-cpufreq: remove dead code
@ 2006-10-20 21:30 akpm
  2006-10-21  5:28 ` Len Brown
  0 siblings, 1 reply; 3+ messages in thread
From: akpm @ 2006-10-20 21:30 UTC (permalink / raw)
  To: len.brown; +Cc: linux-acpi, akpm, lenb

From: Andrew Morton <akpm@osdl.org>

arch/i386/kernel/cpu/cpufreq/speedstep-centrino.c:396: warning: 'sw_any_bug_dmi_table' defined but not used

Cc: Len Brown <lenb@kernel.org>
Signed-off-by: Andrew Morton <akpm@osdl.org>
---

 arch/i386/kernel/cpu/cpufreq/acpi-cpufreq.c |    2 ++
 1 files changed, 2 insertions(+)

diff -puN arch/i386/kernel/cpu/cpufreq/acpi-cpufreq.c~acpi-cpufreq-remove-dead-code arch/i386/kernel/cpu/cpufreq/acpi-cpufreq.c
--- a/arch/i386/kernel/cpu/cpufreq/acpi-cpufreq.c~acpi-cpufreq-remove-dead-code
+++ a/arch/i386/kernel/cpu/cpufreq/acpi-cpufreq.c
@@ -402,6 +402,7 @@ static int sw_any_bug_found(struct dmi_s
 	return 0;
 }
 
+#ifdef CONFIG_SMP
 static struct dmi_system_id sw_any_bug_dmi_table[] = {
 	{
 		.callback = sw_any_bug_found,
@@ -414,6 +415,7 @@ static struct dmi_system_id sw_any_bug_d
 	},
 	{ }
 };
+#endif
 
 static int
 acpi_cpufreq_cpu_init (
_

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [patch 8/8] acpi-cpufreq: remove dead code
  2006-10-20 21:30 [patch 8/8] acpi-cpufreq: remove dead code akpm
@ 2006-10-21  5:28 ` Len Brown
  2006-10-21  5:31   ` Dave Jones
  0 siblings, 1 reply; 3+ messages in thread
From: Len Brown @ 2006-10-21  5:28 UTC (permalink / raw)
  To: akpm; +Cc: linux-acpi

I would expect to see this same change in both speedstep-centrino.c and acpi-cpufreq.c, no?

On Friday 20 October 2006 17:30, akpm@osdl.org wrote:
> From: Andrew Morton <akpm@osdl.org>
> 
> arch/i386/kernel/cpu/cpufreq/speedstep-centrino.c:396: warning: 'sw_any_bug_dmi_table' defined but not used
> 
> Cc: Len Brown <lenb@kernel.org>
> Signed-off-by: Andrew Morton <akpm@osdl.org>
> ---
> 
>  arch/i386/kernel/cpu/cpufreq/acpi-cpufreq.c |    2 ++
>  1 files changed, 2 insertions(+)
> 
> diff -puN arch/i386/kernel/cpu/cpufreq/acpi-cpufreq.c~acpi-cpufreq-remove-dead-code arch/i386/kernel/cpu/cpufreq/acpi-cpufreq.c
> --- a/arch/i386/kernel/cpu/cpufreq/acpi-cpufreq.c~acpi-cpufreq-remove-dead-code
> +++ a/arch/i386/kernel/cpu/cpufreq/acpi-cpufreq.c
> @@ -402,6 +402,7 @@ static int sw_any_bug_found(struct dmi_s
>  	return 0;
>  }
>  
> +#ifdef CONFIG_SMP
>  static struct dmi_system_id sw_any_bug_dmi_table[] = {
>  	{
>  		.callback = sw_any_bug_found,
> @@ -414,6 +415,7 @@ static struct dmi_system_id sw_any_bug_d
>  	},
>  	{ }
>  };
> +#endif
>  
>  static int
>  acpi_cpufreq_cpu_init (
> _
> 

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [patch 8/8] acpi-cpufreq: remove dead code
  2006-10-21  5:28 ` Len Brown
@ 2006-10-21  5:31   ` Dave Jones
  0 siblings, 0 replies; 3+ messages in thread
From: Dave Jones @ 2006-10-21  5:31 UTC (permalink / raw)
  To: Len Brown; +Cc: akpm, linux-acpi

On Sat, Oct 21, 2006 at 01:28:06AM -0400, Len Brown wrote:
 > I would expect to see this same change in both speedstep-centrino.c and acpi-cpufreq.c, no?

Indeed, especially as longterm acpi-cpufreq is going to be the
more interesting of the two.

I'll merge this patch up though, and make the acpi-cpufreq change myself.

Thanks,

	Dave
 
-- 
http://www.codemonkey.org.uk

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2006-10-21  5:32 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2006-10-20 21:30 [patch 8/8] acpi-cpufreq: remove dead code akpm
2006-10-21  5:28 ` Len Brown
2006-10-21  5:31   ` Dave Jones

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox