[OpenWrt-Devel] [PATCH 2/3] lantiq: unify dsl led names
blogic at openwrt.org
Thu Dec 17 08:20:47 EST 2015
On 17/12/2015 13:58, Mathias Kresin wrote:
> 2015-12-17 7:52 GMT+01:00 John Crispin <john at phrozen.org>:
>> 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
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 :)
> openwrt-devel mailing list
> openwrt-devel at lists.openwrt.org
openwrt-devel mailing list
openwrt-devel at lists.openwrt.org
More information about the openwrt-devel