Re: [PATCH v2 07/12] dt-bindings: mmc: Add support for rk3576 eMMC
From: Krzysztof Kozlowski <krzk@kernel.org>
Date: 2024-08-24 07:50:03
Also in:
dri-devel, linux-i2c, linux-iio, linux-mmc, linux-rockchip, linux-serial, linux-spi, linux-watchdog, lkml
On Fri, Aug 23, 2024 at 10:52:34AM -0400, Detlev Casanova wrote:
quoted hunk ↗ jump to hunk
The device is compatible with rk3588, so add an entry for the 2 compatibles together. The rk3576 device has a power-domain that needs to be on for the eMMC to be used. Add it as a requirement. Signed-off-by: Detlev Casanova <detlev.casanova@collabora.com> --- .../bindings/mmc/snps,dwcmshc-sdhci.yaml | 32 +++++++++++++------ 1 file changed, 23 insertions(+), 9 deletions(-)diff --git a/Documentation/devicetree/bindings/mmc/snps,dwcmshc-sdhci.yaml b/Documentation/devicetree/bindings/mmc/snps,dwcmshc-sdhci.yaml index 4d3031d9965f3..7d5e388587027 100644 --- a/Documentation/devicetree/bindings/mmc/snps,dwcmshc-sdhci.yaml +++ b/Documentation/devicetree/bindings/mmc/snps,dwcmshc-sdhci.yaml@@ -12,16 +12,29 @@ maintainers: allOf: - $ref: mmc-controller.yaml# + - if: + properties: + compatible: + contains: + const: rockchip,rk3576-dwcmshc + then: + properties: + power-domains: + minItems: 1
Plaese move the allOf: after the required: block. It grows too much with such if:then: and that's not the most important part of binding we need to see first. With above change: Reviewed-by: Krzysztof Kozlowski <redacted> Best regards, Krzysztof