On 17/12/2015 13:58, Mathias Kresin wrote:
> 2015-12-17 7:52 GMT+01:00 John Crispin <j...@phrozen.org>:
>> Hi,
>>
>> we had a discussion yesterday with a few people and agreed to cleanup
>> the led names on for the lantiq target to match the format expected by
>> the kernel. i like this patch however we need to do the led renaming
>> first i guess.
>
> From my point of view, a configurable DSL led is a precondition for
> renaming the leds. but I'm sure you have a better general view than I.
> I'm fine to skip this patch of the series and fix the DSL led of the
> affected boards after renaming the leds.
>
> I totally agree with you, that the led naming is suboptimal. I'm
> currently working on adding support for systems which are using three
> leds for boot state indication. I had to rewrite the whole diag.sh,
> with the benefit that we can get rid of the static power[0-2] naming
> as well. With having led names like power[0-2] without any indication
> which colour the led has, it's a mess to ensure that I don't break
> existing power led setups.
>
> Am I right that you are talking about a naming scheme like
> "devicename:colour:function" as it's suggested by
> https://www.kernel.org/doc/Documentation/leds/leds-class.txt?
correct, we cleaned up ralink recently and i would like to see the ame
happen for lantiq
>
> Are there any known limitations for the led node names, which are
> available at /sys/class/leds/*/device/of_node/? Maybe we can use node
> names according to the intended (OpenWrt) use and prevent bloating the
> board.d led file this way after switching to the
> devicename:colour:function naming.
>
> Are you working on renaming the leds or are you wait for patches?
not actively working on it now, we made a short todo list for lantiq for
the DD release which will be based on v4.4. if you have the time, then
go for it :)
John
> _______________________________________________
> openwrt-devel mailing list
> openwrt-devel@lists.openwrt.org
> https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel
>
_______________________________________________
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel
