linuxppc-dev.lists.ozlabs.org archive mirror
 help / color / mirror / Atom feed
From: Nico Schottelius <nico-linux-ppc@schottelius.org>
To: Benjamin Herrenschmidt <benh@kernel.crashing.org>,
	Joerg Dorchain <joerg@dorchain.net>
Cc: linuxppc-dev list <linuxppc-dev@ozlabs.org>
Subject: Re: [PATCH] Heartbeat LED for iBook
Date: Tue, 8 Feb 2005 08:54:21 +0100	[thread overview]
Message-ID: <20050208075421.GB21154@schottelius.org> (raw)
In-Reply-To: <20050127163606.GN14866@Redstar.dorchain.net> <1106746946.6250.56.camel@gaston>

[-- Attachment #1: Type: text/plain, Size: 1522 bytes --]

Good morning!

Benjamin Herrenschmidt [Thu, Jan 27, 2005 at 12:42:26AM +1100]:
> One can always control the led by sending PMU command directly
> via /dev/adb :) Ok, that sort-of sucks, especially since the command is
> only supported by some versions of the PMU. On the other hand, I dislike
> cluttering /dev ... maybe I can find some better mecanism via sysfs...

Well, if /dev would fit best, imho, as it represents a device.
And I don't think 'not having enough major/minor numbers' is an argument
not to implement something. We have udev (and before devfs), which 
should clear the old problem.

Btw, howto control it via /dev/adb? Can you point me to a document
which explains that?

Joerg Dorchain [Thu, Jan 27, 2005 at 05:36:06PM +0100]:
> On Wed, Jan 26, 2005 at 12:35:54PM +0100, Nico Schottelius wrote:
> > Joerg Dorchain [Tue, Jan 18, 2005 at 10:42:16AM +0100]:
> > > [heartbeat blink patch]
> > 
> > Well, wouldn't it make much more sense to write a
> > /dev/frontled to access it via userspace? 
> 
> Well, only some hardware supports something that can be used as a HB
> LED, so IMHO yet another (misc-) device node is not completely
> justified.

Imho it would be a nice feature, for testing and for fun.

If someone gives me some pointers on howto do that, I would try to
write a driver/interface to it.

Nico

-- 
Keep it simple & stupid, use what's available.
Please use pgp encryption: 8D0E 27A4 is my id.
http://nico.schotteli.us | http://linux.schottelius.org

[-- Attachment #2: Type: application/pgp-signature, Size: 827 bytes --]

  reply	other threads:[~2005-02-08  7:55 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-01-18  9:42 [PATCH] Heartbeat LED for iBook Joerg Dorchain
2005-01-26 11:35 ` Nico Schottelius
2005-01-26 13:42   ` Benjamin Herrenschmidt
2005-02-08  7:54     ` Nico Schottelius [this message]
2005-02-14 13:14       ` Segher Boessenkool
2005-02-18 11:13         ` Nico Schottelius
2005-02-18 12:11           ` Segher Boessenkool
2005-02-18 12:20             ` Nico Schottelius
2005-02-18 13:00               ` Segher Boessenkool
2005-02-18 17:58                 ` Brad Boyer
2005-02-18 18:10                   ` Segher Boessenkool
2005-02-19 19:17                     ` Brad Boyer
2005-01-27 16:36   ` Joerg Dorchain
2005-02-18 14:57     ` ibook-led (finished) (was: [PATCH] Heartbeat LED for iBook) Nico Schottelius

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=20050208075421.GB21154@schottelius.org \
    --to=nico-linux-ppc@schottelius.org \
    --cc=benh@kernel.crashing.org \
    --cc=joerg@dorchain.net \
    --cc=linuxppc-dev@ozlabs.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).