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

Kelvin Zhang <kelvin.zhang@xxxxxxxxxxx> · Mon, 9 Sep 2024 18:43:50 +0800

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









