[v1] Clean up RPM bus clocks remnants

[PATCH 06/11] dt-bindings: remoteproc: qcom,adsp: Remove AGGRE2 clock

Posted by Konrad Dybcio 2 years, 6 months ago

The AGGRE2 clock is a clock for the entire AGGRE2 bus, managed from
within the interconnect driver. Attaching it to SLPI was a total hack.
Get rid of it.

Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
---
 Documentation/devicetree/bindings/remoteproc/qcom,adsp.yaml | 2 --
 1 file changed, 2 deletions(-)

diff --git a/Documentation/devicetree/bindings/remoteproc/qcom,adsp.yaml b/Documentation/devicetree/bindings/remoteproc/qcom,adsp.yaml
index a2b0079de039..af78f76734db 100644
--- a/Documentation/devicetree/bindings/remoteproc/qcom,adsp.yaml
+++ b/Documentation/devicetree/bindings/remoteproc/qcom,adsp.yaml
@@ -91,11 +91,9 @@ allOf:
         clocks:
           items:
             - description: XO clock
-            - description: AGGRE2 clock
         clock-names:
           items:
             - const: xo
-            - const: aggre2
 
   - if:
       properties:

-- 
2.41.0

Re: [PATCH 06/11] dt-bindings: remoteproc: qcom,adsp: Remove AGGRE2 clock

Posted by Krzysztof Kozlowski 2 years, 6 months ago

On 21/07/2023 17:36, Konrad Dybcio wrote:
> The AGGRE2 clock is a clock for the entire AGGRE2 bus, managed from
> within the interconnect driver. Attaching it to SLPI was a total hack.
> Get rid of it.
> 
> Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
> ---
>  Documentation/devicetree/bindings/remoteproc/qcom,adsp.yaml | 2 --
>  1 file changed, 2 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/remoteproc/qcom,adsp.yaml b/Documentation/devicetree/bindings/remoteproc/qcom,adsp.yaml
> index a2b0079de039..af78f76734db 100644
> --- a/Documentation/devicetree/bindings/remoteproc/qcom,adsp.yaml
> +++ b/Documentation/devicetree/bindings/remoteproc/qcom,adsp.yaml
> @@ -91,11 +91,9 @@ allOf:
>          clocks:
>            items:
>              - description: XO clock
> -            - description: AGGRE2 clock

I think this entry can be squashed now with previous if:then:.

Best regards,
Krzysztof