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

[Roland Hieber <rhi@pengutronix.de>]

On Wed, Nov 22, 2023 at 06:24:21AM +0100, Ulrich Ölmann wrote:
> 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.

Huh yes… these two characters you wrote look the same on my terminal,
but they are indeed different when I paste them somewhere else!

 - Roland

-- 
Roland Hieber, Pengutronix e.K.          | r.hieber@pengutronix.de     |
Steuerwalder Str. 21                     | https://www.pengutronix.de/ |
31137 Hildesheim, Germany                | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686         | Fax:   +49-5121-206917-5555 |