Re: Implementing missing BlinkM LED features

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi!

Please cc lists on communications.

> Thank you. This would be my first time contributing to the kernel, so there are just a few things I wanted to clarify :)
>

Hmm, good luck. 

> I looked into the pattern trigger you mentioned, and from what I found in the sources and mailing list, it seems like an alternative to manually sending a stream of BlinkM script bytes. That is, the user instead sets a brightness and delay and then turns the LED on and off at whatever interval they choose. So it is preferable for the user to send their lighting sequence to the pattern trigger file, rather than with some other file/interface I create?
>

Yes, please. Use existing interface.

> You also mentioned an RGB-aware pattern trigger, which would be necessary for implementing this driver feature. Did you mean I should first patch the existing ledtrig-pattern.c to support changing RGB values, or is that something I would implement within the BlinkM driver?
>

You should patch ledtrig-pattern.c first.

Best regards,
										Pavel
-- 
People of Russia, stop Putin before his war on Ukraine escalates.

Attachment: signature.asc
Description: PGP signature


[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux