All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jean Delvare <khali@linux-fr.org>
To: lm-sensors@vger.kernel.org
Subject: Re: [lm-sensors] [patch 00/36] New w83795 driver
Date: Wed, 15 Sep 2010 16:23:09 +0000	[thread overview]
Message-ID: <20100915182309.144acce6@hyperion.delvare> (raw)
In-Reply-To: <20100915154005.448afabb@hyperion.delvare>

On Wed, 15 Sep 2010 13:07:40 -0300, Henrique de Moraes Holschuh wrote:
> On Wed, 15 Sep 2010, Jean Delvare wrote:
> > On Wed, 15 Sep 2010 12:52:06 -0300, Henrique de Moraes Holschuh wrote:
> > > You've gotten yourself a tester: I have one of those in my main desktop
> > > (SuperMicro C7X58 v1.1).  I just have to backport the driver to 2.6.32.y
> > > to test it there.
> > 
> > You don't even have to, as I did it for you already:
> > http://khali.linux-fr.org/devel/misc/w83795/
> 
> Thanks.  Will use.
> 
> > If you find any problem with the driver, please let me know!
> 
> I will even try to send patches :-)

Thanks :)

> > (Note: monitoring should work OK now, but I didn't have the time to
> > work on fan speed control yet, so I expect bugs in this area.)
> 
> I can try to compare the montoring results with the userspace driver
> included in SuperODoctor II.  The first version of the driver (before
> your patches) had inconsistentices.
> 
> I will have to read the chip's datasheet to check if my hardware setup
> will let me help test PWM control.
> 
> The C7X58/S8SAX BIOS is supposed to put the chip in some sort of
> closed-loop fan control mode if you tell it to, but there are issues
> when mixing 3-pin and 4-pin fans in the same system (or at least
> SuperMicro cautions agains such a setup.  It might be just a lazy BIOS,
> or it might be a chip restriction)... and I have a mix of 3-pin and
> 4-pin fans.

I have similar restrictions on my Asus Z8NA-D6 board. Both CPU fans
have to be either 4-pin or 3-pin for control to work, they can't be
mixed. Same goes for case fans, they have to be all 3-pin or all 4-pin,
otherwise control doesn't work. There are jumpers on the board to
switch between 3-pin and 4-pin setups.

-- 
Jean Delvare

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

  parent reply	other threads:[~2010-09-15 16:23 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-09-15 13:40 [lm-sensors] [patch 00/36] New w83795 driver Jean Delvare
2010-09-15 15:52 ` Henrique de Moraes Holschuh
2010-09-15 15:55 ` Jean Delvare
2010-09-15 16:07 ` Henrique de Moraes Holschuh
2010-09-15 16:23 ` Jean Delvare [this message]
2010-09-17  0:14 ` Guenter Roeck
2010-09-18 11:56 ` Jean Delvare
2010-09-18 13:34 ` Guenter Roeck
2010-09-18 14:16 ` Jean Delvare
2010-09-18 15:00 ` Guenter Roeck

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=20100915182309.144acce6@hyperion.delvare \
    --to=khali@linux-fr.org \
    --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.