All of lore.kernel.org
 help / color / mirror / Atom feed
* Blinking of the paired LEDs
@ 2021-10-26 17:11 Alexander A. Filippov
       [not found] ` <OF324F0F03.50C28783-ON0025877A.00618017-0025877A.00627501@ibm.com>
  0 siblings, 1 reply; 2+ messages in thread
From: Alexander A. Filippov @ 2021-10-26 17:11 UTC (permalink / raw)
  To: openbmc

We have green-red paired LEDs on our hardware. In SysFS they present as a pair of standalone LED. 
And we switch the different color in different cases using separate LED groups in the phosphor-led-manager that turns corresponding physical LEDs.

One of those LED groups turning on the such LED to the yellow blinking by setting the blink mode on the both physical LED. But sometimes these physical LEDs turn with a little time lag. This leads to the blinking of the four states: green (short) -> yellow (long)-> red (short) -> off(long).

I believe this occurs when there is a some traffic through D-Bus (e.g. during the host starting) and LED manager can't set the physical LEDs mode synchronously.

And I see nothing better than trying to combine the phosphor-ledcontroller instances into one process and probably to implement the method for setting the properties with one D-Bus call.

Does anybody have suggestions about any better solution?

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re:  Blinking of the paired LEDs
       [not found] ` <OF324F0F03.50C28783-ON0025877A.00618017-0025877A.00627501@ibm.com>
@ 2021-10-26 18:40   ` Alexander A. Filippov
  0 siblings, 0 replies; 2+ messages in thread
From: Alexander A. Filippov @ 2021-10-26 18:40 UTC (permalink / raw)
  To: Milton Miller II; +Cc: openbmc

> >
> >Does anybody have suggestions about any better solution?
>
> I'd suggest going a level higher by defining the LED to the kernel
> as single instance of the multi-color led class.  While the class is
> described for full RGB controls, I believe a red green led controlled
> by 2 gpios in one bank is well within both the LED and GPIO
> kernel specifications (I don't know if you have chosen gpios
> in the same bank or even if you are using gpios to control it).
> 
> 
> https://github.com/openbmc/linux/blob/dev-5.10/Documentation/leds/leds-class-multicolor.rst
> 


Yes, I've seen this feature but couldn't get it to work with gpio-leds.
I have concluded that gpio-leds does not support multi-led.

The kernel driver modification seems to me a little bit more complex,
but probably you are right, it looks like a better solution.

Thanks for your advice.

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2021-10-26 18:41 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-10-26 17:11 Blinking of the paired LEDs Alexander A. Filippov
     [not found] ` <OF324F0F03.50C28783-ON0025877A.00618017-0025877A.00627501@ibm.com>
2021-10-26 18:40   ` Alexander A. Filippov

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.