Re: [PATCH] Documentation: dt-bindings: leds: document 'default-on' trigger

["Ulrich Ölmann" <u.oelmann@pengutronix.de>]

Hi Roland,

On Tue, Nov 21 2023 at 17:20 +0100, Roland Hieber <rhi@pengutronix.de> wrote:
> The default-on trigger has been supported in barebox since commit
> 767c6b4a814a2a000f3b (2014-02-28, Sascha Hauer: "led: Add default-on
> trigger").
>
> Signed-off-by: Roland Hieber <rhi@pengutronix.de>
> ---
>  Documentation/devicetree/bindings/leds/common.rst | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/Documentation/devicetree/bindings/leds/common.rst b/Documentation/devicetree/bindings/leds/common.rst
> index 09b4e401bc2c..ad94ceb506b1 100644
> --- a/Documentation/devicetree/bindings/leds/common.rst
> +++ b/Documentation/devicetree/bindings/leds/common.rst
> @@ -9,6 +9,7 @@ Common leds properties
>      * ``net`` - LED indicates network activity (tx and rx)
>      * ``net-rx`` - LED indicates network activity (rx only)
>      * ``net-tx`` - LED indicates network activity (tx only)
> +    * ``default-on`` – LED is switched on by default

the lines before utilize the character '-' while in your new line a '–'
is used. I think this should be consistent in one or the other
direction.

Best regards
Ulrich


>  * ``label``: The label for this LED. If omitted, the label is taken
>    from the node name (excluding the unit address).
-- 
Pengutronix e.K.                           | Ulrich Ölmann               |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |