Re: [PATCH v2 5/6] dt-bindings: ufs: qcom,ufs: drop source clock entries

Manivannan Sadhasivam
2024-02-16T14:06:22Z

On Tue, Feb 13, 2024 at 01:22:21PM +0200, Dmitry Baryshkov wrote:
> There is no need to mention and/or to touch in any way the intermediate
> (source) clocks. Drop them from MSM8996 UFSHCD schema, making it follow
> the example lead by all other platforms.
>
> Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>

Acked-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>

- Mani

> ---
> Documentation/devicetree/bindings/ufs/qcom,ufs.yaml | 12 +++++-------
> 1 file changed, 5 insertions(+), 7 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/ufs/qcom,ufs.yaml b/Documentation/devicetree/bindings/ufs/qcom,ufs.yaml
> index 9a4f2b43e155..649f7951ac01 100644
> --- a/Documentation/devicetree/bindings/ufs/qcom,ufs.yaml
> +++ b/Documentation/devicetree/bindings/ufs/qcom,ufs.yaml
> @@ -44,11 +44,11 @@ properties:
>
> clocks:
> minItems: 8
> - maxItems: 11
> + maxItems: 9
>
> clock-names:
> minItems: 8
> - maxItems: 11
> + maxItems: 9
>
> dma-coherent: true
>
> @@ -189,16 +189,14 @@ allOf:
> then:
> properties:
> clocks:
> - minItems: 11
> - maxItems: 11
> + minItems: 9
> + maxItems: 9
> clock-names:
> items:
> - - const: core_clk_src
> - const: core_clk
> - const: bus_clk
> - const: bus_aggr_clk
> - const: iface_clk
> - - const: core_clk_unipro_src
> - const: core_clk_unipro
> - const: core_clk_ice
> - const: ref_clk
> @@ -259,7 +257,7 @@ allOf:
> maxItems: 2
> clocks:
> minItems: 8
> - maxItems: 11
> + maxItems: 9
>
> unevaluatedProperties: false
>
>
> --
> 2.39.2
>

--
மணிவண்ணன் சதாசிவம்