[3/5] dt-bindings: phy: ti,phy-am654-serdes: drop assigned-clocks type

Message ID 20230404190115.546973-3-krzysztof.kozlowski@linaro.org
State New
Headers
Series [1/5] dt-bindings: phy: cadence-sierra: drop assigned-clocks |

Commit Message

Krzysztof Kozlowski April 4, 2023, 7:01 p.m. UTC
  The meta schema from DT schema already defines assigned-clocks, so there
is no need for device schema to mention it at all.  The specific
parenting of the first input and output clock is apparently important,
thus keep them as required, but without defining type.

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
 .../devicetree/bindings/phy/ti,phy-am654-serdes.yaml         | 5 -----
 1 file changed, 5 deletions(-)
  

Comments

Rob Herring April 6, 2023, 4:48 p.m. UTC | #1
On Tue, Apr 04, 2023 at 09:01:13PM +0200, Krzysztof Kozlowski wrote:
> The meta schema from DT schema already defines assigned-clocks, so there
> is no need for device schema to mention it at all.  The specific
> parenting of the first input and output clock is apparently important,
> thus keep them as required, but without defining type.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---
>  .../devicetree/bindings/phy/ti,phy-am654-serdes.yaml         | 5 -----
>  1 file changed, 5 deletions(-)

Reviewed-by: Rob Herring <robh@kernel.org>
  

Patch

diff --git a/Documentation/devicetree/bindings/phy/ti,phy-am654-serdes.yaml b/Documentation/devicetree/bindings/phy/ti,phy-am654-serdes.yaml
index 738c92bb7518..854e554eae67 100644
--- a/Documentation/devicetree/bindings/phy/ti,phy-am654-serdes.yaml
+++ b/Documentation/devicetree/bindings/phy/ti,phy-am654-serdes.yaml
@@ -34,11 +34,6 @@  properties:
       Three input clocks referring to left input reference clock, refclk and right input reference
       clock.
 
-  assigned-clocks:
-    $ref: "/schemas/types.yaml#/definitions/phandle-array"
-  assigned-clock-parents:
-    $ref: "/schemas/types.yaml#/definitions/phandle-array"
-
   '#phy-cells':
     const: 2
     description: