Pavel On 12/15/2017 03:09 AM, Pavel Machek wrote: > Hi! > >> @@ -35,6 +37,7 @@ led-controller@2d { >> led@0 { >> reg = <0>; >> label = "white:display_cluster"; >> + linux,default-trigger = "backlight"; >> }; >> } > > What is "display_cluster"? Is it display backlight? > This device can be configured to drive all 4 LED strings in display mode so that all 4 strings have the same brightness controlled either through PWM or through the brightness register or As cluster string where the brightness of individual strings or clustered strings can be controlled via independent brightness control registers. I am currently working on adding this support to the driver. Dan > Could we use ":backlight" and perhaps start documentation file listing > "common" LED descriptions so they are shared across machines? > > For the record, I'd prefer this to be > "/sys/class/leds/display:white:backlight" in the end... so that we > have nice naming for userspace, consistent between different machines. > > Pavel > -- ------------------ Dan Murphy -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html