Hi Arnd, On Wed, Apr 6, 2022 at 11:31 PM Arnd Bergmann <arnd@xxxxxxxx> wrote: > > On Thu, Apr 7, 2022 at 1:36 AM Brad Larson <brad@xxxxxxxxxxx> wrote: > > > > --- a/Documentation/devicetree/bindings/mmc/cdns,sdhci.yaml > > +++ b/Documentation/devicetree/bindings/mmc/cdns,sdhci.yaml > > @@ -19,10 +19,12 @@ properties: > > - enum: > > - microchip,mpfs-sd4hc > > - socionext,uniphier-sd4hc > > + - pensando,elba-sd4hc > > - const: cdns,sd4hc > > > > reg: > > - maxItems: 1 > > + minItems: 1 > > + maxItems: 2 > > > > Shouldn't the binding describe what the register areas are? If there > is only one of them, it is fairly clear, but when you have the choice > between one and two, it gets ambiguous, and there is a risk that > another SoC might have a different register area in the second entry, > making it incompatible. Thanks for the review. Changing this to allOf:if:then in updated patchset. The second item is particular to Elba SoC. Regards, Brad
