dt-bindings: phy: qmp-ufs: tweak clock and clock-names for sa8775p

[PATCH v2] dt-bindings: phy: qmp-ufs: tweak clock and clock-names for sa8775p

Posted by Bartosz Golaszewski 2 years, 8 months ago

From: Bartosz Golaszewski <bartosz.golaszewski@linaro.org>

maxItems is already globally set to 3. To make the binding easier to read
and remove redundancy, set minItems to 3 for sa8775p as this platform
requires exactly three clocks.

Signed-off-by: Bartosz Golaszewski <bartosz.golaszewski@linaro.org>
---
v1 -> v2:
- rephrased the commit message as this is not a fix but rather
  a readability improvement

 .../devicetree/bindings/phy/qcom,sc8280xp-qmp-ufs-phy.yaml    | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/Documentation/devicetree/bindings/phy/qcom,sc8280xp-qmp-ufs-phy.yaml b/Documentation/devicetree/bindings/phy/qcom,sc8280xp-qmp-ufs-phy.yaml
index 94c0fab065a8..a1897a7606df 100644
--- a/Documentation/devicetree/bindings/phy/qcom,sc8280xp-qmp-ufs-phy.yaml
+++ b/Documentation/devicetree/bindings/phy/qcom,sc8280xp-qmp-ufs-phy.yaml
@@ -78,9 +78,9 @@ allOf:
     then:
       properties:
         clocks:
-          maxItems: 3
+          minItems: 3
         clock-names:
-          maxItems: 3
+          minItems: 3
     else:
       properties:
         clocks:
-- 
2.37.2

Re: [PATCH v2] dt-bindings: phy: qmp-ufs: tweak clock and clock-names for sa8775p

Posted by Vinod Koul 2 years, 7 months ago

On 19-04-23, 14:09, Bartosz Golaszewski wrote:
> From: Bartosz Golaszewski <bartosz.golaszewski@linaro.org>
> 
> maxItems is already globally set to 3. To make the binding easier to read
> and remove redundancy, set minItems to 3 for sa8775p as this platform
> requires exactly three clocks.

Applied, thanks

-- 
~Vinod

Re: [PATCH v2] dt-bindings: phy: qmp-ufs: tweak clock and clock-names for sa8775p

Posted by Rob Herring 2 years, 7 months ago

On Wed, 19 Apr 2023 14:09:14 +0200, Bartosz Golaszewski wrote:
> From: Bartosz Golaszewski <bartosz.golaszewski@linaro.org>
> 
> maxItems is already globally set to 3. To make the binding easier to read
> and remove redundancy, set minItems to 3 for sa8775p as this platform
> requires exactly three clocks.
> 
> Signed-off-by: Bartosz Golaszewski <bartosz.golaszewski@linaro.org>
> ---
> v1 -> v2:
> - rephrased the commit message as this is not a fix but rather
>   a readability improvement
> 
>  .../devicetree/bindings/phy/qcom,sc8280xp-qmp-ufs-phy.yaml    | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 

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