Messages in this thread | | | From | Brad Larson <> | Date | Wed, 25 May 2022 08:46:15 -0700 | Subject | Re: [PATCH 03/11] dt-bindings: mmc: Add Pensando Elba SoC binding |
| |
Hi Arnd,
On Wed, Apr 6, 2022 at 11:31 PM Arnd Bergmann <arnd@arndb.de> wrote: > > On Thu, Apr 7, 2022 at 1:36 AM Brad Larson <brad@pensando.io> 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
| |