| Message ID | 20240311141106.3200743-2-niklas.soderlund+renesas@ragnatech.se |
|---|---|
| State | Not Applicable |
| Headers | show |
| Series | ravb: Support describing the MDIO bus | expand |
| Context | Check | Description |
|---|---|---|
| robh/checkpatch | success | |
| robh/patch-applied | success | |
| robh/dtbs-check | warning | build log |
| robh/dt-meta-schema | success |
On Mon, 11 Mar 2024 15:11:05 +0100, Niklas Söderlund wrote: > The Renesas Ethernet AVB bindings do not allow the MDIO bus to be > described. This has not been needed as only a single PHY is > supported and no MDIO bus properties have been needed. > > Add an optional mdio node to the binding which allows the MDIO bus to be > described and allow bus properties to be set. > > Signed-off-by: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se> > Reviewed-by: Sergey Shtylyov <s.shtylyov@omp.ru> > --- > * Changes since v2 > - Mark the PHY pattern as deprecated and add a comment that new bindings > should describe the PHY inside the MDIO node. > --- > .../devicetree/bindings/net/renesas,etheravb.yaml | 8 ++++++++ > 1 file changed, 8 insertions(+) > Reviewed-by: Rob Herring <robh@kernel.org>
diff --git a/Documentation/devicetree/bindings/net/renesas,etheravb.yaml b/Documentation/devicetree/bindings/net/renesas,etheravb.yaml index de7ba7f345a9..8bfd4fb5f834 100644 --- a/Documentation/devicetree/bindings/net/renesas,etheravb.yaml +++ b/Documentation/devicetree/bindings/net/renesas,etheravb.yaml @@ -93,6 +93,10 @@ properties: description: Number of size cells on the MDIO bus. const: 0 + mdio: + $ref: /schemas/net/mdio.yaml# + unevaluatedProperties: false + renesas,no-ether-link: type: boolean description: @@ -110,9 +114,13 @@ properties: tx-internal-delay-ps: enum: [0, 2000] +# In older bindings there where no mdio child-node to describe the MDIO bus +# and the PHY. To not fail older bindings accept any node with an address. New +# users should describe the PHY inside the mdio child-node. patternProperties: "@[0-9a-f]$": type: object + deprecated: true required: - compatible