Re: [PATCH RFC 1/2] dt-bindings: leds-group-multicolor: Introduce default-intensity

Conor Dooley <[email protected]>
Newsgroups org.kernel.vger.linux-leds,org.kernel.vger.linux-devicetree
Message-ID <20260709-outsell-undamaged-6c8045d6115f@spud>
On Thu, Jul 09, 2026 at 12:46:51AM +0200, Stefan Wahren wrote:
> Currently it's not possible to specify the initial color of a LED group
> during boot. So introduce a new property similar to default-brightness,
> which specifies the intensity of each LED in the group.
> 
> Signed-off-by: Stefan Wahren <[email protected]>
> ---
>  .../devicetree/bindings/leds/leds-group-multicolor.yaml    | 7 +++++++
>  1 file changed, 7 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/leds/leds-group-multicolor.yaml b/Documentation/devicetree/bindings/leds/leds-group-multicolor.yaml
> index 5c9cfa39396b..18b722b807ba 100644
> --- a/Documentation/devicetree/bindings/leds/leds-group-multicolor.yaml
> +++ b/Documentation/devicetree/bindings/leds/leds-group-multicolor.yaml
> @@ -19,6 +19,12 @@ properties:
>  
>    leds: true
>  
> +  default-intensity:
> +    description:
> +      Intensity to be set for each individual LED. Used only during
> +      initialization. If the property is not set then max intensity is used.
> +    $ref: /schemas/types.yaml#/definitions/uint32-array

Hmm, there's already a property proposed and applied for this:
https://lore.kernel.org/linux-leds/[email protected]/
It only supports a single value, I suspect you'll have to rework this so
as not to conflict with existing property?
leds-group-multicolour imports the common leds properties after all.
Probably the one in common.yaml needs to become an array?

> +
>  required:
>    - leds
>  
> @@ -56,6 +62,7 @@ examples:
>          color = <LED_COLOR_ID_RGB>;
>          function = LED_FUNCTION_INDICATOR;
>          leds = <&led0>, <&led1>, <&led2>;
> +        default-intensity = <1 0 0>;
>      };
>  
>  ...
> -- 
> 2.43.0
>
signature.asc (application/pgp-signature, 228 B)
-----BEGIN PGP SIGNATURE-----

iHUEABYKAB0WIQRh246EGq/8RLhDjO14tDGHoIJi0gUCak/igwAKCRB4tDGHoIJi
0kYvAQDFqW0granAQsEhZOoMpPAZ5c+XR6miwCiQip2Plg32FgEAll9FCkvOHIQ+
1XcfbmM057Nfx+7JZO0CESBxAGjoywE=
=tmFt
-----END PGP SIGNATURE-----
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.