[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20131228101622.GA13564@amd.pavel.ucw.cz>
Date: Sat, 28 Dec 2013 11:16:22 +0100
From: Pavel Machek <pavel@....cz>
To: Joe Xue <lgxue@...mail.com>
Cc: "cooloney@...il.com" <cooloney@...il.com>,
"rpurdie@...ys.net" <rpurdie@...ys.net>,
"rob@...dley.net" <rob@...dley.net>,
"milo.kim@...com" <milo.kim@...com>,
"linux-leds@...r.kernel.org" <linux-leds@...r.kernel.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
"linux-doc@...r.kernel.org" <linux-doc@...r.kernel.org>
Subject: Re: [PATCH] Add the LED burst trigger
Hi!
> Good idea.
> I have finished but I'll change it to this way soon and test it in next some days.
>
> What the idea about the character to indicate stop?
>
> I mean this patten maybe indicate just once maybe indicate repeatedly until the next patten.
>
> What about "/"?
> If there is a "/" at end then stop it else repeat it?
Actually, I'd put a do_repeat attribute somewhere instead of using
magic character for automatic repeats.
And while " " for pause and "#" for light would work. Maybe we chould
do "\0" for pause and "\177" for light... and interpret everything
between as an intensity. That will make it useful for LEDs with
variable intensites, too...
Thanks,
Pavel
--
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists