Hi,
thank you for picking this up.
On Sun, Oct 23, 2022 at 11:12:42AM +0200, Frank Wunderlich wrote:
> From: Nícolas F. R. A. Prado <nfraprado@xxxxxxxxxxxxx>
>
> The binding was describing a single clock list for all platforms, but
> that's not really suitable:
>
> Most platforms using at least 2 clocks (source, hclk), some of them
> a third "source_cg". Mt2712 requires an extra 'bus_clk' on some of
> its controllers, while mt8192 requires 8 clocks.
>
> Move the clock definitions inside if blocks that match on the
> compatibles.
>
> I used Patch from Nícolas F. R. A. Prado and modified it to not using
> "not" statement.
>
> Fixes: 59a23395d8aa ("dt-bindings: mmc: Add support for MT8192 SoC")
> Signed-off-by: Nícolas F. R. A. Prado <nfraprado@xxxxxxxxxxxxx>
> Signed-off-by: Frank Wunderlich <frank-w@xxxxxxxxxxxxxxx>
>
> ---
> v2:
> - add this patch
> v3:
> - add blank lines and change "not" to matchlist
> - reorder entries - make generic first then order alphanumeric
> - rewrite commit description
> - drop soc-specific mt8183 - constraints were also set for it above
This is wrong, see below.
> ---
> .../devicetree/bindings/mmc/mtk-sd.yaml | 113 +++++++++++++-----
> 1 file changed, 83 insertions(+), 30 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/mmc/mtk-sd.yaml b/Documentation/devicetree/bindings/mmc/mtk-sd.yaml
> index 3cbf0208f1b4..31bb6dc329d2 100644
> --- a/Documentation/devicetree/bindings/mmc/mtk-sd.yaml
> +++ b/Documentation/devicetree/bindings/mmc/mtk-sd.yaml
[..]
>
> -if:
> - properties:
> - compatible:
> - contains:
> - const: mediatek,mt8183-mmc
> -then:
> - properties:
> - reg:
> - minItems: 2
You can't drop this. Nodes with the mt8183 compatible should keep requiring two
reg values. It's not covered by the branch below.
Thanks,
Nícolas
> +allOf:
> + - $ref: mmc-controller.yaml#
> + - if:
> + properties:
> + compatible:
> + enum:
> + - mediatek,mt2701-mmc
> + - mediatek,mt6779-mmc
> + - mediatek,mt6795-mmc
> + - mediatek,mt7620-mmc
> + - mediatek,mt7622-mmc
> + - mediatek,mt7623-mmc
> + - mediatek,mt8135-mmc
> + - mediatek,mt8173-mmc
> + - mediatek,mt8183-mmc
> + - mediatek,mt8186-mmc
> + - mediatek,mt8188-mmc
> + - mediatek,mt8195-mmc
> + - mediatek,mt8516-mmc
> + then:
> + properties:
> + clocks:
> + minItems: 2
> + items:
> + - description: source clock
> + - description: HCLK which used for host
> + - description: independent source clock gate
> + clock-names:
> + minItems: 2
> + items:
> + - const: source
> + - const: hclk
> + - const: source_cg
> +
[..]
