linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Greg KH <greg@kroah.com>
To: Andrey Borzenkov <arvidjaar@mail.ru>
Cc: linux-kernel@vger.kernel.org
Subject: Re: 2.6 - sysfs sensor nameing inconsistency
Date: Tue, 15 Jul 2003 13:18:22 -0700	[thread overview]
Message-ID: <20030715201822.GA5040@kroah.com> (raw)
In-Reply-To: <200307152214.38825.arvidjaar@mail.ru>

On Tue, Jul 15, 2003 at 10:14:38PM +0400, Andrey Borzenkov wrote:
> In 2.4 all sensor chip got a subdirectory with name derived from type_name - a 
> single word describing sensor, like

And they used sysctl and proc, ugh, ugly :)

> adm1021.c:              type_name = "max1617";
> adm1021.c:              type_name = "max1617a";
> adm1021.c:              type_name = "adm1021";
> adm1021.c:              type_name = "adm1023";
> adm1021.c:              type_name = "thmc10";
> adm1021.c:              type_name = "lm84";
> adm1021.c:              type_name = "gl523sm";
> adm1021.c:              type_name = "mc1066";
> ...
> 
> etc. All user-level configuration (sensors, gkrellm) have been using these 
> names to match available sensors and configuration data.
> 
> In 2.6 sensors appear under /sysfs, type_name no more used and the only 
> identification available is .../name, but it seems to be arbitrary chosen 
> like
> 
> - single word ("it87") - lm87.c
> - "name chip" or "name subclient" - most others (lm78.c, wd83781d.c etc)
> - completely arbitrary shiny description - "Generic LM85", "National LM85-B" 
> etc in lm85.c
> 
> This means, any user program accessing sensors need incompatible changes and 
> comfiuration cannot be shared between 2.4 and 2.6 without serious redesign 
> and/or some translation layer.

The "translation layer" is libsensors.  libsensors needs to be rewritten
for 2.6.  The sensors people know this, and it's even detailed on their
web page.  Any help with this is greatly appreciated.

> If there are serious reasons to keep current names in "name" - what about 
> adding extra type_name property that will hold type_name compatible with 2.4, 
> at least for those drivers that are also available there. This would allow 
> easily reuse existing sensors configuration.

Patches to help do this are always welcome :)

thanks,

greg k-h

  reply	other threads:[~2003-07-15 20:03 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2003-07-15 18:14 2.6 - sysfs sensor nameing inconsistency Andrey Borzenkov
2003-07-15 20:18 ` Greg KH [this message]
2003-07-26 18:00   ` Andrey Borzenkov
2003-08-15 20:51     ` Greg KH
2003-08-16 15:38       ` Andrey Borzenkov
2003-08-16 16:50         ` Greg KH
2003-08-18 16:49           ` Andrey Borzenkov
2003-08-18 21:31             ` Greg KH
2003-08-19 19:19           ` Andrey Borzenkov
2003-08-19 19:45             ` Greg KH
2003-08-31 16:25               ` Andrey Borzenkov
2003-09-22 22:29                 ` Greg KH
2003-11-02 18:50                   ` Andrey Borzenkov
2003-07-27  4:42 Margit Schubert-While
2003-07-27  6:23 Andrey Borzenkov

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=20030715201822.GA5040@kroah.com \
    --to=greg@kroah.com \
    --cc=arvidjaar@mail.ru \
    --cc=linux-kernel@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).