| Date | Fri, 15 Sep 2023 12:49:48 +0200 | Subject | Re: [PATCH v4 21/42] dt-bindings: net: Add Cirrus EP93xx | From | Krzysztof Kozlowski <> |
| |
On 15/09/2023 10:11, Nikita Shubin via B4 Relay wrote: > From: Nikita Shubin <nikita.shubin@maquefel.me> >
> + mdio: > + $ref: mdio.yaml# > + unevaluatedProperties: false > + description: optional node for embedded MDIO controller > + > +required: > + - compatible > + - reg > + - interrupts > + - phy-handle > + > +additionalProperties: false > + > +examples: > + - | > + ethernet@80010000 { > + compatible = "cirrus,ep9301-eth"; > + reg = <0x80010000 0x10000>; > + interrupt-parent = <&vic1>; > + interrupts = <7>; > + phy-handle = <&phy0>;
Would be nice to extend the example with mdio, to be complete as much as possible. Anyway:
Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Best regards, Krzysztof
|