dt-bindings: ufs: qcom: Drop redundant "reg" constraints

[PATCH] dt-bindings: ufs: qcom: Drop redundant "reg" constraints

Posted by Krzysztof Kozlowski 3 months, 2 weeks ago

The "reg" in top-level has maxItems:2, thus repeating this in "if:then:"
blocks is redundant.  Similarly number of items cannot be less than 1.

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
 Documentation/devicetree/bindings/ufs/qcom,ufs.yaml | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/Documentation/devicetree/bindings/ufs/qcom,ufs.yaml b/Documentation/devicetree/bindings/ufs/qcom,ufs.yaml
index 1dd41f6d5258..516bb61a4624 100644
--- a/Documentation/devicetree/bindings/ufs/qcom,ufs.yaml
+++ b/Documentation/devicetree/bindings/ufs/qcom,ufs.yaml
@@ -88,7 +88,6 @@ allOf:
             - const: ice_core_clk
         reg:
           minItems: 2
-          maxItems: 2
         reg-names:
           minItems: 2
       required:
@@ -117,7 +116,6 @@ allOf:
             - const: tx_lane0_sync_clk
             - const: rx_lane0_sync_clk
         reg:
-          minItems: 1
           maxItems: 1
         reg-names:
           maxItems: 1
@@ -147,7 +145,6 @@ allOf:
             - const: ice_core_clk
         reg:
           minItems: 2
-          maxItems: 2
         reg-names:
           minItems: 2
       required:
-- 
2.48.1

Re: [PATCH] dt-bindings: ufs: qcom: Drop redundant "reg" constraints

Posted by Martin K. Petersen 3 months, 1 week ago

On Mon, 27 Oct 2025 12:31:08 +0100, Krzysztof Kozlowski wrote:

> The "reg" in top-level has maxItems:2, thus repeating this in "if:then:"
> blocks is redundant.  Similarly number of items cannot be less than 1.
> 
> 

Applied to 6.19/scsi-queue, thanks!

[1/1] dt-bindings: ufs: qcom: Drop redundant "reg" constraints
      https://git.kernel.org/mkp/scsi/c/525a411f9a5f

-- 
Martin K. Petersen

Re: [PATCH] dt-bindings: ufs: qcom: Drop redundant "reg" constraints

Posted by Martin K. Petersen 3 months, 1 week ago

Krzysztof,

> The "reg" in top-level has maxItems:2, thus repeating this in "if:then:"
> blocks is redundant.  Similarly number of items cannot be less than 1.

Applied to 6.19/scsi-staging, thanks!

-- 
Martin K. Petersen

Re: [PATCH] dt-bindings: ufs: qcom: Drop redundant "reg" constraints

Posted by Rob Herring (Arm) 3 months, 2 weeks ago

On Mon, 27 Oct 2025 12:31:08 +0100, Krzysztof Kozlowski wrote:
> The "reg" in top-level has maxItems:2, thus repeating this in "if:then:"
> blocks is redundant.  Similarly number of items cannot be less than 1.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---
>  Documentation/devicetree/bindings/ufs/qcom,ufs.yaml | 3 ---
>  1 file changed, 3 deletions(-)
> 

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