All of lore.kernel.org
 help / color / mirror / Atom feed
From: khali@linux-fr.org (Jean Delvare)
To: lm-sensors@vger.kernel.org
Subject: [lm-sensors] [PATCH] Add MAX6650 support
Date: Sat, 17 Mar 2007 13:39:53 +0000	[thread overview]
Message-ID: <20070317143953.cbb80c83.khali@linux-fr.org> (raw)
In-Reply-To: <200701171359.46334.hjk@linutronix.de>

Hi Hans-J?rgen,

On Fri, 16 Mar 2007 22:07:52 +0100, Hans-J?rgen Koch wrote:
> Well, I really appreciate good code quality. If this is the price, I'm willing 
> to pay it. Actually, I thank you for helping me so much.

Patch accepted, will be in next -mm, and from there will go to 2.6.22.
Thanks :)

> > Can you please test the user-space support now? I suspect that we need
> > to update libsensors at least to match the name change of the "speed"
> > file. We're about to release lm_sensors 2.10.3 (in a few days now) and
> > I'd like it to support your new driver properly.
> 
> Yes, but not so soon, sorry. I'm not at home the whole next week and will not 
> have any sensors hardware available. After that, I might find some time to 
> have a look. I have an LM93 driver almost ready, so that'll be the next thing 
> I'll keep you busy with :-) In the course of that development I can have a 
> short look at user space stuff, too.

I've made the minimum fix for the 2.10.3 release, so that at least no
error is displayed. Feel free to propose further improvements later.
I've also updated the Devices on our wiki to reflect the current state
of the MAX6650/MAX6651 support.

If you start working on the lm93 driver, make sure to search the
mailing list archives first. I remember that the driver was already
ported to Linux 2.6, but never reviewed. This was some times ago
though, many things changed in the kernel meanwhile, so it's up to you
whether you want to start from an old port or from scratch. Beware, the
lm93 driver is a fat beast.

-- 
Jean Delvare


  parent reply	other threads:[~2007-03-17 13:39 UTC|newest]

Thread overview: 38+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-01-17 12:59 [lm-sensors] [PATCH] Add MAX6650 support Hans-Jürgen Koch
2007-01-22  8:36 ` Jean Delvare
2007-02-11 15:44 ` corentin.labbe
2007-02-11 16:22 ` Hans-Jürgen Koch
2007-02-12 13:48 ` Hans-Jürgen Koch
2007-02-27 15:21 ` [lm-sensors] " Matej Kenda
2007-02-27 15:35 ` [lm-sensors] " Jean Delvare
2007-02-27 16:17 ` Hans-Jürgen Koch
2007-02-27 16:29 ` Matej Kenda
2007-02-27 17:03 ` Matej Kenda
2007-02-27 17:13 ` Hans-Jürgen Koch
2007-02-27 18:07 ` Jean Delvare
2007-02-27 19:58 ` Hans-Jürgen Koch
2007-02-28 11:44 ` Hans-Jürgen Koch
2007-02-28 15:57 ` Matej Kenda
2007-02-28 16:08 ` Hans-Jürgen Koch
2007-03-01  7:15 ` Jean Delvare
2007-03-01  7:34 ` Jean Delvare
2007-03-01 10:11 ` Hans-Jürgen Koch
2007-03-01 17:34 ` corentin.labbe
2007-03-02 11:32 ` Jean Delvare
2007-03-05 11:34 ` Hans-Jürgen Koch
2007-03-11 14:40 ` Jean Delvare
2007-03-11 15:21 ` Jean Delvare
2007-03-11 21:08 ` Hans-Jürgen Koch
2007-03-12 13:50 ` Jean Delvare
2007-03-12 14:44 ` Hans-Jürgen Koch
2007-03-12 16:49 ` Jean Delvare
2007-03-13 13:25 ` Hans-Jürgen Koch
2007-03-15 20:10 ` Jean Delvare
2007-03-16 16:44 ` Hans-Jürgen Koch
2007-03-16 19:17 ` Jean Delvare
2007-03-16 21:07 ` Hans-Jürgen Koch
2007-03-16 21:19 ` Hans-Jürgen Koch
2007-03-16 21:19 ` Thomas Gleixner
2007-03-16 21:33 ` Thomas Gleixner
2007-03-17 13:39 ` Jean Delvare [this message]
2007-03-17 14:23 ` Hans-Jürgen Koch

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=20070317143953.cbb80c83.khali@linux-fr.org \
    --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.