mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Philip Graham Willoughby <pgw99@doc.ic.ac.uk>
To: Ghozlane Toumi <gtoumi@laposte.net>
Cc: Jamey Hicks <jamey.hicks@hp.com>, Pavel Machek <pavel@ucw.cz>,
	Linux Kernel Mailing List <linux-kernel@vger.kernel.org>
Subject: Re: PATCH : LEDs - possibly the most pointless kernel subsystemever
Date: Wed, 30 Jul 2003 13:27:49 +0100	[thread overview]
Message-ID: <20030730122749.GA31982@bodmin.doc.ic.ac.uk> (raw)
In-Reply-To: <001101c35693$8e97cc40$0a00a8c0@toumi>

On 2003-07-30 14:06:43 +0000, Ghozlane Toumi wrote:
> > I would have thought that leds are output?  Why would output devices be
> > integrated into the input subsystem?
> 
> Perhaps because the input subsystem could/should be renamed to event
> subsytem ?

The LED devices I'm using are nothing like any of the devices in the
input subsystem at all -- they don't generate events _ever_ for
instance.  /dev/input/event## for a pure LED device would be rather
boring.

IMO it's cleaner to have a LED subsystem and proxy drivers to expose
LEDs provided by other subsystems to the generic interface, rather than
trying to ram everything onto the input subsystem regardless of how well
it fits.

Of course, other opinions may (will) differ ;-).

If you did want to get the LED change events from the LED subsystem
through the input layer, you could add a hook into led_set_real in
leds.c, but since all changes are software triggered I don't really see
the value in that.

Regards,

Philip Willoughby

Systems Programmer, Department of Computing, Imperial College, London, UK
-- 
echo bzidd@nfo.ho.co.se | tr "bizndfohces" "pwgd9ociaku"
Why reinvent the wheel?                 Because we can make it rounder...

  reply	other threads:[~2003-07-30 12:28 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2003-07-29 15:17 PATCH : LEDs - possibly the most pointless kernel subsystem ever Philip Graham Willoughby
2003-07-29 18:00 ` Pavel Machek
2003-07-30 11:45   ` Jamey Hicks
2003-07-30 12:06     ` PATCH : LEDs - possibly the most pointless kernel subsystemever Ghozlane Toumi
2003-07-30 12:27       ` Philip Graham Willoughby [this message]
2003-07-30 12:39         ` Wichert Akkerman
2003-07-30 12:27     ` PATCH : LEDs - possibly the most pointless kernel subsystem ever Pavel Machek
2003-07-30  6:36 ` CaT
2003-07-30 13:00   ` Andries Brouwer
2003-07-30 13:20     ` CaT
2003-07-30 23:28 ` Greg KH

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=20030730122749.GA31982@bodmin.doc.ic.ac.uk \
    --to=pgw99@doc.ic.ac.uk \
    --cc=gtoumi@laposte.net \
    --cc=jamey.hicks@hp.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=pavel@ucw.cz \
    /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®