Re: [PATCH 1/2] dt-bindings: pwm: amlogic: Document C3 PWM

[Kelvin Zhang]

On 2024/9/6 20:50, Krzysztof Kozlowski wrote:
> [ EXTERNAL EMAIL ]
>
> On 06/09/2024 14:46, Kelvin Zhang via B4 Relay wrote:
> > From: Kelvin Zhang <kelvin.zhang@xxxxxxxxxxx>
> >
> > Document amlogic,c3-pwm compatible, which falls back to the meson-s4-pwm
> > group.
> >
> > Signed-off-by: Kelvin Zhang <kelvin.zhang@xxxxxxxxxxx>
> > ---
> >   Documentation/devicetree/bindings/pwm/pwm-amlogic.yaml | 4 ++++
> >   1 file changed, 4 insertions(+)
> >
> > diff --git a/Documentation/devicetree/bindings/pwm/pwm-amlogic.yaml b/Documentation/devicetree/bindings/pwm/pwm-amlogic.yaml
> > index 1d71d4f8f328..356371164acd 100644
> > --- a/Documentation/devicetree/bindings/pwm/pwm-amlogic.yaml
> > +++ b/Documentation/devicetree/bindings/pwm/pwm-amlogic.yaml
> > @@ -44,6 +44,10 @@ properties:
> >                 - amlogic,meson-axg-pwm-v2
> >                 - amlogic,meson-g12-pwm-v2
> >             - const: amlogic,meson8-pwm-v2
> > +      - items:
> > +          - enum:
> > +              - amlogic,c3-pwm
>
> Assuming that lack of meson fits all other instances of c3:
That's right. C3 does not belong to the meson series.
Thanks for your review!

> Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx>
>
> Best regards,
> Krzysztof