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 2/2] hwmon: Add documentation for tmp421
Date: Tue, 16 Jun 2009 11:48:58 +0000	[thread overview]
Message-ID: <20090616134858.0e417fe4@hyperion.delvare> (raw)
In-Reply-To: <20090615074339.GC4667@ubuntu>

On Mon, 15 Jun 2009 09:43:39 +0200, Andre Prendel wrote:
> This patch adds documentation for driver tmp421.
> 
> Signed-off-by: Andre Prendel <andre.prendel@gmx.de>
> ---
>  tmp421 |   36 ++++++++++++++++++++++++++++++++++++
>  1 file changed, 36 insertions(+)
> 
> Index: linux-2.6/Documentation/hwmon/tmp421
> =================================> --- /dev/null	1970-01-01 00:00:00.000000000 +0000
> +++ linux-2.6/Documentation/hwmon/tmp421	2009-06-11 21:50:46.000000000 +0200
> @@ -0,0 +1,36 @@
> +Kernel driver tmp421
> +==========
> +
> +Supported chips:
> +  * Texas Instruments TMP421
> +    Prefix: 'tmp421'
> +    Addresses scanned: I2C 0x2a, 0x4c, 0x4d, 0x4e and 0x4f
> +    Datasheet: http://focus.ti.com/docs/prod/folders/print/tmp421.html
> +  * Texas Instruments TMP422
> +    Prefix: 'tmp422'
> +    Addresses scanned: I2C 0x2a, 0x4c, 0x4d, 0x4e and 0x4f
> +    Datasheet: http://focus.ti.com/docs/prod/folders/print/tmp421.html
> +  * Texas Instruments TMP423
> +    Prefix: 'tmp423'
> +    Addresses scanned: I2C 0x2a, 0x4c, 0x4d, 0x4e and 0x4f
> +    Datasheet: http://focus.ti.com/docs/prod/folders/print/tmp421.html
> +
> +Authors:
> +	Andre Prendel <andre.prendel@gmx.de>
> +
> +Description
> +-----------
> +
> +This driver implements support for Texas Instruments TMP421, TMP422
> +and TMP423 temperature sensor chips. These chips implements one local

Spelling: implement.

> +and up to one (TMP421), up to two (TMP422) or up to three (TMP423)
> +remote sensors. Temperature is measured in degrees Celsius. The chips
> +are wired over I2C/SMBus and specified over a temperature range of
> +-40°C to +125°C. Resolution for both the local and remote channels is
> +0.0625°C.

The degree symbol is not ASCII, I'd suggest spelling it "degree C" to
avoid charset headaches.

> +
> +The chips supports only temperature measurement and exports these

Spelling: support. Also, the drivers exports the values to sysfs, not
the chips.

> +values via following sysfs-interface:
> +
> +temp[1-4]_input
> +temp[2-4]_fault

Maybe you could add a section explaining that not all possible
addresses are probed by default, why and how to work around it if
needed.

-- 
Jean Delvare

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

      reply	other threads:[~2009-06-16 11:48 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-06-15  7:43 [lm-sensors] [PATCH 2/2] hwmon: Add documentation for tmp421 driver Andre Prendel
2009-06-16 11:48 ` Jean Delvare [this message]

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=20090616134858.0e417fe4@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.