On 05/08/2023 05:14, Drew Fustini wrote: > Add compatible value for the T-Head TH1520 dwcmshc controller and > thead,io-fixed-1v8 and thead,pull-up properties. > > Signed-off-by: Drew Fustini <dfustini@xxxxxxxxxxxx> > --- > Documentation/devicetree/bindings/mmc/snps,dwcmshc-sdhci.yaml | 9 +++++++++ > 1 file changed, 9 insertions(+) > > diff --git a/Documentation/devicetree/bindings/mmc/snps,dwcmshc-sdhci.yaml b/Documentation/devicetree/bindings/mmc/snps,dwcmshc-sdhci.yaml > index a43eb837f8da..57602c345cab 100644 > --- a/Documentation/devicetree/bindings/mmc/snps,dwcmshc-sdhci.yaml > +++ b/Documentation/devicetree/bindings/mmc/snps,dwcmshc-sdhci.yaml > @@ -19,6 +19,7 @@ properties: > - rockchip,rk3568-dwcmshc > - rockchip,rk3588-dwcmshc > - snps,dwcmshc-sdhci > + - thead,th1520-dwcmshc > > reg: > maxItems: 1 > @@ -60,6 +61,14 @@ properties: > description: Specify the number of delay for tx sampling. > $ref: /schemas/types.yaml#/definitions/uint8 > > + thead,io-fixed-1v8: > + description: SoC PHY pad is fixed 1.8V > + type: boolean Isn't this duplicating existing properties for MMC modes with 1.8 V? > + > + thead,pull-up: > + description: True if pull-up, false if pull-down This explains me nothing. No clue what you are pulling and why do you need it. Pin pulls should be done via pin controller, not MMC. Anyway you should have here allOf:if:then (move the allOf: from top to behind "required:") which will disallow these properties for other variants. > + type: boolean > + > > required: > - compatible > Best regards, Krzysztof
