> -----Original Message----- > From: Zhang Rui [mailto:rui.zhang@xxxxxxxxx] > Sent: Tuesday, July 28, 2009 12:24 AM > To: Thomas Renninger > Cc: davej@xxxxxxxxxx; linux-acpi; cpufreq@xxxxxxxxxxxxxxx; > Len Brown; howl.nsp@xxxxxxxxx; Langsdorf, Mark > Subject: Re: [PATCH] powernow-k8: don't load powernow-k8 > driver whenacpi=off > > On Thu, 2009-07-23 at 18:09 +0800, Thomas Renninger wrote: > > On Thursday 23 July 2009 04:11:10 Zhang Rui wrote: > > > when acpi=off, powernow-k8 driver is still loaded, and fails with > > > a firmware bug warning message. > > > http://bugzilla.kernel.org/show_bug.cgi?id=13695#c5 > > > > > > this is misleading, > > > we should not load powernow-k8 driver when acpi=off. > > > http://bugzilla.kernel.org/show_bug.cgi?id=13695 > > > > Maybe this one is better: > > - It allows these special, deprecated PSB BIOS structure > > use in acpi=off case > > - It complains that powernow-k8 won't work without ACPI, instead > > of complaining about the BIOS missing _PSS ACPI structures > > (in acpi=off case). > > > > Compile tested on latest 2.6.31-rc2. > > > > Thomas > > > > ---- > > CPUFREQ: Avoid powernow-k8 BIOS complains when acpi=off > > > > - It still allows these special, deprecated PSB BIOS structure > > use in acpi=off case > > - It complains that powernow-k8 won't work without ACPI, instead > > of complaining about the BIOS missing _PSS ACPI structures > > (in acpi=off case). > > > > Signed-off-by: Thomas Renninger <trenn@xxxxxxx> > > Signed-off-by: Zhang Rui <rui.zhang@xxxxxxxxx> Signed-off-by: Mark Langsdorf <mark.langsdorf@xxxxxxx> > > --- > > arch/x86/kernel/cpu/cpufreq/powernow-k8.c | 6 +++++- > > 1 file changed, 5 insertions(+), 1 deletion(-) > > > > Index: > cpufreq_governor_cleanups/arch/x86/kernel/cpu/cpufreq/powernow-k8.c > > =================================================================== > > --- > cpufreq_governor_cleanups.orig/arch/x86/kernel/cpu/cpufreq/pow > ernow-k8.c > > +++ > cpufreq_governor_cleanups/arch/x86/kernel/cpu/cpufreq/powernow-k8.c > > @@ -1286,7 +1286,11 @@ static int __cpuinit powernowk8_cpu_init > > * an UP version, and is deprecated by AMD. > > */ > > if (num_online_cpus() != 1) { > > - printk_once(ACPI_PSS_BIOS_BUG_MSG); > > + if (acpi_disabled) > > + printk_once(KERN_WARN PFX > > + "ACPI disabled, exiting\n"); > > + else > > + printk_once(ACPI_PSS_BIOS_BUG_MSG); > > goto err_out; > > } > > if (pol->cpu != 0) { > > > > > -- 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
