From: khali@linux-fr.org (Jean Delvare)
To: lm-sensors@vger.kernel.org
Subject: [lm-sensors] Sensors-detect with DMI detection
Date: Fri, 23 Mar 2007 08:59:45 +0000 [thread overview]
Message-ID: <20070323095945.ea7cf3af.khali@linux-fr.org> (raw)
In-Reply-To: <00c901c7568f$60a3ad40$0801a8c0@pinkeltje>
Hi Ivo,
On Thu, 22 Mar 2007 11:09:14 +0100, Ivo Manca wrote:
> Just a quick question (again ;)). Is it necesary for us to even think about
> supporting the 2.4 kernels?
> I'm asking this, because if our addition doesn't get included before
> libsensors 3.x, it seems like we only get to work with 2.6, isn't it?
Side note: we are already at libsensors 3.x (unfortunately.) What you
refer to is lm-sensors 3.x, for which the libsensors version will be
4.x. Actually I think we should jump to lm-sensors 4.x directly to sync
the lib version with the package version again - but Mark seems to like
lm-sensors 3.x.
> If that is so, we can drop a table in our database. This is because we want
> to add the kernel versions a configuration is working for. The easiest way
> to implement this, is to just modify the minimum kernel version for that
> configuration. If there's no support for 2.4, we can just add a field with
> the minimum working 2.6 kernel verson, like 2.6.10. Else, we must also
> register the minimum 2.4 version, either as a field, or with a
> table/relationship.
>
> Just wondering :)
We don't really care about 2.4 anymore. If you can support it for free,
alright. If it has a cost, just don't do it.
One thing which you will have to handle though are the syntax changes
to sensors.conf. Two things are going to happen in a (more or less)
near future:
* Mark Hoffman will add support for the "include" statement. Obviously,
the user will need libsensors 4.x if they download a configuration file
which uses such a statement. OTOH, I can't see any reason why an
include statement would be used in the configuration files for
motherboards, so maybe all you have to do is strip any include
statement before storing the configuration file in the database.
* Once the library discovers the device features autmatically,
sensors.conf will have to use the standard feature names, instead of
the custom ones. This means we'll have to fork sensors.conf.eg into an
old-style version and a new-style version. The configuration files
stored in your database will have the same problem - they will work
with either the old library, or the new one, but not both.
Note that it should be possible to translate old-style configuration
files to the new style automatically, using the same translation
rules which are used by libsensors today. A perl script would do. The
other way around is much more difficult, though, as you would need to
write an extensive reverse mapping table for every supported chip.
--
Jean Delvare
next prev parent reply other threads:[~2007-03-23 8:59 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-02-22 14:40 [lm-sensors] Sensors-detect with DMI detection Ivo Manca
2007-02-22 17:19 ` Jean Delvare
2007-02-23 7:08 ` Hans de Goede
2007-02-27 8:50 ` Jean Delvare
2007-02-27 10:59 ` Ivo Manca
2007-02-27 11:43 ` Jean Delvare
2007-02-27 12:24 ` Ivo Manca
2007-02-27 20:41 ` Rudolf Marek
2007-02-28 13:35 ` Hans de Goede
2007-02-28 13:44 ` Jasper Alias
2007-03-01 7:46 ` Jean Delvare
2007-03-01 8:13 ` Hans de Goede
2007-03-01 8:54 ` Jean Delvare
2007-03-16 13:06 ` Ivo Manca
2007-03-18 20:29 ` Jean Delvare
2007-03-20 12:39 ` Ivo Manca
2007-03-22 10:09 ` Ivo Manca
2007-03-23 8:59 ` Jean Delvare [this message]
2007-03-23 9:06 ` Hans de Goede
2007-03-23 15:43 ` 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=20070323095945.ea7cf3af.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.