Messages in this thread | | | Date | Sat, 27 Mar 2021 23:19:25 +0100 | From | Pavel Machek <> | Subject | Re: [PATCH 17/17] auxdisplay: ht16k33: Add segment display LED support |
| |
Hi!
> > > > > - err = ht16k33_brightness_set(priv, MAX_BRIGHTNESS); > > > > > + of_property_read_u32(node, "color", &color); > > > > > + seg->led.name = devm_kasprintf(dev, GFP_KERNEL, > > > > > + DRIVER_NAME ":%s:" LED_FUNCTION_BACKLIGHT, > > > > > + color < LED_COLOR_ID_MAX ? led_colors[color] : ""); > > > > And would prefer not to see driver_name as part of LED name. > > OK. So what should I use instead? Or just leave the part before the > first colon empty?
I'd suggest auxdisplay:...:backlight.
And we should start documenting this somewhere.
Best regards, Pavel -- http://www.livejournal.com/~pavelmachek [unhandled content-type:application/pgp-signature] | |