[PATCH v5 4/6] dt-bindings: display/msm: drop assigned-clock-parents for dp controller

Dmitry Baryshkov posted 6 patches 1 month, 3 weeks ago
There is a newer version of this series
[PATCH v5 4/6] dt-bindings: display/msm: drop assigned-clock-parents for dp controller
Posted by Dmitry Baryshkov 1 month, 3 weeks ago
From: Abhinav Kumar <quic_abhinavk@quicinc.com>

DP controller schema documents assigned-clocks and
assigned-clock-parents. However these assignments should not be a part
of the ABI: there are no actual requirements on the order of the
assignments, MST cases require different number of clocks to be
assigned, etc.

Instead of fixing up the documentation, drop the assigned-clock-parents
and assigned-clocks from the bindings. The generic clock/clock.yaml
already covers these properties.

Suggested-by: Krzysztof Kozlowski <krzk@kernel.org>
Signed-off-by: Abhinav Kumar <quic_abhinavk@quicinc.com>
Signed-off-by: Jessica Zhang <jessica.zhang@oss.qualcomm.com>
Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com>
---
 .../devicetree/bindings/display/msm/dp-controller.yaml         | 10 ----------
 1 file changed, 10 deletions(-)

diff --git a/Documentation/devicetree/bindings/display/msm/dp-controller.yaml b/Documentation/devicetree/bindings/display/msm/dp-controller.yaml
index 55e37ec74591af0a1329598f6059475926fdd64e..352824c245224d7b5e096770684795625fc9f146 100644
--- a/Documentation/devicetree/bindings/display/msm/dp-controller.yaml
+++ b/Documentation/devicetree/bindings/display/msm/dp-controller.yaml
@@ -73,16 +73,6 @@ properties:
       - const: ctrl_link_iface
       - const: stream_pixel
 
-  assigned-clocks:
-    items:
-      - description: link clock source
-      - description: pixel clock source
-
-  assigned-clock-parents:
-    items:
-      - description: phy 0 parent
-      - description: phy 1 parent
-
   phys:
     maxItems: 1
 

-- 
2.39.5
Re: [PATCH v5 4/6] dt-bindings: display/msm: drop assigned-clock-parents for dp controller
Posted by Rob Herring (Arm) 1 month, 3 weeks ago
On Sat, 09 Aug 2025 12:16:18 +0300, Dmitry Baryshkov wrote:
> From: Abhinav Kumar <quic_abhinavk@quicinc.com>
> 
> DP controller schema documents assigned-clocks and
> assigned-clock-parents. However these assignments should not be a part
> of the ABI: there are no actual requirements on the order of the
> assignments, MST cases require different number of clocks to be
> assigned, etc.
> 
> Instead of fixing up the documentation, drop the assigned-clock-parents
> and assigned-clocks from the bindings. The generic clock/clock.yaml
> already covers these properties.
> 
> Suggested-by: Krzysztof Kozlowski <krzk@kernel.org>
> Signed-off-by: Abhinav Kumar <quic_abhinavk@quicinc.com>
> Signed-off-by: Jessica Zhang <jessica.zhang@oss.qualcomm.com>
> Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com>
> ---
>  .../devicetree/bindings/display/msm/dp-controller.yaml         | 10 ----------
>  1 file changed, 10 deletions(-)
> 

Acked-by: Rob Herring (Arm) <robh@kernel.org>