Re: [PATCH] hwmon: (it87) Add support for IT8781F, IT8782F, IT8783E/F

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

 



HI Guenter,

Am 26.03.2012 um 21:05 schrieb Guenter Roeck:
> On Mon, 2012-03-26 at 12:08 -0400, Björn Gerhart wrote:
>> Hi Guenter,
>> 
>> Am 24.03.2012 um 19:07 schrieb Guenter Roeck:
>>> On Sat, Mar 24, 2012 at 12:23:13PM -0400, Guenter Roeck wrote:
>>>> Hi Bjoern,
>>>> 
>>>> At 01:11 AM 3/24/2012, Björn Gerhart wrote:
>>>> [ ... ]
>>>> 
>>>>>   label temp3 "CPU T."
>>>>>   compute temp3 100+@,-1*(100-@)
>>>> 
>>>> Is it possible that this is due to a bad sensor type configuration ?
>>>> 
>>>> Thermal diodes have a "current mode" and a 
>>>> "voltage mode", and if misconfigured (or 
>>>> miswired) the reported temperature difference is 
>>>> just about in that range. Difference in wiring, 
>>>> if I understand it correctly, is that the + pin 
>>>> is pulled high with a resistor in voltage mode, 
>>>> which is not the case in current mode.
>>>> 
>> As I have learned this morning from our hardware developers, neither "current" nor "voltage" mode is used with temp3, but digital "PECI" mode. In this case, temp3's value describes the _difference_ between the actual temperature and the CPU's maximum temperature (where the CPU will begin to throttle).
>> So actually, I think the static sensors.conf is the wrong place to specify this value, because different CPU models have different maximum temperatures. Therefore, dealing with 100 in our own sensors.conf (as implemented by now) is just a very rough inexact value.
>> 
>> Afaik, only temp3@IT8783 can be used in PECI mode, and in general other ITE sensor chip models can be driven in PECI mode also.
>> 
>> So, how could the PECI mode be handled on the driver side?
>> Detecting if it8783 is in PECI mode:
>> - Index register 0x2C, bit 6: set to 1 (Enable PECI)
>> - temp3 value is negative until CPU is not throttling
>> 
> Possibly also register 0x55 (ADC Temperature Extra Channel Enable
> Register), bit 7.
> 
> Can you check with your HW folks ?
> 
Your're right, checked that point with my HW colleagues this morning. They say when bit7 is 1 (as it is in our design), then the chip is in PECI resp. SST mode for getting digital temperatures @temp3.

Björn


_______________________________________________
lm-sensors mailing list
lm-sensors@xxxxxxxxxxxxxx
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors



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

  Powered by Linux