Test of k8temp

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hello Rudolf,

On Mon, 2009-07-06 at 16:16 +0200, Rudolf Marek wrote:
> Hi again,
> 
> 
>  >Anyway, In the patch, replace :
> 
>  >+       /* Some AMD 10H cpus reports Inaccurate Temperature Measurement */
>  >+       if ((temp < 21000) || (temp > 90000))
> 
>  >+       /* Some AMD 10H cpus reports Inaccurate Temperature Measurement */
>  >+       if ((temp < 10000) || (temp > 90000))
> 
> So, now you see that workaround on some arbitrary temperature simply does not 
> make sense.
> 
> Please don't add  support for fam10h CPUs. This is not a point of HW monitoring, 
> to rule out  arbitrary temperatures.
> 

I totally agree with you that's why I removed the support for 10h as per
your suggestion.

> I'm perfectly fine if you come up with a patch for fam 11h because the Errata 
> seems to have vanished...
> 

x86: hwmon/k8temp.c Add support for AMD 11H

http://git.kernel.org/?p=linux/kernel/git/x86/linux-2.6-tip.git;a=commit;h=318e6a08d340d853459a39b4104eab0334568427

is still waiting for your approval.

Thanks,
--
JSR




[Index of Archives]     [Linux Kernel]     [Linux Hardware Monitoring]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]

  Powered by Linux