RE: [PATCH] Add the LED burst trigger
From: Joe Xue
Date: Sat Dec 28 2013 - 19:23:55 EST
I have finished the patten trigger and have tested it.
Do you mind me putting your name as suggest-by? For part idea is from you.
> Date: Sat, 28 Dec 2013 20:26:30 +0100
> From: pavel@xxxxxx
> To: lgxue@xxxxxxxxxxx
> CC: cooloney@xxxxxxxxx; rpurdie@xxxxxxxxx; rob@xxxxxxxxxxx; milo.kim@xxxxxx; linux-leds@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx; linux-doc@xxxxxxxxxxxxxxx
> Subject: Re: [PATCH] Add the LED burst trigger
>>> 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...
>> "\0" is not easy to give if application is a script.
>> intensity is a good idea, but makes it more complex, and user need to convert the "\num" to a ASCII code.
> OTOH it is easier for C programs _and_ we can support different
> brightness levels.
> It is not that bad from script:
> echo -ne '\0\051\0377'> foo...
> (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@xxxxxxxxxxxxxxx
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/ --
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/