Re: patch: p4tcc and speedstep cpufreq drivers

On Thu, Feb 17, 2005 at 04:33:45PM -0800, Nate Lawson wrote:
> Pawel Worach wrote:
> >Nate Lawson wrote:
> >
> >>Attached is a patch that I'd like to get tested.  After applying it,
> >>rebuild and load the cpufreq.ko module.  Be sure you do _not_ have
> >>"options CPU_ENABLE_TCC" in your kernel config or the new driver will
> >>conflict with the old.
> >
> >
> >Hi Nate,
> >
> >This is what I get on a TP T41, do the cpufreq results below look right?
> >Also if I loaded both modules all I got for dev.cpu.0.freq was -1 or 
> >1700 and no levels.
> >
> >CPU: Intel(R) Pentium(R) M processor 1700MHz (1698.56-MHz 686-class CPU)
> >  Origin = "GenuineIntel"  Id = 0x695  Stepping = 5
> >Features=0xa7e9f9bf<FPU,VME,DE,PSE,TSC,MSR,MCE,CX8,SEP,MTRR,PGE,MCA,CMOV,PAT,CLFLUSH,DTS,ACPI,MMX,FXSR,SSE,SSE2,TM,PBE> 
> 
> Apologies.  I found 2 bugs, one was not calling cpufreq_register() and 
> the other was that the code to detect acpi_perf (in ichss and est) was 
> incorrect.  I've committed fixes for that and have updated the patch. 
> Please ues this version and test again.

Hi,

This is an TP X31.

CPU: Intel(R) Pentium(R) M processor 1400MHz (1398.82-MHz 686-class CPU)
  Origin = "GenuineIntel"  Id = 0x695  Stepping = 5
  Features=0xa7e9f9bf<FPU,VME,DE,PSE,TSC,MSR,MCE,CX8,SEP,MTRR,PGE,MCA,CMOV,PAT,C
LFLUSH,DTS,ACPI,MMX,FXSR,SSE,SSE2,TM,PBE>

freq_levels looks strange.. and changes...
(only cpufreq is loaded at boot)

rafan_at_woodstock|UTF-8 [~] (15:57)$ sysctl dev.cpu
dev.cpu.0.%desc: ACPI CPU (3 Cx states)
dev.cpu.0.%driver: cpu
dev.cpu.0.%location: handle=\_PR_.CPU_
dev.cpu.0.%pnpinfo: _HID=none _UID=0
dev.cpu.0.%parent: acpi0
dev.cpu.0.freq: 1399
dev.cpu.0.freq_levels: -738340620/0 0/0 0/0
rafan_at_woodstock|UTF-8 [~] (15:57)$ sysctl dev.cpu
dev.cpu.0.%desc: ACPI CPU (3 Cx states)
dev.cpu.0.%driver: cpu
dev.cpu.0.%location: handle=\_PR_.CPU_
dev.cpu.0.%pnpinfo: _HID=none _UID=0
dev.cpu.0.%parent: acpi0
dev.cpu.0.freq: 1399
dev.cpu.0.freq_levels: 1398/-1 1398/-1 0/0
rafan_at_woodstock|UTF-8 [~] (15:57)$ sysctl dev.cpu
dev.cpu.0.%desc: ACPI CPU (3 Cx states)
dev.cpu.0.%driver: cpu
dev.cpu.0.%location: handle=\_PR_.CPU_
dev.cpu.0.%pnpinfo: _HID=none _UID=0
dev.cpu.0.%parent: acpi0
dev.cpu.0.freq: 1399
dev.cpu.0.freq_levels: 1398/-1 1398/-1 0/0


Regarad,
Rong-En Fan