On 08/02/2023 17:42, Clément Léger wrote: > Add "renesas,rzn1-gmac" binding documentation which is compatible with > "snps,dwmac" compatible driver but uses a custom PCS to communicate > with the phy. > > Signed-off-by: Clément Léger <clement.leger@xxxxxxxxxxx> > --- > .../bindings/net/renesas,rzn1-gmac.yaml | 67 +++++++++++++++++++ > 1 file changed, 67 insertions(+) > create mode 100644 Documentation/devicetree/bindings/net/renesas,rzn1-gmac.yaml > > diff --git a/Documentation/devicetree/bindings/net/renesas,rzn1-gmac.yaml b/Documentation/devicetree/bindings/net/renesas,rzn1-gmac.yaml > new file mode 100644 > index 000000000000..944fd0d97d79 > --- /dev/null > +++ b/Documentation/devicetree/bindings/net/renesas,rzn1-gmac.yaml > @@ -0,0 +1,67 @@ > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/net/renesas,rzn1-gmac.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: Renesas GMAC > + > +maintainers: > + - Clément Léger <clement.leger@xxxxxxxxxxx> > + > +select: > + properties: > + compatible: > + contains: > + enum: > + - renesas,r9a06g032-gmac > + - renesas,rzn1-gmac > + required: > + - compatible > + > +allOf: > + - $ref: snps,dwmac.yaml# > + > +properties: > + compatible: > + items: > + - enum: > + - renesas,r9a06g032-gmac > + - renesas,rzn1-gmac > + - snps,dwmac This is still not correct and does not make any sense. What do you want to say here with such binding? That you describe "snps,dwmac" here? Then it's duplicated with snps,dwmac.yaml... Drop that enum and make it a proper list. Best regards, Krzysztof
