diff options
author | jbrunet <jbrunet@baylibre.com> | 2016-12-19 10:05:38 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2016-12-20 13:50:50 -0500 |
commit | 308d3165d8b2b98d3dc3d97d6662062735daea67 (patch) | |
tree | aa26f239778a5429318bb40b26ac3479fa5e45e5 | |
parent | 57f3986231bb2c69a55ccab1d2b30a00818027ac (diff) |
dt: bindings: net: use boolean dt properties for eee broken modes
The patches regarding eee-broken-modes was merged before all people
involved could find an agreement on the best way to move forward.
While we agreed on having a DT property to mark particular modes as broken,
the value used for eee-broken-modes mapped the phy register in very direct
way. Because of this, the concern is that it could be used to implement
configuration policies instead of describing a broken HW.
In the end, having a boolean property for each mode seems to be preferred
over one bit field value mapping the register (too) directly.
Cc: Florian Fainelli <f.fainelli@gmail.com>
Signed-off-by: Jerome Brunet <jbrunet@baylibre.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r-- | Documentation/devicetree/bindings/net/phy.txt | 10 | ||||
-rw-r--r-- | include/dt-bindings/net/mdio.h | 19 |
2 files changed, 8 insertions, 21 deletions
diff --git a/Documentation/devicetree/bindings/net/phy.txt b/Documentation/devicetree/bindings/net/phy.txt index 54749b60a466..ff1bc4b1bb3b 100644 --- a/Documentation/devicetree/bindings/net/phy.txt +++ b/Documentation/devicetree/bindings/net/phy.txt | |||
@@ -38,8 +38,14 @@ Optional Properties: | |||
38 | - enet-phy-lane-swap: If set, indicates the PHY will swap the TX/RX lanes to | 38 | - enet-phy-lane-swap: If set, indicates the PHY will swap the TX/RX lanes to |
39 | compensate for the board being designed with the lanes swapped. | 39 | compensate for the board being designed with the lanes swapped. |
40 | 40 | ||
41 | - eee-broken-modes: Bits to clear in the MDIO_AN_EEE_ADV register to | 41 | - eee-broken-100tx: |
42 | disable EEE broken modes. | 42 | - eee-broken-1000t: |
43 | - eee-broken-10gt: | ||
44 | - eee-broken-1000kx: | ||
45 | - eee-broken-10gkx4: | ||
46 | - eee-broken-10gkr: | ||
47 | Mark the corresponding energy efficient ethernet mode as broken and | ||
48 | request the ethernet to stop advertising it. | ||
43 | 49 | ||
44 | Example: | 50 | Example: |
45 | 51 | ||
diff --git a/include/dt-bindings/net/mdio.h b/include/dt-bindings/net/mdio.h deleted file mode 100644 index 99c6d903d439..000000000000 --- a/include/dt-bindings/net/mdio.h +++ /dev/null | |||
@@ -1,19 +0,0 @@ | |||
1 | /* | ||
2 | * This header provides generic constants for ethernet MDIO bindings | ||
3 | */ | ||
4 | |||
5 | #ifndef _DT_BINDINGS_NET_MDIO_H | ||
6 | #define _DT_BINDINGS_NET_MDIO_H | ||
7 | |||
8 | /* | ||
9 | * EEE capability Advertisement | ||
10 | */ | ||
11 | |||
12 | #define MDIO_EEE_100TX 0x0002 /* 100TX EEE cap */ | ||
13 | #define MDIO_EEE_1000T 0x0004 /* 1000T EEE cap */ | ||
14 | #define MDIO_EEE_10GT 0x0008 /* 10GT EEE cap */ | ||
15 | #define MDIO_EEE_1000KX 0x0010 /* 1000KX EEE cap */ | ||
16 | #define MDIO_EEE_10GKX4 0x0020 /* 10G KX4 EEE cap */ | ||
17 | #define MDIO_EEE_10GKR 0x0040 /* 10G KR EEE cap */ | ||
18 | |||
19 | #endif | ||