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

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


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@xxxxxxxxxxxx>:
> >>> Assume all three are compatible and have the same functionality.
> >>> 
> >>> Signed-off-by: Guenter Roeck <linux@xxxxxxxxxxxx>
> >> Tested-by: Bjoern Gerhart <oss@xxxxxxxxxxxxx>
> > 
> > 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.

> 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.

-- 
Jean Delvare

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



[Video for Linux]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Photos]     [Yosemite Photos]     [Free Singles Community]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]

Add to Google Powered by Linux