Re: [PATCH 1/2] dt-bindings: leds: sgm3140: Document richtek,rt5033 compatible

[Lee Jones]

On Wed, 21 Jun 2023, Conor Dooley wrote:

> Raymond,
> 
> On Wed, Jun 21, 2023 at 04:03:02PM +0100, Lee Jones wrote:
> > On Wed, 21 Jun 2023, Lee Jones wrote:
> > 
> > > On Fri, 02 Jun 2023, Raymond Hackley wrote:
> > > 
> > > > Add devicetree binding for Richtek RT5033 Flash LED charge pump used for
> > > > camera flash LEDs.
> > > > 
> > > > Signed-off-by: Raymond Hackley <raymondhackley@xxxxxxxxxxxxxx>
> > > > ---
> > > >  Documentation/devicetree/bindings/leds/leds-sgm3140.yaml | 1 +
> > > >  1 file changed, 1 insertion(+)
> > > > 
> > > > diff --git a/Documentation/devicetree/bindings/leds/leds-sgm3140.yaml b/Documentation/devicetree/bindings/leds/leds-sgm3140.yaml
> > > > index 4d2ffe5fcfc7..37d2a93780ab 100644
> > > > --- a/Documentation/devicetree/bindings/leds/leds-sgm3140.yaml
> > > > +++ b/Documentation/devicetree/bindings/leds/leds-sgm3140.yaml
> > > > @@ -20,6 +20,7 @@ properties:
> > > >    compatible:
> > > >      enum:
> > > >        - ocs,ocp8110
> > > > +      - richtek,rt5033-led
> > > 
> > > Why is "-led" appended on to this one and not the others?
> > 
> > It's unusual for me to have to dump through a bunch of hoops just to
> > read and then subsequently reply to mail.  Can you consider removing
> > whatever encryption you have enabled please?
> 
> https://www.kernel.org/doc/html/latest/process/email-clients.html#proton-mail

Thanks for the link.  This is perfect!

At least I know it's not just me being unreasonable.

> I was under the impression that there was no way to disable this
> behaviour, but I saw mention somewhere that they managed to patch the
> open source version of the protonmail bridge to function. I cannot find
> this on lore, because the person's domain was not a proton one and I
> completely forget their name as it was several months ago.
> I'll reply here if I find it.

Thanks.

-- 
Lee Jones [李琼斯]