diff mbox series

[3/3] dt-bindings: mmc: marvell,xenon-sdhci: Drop requiring 2 clocks

Message ID 20250320-dt-marvell-mmc-v1-3-e51002ea0238@kernel.org
State New
Headers show
Series dt-bindings: mmc: marvell,xenon-sdhci: Fix several validation warnings | expand

Commit Message

Rob Herring (Arm) March 20, 2025, 9:35 p.m. UTC 
The "axi" clock is optional on at least the "marvell,armada-ap806-sdhci"
variant. Not sure what's correct here, so just drop the constraint.

Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
---
 .../devicetree/bindings/mmc/marvell,xenon-sdhci.yaml | 20 --------------------
 1 file changed, 20 deletions(-)

Comments

Krzysztof Kozlowski March 26, 2025, 8:14 a.m. UTC | #1 
On Thu, Mar 20, 2025 at 04:35:48PM -0500, Rob Herring (Arm) wrote:
> The "axi" clock is optional on at least the "marvell,armada-ap806-sdhci"
> variant. Not sure what's correct here, so just drop the constraint.
> 
> Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
> ---
>  .../devicetree/bindings/mmc/marvell,xenon-sdhci.yaml | 20 --------------------
>  1 file changed, 20 deletions(-)

Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

Best regards,
Krzysztof
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/mmc/marvell,xenon-sdhci.yaml b/Documentation/devicetree/bindings/mmc/marvell,xenon-sdhci.yaml
index e7df7a2a0331..ee2ddef36369 100644
--- a/Documentation/devicetree/bindings/mmc/marvell,xenon-sdhci.yaml
+++ b/Documentation/devicetree/bindings/mmc/marvell,xenon-sdhci.yaml
@@ -169,26 +169,6 @@  allOf:
 
         marvell,pad-type: false
 
-  - if:
-      properties:
-        compatible:
-          contains:
-            enum:
-              - marvell,armada-cp110-sdhci
-              - marvell,armada-ap807-sdhci
-              - marvell,armada-ap806-sdhci
-
-    then:
-      properties:
-        clocks:
-          minItems: 2
-
-        clock-names:
-          items:
-            - const: core
-            - const: axi
-
-
 required:
   - compatible
   - reg