On Fri, Mar 24, 2023 at 08:41:29AM +0100, Luca Weiss wrote: > The code in ufs-qcom-ice.c needs the ICE reg to be named "ice". Add this > in the bindings so the existing dts can validate successfully. > > Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx> > Signed-off-by: Luca Weiss <luca.weiss@xxxxxxxxxxxxx> > --- > Documentation/devicetree/bindings/ufs/qcom,ufs.yaml | 20 ++++++++++++++++++++ > 1 file changed, 20 insertions(+) > > diff --git a/Documentation/devicetree/bindings/ufs/qcom,ufs.yaml b/Documentation/devicetree/bindings/ufs/qcom,ufs.yaml > index 23447281deec..ebc8e1adbc6f 100644 > --- a/Documentation/devicetree/bindings/ufs/qcom,ufs.yaml > +++ b/Documentation/devicetree/bindings/ufs/qcom,ufs.yaml > @@ -75,6 +75,10 @@ properties: > minItems: 1 > maxItems: 2 > > + reg-names: > + minItems: 1 > + maxItems: 2 > + > required-opps: > maxItems: 1 > > @@ -129,6 +133,8 @@ allOf: > reg: > minItems: 1 > maxItems: 1 > + reg-names: > + maxItems: 1 > > - if: > properties: > @@ -157,6 +163,12 @@ allOf: > reg: > minItems: 2 > maxItems: 2 > + reg-names: > + items: > + - const: std > + - const: ice > + required: > + - reg-names > > - if: > properties: > @@ -185,6 +197,8 @@ allOf: > reg: > minItems: 1 > maxItems: 1 > + reg-names: > + maxItems: 1 > > - if: > properties: > @@ -210,6 +224,12 @@ allOf: > reg: > minItems: 2 > maxItems: 2 > + reg-names: > + items: > + - const: std > + - const: ice > + required: > + - reg-names > > # TODO: define clock bindings for qcom,msm8994-ufshc > Looks good to me, though I'm not an expert in device tree bindings. It's unfortunate that whether ICE is present needs to be explicitly declared for each SoC here (I would have expected that to only be done in the device tree files themselves), and that the order of the regs and clocks is fixed (I would have expected them to act as maps keyed by name). But it seems that's the way it is. Reviewed-by: Eric Biggers <ebiggers@xxxxxxxxxx> - Eric
