On 22/10/2022 05:05, Frank Wunderlich wrote: > From: Frank Wunderlich <frank-w@xxxxxxxxxxxxxxx> > > Add SoC specific section for defining clock configuration. > > Add compatible in "not" section to avoid override the settings again. > > Signed-off-by: Frank Wunderlich <frank-w@xxxxxxxxxxxxxxx> > --- > .../devicetree/bindings/mmc/mtk-sd.yaml | 28 +++++++++++++++++++ > 1 file changed, 28 insertions(+) > > diff --git a/Documentation/devicetree/bindings/mmc/mtk-sd.yaml b/Documentation/devicetree/bindings/mmc/mtk-sd.yaml > index c7bcf0c3dd5d..9df63c461f1f 100644 > --- a/Documentation/devicetree/bindings/mmc/mtk-sd.yaml > +++ b/Documentation/devicetree/bindings/mmc/mtk-sd.yaml > @@ -229,6 +229,33 @@ allOf: > - const: hclk > - const: source_cg > - const: bus_clk Blank line > + - if: > + properties: > + compatible: > + contains: > + enum: > + - mediatek,mt7986-mmc > + then: > + properties: > + assigned-clock-parents: > + maxItems: 2 > + assigned-clocks: > + maxItems: 2 Define these two only if they are required for the hardware to operate and only such setup (two assigned parents) is allowed. Otherwise you put some constraints on board design. > + clocks: > + minItems: 3 Best regards, Krzysztof
