From: mds@paradyne.com (Mark D. Studebaker )
To: lm-sensors@vger.kernel.org
Subject: LM83 support
Date: Thu, 19 May 2005 06:24:03 +0000 [thread overview]
Message-ID: <3F0388E7.3000404@paradyne.com> (raw)
In-Reply-To: <20030702090820.2c39fd00.khali@linux-fr.org>
agreed that a separate driver for lm83 is best, and that
eventually adm1023 support should be moved out of adm1021.
BTW Nat. Semi has always had the best datasheets in the industry.
Also, if you ever have a chance to see Bob Pease on his lecture tour
don't miss it...
http://www.national.com/rap/
Jean Delvare wrote:
>>I would recommend extending the adm1021 driver. It's a relatively
>>simple driver and not nearly as complex as the w83781d. And the
>>supported chips have fewer differences.
>
>
> After taking a look at the code, I had come to the opposite conclusion.
> The adm1021 driver handles only 2 temperature sensors while the LM83
> specificity is that it has 4 temperature sensors. If we are to limit the
> LM83 to 2 temperature readings, then it becomes a LM84 that is already
> supported. And changing the adm1021 driver code to support 4 temperature
> sensors probably makes no sense. So I was planing to write a new driver
> for the LM83.
>
> Wether or not existing support should then be moved from adm1021 to lm83
> could be discussed later (although my arguments right above make it
> quite obvious that older chipsets should be kept supported by adm1021).
>
>
>>You should probably look at modularizing the extended features
>>somewhat.
>> You can look at my lm85.c driver for an example.
>
>
> I'll take a look at the code and see if it can change my mind. As said
> in a previous post, I never write a driver myself so my opinion may not
> be the wisest.
>
> One thing that should be taken into account however is the fact that the
> adm1021 driver already supports 8 devices, which is our current max.
> Adding a ninth chipset to this driver would introduce (again)
> incompatibilities.
>
next prev parent reply other threads:[~2005-05-19 6:24 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2005-05-19 6:24 LM83 support Jean Delvare
2005-05-19 6:24 ` Philip Pokorny
2005-05-19 6:24 ` Mark D. Studebaker [this message]
2005-05-19 6:24 ` Jean Delvare
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=3F0388E7.3000404@paradyne.com \
--to=mds@paradyne.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.