Commit 9e5c8d39 authored by Alexandru Ardelean's avatar Alexandru Ardelean Committed by Jakub Kicinski

dt-bindings: net: dwmac: fix 'mac-mode' type

The 'mac-mode' property is similar to 'phy-mode' and 'phy-connection-type',
which are enums of mode strings.

The 'dwmac' driver supports almost all modes declared in the 'phy-mode'
enum (except for 1 or 2). But in general, there may be a case where
'mac-mode' becomes more generic and is moved as part of phylib or netdev.

In any case, the 'mac-mode' field should be made an enum, and it also makes
sense to just reference the 'phy-connection-type' from
'ethernet-controller.yaml'. That will also make it more future-proof for new
modes.

Fixes: 9c15d359 ("dt-bindings: net: dwmac: document 'mac-mode' property")
Signed-off-by: default avatarAlexandru Ardelean <alexandru.ardelean@analog.com>
Reviewed-by: default avatarAndrew Lunn <andrew@lunn.ch>
Reviewed-by: default avatarRob Herring <robh@kernel.org>
Signed-off-by: default avatarJakub Kicinski <jakub.kicinski@netronome.com>
parent 92974a1d
...@@ -113,7 +113,7 @@ properties: ...@@ -113,7 +113,7 @@ properties:
const: stmmaceth const: stmmaceth
mac-mode: mac-mode:
maxItems: 1 $ref: ethernet-controller.yaml#/properties/phy-connection-type
description: description:
The property is identical to 'phy-mode', and assumes that there is mode The property is identical to 'phy-mode', and assumes that there is mode
converter in-between the MAC & PHY (e.g. GMII-to-RGMII). This converter converter in-between the MAC & PHY (e.g. GMII-to-RGMII). This converter
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment