* [PATCH] dt-bindings: ufs: qcom: Drop redundant "reg" constraints
@ 2025-10-27 11:31 Krzysztof Kozlowski
2025-10-27 20:28 ` Rob Herring (Arm)
` (2 more replies)
0 siblings, 3 replies; 4+ messages in thread
From: Krzysztof Kozlowski @ 2025-10-27 11:31 UTC (permalink / raw)
To: Manivannan Sadhasivam, Alim Akhtar, Avri Altman, Bart Van Assche,
Rob Herring, Krzysztof Kozlowski, Conor Dooley, Bjorn Andersson,
Andy Gross, linux-arm-msm, linux-scsi, devicetree, linux-kernel
Cc: Krzysztof Kozlowski
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
^ permalink raw reply related [flat|nested] 4+ messages in thread* Re: [PATCH] dt-bindings: ufs: qcom: Drop redundant "reg" constraints
2025-10-27 11:31 [PATCH] dt-bindings: ufs: qcom: Drop redundant "reg" constraints Krzysztof Kozlowski
@ 2025-10-27 20:28 ` Rob Herring (Arm)
2025-10-30 3:23 ` Martin K. Petersen
2025-11-03 3:29 ` Martin K. Petersen
2 siblings, 0 replies; 4+ messages in thread
From: Rob Herring (Arm) @ 2025-10-27 20:28 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Krzysztof Kozlowski, Conor Dooley, Bjorn Andersson, devicetree,
Avri Altman, linux-arm-msm, Alim Akhtar, linux-kernel,
Bart Van Assche, linux-scsi, Manivannan Sadhasivam, Andy Gross
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>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] dt-bindings: ufs: qcom: Drop redundant "reg" constraints
2025-10-27 11:31 [PATCH] dt-bindings: ufs: qcom: Drop redundant "reg" constraints Krzysztof Kozlowski
2025-10-27 20:28 ` Rob Herring (Arm)
@ 2025-10-30 3:23 ` Martin K. Petersen
2025-11-03 3:29 ` Martin K. Petersen
2 siblings, 0 replies; 4+ messages in thread
From: Martin K. Petersen @ 2025-10-30 3:23 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Manivannan Sadhasivam, Alim Akhtar, Avri Altman, Bart Van Assche,
Rob Herring, Krzysztof Kozlowski, Conor Dooley, Bjorn Andersson,
Andy Gross, linux-arm-msm, linux-scsi, devicetree, linux-kernel
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
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] dt-bindings: ufs: qcom: Drop redundant "reg" constraints
2025-10-27 11:31 [PATCH] dt-bindings: ufs: qcom: Drop redundant "reg" constraints Krzysztof Kozlowski
2025-10-27 20:28 ` Rob Herring (Arm)
2025-10-30 3:23 ` Martin K. Petersen
@ 2025-11-03 3:29 ` Martin K. Petersen
2 siblings, 0 replies; 4+ messages in thread
From: Martin K. Petersen @ 2025-11-03 3:29 UTC (permalink / raw)
To: Manivannan Sadhasivam, Alim Akhtar, Avri Altman, Bart Van Assche,
Rob Herring, Krzysztof Kozlowski, Conor Dooley, Bjorn Andersson,
Andy Gross, linux-arm-msm, linux-scsi, devicetree, linux-kernel,
Krzysztof Kozlowski
Cc: Martin K . Petersen
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
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2025-11-03 3:29 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-10-27 11:31 [PATCH] dt-bindings: ufs: qcom: Drop redundant "reg" constraints Krzysztof Kozlowski
2025-10-27 20:28 ` Rob Herring (Arm)
2025-10-30 3:23 ` Martin K. Petersen
2025-11-03 3:29 ` Martin K. Petersen
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox