undefined reference to `acpi_processor_register_performance

undefined reference to `acpi_processor_register_performance

[water modem]

2.6.7 compile error

   LD      .tmp_vmlinux1
arch/i386/kernel/built-in.o(.text+0xcfeb): In function 
`powernow_acpi_init':
: undefined reference to `acpi_processor_register_performance'
arch/i386/kernel/built-in.o(.text+0xd01c): In function 
`powernow_acpi_init':
: undefined reference to `acpi_processor_unregister_performance'
arch/i386/kernel/built-in.o(.exit.text+0x32): In function `powernow_exit':
: undefined reference to `acpi_processor_unregister_performance'
make: *** [.tmp_vmlinux1] Error 1
[root@enlaptop linux-2.6.7]#

Re: undefined reference to `acpi_processor_register_performance

[Randy.Dunlap]

On Thu, 17 Jun 2004 20:35:12 +0000 water modem wrote:

| 2.6.7 compile error
| 
|    LD      .tmp_vmlinux1
| arch/i386/kernel/built-in.o(.text+0xcfeb): In function 
| `powernow_acpi_init':
| : undefined reference to `acpi_processor_register_performance'
| arch/i386/kernel/built-in.o(.text+0xd01c): In function 
| `powernow_acpi_init':
| : undefined reference to `acpi_processor_unregister_performance'
| arch/i386/kernel/built-in.o(.exit.text+0x32): In function `powernow_exit':
| : undefined reference to `acpi_processor_unregister_performance'
| make: *** [.tmp_vmlinux1] Error 1
| [root@enlaptop linux-2.6.7]#

got a .config for this?  i'm having a little trouble duplicating it.

thanks,
--
~Randy

[PATCH] undefined reference to `acpi_processor_register_performance

[Randy.Dunlap]

Several CPU_FREQ options need ACPI_PROCESSOR interfaces
to build.

Signed-off-by: Randy Dunlap <rddunlap@osdl.org>


diffstat:=
 arch/i386/kernel/cpu/cpufreq/Kconfig |    6 +++---
 1 files changed, 3 insertions(+), 3 deletions(-)

diff -Naurp ./arch/i386/kernel/cpu/cpufreq/Kconfig~cpu_freqs ./arch/i386/kernel/cpu/cpufreq/Kconfig
--- ./arch/i386/kernel/cpu/cpufreq/Kconfig~cpu_freqs	2004-06-15 22:19:01.000000000 -0700
+++ ./arch/i386/kernel/cpu/cpufreq/Kconfig	2004-06-17 21:52:21.000000000 -0700
@@ -80,7 +80,7 @@ config X86_POWERNOW_K6
 
 config X86_POWERNOW_K7
 	tristate "AMD Mobile Athlon/Duron PowerNow!"
-	depends on CPU_FREQ_TABLE
+	depends on CPU_FREQ_TABLE && ACPI_PROCESSOR
 	help
 	  This adds the CPUFreq driver for mobile AMD K7 mobile processors.
 
@@ -90,7 +90,7 @@ config X86_POWERNOW_K7
 
 config X86_POWERNOW_K8
 	tristate "AMD Opteron/Athlon64 PowerNow!"
-	depends on CPU_FREQ && EXPERIMENTAL
+	depends on CPU_FREQ && ACPI_PROCESSOR && EXPERIMENTAL
 	help
 	  This adds the CPUFreq driver for mobile AMD Opteron/Athlon64 processors.
 
@@ -111,7 +111,7 @@ config X86_GX_SUSPMOD
 
 config X86_SPEEDSTEP_CENTRINO
 	tristate "Intel Enhanced SpeedStep"
-	depends on CPU_FREQ_TABLE
+	depends on CPU_FREQ_TABLE && ACPI_PROCESSOR
 	help
 	  This adds the CPUFreq driver for Enhanced SpeedStep enabled
 	  mobile CPUs.  This means Intel Pentium M (Centrino) CPUs.

Re: [PATCH] undefined reference to `acpi_processor_register_performance

[Dave Jones]

On Thu, Jun 17, 2004 at 09:58:12PM -0700, Randy.Dunlap wrote:
 > 
 > Several CPU_FREQ options need ACPI_PROCESSOR interfaces
 > to build.

Hmm, it should be optional. Ie, if you don't have ACPI enabled,
you should still be able to use these drivers (the k7 one at least),
you just won't be able to use the ACPI fallback.

		Dave

Re: [PATCH] undefined reference to `acpi_processor_register_performance

[Randy.Dunlap]

On Fri, 18 Jun 2004 12:25:49 +0100 Dave Jones wrote:

| On Thu, Jun 17, 2004 at 09:58:12PM -0700, Randy.Dunlap wrote:
|  > 
|  > Several CPU_FREQ options need ACPI_PROCESSOR interfaces
|  > to build.
| 
| Hmm, it should be optional. Ie, if you don't have ACPI enabled,
| you should still be able to use these drivers (the k7 one at least),
| you just won't be able to use the ACPI fallback.

Hm, I see what you mean, but how do we handle source files that
call an optional kernel API (depending on CONFIG_xyz)?

IOW, building a cpufreq driver in-kernel that wants to call
acpi_processor_register_performance(), where CONFIG_ACPI_PROCESSOR=m,
needs to be prevented....

--
~Randy