diff mbox series

[v2,01/16] dt-bindings: clock: qcom,gcc-msm8998: drop core_bi_pll_test_se

Message ID 20221228185237.3111988-2-dmitry.baryshkov@linaro.org (mailing list archive)
State Rejected, archived
Headers show
Series clk: qcom: get rid of core_bi_pll_test_se | expand

Commit Message

Dmitry Baryshkov Dec. 28, 2022, 6:52 p.m. UTC 
The test clock apparently it's not used by anyone upstream. Remove it.

Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
---
 .../devicetree/bindings/clock/qcom,gcc-msm8998.yaml         | 6 +-----
 1 file changed, 1 insertion(+), 5 deletions(-)

Comments

Rob Herring (Arm) Jan. 8, 2023, 6:41 p.m. UTC | #1 
On Wed, 28 Dec 2022 20:52:22 +0200, Dmitry Baryshkov wrote:
> The test clock apparently it's not used by anyone upstream. Remove it.
> 
> Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
> ---
>  .../devicetree/bindings/clock/qcom,gcc-msm8998.yaml         | 6 +-----
>  1 file changed, 1 insertion(+), 5 deletions(-)
> 

Reviewed-by: Rob Herring <robh@kernel.org>
Dmitry Baryshkov Jan. 8, 2023, 10:17 p.m. UTC | #2 
On 08/01/2023 20:41, Rob Herring wrote:
> 
> On Wed, 28 Dec 2022 20:52:22 +0200, Dmitry Baryshkov wrote:
>> The test clock apparently it's not used by anyone upstream. Remove it.
>>
>> Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
>> ---
>>   .../devicetree/bindings/clock/qcom,gcc-msm8998.yaml         | 6 +-----
>>   1 file changed, 1 insertion(+), 5 deletions(-)
>>
> 
> Reviewed-by: Rob Herring <robh@kernel.org>

Just at the point when we decided to keep the clock in bindings and just 
drop it from the driver :-(
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/clock/qcom,gcc-msm8998.yaml b/Documentation/devicetree/bindings/clock/qcom,gcc-msm8998.yaml
index 2d5355cf9def..3c9729050d6f 100644
--- a/Documentation/devicetree/bindings/clock/qcom,gcc-msm8998.yaml
+++ b/Documentation/devicetree/bindings/clock/qcom,gcc-msm8998.yaml
@@ -25,7 +25,6 @@  properties:
       - description: Board XO source
       - description: Sleep clock source
       - description: Audio reference clock (Optional clock)
-      - description: PLL test clock source (Optional clock)
     minItems: 2
 
   clock-names:
@@ -33,7 +32,6 @@  properties:
       - const: xo
       - const: sleep_clk
       - const: aud_ref_clk # Optional clock
-      - const: core_bi_pll_test_se # Optional clock
     minItems: 2
 
 required:
@@ -57,11 +55,9 @@  examples:
       reg = <0x00100000 0xb0000>;
       clocks = <&rpmcc RPM_SMD_XO_CLK_SRC>,
                <&sleep>,
-               <0>,
                <0>;
       clock-names = "xo",
                     "sleep_clk",
-                    "aud_ref_clk",
-                    "core_bi_pll_test_se";
+                    "aud_ref_clk";
     };
 ...