Re: [PATCH 1/2] ARM: dts: omap3-n900: Fix lp5523 for multi color

From: Merlijn Wajer
Date: Thu Dec 16 2021 - 06:45:05 EST


Hi Pavel,

On 15/12/2021 21:31, Pavel Machek wrote:
> Hi!
>
>> From: "Sicelo A. Mhlongo" <absicsz@xxxxxxxxx>
>>
>> Since the LED multicolor framework support was added in commit
>> 92a81562e695 ("leds: lp55xx: Add multicolor framework support to lp55xx")
>> LEDs on this platform stopped working.
>
> Does this change names is /sys/class/leds?

I booted my 5.15 with this series (and some other patches) added and
these are the led names:

> root@devuan-n900:~# uname -a
> Linux devuan-n900 5.15.2-00596-g94bfc7de5522-dirty #58 SMP PREEMPT Thu Dec 16 11:53:53 CET 2021 armv7l GNU/Linux
> root@devuan-n900:~# ls /sys/class/leds
> debug::sleep lp5523:g lp5523:kb2 lp5523:kb4 lp5523:kb6
> lp5523:b lp5523:kb1 lp5523:kb3 lp5523:kb5 lp5523:r

On 5.1 (my previous 'stable') kernel, these are the led names:

> root@devuan-n900:~# uname -a
> Linux devuan-n900 5.1.21 #1 Mon Jul 19 16:32:44 UTC 2021 armv7l GNU/Linux
> root@devuan-n900:~# ls /sys/class/leds
> debug::sleep lp5523:g lp5523:kb2 lp5523:kb4 lp5523:kb6
> lp5523:b lp5523:kb1 lp5523:kb3 lp5523:kb5 lp5523:r

So I don't think the names are changed.

Regards,
Merlijn