Problem reading vitals from Gigabyte H77-DH3H

[Andriy Gapon]

on 21/10/2012 10:11 Scot Hetzel said the following:
> On Sat, Oct 20, 2012 at 3:09 PM, Andriy Gapon <avg at freebsd.org> wrote:
>> on 20/10/2012 22:42 Andriy Gapon said the following:
>>> on 20/10/2012 22:20 Derek Kulinski said the following:
>>>> I have three questions though:
>>>> 1. The motherboard has 4 fan sockets (as far as I can tell), CPU_FAN,
>>>>    and SYS_FAN[1-3]. SYS_FAN1 currently is not connected.
>>>>    Seems like:
>>>>      fan0 -> CPU_FAN (did not try to disconnect it to check :)
>>>>      fan1 -> SYS_FAN1
>>>>      fan2 -> SYS_FAN2
>>>>    There is no entry for SYS_FAN3. I disconnected it temporarily but
>>>>    it did not seem to affect the output. Is it possible to get that
>>>>    information from the motherboard?
>>>
>>> The driver would have to be updated for that.
>>> Unfortunately ITE does not provide public datasheets.
>>> We could pick up some new bits from the Linux driver though.
>>> http://lxr.linux.no/#linux+v3.6.2/drivers/hwmon/it87.c
>>
>> In fact, here is a completely untested patch:
>> http://people.freebsd.org/~avg/it-fans-0x80.diff
>>
> 
> @@ -354,12 +372,15 @@ static void
>  it_refresh_sensor_data(struct it_softc *sc)
>  {
>  	/* Refresh our stored data for every sensor */
> -	it_generic_stemp(sc, &sc->sensors[12]);
> -	it_generic_svolt(sc, &sc->sensors[3]);
> -	if (sc->fan16bit)
> +	if (sc->fan16bit) {
>  		it_16bit_fanrpm(sc, &sc->sensors[0]);
> -	else
> +		it_generic_svolt(sc, &sc->sensors[5]);
> +		it_generic_svolt(sc, &sc->sensors[14]);  <- Looks to be a copy/paste bug ;-)

Indeed.  Should be "stemp" of course.
Thank you!

> +	} else {
>  		it_generic_fanrpm(sc, &sc->sensors[0]);
> +		it_generic_svolt(sc, &sc->sensors[3]);
> +		it_generic_stemp(sc, &sc->sensors[12]);
> +	}
>  }
> 


-- 
Andriy Gapon