Christian Lamparter <[email protected]> [2018-12-31 17:41:34]:
> I hope you know what you are up against because unless you also do the changes
> upstream this will happen again and again. :\ / :)
My plan is to first wait for comments here, see if it gets merged eventualy
and then start poking upstream. I still didn't received any feedback yet(good
sign?) on my last `treewide: dts: Remove default-state=off property...`[1]
upstream attempt so I don't know if it's worth the effort.
Anyway, I guess, that in most of the cases, people are just copy&pasting from
the DTS files from the OpenWrt repository and some of them even wonder[2] why
they need to use generic `leds` node names if it's not the case in the rest of
the DTS files in OpenWrt tree:
@mkresin: would you please rename the node to the generic "leds".
@arapov: @mkresin, by the way, the rest of *.dts in ramips/ using gpio-leds.
Do you still think it is good to deviate from the rest here?
1. https://patchwork.kernel.org/patch/10732465/
2. https://github.com/openwrt/openwrt/pull/1686#discussion_r244512451
3. https://openwrt.org/submitting-patches#dts_checklist
-- ynezz
_______________________________________________
openwrt-devel mailing list
[email protected]
https://lists.openwrt.org/mailman/listinfo/openwrt-devel
