on 21/10/2012 10:11 Scot Hetzel said the following:
> On Sat, Oct 20, 2012 at 3:09 PM, Andriy Gapon <a...@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
_______________________________________________
freebsd-stable@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-stable
To unsubscribe, send any mail to "freebsd-stable-unsubscr...@freebsd.org"
