All of lore.kernel.org
 help / color / mirror / Atom feed
From: Guenter Roeck <guenter.roeck@ericsson.com>
To: lm-sensors@vger.kernel.org
Subject: Re: [lm-sensors] [PATCH] hwmon: (w83627ehf) Fix documentation of
Date: Tue, 01 Nov 2011 13:57:36 +0000	[thread overview]
Message-ID: <20111101135736.GA6669@ericsson.com> (raw)
In-Reply-To: <20111101111936.416dd882@endymion.delvare>

On Tue, Nov 01, 2011 at 06:19:36AM -0400, Jean Delvare wrote:
> The NCT6775F and NCT6776F have specific temperature sensor resolutions
> for sensors above 3.
> 
> Signed-off-by: Jean Delvare <khali@linux-fr.org>
> Cc: Guenter Roeck <guenter.roeck@ericsson.com>
> ---
> Guenter, did I get it right?
> 
Did I add the 0.0 degrees note ? Brr.

Anyway, it is kind of correct. temp7 to temp9 resolution is 0.5 degrees C, but the code
only reads full degrees because it was too complicated to merge the lowest bit.

Acked-by: Guenter Roeck <guenter.roeck@ericsson.com>

Thanks,
Guenter

>  Documentation/hwmon/w83627ehf |    5 +++--
>  1 file changed, 3 insertions(+), 2 deletions(-)
> 
> --- linux-3.2-rc0.orig/Documentation/hwmon/w83627ehf	2011-10-31 17:00:50.000000000 +0100
> +++ linux-3.2-rc0/Documentation/hwmon/w83627ehf	2011-11-01 10:55:03.000000000 +0100
> @@ -62,8 +62,9 @@ sensors. The configured source for each
>  in tempX_label.
>  
>  Temperatures are measured in degrees Celsius and measurement resolution is 1
> -degC for temp1 and and 0.5 degC for temp2 and temp3. For temp4 and higher,
> -resolution is 1 degC for W83667HG-B and 0.0 degC for NCT6775F and NCT6776F.
> +degC for temp1 and and 0.5 degC for temp2 and temp3. For W83667HG-B, temp4
> +has resolution 1 degC. For NCT6775F and NCT6776F, temp4 to temp6 have
> +resolution 0.5 degC, while temp7 to temp9 have resolution 1 degC.
>  An alarm is triggered when the temperature gets higher than high limit;
>  it stays on until the temperature falls below the hysteresis value.
>  Alarms are only supported for temp1, temp2, and temp3.
> 
> 
> -- 
> Jean Delvare

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

  reply	other threads:[~2011-11-01 13:57 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-11-01 10:19 [lm-sensors] [PATCH] hwmon: (w83627ehf) Fix documentation of Jean Delvare
2011-11-01 13:57 ` Guenter Roeck [this message]
2011-11-05 13:57 ` Jean Delvare
2011-11-06 16:01 ` 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=20111101135736.GA6669@ericsson.com \
    --to=guenter.roeck@ericsson.com \
    --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.