From: "Björn Gerhart" <oss@extracloud.de>
To: lm-sensors@vger.kernel.org
Subject: Re: [lm-sensors] [PATCH] hwmon: (it87) Add support for IT8781F, IT8782F, IT8783E/F
Date: Sat, 24 Mar 2012 08:11:53 +0000 [thread overview]
Message-ID: <982E6473-EAC9-4FB7-9A0D-3CF70AB5D5AA@extracloud.de> (raw)
In-Reply-To: <1331180739-19470-1-git-send-email-linux@roeck-us.net>
Salut Jean,
Am 24.03.2012 um 08:05 schrieb Jean Delvare:
> Hallo Björn,
>
> On Sat, 24 Mar 2012 00:24:58 +0100, Björn Gerhart wrote:
>> Am 23.03.2012 um 22:02 schrieb Jean Delvare:
>>
>>> On Tue, 13 Mar 2012 19:43:15 +0100, Bjoern Gerhart wrote:
>>>> 2012/3/8 Guenter Roeck <linux@roeck-us.net>:
>>>>> Assume all three are compatible and have the same functionality.
>>>>>
>>>>> Signed-off-by: Guenter Roeck <linux@roeck-us.net>
>>>> Tested-by: Bjoern Gerhart <oss@extracloud.de>
>>>
>>> Care to share the output of "sensors" with us?
>>>
>> Sure. To be more precise, I tested with an IT8783 chip configured and wired by the hardware team at the company I'm working for on a proprietary mainboard. With the sensors.conf created therefore, the sensors output looks like this:
>>
>> [root@localhost ~]# LANG=C sensors
>> it8783-isa-0a10
>> Adapter: ISA adapter
>> CPU Core: +1.15 V (min = +0.70 V, max = +1.70 V)
>> +1.50V: +1.50 V (min = +1.10 V, max = +1.90 V)
>> +3.30V: +3.38 V (min = +2.70 V, max = +4.00 V)
>> +12.0V: +12.16 V (min = +10.69 V, max = +13.70 V)
>> -12.0V: -11.95 V (min = -13.68 V, max = -10.30 V)
>> +1.10V: +1.12 V (min = +0.90 V, max = +1.22 V)
>> VBat: +3.09 V
>> Fan1: 1662 RPM (min = 300 RPM)
>> PSU: 1245 RPM (min = 500 RPM)
>> Board T.: +30.0 C (low = +0.0 C, high = +55.0 C) sensor = thermal diode
>> CPU T.: +47.0 C (low = +0.0 C, high = +63.0 C) sensor = thermal diode
>>
>> coretemp-isa-0000
>> Adapter: ISA adapter
>> Core 0: +40.0 C (high = +86.0 C, crit = +100.0 C)
>> Core 1: +43.0 C (high = +86.0 C, crit = +100.0 C)
>>
>>
>> These are about the same values as the "Hardware Monitor" section at the BIOS setup shows also. So all values look reasonable.
>
> Thanks. Can we see the configuration file associated with this? What I
> am particularly interested in are in3, in7 and in8 (Vbat) as they are
> treated specially by the chip and the driver.
>
Ok. in3 gets ignored, maybe not wired on that design. in7 and in8 don't get computed.
chip "it8783-isa-0a10"
#=== voltages
label in0 "CPU Core"
compute in0 @, @
set in0_min 0.7
set in0_max 1.7
label in1 "+1.50V"
compute in1 @,@
set in1_min 1.1
set in1_max 1.9
label in2 "+3.30V"
compute in2 @, @
set in2_min 2.7
set in2_max 4.0
label in4 "+12.0V"
compute in4 ((30/10) +1)*@ , @/((30/10) +1)
set in4_min 10.7
set in4_max 13.7
label in6 "-12.0V"
compute in6 (1+232/56)*@ - 4.096*232/56, (@ + 4.096*232/56)/(1+232/56)
set in6_min -13.7
set in6_max -10.3
label in7 "+1.10V"
set in7_min 0.89
set in7_max 1.21
label in8 "VBat"
ignore in3
ignore in5
ignore vid
#=== temps
label temp2 "Board T."
set temp2_min 0
set temp2_max 55
label temp3 "CPU T."
compute temp3 100+@,-1*(100-@)
set temp3_min 0
set temp3_max 63
ignore temp1
ignore temp4
#=== fans
label fan1 "Fan1"
set fan1_min 300
label fan3 "PSU"
set fan3_min 500
ignore fan2
ignore fan4
>> In general, for future Tested-bys: does it make sense to append a kind of test report (like the above one) directly after the Tested-by statement, or should it be located in a separate mail?
>
> Yes, test results are always welcome, and including them in the same
> e-mail is fine.
>
ok, perfect ;-)
Björn
_______________________________________________
lm-sensors mailing list
lm-sensors@lm-sensors.org
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors
next prev parent reply other threads:[~2012-03-24 8:11 UTC|newest]
Thread overview: 28+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-03-08 4:25 [lm-sensors] [PATCH] hwmon: (it87) Add support for IT8781F, IT8782F, IT8783E/F Guenter Roeck
2012-03-13 18:43 ` Bjoern Gerhart
2012-03-15 21:13 ` Guenter Roeck
2012-03-15 21:18 ` Jean Delvare
2012-03-23 21:02 ` Jean Delvare
2012-03-23 21:17 ` Jean Delvare
2012-03-23 23:24 ` Björn Gerhart
2012-03-24 4:18 ` Guenter Roeck
2012-03-24 7:05 ` Jean Delvare
2012-03-24 8:11 ` Björn Gerhart [this message]
2012-03-24 8:37 ` Jean Delvare
2012-03-24 15:22 ` Guenter Roeck
2015-08-04 19:45 ` [lm-sensors] [PATCH] hwmon: (it87) Add support for IT8732F Justin Maggard
2015-08-04 20:08 ` Guenter Roeck
2015-08-04 21:03 ` Jean Delvare
2015-08-04 21:24 ` Guenter Roeck
2015-08-04 21:29 ` Justin Maggard
2015-08-04 21:50 ` Guenter Roeck
2012-03-24 16:23 [lm-sensors] [PATCH] hwmon: (it87) Add support for IT8781F, IT8782F, IT8783E/F Guenter Roeck
2012-03-24 18:07 ` Guenter Roeck
2012-03-26 16:08 ` Björn Gerhart
2012-03-26 17:01 ` Jean Delvare
2012-03-26 17:38 ` Guenter Roeck
2012-03-26 19:05 ` Guenter Roeck
2012-03-27 19:12 ` Björn Gerhart
2012-03-27 19:21 ` Björn Gerhart
2012-03-27 23:28 ` Guenter Roeck
2012-03-29 19:46 ` Björn Gerhart
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=982E6473-EAC9-4FB7-9A0D-3CF70AB5D5AA@extracloud.de \
--to=oss@extracloud.de \
--cc=lm-sensors@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.