Subject: Re: CVS commit: src/sys/arch (est stuff)
To: None <mason@primenet.com.au>
From: Juan RP <juan@xtrarom.org>
List: source-changes
Date: 09/03/2006 11:08:07
On Sun, 3 Sep 2006 09:01:12 +0000 (UTC)
Geoff Wing <mason@primenet.com.au> wrote:

> Juan Romero Pardines <xtraeme@netbsd.org> typed:
> : Module Name:	src
> : Committed By:	xtraeme
> : Date:		Sun Sep  3 06:49:57 UTC 2006
> :
> : Modified Files:
> : 	src/sys/arch/i386/i386: est.c identcpu.c
> : 	src/sys/arch/i386/include: cpu.h
> : 	src/sys/arch/x86/include: specialreg.h
> : Log Message:
> : Update the enhanced speedstep driver and sync the code with OpenBSD:
> 
> And if you don't have speedstep option (to get est.c) you'll get ``bus_clock''
> not defined in indentcpu.c
> 
> Please fix this.
> 
> Regards,
> Geoff

Thanks for catching it... does the following patch work for you?

Index: cpu.h
===================================================================
RCS file: /cvsroot/src/sys/arch/i386/include/cpu.h,v
retrieving revision 1.128
diff -u -p -r1.128 cpu.h
--- cpu.h       3 Sep 2006 06:49:57 -0000       1.128
+++ cpu.h       3 Sep 2006 09:07:19 -0000
@@ -39,6 +39,7 @@
 
 #ifdef _KERNEL
 #if defined(_KERNEL_OPT)
+#include "opt_enhanced_speedstep.h"
 #include "opt_multiprocessor.h"
 #include "opt_math_emulate.h"
 #include "opt_user_ldt.h"
@@ -351,7 +352,9 @@ void        i386_init_pcb_tss_ldt(struct cpu_in
 void   i386_proc0_tss_ldt_init(void);
 
 /* identcpu.c */
+#ifdef ENHANCED_SPEEDSTEP
 extern int bus_clock;
+#endif
 extern int tmx86_has_longrun;
 extern u_int crusoe_longrun;
 extern u_int crusoe_frequency;