mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Henrique de Moraes Holschuh <hmh@hmh.eng.br>
To: Richard Hughes <hughsient@gmail.com>
Cc: Pavel Machek <pavel@ucw.cz>, Greg KH <greg@kroah.com>,
	Richard Purdie <rpurdie@openedhand.com>,
	linux-kernel <linux-kernel@vger.kernel.org>,
	"kay.sievers" <kay.sievers@vrfy.org>,
	"Bryn M. Reeves" <breeves@redhat.com>,
	John Lenz <lenz@cs.wisc.edu>
Subject: Re: [patch] Move led attributes out of device name and into sysfs attributes, was Re: LED devices
Date: Sun, 10 Jun 2007 22:57:49 -0300	[thread overview]
Message-ID: <20070611015749.GA24151@khazad-dum.debian.net> (raw)
In-Reply-To: <1181505736.5241.1.camel@work>

Hi Richard!

On Sun, 10 Jun 2007, Richard Hughes wrote:

> On Sun, 2007-06-10 at 12:11 +0200, Pavel Machek wrote:
> > Can we keep the original naming? spitz:disk is as unique as led02, and
> > it is _way_ easier to use.
> > Come on, I want to use the led subsystem from the scripts... 
> 
> I don't see a problem with spitz_disk, which is just as easy to use in
> scripts.

It is the old problem with mV in battery_level_mV.  How do you know for sure
what is the designator, and what is the unit (or in this case, color)?  You
could take the last _[^_]+, but what if there is no unit/color? etc.

There is a BIIIIG thread about it (or more than one, actually) in the
archives, and I don't recall a consensus being reached.

-- 
  "One disk to rule them all, One disk to find them. One disk to bring
  them all and in the darkness grind them. In the Land of Redmond
  where the shadows lie." -- The Silicon Valley Tarot
  Henrique Holschuh

      reply	other threads:[~2007-06-11  1:58 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-06-01 15:04 Richard Hughes
2007-06-01 15:43 ` Richard Purdie
2007-06-01 15:59   ` Richard Hughes
2007-06-01 16:23     ` Richard Purdie
2007-06-08 18:57       ` Greg KH
2007-06-08 23:02         ` Richard Purdie
2007-06-08 23:46           ` Greg KH
2007-06-09  9:25             ` Richard Purdie
2007-06-25  5:06               ` Greg KH
2007-06-26 10:02                 ` Richard Purdie
2007-06-26 10:46                   ` Richard Hughes
2007-06-28 19:15                     ` Kay Sievers
2007-06-10 10:11         ` Pavel Machek
2007-06-10 20:02           ` Richard Hughes
2007-06-11  1:57             ` Henrique de Moraes Holschuh [this message]

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=20070611015749.GA24151@khazad-dum.debian.net \
    --to=hmh@hmh.eng.br \
    --cc=breeves@redhat.com \
    --cc=greg@kroah.com \
    --cc=hughsient@gmail.com \
    --cc=kay.sievers@vrfy.org \
    --cc=lenz@cs.wisc.edu \
    --cc=linux-kernel@vger.kernel.org \
    --cc=pavel@ucw.cz \
    --cc=rpurdie@openedhand.com \
    /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

all inboxes | Powered by JetHome®