All of lore.kernel.org
 help / color / mirror / Atom feed
From: Andrew Lutomirski <luto@mit.edu>
To: lm-sensors@vger.kernel.org
Subject: Re: [lm-sensors] Standalone driver for W83677HG-I, NCT6775F,
Date: Mon, 07 Feb 2011 18:20:22 +0000	[thread overview]
Message-ID: <AANLkTimzesTKSf-RWyukcgxu4su-R8MJR-JcDgeWDGvH@mail.gmail.com> (raw)
In-Reply-To: <20110205175852.GA26672@ericsson.com>

On Mon, Feb 7, 2011 at 12:58 PM, Guenter Roeck
<guenter.roeck@ericsson.com> wrote:
> Hi all,
>
> I published a new version of the standalone driver for W83667HG-I (NCT6771F, NCT6775F)
> and NCT6776F at http://roeck-us.net/linux/drivers/w83627ehf/.
>
> Key changes are listed in the README file. Most of the reported bugs have been fixed,
> though I am not sure about PWM configuration on NCT6775F and fan min limits on NCT6776F.
>

I don't see a README.  What's w83627ehf.c.18?

> For PWM configuration on NCT6775F, I did not find a problem
. I can set and read PWM values,
> but since my board does not connect the chip to actual fans I have no idea if the settings
> have any effect. So this will need to be re-tested on a board which actually uses the chip
> for fan control.

Something's weird.

# cat pwm2_enable
5
# cat pwm2_mode
1
# echo 1 >pwm2_enable
# cat pwm2_enable
5
# echo 1 >pwm2_enable
# cat pwm2_enable
5
# echo 2 >pwm2_enable
# cat pwm2_enable
6

[fiddle with pwm2 here.  values written stick but have no effect.]

# echo 3 >pwm2_enable
[root@midnight w83627ehf.656]# cat pwm2_enable
8
[root@midnight w83627ehf.656]# echo 1 >pwm2_enable
[root@midnight w83627ehf.656]# cat pwm2_enable
8

Note: I won't swear that my motherboard uses this particular chip to
drive the PWMs, but I assume it does since it can certainly measure
fan speed w/ this chip and it controls the PWMs somehow.

>
> For the fan min limits on NCT6776F, the code still uses the set of registers starting at
> 0x63a. The prototype driver we got from Nuvoton does set those registers, so presumably
> it should work unless that driver has a bug. Worth to re-test, especially since
> I fixed a problem in the speed setting code.

Alarm code still seems to work.

--Andy

>
> Please test the new code and let me know if there are any problems. And thanks a lot
> to all the feedback I got for the first revision of the code.
>
> Guenter
>
>

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

  parent reply	other threads:[~2011-02-07 18:20 UTC|newest]

Thread overview: 40+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-02-05 17:58 [lm-sensors] Standalone driver for W83677HG-I, NCT6775F, NCT6776F Guenter Roeck
2011-02-05 18:04 ` [lm-sensors] Standalone driver for W83677HG-I, NCT6775F, Guenter Roeck
2011-02-05 19:15 ` Guenter Roeck
2011-02-05 21:46 ` Ian Dobson
2011-02-06  1:00 ` Guenter Roeck
2011-02-06  9:36 ` Ian Dobson
2011-02-06  9:56 ` Jean Delvare
2011-02-06 10:00 ` Ian Dobson
2011-02-06 10:00 ` Jean Delvare
2011-02-06 10:14 ` Ian Dobson
2011-02-06 11:59 ` Ian Dobson
2011-02-06 13:37 ` Ian Dobson
2011-02-06 13:55 ` Ian Dobson
2011-02-06 15:45 ` Ian Dobson
2011-02-06 15:53 ` Jean Delvare
2011-02-06 16:03 ` Ian Dobson
2011-02-06 16:04 ` Guenter Roeck
2011-02-06 16:13 ` Guenter Roeck
2011-02-06 16:28 ` Guenter Roeck
2011-02-07  3:50 ` Andy Lutomirski
2011-02-07  4:50 ` Guenter Roeck
2011-02-07  7:33 ` Jean Delvare
2011-02-07  7:40 ` Jean Delvare
2011-02-07 13:56 ` Andrew Lutomirski
2011-02-07 14:39 ` Guenter Roeck
2011-02-07 14:47 ` Guenter Roeck
2011-02-07 14:57 ` Guenter Roeck
2011-02-07 16:03 ` Guenter Roeck
2011-02-07 16:50 ` Andrew Lutomirski
2011-02-07 17:44 ` Guenter Roeck
2011-02-07 17:58 ` Guenter Roeck
2011-02-07 18:20 ` Andrew Lutomirski [this message]
2011-02-07 18:48 ` Guenter Roeck
2011-02-07 19:53 ` Andrew Lutomirski
2011-02-07 20:26 ` Guenter Roeck
2011-02-07 20:32 ` Andrew Lutomirski
2011-02-07 20:35 ` Jean Delvare
2011-02-07 21:27 ` Guenter Roeck
2011-04-10 15:28 ` Ian Pilcher
2011-04-12 16:49 ` Ian Dobson

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=AANLkTimzesTKSf-RWyukcgxu4su-R8MJR-JcDgeWDGvH@mail.gmail.com \
    --to=luto@mit.edu \
    --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.