RE: [PATCH V2 1/2] dt-bindings: leds: pwm: Add enable-gpios property

[LI Qingwu]

> -----Original Message-----
> From: Krzysztof Kozlowski <krzk@xxxxxxxxxx>
> Sent: Thursday, July 3, 2025 3:12 PM
> To: LI Qingwu <Qing-wu.Li@xxxxxxxxxxxxxxxxxxxxxxx>; Rob Herring (Arm)
> <robh@xxxxxxxxxx>
> Cc: krzk+dt@xxxxxxxxxx; conor+dt@xxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx;
> GEO-CHHER-bsp-development <bsp-development.geo@xxxxxxxxxxxxxxxxxxxx>;
> lee@xxxxxxxxxx; devicetree@xxxxxxxxxxxxxxx; pavel@xxxxxxxxxx;
> linux-leds@xxxxxxxxxxxxxxx
> Subject: Re: [PATCH V2 1/2] dt-bindings: leds: pwm: Add enable-gpios property
> 
> This email is not from Hexagon’s Office 365 instance. Please be careful while
> clicking links, opening attachments, or replying to this email.
> 
> 
> On 03/07/2025 05:54, LI Qingwu wrote:
> >>
> >> On Wed, 02 Jul 2025 19:47:58 +0800, LI Qingwu wrote:
> >>> Some PWM LED chips have a dedicated enable GPIO.
> >>> This commit adds the support to specify such GPIO.
> >>>
> >>> Signed-off-by: LI Qingwu <Qing-wu.Li@xxxxxxxxxxxxxxxxxxxxxxx>
> >>> ---
> >>>  Documentation/devicetree/bindings/leds/leds-pwm.yaml | 8 ++++++++
> >>>  1 file changed, 8 insertions(+)
> >>>
> >>
> >> My bot found errors running 'make dt_binding_check' on your patch:
> >>
> >> yamllint warnings/errors:
> >> ./Documentation/devicetree/bindings/leds/leds-pwm.yaml:44:21: [error]
> >> empty value in block mapping (empty-values)
> >
> > Fixed in V3
> 
> So your answer to my "never tested" was to send again the same as v2 and still
> not tested?

Nop,  "Fixed in V3" answer "bot found errors running 'make dt_binding_check'"
Testing info comes later in v4

> 
> Best regards,
> Krzysztof