| Message ID | 20230623113009.2512206-2-abel.vesa@linaro.org (mailing list archive) |
|---|---|
| State | Deferred |
| Headers | show |
| Series | scsi: dt-bindings: ufs: qcom: Some fixes to clear all dtbs_check warnings | expand |
On 23/06/2023 13:30, Abel Vesa wrote: > The check for 'qcom,ice' property is wrong. Fix it by checking using > if-required clause and expand the clocks minItems and maxItems for > platforms where 'qcom,ice' is not required so that it includes platforms > with single reg entry and clocks that do not provide an ICE one. > > Fixes: 29a6d1215b7c ("scsi: ufs: dt-bindings: qcom: Add ICE phandle") > Signed-off-by: Abel Vesa <abel.vesa@linaro.org> > --- > Documentation/devicetree/bindings/ufs/qcom,ufs.yaml | 9 ++++---- Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> Best regards, Krzysztof
diff --git a/Documentation/devicetree/bindings/ufs/qcom,ufs.yaml b/Documentation/devicetree/bindings/ufs/qcom,ufs.yaml index 943dafb69529..bdfa86a0cc98 100644 --- a/Documentation/devicetree/bindings/ufs/qcom,ufs.yaml +++ b/Documentation/devicetree/bindings/ufs/qcom,ufs.yaml @@ -194,9 +194,8 @@ allOf: # TODO: define clock bindings for qcom,msm8994-ufshc - if: - properties: - qcom,ice: - maxItems: 1 + required: + - qcom,ice then: properties: reg: @@ -207,10 +206,10 @@ allOf: else: properties: reg: - minItems: 2 + minItems: 1 maxItems: 2 clocks: - minItems: 9 + minItems: 8 maxItems: 11 unevaluatedProperties: false
The check for 'qcom,ice' property is wrong. Fix it by checking using if-required clause and expand the clocks minItems and maxItems for platforms where 'qcom,ice' is not required so that it includes platforms with single reg entry and clocks that do not provide an ICE one. Fixes: 29a6d1215b7c ("scsi: ufs: dt-bindings: qcom: Add ICE phandle") Signed-off-by: Abel Vesa <abel.vesa@linaro.org> --- Documentation/devicetree/bindings/ufs/qcom,ufs.yaml | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-)