On Thu, Jan 23, 2025 at 12:49:06PM +0100, Heiko Schocher wrote:
in linux we have the option to create the name of a led optionally through the following properties:
- function
- color
- function-enumerator
This patch adds support for parsing this properties if there is no label property.
To be as close as possible to linux import the following files from it:
include/dt-bindings/leds/common.h include/linux/uapi/linux/uleds.h
[snip]
as we introduce a new element in struct led_uc_priv, may we should add this functionality through a new Kconfig option? (Which also would make this optional and save some bytes for boards who do not need this...)
[snip]
arch/sandbox/dts/test.dts | 37 ++++++++++++- doc/device-tree-bindings/leds/common.txt | 28 ++++++++++ drivers/led/led-uclass.c | 70 ++++++++++++++++++++++-- include/led.h | 6 ++ test/dm/led.c | 22 +++++++- test/dm/ofnode.c | 2 +- 6 files changed, 158 insertions(+), 7 deletions(-)
Getting things out of order slightly, sorry. The commit message should be updated a bit more still since we aren't copying files in. And we should delete doc/device-tree-bindings/leds/common.txt in a pre-patch because we have that from upstream now. As for size, how much is the growth? You can use tools/qconfig.py -b, and then -f LED to find some platforms to size compare before/after, and if you've not used buildman for size checking before I've put my script up over at https://source.denx.de/u-boot/u-boot-extras/-/raw/master/contrib/trini/u-boo... now for easier sharing.