From: Jordan Crouse <jordan.crouse@amd.com>
To: lm-sensors@vger.kernel.org
Subject: Re: [lm-sensors] adt7475 support (Asus M2N-SLI Deluxe)
Date: Tue, 22 Jan 2008 22:55:22 +0000 [thread overview]
Message-ID: <fn5s8p$d0v$1@ger.gmane.org> (raw)
In-Reply-To: <Pine.LNX.4.64.0801211406570.32720@birdbird.example.com>
On Tue, 22 Jan 2008 22:10:32 +0100, Hans de Goede wrote:
> Matt Roberds wrote:
>> Hello all!
>>
>> I recently built a system using an Asus M2N-SLI Deluxe motherboard,
>> which has an Analog Devices ADT7475 thermal monitor and fan control
>> chip. sensors-detect knows about it but says there is no driver. I've
>> been looking through the mailing list archives and found a post from
>> Hans de Goede in December with a partial driver (see
>> http://lists.lm-sensors.org/pipermail/lm-sensors/2007-
December/022169.html
>> ).
>>
>> I downloaded that driver and it built as a module under kernel 2.6.18,
>> after commenting out a few lines of code. sensors (from lm-sensors
>> 3.0.0) says:
>>
>>
> Hi,
>
> Yes that Hans de Goede would be me. Unfortunately I'm still waiting for
> my Asus M2N-SLI Deluxe to return from the Asus repair center as it broke
> down.
>
> Notice that beside my driver WIP, there also is one written by Jordan
> Crouse I just noticed when looking it up in my archive that he hasn't
> send it to the list yet, so I've attached it here. Both drivers need
> quite some work before being acceptable for mainline inclusion. The one
> done by my students is incomplete, and the one done by Jordan Crouse
> (attached) needs to have atleast the following issues fixed:
>
> * remote vcc and vcpp are not valid input names, please see
> Documentation/hwmon/sysfs-interface for valid sysfs attr names.
>
> * You're handling of the alarms is wrong, now a days we have one alarm
> file per inpuit, again see: Documentation/hwmon/sysfs-interface
>
> * Don't call a driver adt74xx even if it will support multiple chips, it
> may very well never support all chips matching the xx mask, so instead
> just call it after the first support chips, other documentation like the
> wiki-page and sensors-detect will show to use this driver for releated
> chips.
I have some updates for that, but I have been very lazy in sending it
out. I'll send it out today, if it will help.
Thanks for the kick,
Jordan
_______________________________________________
lm-sensors mailing list
lm-sensors@lm-sensors.org
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors
next prev parent reply other threads:[~2008-01-22 22:55 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-01-21 20:52 [lm-sensors] adt7475 support (Asus M2N-SLI Deluxe) Matt Roberds
2008-01-22 21:10 ` Hans de Goede
2008-01-22 22:55 ` Jordan Crouse [this message]
2008-01-22 23:47 ` Jordan Crouse
2008-01-24 16:37 ` Matt Roberds
2008-01-24 21:57 ` Jordan Crouse
2008-02-13 16:53 ` Artur Szymiec
2008-02-18 17:00 ` Jordan Crouse
2008-02-18 17:34 ` Artur Szymiec
2008-02-18 18:00 ` Hans de Goede
2008-02-18 19:01 ` Artur Szymiec
2008-02-18 19:50 ` 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='fn5s8p$d0v$1@ger.gmane.org' \
--to=jordan.crouse@amd.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.