Skip to content

Commit 4dc4dc1

Browse files
committed
dts: bindings: remove legacy MDIO-related properties/defines
Remove any properties from the Xilinx GEM DT node specification and the associated header in dt-bindings that relate to the legacy MDIO/PHY support. Signed-off-by: Immo Birnbaum <mail@birnbaum.immo>
1 parent cf4dbad commit 4dc4dc1

File tree

2 files changed

+0
-98
lines changed

2 files changed

+0
-98
lines changed

dts/bindings/ethernet/xlnx,gem.yaml

Lines changed: 0 additions & 65 deletions
Original file line numberDiff line numberDiff line change
@@ -28,66 +28,6 @@ properties:
2828
item must be set to the clock frequency of the PLL supplying the
2929
respective GEM's TX clock - by default, this is the IO PLL.
3030
31-
mdc-divider:
32-
type: int
33-
required: true
34-
description: |
35-
The MDC clock divider for the respective GEM. This is the divider
36-
applied to the LPD_LSBUS clock in order to derive MDIO interface
37-
clock driving communications with the attached PHY. Refer to the
38-
ZynqMP register documentation (ug1087), network_config (GEM) Register
39-
Description, bits [20:18] to determine the appropriate divider for
40-
the current target's LPD LSBUS clock frequency.
41-
42-
init-mdio-phy:
43-
type: boolean
44-
description: |
45-
Activates the management of a PHY associated with the controller in-
46-
stance. If this parameter is activated at the board level, the de-
47-
fault values of the associated parameters mdio-phy-address, phy-poll-
48-
interval, link-speed and advertise-lower-link-speeds should be checked
49-
and overwritten at the board level if required.
50-
51-
mdio-phy-address:
52-
type: int
53-
required: true
54-
description: |
55-
The address on the MDIO bus of the PHY associated with the controller
56-
instance. Set the address to 0 for auto-detection (first responding
57-
PHY will be claimed by the driver, watch out in case of shared MDIO
58-
use), or to a fixed address between 1 and 32.
59-
60-
phy-poll-interval:
61-
type: int
62-
required: true
63-
description: |
64-
PHY status polling interval in milliseconds for a driver instance
65-
managing an associated PHY.
66-
67-
link-speed:
68-
type: int
69-
required: true
70-
description: |
71-
Nominal link speed. If no PHY is managed by an instance of this driver,
72-
the respective controller will be configured to match the link speed
73-
specified here. If a PHY is managed by the driver, advertisement of
74-
the link speed specified here will be requested. If the optional pro-
75-
perty advertise-lower-link-speeds is set, advertisement of the link
76-
speed specified here plus any valid link speed below this value will
77-
be requested.
78-
enum:
79-
- 1
80-
- 2
81-
- 3
82-
83-
advertise-lower-link-speeds:
84-
type: boolean
85-
description: |
86-
Indicates to a driver instance which manages an associated PHY on
87-
the MDIO bus to include link speeds lower than the nominal value
88-
set in the link-speed property in the advertisement when requesting
89-
link speed auto-negotiation with a peer system.
90-
9131
handle-rx-in-isr:
9232
type: boolean
9333
description: |
@@ -318,11 +258,6 @@ properties:
318258
description: Optional feature flag - Discard non-VLAN frames. When set,
319259
only VLAN tagged frames will be passed to the address matching logic.
320260

321-
full-duplex:
322-
type: boolean
323-
description: |
324-
Optional feature flag - Enables full duplex reception and transmission.
325-
326261
discard-rx-frame-ahb-unavail:
327262
type: boolean
328263
description: |

include/zephyr/dt-bindings/ethernet/xlnx_gem.h

Lines changed: 0 additions & 33 deletions
Original file line numberDiff line numberDiff line change
@@ -6,39 +6,6 @@
66
#ifndef ZEPHYR_INCLUDE_DT_BINDINGS_ETHERNET_XLNX_GEM_H_
77
#define ZEPHYR_INCLUDE_DT_BINDINGS_ETHERNET_XLNX_GEM_H_
88

9-
/* PHY auto-detection alias */
10-
#define XLNX_GEM_PHY_AUTO_DETECT 0
11-
12-
/*
13-
* MDC divider values
14-
*
15-
* According to the ZynqMP's gem.network_config register documentation (UG1087),
16-
* divider /32 is the reset value. The network_config[mdc_clock_division]
17-
* documentation in UG1087 is likely wrong (copied directly from the Zynq-7000),
18-
* as it claims that the MDC clock division is applied to the cpu_1x clock
19-
* which the UltraScale doesn't have. Contradicting information is provided in
20-
* the UltraScale TRM (UG1085), which mentions in chapter 34, section "Configure
21-
* the PHY", p. 1074, that the MDC clock division is applied to the IOU_SWITCH_CLK.
22-
* Xilinx's emacps driver doesn't (or no longer does) limit the range of dividers
23-
* on the UltraScale compared to the Zynq-7000.
24-
* -> Contrary to earlier revisions of this driver, all dividers are available
25-
* to both the UltraScale and the Zynq-7000.
26-
*/
27-
28-
#define XLNX_GEM_MDC_DIVIDER_8 0 /* cpu_1x or IOU_SWITCH_CLK < 20 MHz */
29-
#define XLNX_GEM_MDC_DIVIDER_16 1 /* cpu_1x or IOU_SWITCH_CLK 20 - 40 MHz */
30-
#define XLNX_GEM_MDC_DIVIDER_32 2 /* cpu_1x or IOU_SWITCH_CLK 40 - 80 MHz */
31-
#define XLNX_GEM_MDC_DIVIDER_48 3 /* cpu_1x or IOU_SWITCH_CLK 80 - 120 MHz */
32-
#define XLNX_GEM_MDC_DIVIDER_64 4 /* cpu_1x or IOU_SWITCH_CLK 120 - 160 MHz */
33-
#define XLNX_GEM_MDC_DIVIDER_96 5 /* cpu_1x or IOU_SWITCH_CLK 160 - 240 MHz */
34-
#define XLNX_GEM_MDC_DIVIDER_128 6 /* cpu_1x or IOU_SWITCH_CLK 240 - 320 MHz */
35-
#define XLNX_GEM_MDC_DIVIDER_224 7 /* cpu_1x or IOU_SWITCH_CLK 320 - 540 MHz */
36-
37-
/* Link speed values */
38-
#define XLNX_GEM_LINK_SPEED_10MBIT 1
39-
#define XLNX_GEM_LINK_SPEED_100MBIT 2
40-
#define XLNX_GEM_LINK_SPEED_1GBIT 3
41-
429
/* AMBA AHB data bus width */
4310
#define XLNX_GEM_AMBA_AHB_DBUS_WIDTH_32BIT 0
4411
#define XLNX_GEM_AMBA_AHB_DBUS_WIDTH_64BIT 1

0 commit comments

Comments
 (0)