[lm-sensors] [PATCH] coretemp driver and Celeron 430 - works with minor modification

[Rudolf Marek]

Hello Jan,

Sorry for the delay, I'm busy with other stuff too.

Thank you for the report, I check the revision guide and everything seems ok. I
would like you to test attach patch. I think it does only what you did + updates
the documentation.

Please can you test that patch?

Signed-off-by: Rudolf Marek <r.marek at assembler.cz>

This patch adds support for the Celeron 4xx based on Core 2 core.

Thanks,

Rudolf




Jan Richling wrote:
> Hi,
> 
> I just tested your coretemp driver on a Celeron 430 (Core 2 based) and
> got unknown CPU model 16 which is triggered by the condition that you
> only accept model 0xe or 0xf as valid devices. The Celeron 4xx is model
> 22 (0x16) so I patched the driver by adding (|| (c->x86_model == 0x16)).
> This works out of the box with no further modifications and shows
> temperatures that are equal to those displayed by a Windows utility on
> the same machine.
> 
> So I can tell you that your driver works for family 6, model 22 also.
> 
> Just to be complete here is the cpuinfo:
> 
> cat /proc/cpuinfo
> processor       : 0
> vendor_id       : GenuineIntel
> cpu family      : 6
> model           : 22
> model name      : Intel(R) Celeron(R) CPU          430  @ 1.80GHz
> stepping        : 1
> cpu MHz         : 2400.147
> cache size      : 512 KB
> fpu             : yes
> fpu_exception   : yes
> cpuid level     : 10
> wp              : yes
> flags           : fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge
> mca cmov pat pse36 clflush dts acpi mmx fxsr sse sse2 ss tm syscall lm
> constant_tsc pni monitor ds_cpl tm2 ssse3 cx16 xtpr lahf_lm
> bogomips        : 4802.79
> clflush size    : 64
> cache_alignment : 64
> address sizes   : 36 bits physical, 48 bits virtual
> power management:
> 
> (it shows 2400 MHz due to FSB-mod (200->266) and the resulting
> overclock, stock is 9x200=1,8 GHz)
> 
> Greetings and many thanks for the driver,
> 
> Jan
> 
>    ------------------------------------------------------------------
>   / Dr. Jan Richling - http://www.richling.de                      /
>  / Member of Computer Architecture and Communication Group -      /
> / Department of Computer Science - Humboldt University Berlin    /
-------------- next part --------------
A non-text attachment was scrubbed...
Name: add_support_for_sc_celerons.patch
Type: text/x-diff
Size: 1367 bytes
Desc: not available
Url : http://lists.lm-sensors.org/pipermail/lm-sensors/attachments/20071007/3a003630/attachment.bin