[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <3kww5et2q2mqddpvtqzuj3jqzvfds66qrufawcmumamrqoaugk@tiq6zoe5psom>
Date: Wed, 24 Sep 2025 22:45:37 -0700
From: Dmitry Torokhov <dmitry.torokhov@...il.com>
To: Luca Weiss <luca@...aweiss.eu>
Cc: Krzysztof Kozlowski <krzk@...nel.org>, Rob Herring <robh@...nel.org>,
Krzysztof Kozlowski <krzk+dt@...nel.org>, Conor Dooley <conor+dt@...nel.org>,
Courtney Cavin <courtney.cavin@...ymobile.com>, Vinod Koul <vkoul@...nel.org>, Bhushan Shah <bshah@....org>,
~postmarketos/upstreaming@...ts.sr.ht, phone-devel@...r.kernel.org, linux-arm-msm@...r.kernel.org,
linux-input@...r.kernel.org, devicetree@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH 1/2] dt-bindings: input: pm8941-pwrkey: Document
wakeup-source property
On Tue, Sep 09, 2025 at 07:54:33AM -0700, Dmitry Torokhov wrote:
> On Tue, Sep 09, 2025 at 04:41:26PM +0200, Luca Weiss wrote:
> > On 2025-09-09 16:33, Krzysztof Kozlowski wrote:
> > > On 09/09/2025 16:08, Dmitry Torokhov wrote:
> > > > > > compatible:
> > > > > > enum:
> > > > > > @@ -36,6 +33,11 @@ properties:
> > > > > > pin should be configured for pull up.
> > > > > > $ref: /schemas/types.yaml#/definitions/flag
> > > > > >
> > > > > > + wakeup-source:
> > > > > > + description: |
> > > > > > + Button can wake-up the system. Only applicable
> > > > > > for 'resin',
> > > > > > + 'pwrkey' always wakes the system by default.
> > > > >
> > > > >
> > > > > I'll fix existing code, so don't repeat that style.
> > > >
> > > > If you ack I can reformat on my side to match the patch you just sent.
> > >
> > > Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
> >
> > Thanks for fixing that up Krzysztof! I noticed but didn't want to deviate
> > from the style just for this description. Of course better to fix the
> > formatting in the first place.
> >
> > @Dmitry: Maybe give this patch some time (1-2 weeks?) to gather more
> > feedback,
> > given the reasons outlined in the cover letter. Also on the driver patch.
>
> OK, I'll hold on to this for a couple of weeks.
Nobody voiced any objections so far, so applied both.
--
Dmitry
Powered by blists - more mailing lists