Messages in this thread | | | Date | Wed, 28 Sep 2022 19:15:51 +0200 | Subject | Re: [PATCHv4 1/3] dt-bindings: mmc: synopsys-dw-mshc: document "altr,sysmgr-syscon" | From | Krzysztof Kozlowski <> |
| |
On 28/09/2022 18:54, Dinh Nguyen wrote: > Document the optional "altr,sysmgr-syscon" binding that is used to > access the System Manager register that controls the SDMMC clock > phase. > > Signed-off-by: Dinh Nguyen <dinguyen@kernel.org> > ---
Thank you for your patch. There is something to discuss/improve.
> + > +allOf: > + - $ref: "synopsys-dw-mshc-common.yaml#" > + > + - if: > + properties: > + compatible: > + contains: > + const: > + - altr,socfpga-dw-mshc
It still should not be an array, even if there is no warning.
> + then: > + required: > + - altr,sysmgr-syscon > + else: > + properties: > + altr,sysmgr-syscon: false
Best regards, Krzysztof
| |