Skip site navigation (1)Skip section navigation (2)
Date:      Sun, 21 Oct 2012 11:53:51 +0300
From:      Andriy Gapon <avg@FreeBSD.org>
To:        Scot Hetzel <swhetzel@gmail.com>
Cc:        Derek Kulinski <takeda@takeda.tk>, freebsd-stable@FreeBSD.org
Subject:   Re: Problem reading vitals from Gigabyte H77-DH3H
Message-ID:  <5083B81F.2080909@FreeBSD.org>
In-Reply-To: <CACdU%2Bf_v21uidfTmOJd_yjP=HGcZ6vWxMnLeLwhHOv3ZEzgq3Q@mail.gmail.com>
References:  <1286515493.20121017131543@takeda.tk> <507F1761.1010202@FreeBSD.org> <20121017205147.GB36106@chinatsu.takeda.tk> <5081552F.2050303@FreeBSD.org> <771658188.20121019205010@takeda.tk> <508254AF.7040709@FreeBSD.org> <994545137.20121020010809@takeda.tk> <50826040.6010106@FreeBSD.org> <55342693.20121020103732@takeda.tk> <5082EEC8.2080307@FreeBSD.org> <198684285.20121020122034@takeda.tk> <5082FEA8.3050600@FreeBSD.org> <508304E6.6020202@FreeBSD.org> <CACdU%2Bf_v21uidfTmOJd_yjP=HGcZ6vWxMnLeLwhHOv3ZEzgq3Q@mail.gmail.com>

next in thread | previous in thread | raw e-mail | index | archive | help
on 21/10/2012 10:11 Scot Hetzel said the following:
> On Sat, Oct 20, 2012 at 3:09 PM, Andriy Gapon <avg@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



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?5083B81F.2080909>