* [PATCH net-next 0/2] DP83TD510 Single Pair 10Mbps Ethernet PHY @ 2020-10-08 16:23 Dan Murphy 2020-10-08 16:23 ` [PATCH net-next 1/2] dt-bindings: dp83td510: Add binding for DP83TD510 " Dan Murphy 2020-10-08 16:23 ` [PATCH net-next 2/2] net: phy: dp83td510: Add support for the " Dan Murphy 0 siblings, 2 replies; 13+ messages in thread From: Dan Murphy @ 2020-10-08 16:23 UTC (permalink / raw) To: davem, andrew, f.fainelli, hkallweit1; +Cc: netdev, linux-kernel, Dan Murphy Hello The DP83TD510 is an Ethernet PHY supporting single pair of twisted wires. The PHY is capable of 10Mbps communication over long distances and exceeds the IEEE 802.3cg 10BASE-T1L single-pair Ethernet specification. The PHY supports various voltage level signalling and can be forced to support a specific voltage or allowed to perfrom auto negotiation on the voltage level. The default for the PHY is auto negotiation but if the PHY is forced to a specific voltage then the LP must also support the same voltage. Dan Dan Murphy (2): dt-bindings: dp83td510: Add binding for DP83TD510 Ethernet PHY net: phy: dp83td510: Add support for the DP83TD510 Ethernet PHY .../devicetree/bindings/net/ti,dp83td510.yaml | 70 +++ drivers/net/phy/Kconfig | 6 + drivers/net/phy/Makefile | 1 + drivers/net/phy/dp83td510.c | 583 ++++++++++++++++++ 4 files changed, 660 insertions(+) create mode 100644 Documentation/devicetree/bindings/net/ti,dp83td510.yaml create mode 100644 drivers/net/phy/dp83td510.c -- 2.28.0.585.ge1cfff676549 ^ permalink raw reply [flat|nested] 13+ messages in thread
* [PATCH net-next 1/2] dt-bindings: dp83td510: Add binding for DP83TD510 Ethernet PHY 2020-10-08 16:23 [PATCH net-next 0/2] DP83TD510 Single Pair 10Mbps Ethernet PHY Dan Murphy @ 2020-10-08 16:23 ` Dan Murphy 2020-10-08 17:11 ` Florian Fainelli 2020-10-08 16:23 ` [PATCH net-next 2/2] net: phy: dp83td510: Add support for the " Dan Murphy 1 sibling, 1 reply; 13+ messages in thread From: Dan Murphy @ 2020-10-08 16:23 UTC (permalink / raw) To: davem, andrew, f.fainelli, hkallweit1; +Cc: netdev, linux-kernel, Dan Murphy The DP83TD510 is a 10M single twisted pair Ethernet PHY Signed-off-by: Dan Murphy <dmurphy@ti.com> --- .../devicetree/bindings/net/ti,dp83td510.yaml | 70 +++++++++++++++++++ 1 file changed, 70 insertions(+) create mode 100644 Documentation/devicetree/bindings/net/ti,dp83td510.yaml diff --git a/Documentation/devicetree/bindings/net/ti,dp83td510.yaml b/Documentation/devicetree/bindings/net/ti,dp83td510.yaml new file mode 100644 index 000000000000..0f0eac77a11a --- /dev/null +++ b/Documentation/devicetree/bindings/net/ti,dp83td510.yaml @@ -0,0 +1,70 @@ +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) +# Copyright (C) 2020 Texas Instruments Incorporated +%YAML 1.2 +--- +$id: "http://devicetree.org/schemas/net/ti,dp83td510.yaml#" +$schema: "http://devicetree.org/meta-schemas/core.yaml#" + +title: TI DP83TD510 ethernet PHY + +allOf: + - $ref: "ethernet-controller.yaml#" + +maintainers: + - Dan Murphy <dmurphy@ti.com> + +description: | + The PHY is an twisted pair 10Mbps Ethernet PHY that support MII, RMII and + RGMII interfaces. + + Specifications about the Ethernet PHY can be found at: + http://www.ti.com/lit/ds/symlink/dp83td510e.pdf + +properties: + reg: + maxItems: 1 + + tx-fifo-depth: + description: | + Transmitt FIFO depth for RMII mode. The PHY only exposes 4 nibble + depths. The valid nibble depths are 4, 5, 6 and 8. + default: 5 + + rx-internal-delay-ps: + description: | + Setting this property to a non-zero number sets the RX internal delay + for the PHY. The internal delay for the PHY is fixed to 30ns relative + to receive data. + + tx-internal-delay-ps: + description: | + Setting this property to a non-zero number sets the TX internal delay + for the PHY. The internal delay for the PHY has a range of -4 to 4ns + relative to transmit data. + + ti,master-slave-mode: + $ref: /schemas/types.yaml#definitions/uint32 + default: 0 + description: | + Force the PHY to be configured to a specific mode. + Force Auto Negotiation - 0 + Force Master mode at 1v p2p - 1 + Force Master mode at 2.4v p2p - 2 + Force Slave mode at 1v p2p - 3 + Force Slave mode at 2.4v p2p - 4 + +required: + - reg + +examples: + - | + mdio0 { + #address-cells = <1>; + #size-cells = <0>; + ethphy0: ethernet-phy@0 { + reg = <0>; + tx-fifo-depth = <5>; + rx-internal-delay-ps = <1>; + tx-internal-delay-ps = <1>; + }; + }; -- 2.28.0.585.ge1cfff676549 ^ permalink raw reply related [flat|nested] 13+ messages in thread
* Re: [PATCH net-next 1/2] dt-bindings: dp83td510: Add binding for DP83TD510 Ethernet PHY 2020-10-08 16:23 ` [PATCH net-next 1/2] dt-bindings: dp83td510: Add binding for DP83TD510 " Dan Murphy @ 2020-10-08 17:11 ` Florian Fainelli 2020-10-08 18:18 ` Dan Murphy 0 siblings, 1 reply; 13+ messages in thread From: Florian Fainelli @ 2020-10-08 17:11 UTC (permalink / raw) To: Dan Murphy, davem, andrew, hkallweit1; +Cc: netdev, linux-kernel On 10/8/2020 9:23 AM, Dan Murphy wrote: > The DP83TD510 is a 10M single twisted pair Ethernet PHY > > Signed-off-by: Dan Murphy <dmurphy@ti.com> > --- > .../devicetree/bindings/net/ti,dp83td510.yaml | 70 +++++++++++++++++++ > 1 file changed, 70 insertions(+) > create mode 100644 Documentation/devicetree/bindings/net/ti,dp83td510.yaml > > diff --git a/Documentation/devicetree/bindings/net/ti,dp83td510.yaml b/Documentation/devicetree/bindings/net/ti,dp83td510.yaml > new file mode 100644 > index 000000000000..0f0eac77a11a > --- /dev/null > +++ b/Documentation/devicetree/bindings/net/ti,dp83td510.yaml > @@ -0,0 +1,70 @@ > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > +# Copyright (C) 2020 Texas Instruments Incorporated > +%YAML 1.2 > +--- > +$id: "http://devicetree.org/schemas/net/ti,dp83td510.yaml#" > +$schema: "http://devicetree.org/meta-schemas/core.yaml#" > + > +title: TI DP83TD510 ethernet PHY > + > +allOf: > + - $ref: "ethernet-controller.yaml#" > + > +maintainers: > + - Dan Murphy <dmurphy@ti.com> > + > +description: | > + The PHY is an twisted pair 10Mbps Ethernet PHY that support MII, RMII and > + RGMII interfaces. > + > + Specifications about the Ethernet PHY can be found at: > + http://www.ti.com/lit/ds/symlink/dp83td510e.pdf > + > +properties: > + reg: > + maxItems: 1 > + > + tx-fifo-depth: > + description: | > + Transmitt FIFO depth for RMII mode. The PHY only exposes 4 nibble > + depths. The valid nibble depths are 4, 5, 6 and 8. > + default: 5 > + > + rx-internal-delay-ps: > + description: | > + Setting this property to a non-zero number sets the RX internal delay > + for the PHY. The internal delay for the PHY is fixed to 30ns relative > + to receive data. > + > + tx-internal-delay-ps: > + description: | > + Setting this property to a non-zero number sets the TX internal delay > + for the PHY. The internal delay for the PHY has a range of -4 to 4ns > + relative to transmit data. Those two properties are already defined as part of Documentation/devicetree/bindings/net/ethernet-phy.yaml, so you can reference that binding, too. > + > + ti,master-slave-mode: > + $ref: /schemas/types.yaml#definitions/uint32 > + default: 0 > + description: | > + Force the PHY to be configured to a specific mode. > + Force Auto Negotiation - 0 > + Force Master mode at 1v p2p - 1 > + Force Master mode at 2.4v p2p - 2 > + Force Slave mode at 1v p2p - 3 > + Force Slave mode at 2.4v p2p - 4 If you accept different values you should be indicating which values are supported with an enumeration. -- Florian ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH net-next 1/2] dt-bindings: dp83td510: Add binding for DP83TD510 Ethernet PHY 2020-10-08 17:11 ` Florian Fainelli @ 2020-10-08 18:18 ` Dan Murphy 0 siblings, 0 replies; 13+ messages in thread From: Dan Murphy @ 2020-10-08 18:18 UTC (permalink / raw) To: Florian Fainelli, davem, andrew, hkallweit1; +Cc: netdev, linux-kernel Florian Thanks for the review On 10/8/20 12:11 PM, Florian Fainelli wrote: > > > On 10/8/2020 9:23 AM, Dan Murphy wrote: >> The DP83TD510 is a 10M single twisted pair Ethernet PHY >> >> Signed-off-by: Dan Murphy <dmurphy@ti.com> >> --- >> .../devicetree/bindings/net/ti,dp83td510.yaml | 70 +++++++++++++++++++ >> 1 file changed, 70 insertions(+) >> create mode 100644 >> Documentation/devicetree/bindings/net/ti,dp83td510.yaml >> >> diff --git a/Documentation/devicetree/bindings/net/ti,dp83td510.yaml >> b/Documentation/devicetree/bindings/net/ti,dp83td510.yaml >> new file mode 100644 >> index 000000000000..0f0eac77a11a >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/net/ti,dp83td510.yaml >> @@ -0,0 +1,70 @@ >> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) >> +# Copyright (C) 2020 Texas Instruments Incorporated >> +%YAML 1.2 >> +--- >> +$id: "http://devicetree.org/schemas/net/ti,dp83td510.yaml#" >> +$schema: "http://devicetree.org/meta-schemas/core.yaml#" >> + >> +title: TI DP83TD510 ethernet PHY >> + >> +allOf: >> + - $ref: "ethernet-controller.yaml#" >> + >> +maintainers: >> + - Dan Murphy <dmurphy@ti.com> >> + >> +description: | >> + The PHY is an twisted pair 10Mbps Ethernet PHY that support MII, >> RMII and >> + RGMII interfaces. >> + >> + Specifications about the Ethernet PHY can be found at: >> + http://www.ti.com/lit/ds/symlink/dp83td510e.pdf >> + >> +properties: >> + reg: >> + maxItems: 1 >> + >> + tx-fifo-depth: >> + description: | >> + Transmitt FIFO depth for RMII mode. The PHY only exposes 4 >> nibble >> + depths. The valid nibble depths are 4, 5, 6 and 8. >> + default: 5 >> + >> + rx-internal-delay-ps: >> + description: | >> + Setting this property to a non-zero number sets the RX >> internal delay >> + for the PHY. The internal delay for the PHY is fixed to 30ns >> relative >> + to receive data. >> + >> + tx-internal-delay-ps: >> + description: | >> + Setting this property to a non-zero number sets the TX >> internal delay >> + for the PHY. The internal delay for the PHY has a range of >> -4 to 4ns >> + relative to transmit data. > > Those two properties are already defined as part of > Documentation/devicetree/bindings/net/ethernet-phy.yaml, so you can > reference that binding, too. OK I referenced the ethernet-controller.yaml for the delay. I am wondering if we should add rx/tx-fifo-depth to the ethernet-phy.yaml as well. That way PHYs only have to reference ethernet-phy.yaml. Or maybe remove the internal-delay from the ethernet-phy.yaml and reference the ethernet-controller.yaml in the ethernet-phy.yaml so we don't have to maintain duplicate properties > >> + >> + ti,master-slave-mode: >> + $ref: /schemas/types.yaml#definitions/uint32 >> + default: 0 >> + description: | >> + Force the PHY to be configured to a specific mode. >> + Force Auto Negotiation - 0 >> + Force Master mode at 1v p2p - 1 >> + Force Master mode at 2.4v p2p - 2 >> + Force Slave mode at 1v p2p - 3 >> + Force Slave mode at 2.4v p2p - 4 > > If you accept different values you should be indicating which values > are supported with an enumeration. Ah yes forgot the min/max ^ permalink raw reply [flat|nested] 13+ messages in thread
* [PATCH net-next 2/2] net: phy: dp83td510: Add support for the DP83TD510 Ethernet PHY 2020-10-08 16:23 [PATCH net-next 0/2] DP83TD510 Single Pair 10Mbps Ethernet PHY Dan Murphy 2020-10-08 16:23 ` [PATCH net-next 1/2] dt-bindings: dp83td510: Add binding for DP83TD510 " Dan Murphy @ 2020-10-08 16:23 ` Dan Murphy 2020-10-08 16:51 ` Heiner Kallweit 2020-10-16 22:02 ` Andrew Lunn 1 sibling, 2 replies; 13+ messages in thread From: Dan Murphy @ 2020-10-08 16:23 UTC (permalink / raw) To: davem, andrew, f.fainelli, hkallweit1; +Cc: netdev, linux-kernel, Dan Murphy The DP83TD510E is an ultra-low power Ethernet physical layer transceiver that supports 10M single pair cable. The device supports both 2.4-V p2p and 1-V p2p output voltage as defined by IEEE 802.3cg 10Base-T1L specfications. These modes can be forced via the device tree or the device is defaulted to auto negotiation to determine the proper p2p voltage. Signed-off-by: Dan Murphy <dmurphy@ti.com> --- drivers/net/phy/Kconfig | 6 + drivers/net/phy/Makefile | 1 + drivers/net/phy/dp83td510.c | 583 ++++++++++++++++++++++++++++++++++++ 3 files changed, 590 insertions(+) create mode 100644 drivers/net/phy/dp83td510.c diff --git a/drivers/net/phy/Kconfig b/drivers/net/phy/Kconfig index 698bea312adc..017252e1504c 100644 --- a/drivers/net/phy/Kconfig +++ b/drivers/net/phy/Kconfig @@ -302,6 +302,12 @@ config DP83869_PHY Currently supports the DP83869 PHY. This PHY supports copper and fiber connections. +config DP83TD510_PHY + tristate "Texas Instruments DP83TD510 10M Single Pair Ethernet PHY" + help + Support for the DP83TD510 Ethernet PHY. This PHY supports a 10M single + pair Ethernet connection. + config VITESSE_PHY tristate "Vitesse PHYs" help diff --git a/drivers/net/phy/Makefile b/drivers/net/phy/Makefile index a13e402074cf..bf62ce211eb4 100644 --- a/drivers/net/phy/Makefile +++ b/drivers/net/phy/Makefile @@ -56,6 +56,7 @@ obj-$(CONFIG_DP83848_PHY) += dp83848.o obj-$(CONFIG_DP83867_PHY) += dp83867.o obj-$(CONFIG_DP83869_PHY) += dp83869.o obj-$(CONFIG_DP83TC811_PHY) += dp83tc811.o +obj-$(CONFIG_DP83TD510_PHY) += dp83td510.o obj-$(CONFIG_FIXED_PHY) += fixed_phy.o obj-$(CONFIG_ICPLUS_PHY) += icplus.o obj-$(CONFIG_INTEL_XWAY_PHY) += intel-xway.o diff --git a/drivers/net/phy/dp83td510.c b/drivers/net/phy/dp83td510.c new file mode 100644 index 000000000000..0cce9063fd1c --- /dev/null +++ b/drivers/net/phy/dp83td510.c @@ -0,0 +1,583 @@ +// SPDX-License-Identifier: GPL-2.0 +/* Driver for the Texas Instruments DP83TD510 PHY + * Copyright (C) 2020 Texas Instruments Incorporated - https://www.ti.com/ + */ + +#include <linux/ethtool.h> +#include <linux/etherdevice.h> +#include <linux/kernel.h> +#include <linux/mii.h> +#include <linux/module.h> +#include <linux/of.h> +#include <linux/phy.h> +#include <linux/netdevice.h> + +#define DP83TD510E_PHY_ID 0x20000180 +#define DP83TD510_DEVADDR_AN 0x7 +#define DP83TD510_DEVADDR 0x1f + +#define DP83TD510_MII_REG 0x0 +#define DP83TD510_PHY_STAT 0x10 +#define DP83TD510_GEN_CFG 0x11 +#define DP83TD510_INT_REG1 0x12 +#define DP83TD510_INT_REG2 0x13 +#define DP83TD510_MAC_CFG_1 0x17 + +#define DP83TD510_SOR_1 0x467 + +#define DP83TD510_HW_RESET BIT(15) +#define DP83TD510_SW_RESET BIT(14) + +/* GEN CFG bits */ +#define DP83TD510_INT_OE BIT(0) +#define DP83TD510_INT_EN BIT(1) + +/* INT REG 1 bits */ +#define DP83TD510_INT1_ESD_EN BIT(3) +#define DP83TD510_INT1_LINK_EN BIT(5) +#define DP83TD510_INT1_RHF_EN BIT(7) +#define DP83TD510_INT1_ESD BIT(11) +#define DP83TD510_INT1_LINK BIT(13) +#define DP83TD510_INT1_RHF BIT(15) + +/* INT REG 2 bits */ +#define DP83TD510_INT2_POR_EN BIT(0) +#define DP83TD510_INT2_POL_EN BIT(1) +#define DP83TD510_INT2_PAGE_EN BIT(5) +#define DP83TD510_INT2_POR BIT(8) +#define DP83TD510_INT2_POL BIT(9) +#define DP83TD510_INT2_PAGE BIT(13) + +/* MAC CFG bits */ +#define DP83TD510_RX_CLK_SHIFT BIT(12) +#define DP83TD510_TX_CLK_SHIFT BIT(11) + +#define DP83TD510_MASTER_MODE BIT(2) +#define DP83TD510_2_4V BIT(7) +#define DP83TD510_RGMII BIT(8) + +#define DP83TD510_FIFO_DEPTH_MASK GENMASK(6, 5) +#define DP83TD510_FIFO_DEPTH_4_B_NIB 0 +#define DP83TD510_FIFO_DEPTH_5_B_NIB BIT(5) +#define DP83TD510_FIFO_DEPTH_6_B_NIB BIT(6) +#define DP83TD510_FIFO_DEPTH_8_B_NIB (BIT(5) | BIT(6)) + +enum dp83td510_mode_config { + DP83TD510_AUTO_NEG = 0, + DP83TD510_MASTER_1 = 1, + DP83TD510_MASTER_24 = 2, + DP83TD510_SLAVE_1 = 3, + DP83TD510_SLAVE_24 = 4, +}; + +struct dp83td510_private { + u32 forced_mode; + u32 tx_fifo_depth; + u32 rgmii_delay; + bool is_rgmii; +}; + +struct dp83td510_init_reg { + int reg; + int val; +}; + +static struct dp83td510_init_reg dp83td510_master_1_0[] = { + { 0x000d, 0x0007 }, /* disable auto-neg */ + { 0x000e, 0x0200 }, + { 0x000d, 0x4007 }, + { 0x000e, 0x0000 }, + { 0x000d, 0x0001 }, /* force master mode */ + { 0x000e, 0x0834 }, + { 0x000d, 0x4001 }, + { 0x000e, 0x4000 }, + { 0x000d, 0x0001 }, /* force 1.0v swing */ + { 0x000e, 0x08f6 }, + { 0x000d, 0x4001 }, + { 0x000e, 0x0000 }, + { 0x0608, 0x003b }, /* disable_0_transition */ + { 0x0862, 0x39f8 }, /* AGC Gain during Autoneg */ + { 0x081a, 0x67c0 }, /* deq offset for 1V swing */ + { 0x081c, 0xfb62 }, /* deq offset for 2.4V swing */ + { 0x0830, 0x05a3 }, /* Enable energy lost fallback */ + { 0x0855, 0x1b55 }, /* MSE Threshold change */ + { 0x0831, 0x0403 }, /* energy detect threshold */ + { 0x0856, 0x1800 }, /* good1 MSE threshold change */ + { 0x0857, 0x8fa0 }, /* Enable fallback to phase 1 on watchdog trigger */ + { 0x0871, 0x000c }, /* TED input changed to slicer_in without FFE */ + { 0x0883, 0x022e }, /* Enable Rx Filter, Change PGA threshold for Short Cable detection */ + { 0x0402, 0x1800 }, /* Adjusr LD swing */ + { 0x0878, 0x2248 }, /* Change PI up/down polarity */ + { 0x010c, 0x0008 }, /* tx filter coefficient */ + { 0x0112, 0x1212 }, /* tx filter scaling factor */ + { 0x0809, 0x5c80 }, /* AGC retrain */ + { 0x0803, 0x1529 }, /* Master Ph1 Back-off */ + { 0x0804, 0x1a33 }, /* Master Ph1 Back-off */ + { 0x0805, 0x1f3d }, /* Master Ph1 Back-off */ + { 0x0850, 0x045b }, /* hybrid gain & delay */ + { 0x0874, 0x6967 }, /* kp step 0 for master */ + { 0x0852, 0x7800 }, /* FAGC init gain */ + { 0x0806, 0x1e1e }, /* Master/Slave Ph2 Back-off */ + { 0x0807, 0x2525 }, /* Master/Slave Ph2 Back-off */ + { 0x0808, 0x2c2c }, /* Master/Slave Ph2 Back-off */ + { 0x0850, 0x0590 }, /* Hybrid Gain/Delay Code */ + { 0x0827, 0x4000 }, /* Echo Fixed Delay */ + { 0x0849, 0x0fe4 }, /* Hybrid Cal enable */ + { 0x084b, 0x04b5 }, /* Echo Score Sel */ + { 0x0018, 0x0043 }, /* Set CRS/RX_DV pin as RX_DV for RMII repeater mode */ +}; + +static struct dp83td510_init_reg dp83td510_slave_1_0[] = { + { 0x000d, 0x0007 }, /* disable auto-neg */ + { 0x000e, 0x0200 }, + { 0x000d, 0x4007 }, + { 0x000e, 0x0000 }, + { 0x000d, 0x0001 }, /* force slave mode */ + { 0x000e, 0x0834 }, + { 0x000d, 0x4001 }, + { 0x000e, 0x0000 }, + { 0x000d, 0x0001 }, /* force 1.0v swing */ + { 0x000d, 0x4001 }, + { 0x000e, 0x0000 }, + { 0x0608, 0x003b }, /* disable_0_transition */ + { 0x0862, 0x39f8 }, /* AGC Gain during Autoneg */ + { 0x081a, 0x67c0 }, /* deq offset for 1V swing */ + { 0x081c, 0xfb62 }, /* deq offset for 2.4V swing */ + { 0x0830, 0x05a3 }, /* Enable energy lost fallback */ + { 0x0855, 0x1b55 }, /* MSE Threshold change */ + { 0x0831, 0x0403 }, /* energy detect threshold */ + { 0x0856, 0x1800 }, /* good1 MSE threshold change */ + { 0x0857, 0x8fa0 }, /* Enable fallback to phase 1 on watchdog trigger */ + { 0x0871, 0x000c }, /* TED input changed to slicer_in without FFE */ + { 0x0883, 0x022e }, /* Enable Rx Filter, PGA threshold for Short Cable detection */ + { 0x0402, 0x1800 }, /* Adjusr LD swing */ + { 0x0878, 0x2248 }, /* Change PI up/down polarity */ + { 0x010c, 0x0008 }, /* tx filter coefficient */ + { 0x0112, 0x1212 }, /* tx filter scaling factor */ + { 0x0809, 0x5c80 }, /* AGC retrain */ + { 0x0803, 0x1529 }, /* Master Ph1 Back-off */ + { 0x0804, 0x1a33 }, /* Master Ph1 Back-off */ + { 0x0805, 0x1f3d }, /* Master Ph1 Back-off */ + { 0x0850, 0x045b }, /* hybrid gain & delay */ + { 0x0874, 0x6967 }, /* kp step 0 for master */ + { 0x0852, 0x7800 }, /* FAGC init gain */ + { 0x0806, 0x1e1e }, /* Master/Slave Ph2 Back-off */ + { 0x0807, 0x2525 }, /* Master/Slave Ph2 Back-off */ + { 0x0808, 0x2c2c }, /* Master/Slave Ph2 Back-off */ + { 0x0850, 0x0590 }, /* Hybrid Gain/Delay Code */ + { 0x0827, 0x4000 }, /* Echo Fixed Delay */ + { 0x0849, 0x0fe4 }, /* Hybrid Cal enable */ + { 0x084b, 0x04b5 }, /* Echo Score Sel */ + { 0x0018, 0x0043 }, /* Set CRS/RX_DV pin as RX_DV for RMII repeater mode */ +}; + +static struct dp83td510_init_reg dp83td510_master_2_4[] = { + { 0x000d, 0x0007 }, /* disable auto-neg */ + { 0x000e, 0x0200 }, + { 0x000d, 0x4007 }, + { 0x000e, 0x0000 }, + { 0x000d, 0x0001 }, /* force Master mode */ + { 0x000e, 0x0834 }, + { 0x000d, 0x4001 }, + { 0x000e, 0x4000 }, + { 0x000d, 0x0001 }, /* force 2.4v swing */ + { 0x000e, 0x08f6 }, + { 0x000d, 0x4001 }, + { 0x000e, 0x1000 }, + { 0x0608, 0x003b }, /* disable_0_transition */ + { 0x0862, 0x39f8 }, /* AGC Gain during Autoneg */ + { 0x081a, 0x67c0 }, /* deq offset for 1V swing */ + { 0x081c, 0xfb62 }, /* deq offset for 2.4V swing */ + { 0x0830, 0x05a3 }, /* Enable energy lost fallback */ + { 0x0855, 0x1b55 }, /* MSE Threshold change */ + { 0x0831, 0x0403 }, /* energy detect threshold */ + { 0x0856, 0x1800 }, /* good1 MSE threshold change */ + { 0x0857, 0x8fa0 }, /* Enable fallback to phase 1 on watchdog trigger */ + { 0x0871, 0x000c }, /* TED input changed to slicer_in without FFE */ + { 0x0883, 0x022e }, /* Enable Rx Filter, Change PGA threshold for Short Cable detection */ + { 0x0402, 0x1800 }, /* Adjusr LD swing */ + { 0x0878, 0x2248 }, /* Change PI up/down polarity */ + { 0x010c, 0x0008 }, /* tx filter coefficient */ + { 0x0112, 0x1212 }, /* tx filter scaling factor */ + { 0x0809, 0x5c80 }, /* AGC retrain */ + { 0x0803, 0x1529 }, /* Master Ph1 Back-off */ + { 0x0804, 0x1a33 }, /* Master Ph1 Back-off */ + { 0x0805, 0x1f3d }, /* Master Ph1 Back-off */ + { 0x0850, 0x045b }, /* hybrid gain & delay */ + { 0x0874, 0x6967 }, /* kp step 0 for master */ + { 0x0852, 0x7800 }, /* FAGC init gain */ + { 0x0806, 0x1e1e }, /* Master/Slave Ph2 Back-off */ + { 0x0807, 0x2525 }, /* Master/Slave Ph2 Back-off */ + { 0x0808, 0x2c2c }, /* Master/Slave Ph2 Back-off */ + { 0x0850, 0x0590 }, /* Hybrid Gain/Delay Code */ + { 0x0827, 0x4000 }, /* Echo Fixed Delay */ + { 0x0849, 0x0fe4 }, /* Hybrid Cal enable */ + { 0x084b, 0x04b5 }, /* Echo Score Sel */ + { 0x0018, 0x0043 }, /* Set CRS/RX_DV pin as RX_DV for RMII repeater mode */ +}; + +static struct dp83td510_init_reg dp83td510_slave_2_4[] = { + { 0x000d, 0x0007}, /* disable auto-neg */ + { 0x000e, 0x0200}, + { 0x000d, 0x4007}, + { 0x000e, 0x0000}, + { 0x000d, 0x0001}, /* force slave mode */ + { 0x000e, 0x0834}, + { 0x000d, 0x4001}, + { 0x000e, 0x0000}, + { 0x000d, 0x0001}, /* force 2.4v swing */ + { 0x000e, 0x08f6}, + { 0x000d, 0x4001}, + { 0x000e, 0x1000}, + { 0x0608, 0x003b}, /* disable_0_transition */ + { 0x0862, 0x39f8}, /* AGC Gain during Autoneg */ + { 0x081a, 0x67c0}, /* deq offset for 1V swing */ + { 0x081c, 0xfb62}, /* deq offset for 2.4V swing */ + { 0x0830, 0x05a3}, /* Enable energy lost fallback */ + { 0x0855, 0x1b55}, /* MSE Threshold change */ + { 0x0831, 0x0403}, /* energy detect threshold */ + { 0x0856, 0x1800}, /* good1 MSE threshold change */ + { 0x0857, 0x8fa0}, /* Enable fallback to phase 1 on watchdog trigger */ + { 0x0871, 0x000c}, /* TED input changed to slicer_in without FFE */ + { 0x0883, 0x022e}, /* Enable Rx Filter, Change PGA threshold for Short Cable detection */ + { 0x0402, 0x1800}, /* Adjusr LD swing */ + { 0x0878, 0x2248}, /* Change PI up/down polarity */ + { 0x010c, 0x0008}, /* tx filter coefficient */ + { 0x0112, 0x1212}, /* tx filter scaling factor */ + { 0x0809, 0x5c80}, /* AGC retrain */ + { 0x0803, 0x1529}, /* Master Ph1 Back-off */ + { 0x0804, 0x1a33}, /* Master Ph1 Back-off */ + { 0x0805, 0x1f3d}, /* Master Ph1 Back-off */ + { 0x0850, 0x045b}, /* hybrid gain & delay */ + { 0x0874, 0x6967}, /* kp step 0 for master */ + { 0x0852, 0x7800}, /* FAGC init gain */ + { 0x0806, 0x1e1e}, /* Master/Slave Ph2 Back-off */ + { 0x0807, 0x2525}, /* Master/Slave Ph2 Back-off */ + { 0x0808, 0x2c2c}, /* Master/Slave Ph2 Back-off */ + { 0x0850, 0x0590}, /* Hybrid Gain/Delay Code */ + { 0x0827, 0x4000}, /* Echo Fixed Delay */ + { 0x0849, 0x0fe4}, /* Hybrid Cal enable */ + { 0x084b, 0x04b5}, /* Echo Score Sel */ + { 0x0018, 0x0043}, /* Set CRS/RX_DV pin as RX_DV for RMII repeater mode */ +}; + +static struct dp83td510_init_reg dp83td510_auto_neg[] = { + { 0x608, 0x003b }, /* disable_0_transition */ + { 0x862, 0x39f8 }, /* AGC Gain during Autoneg */ + { 0x81a, 0x67c0 }, /* deq offset for 1V swing */ + { 0x81c, 0xfb62 }, /* deq offset for 2.4V swing */ + { 0x830, 0x05a3 }, /* Enable energy lost fallback */ + { 0x855, 0x1b55 }, /* MSE Threshold change */ + { 0x831, 0x0403 }, /* energy detect threshold */ + { 0x856, 0x1800 }, /* good1 MSE threshold change */ + { 0x857, 0x8fa0 }, /* Enable fallback to phase 1 on watchdog trigger */ + { 0x871, 0x000c }, /* TED input changed to slicer_in without FFE */ + { 0x883, 0x022e }, /* Enable Rx Filter Change PGA threshold for Short Cable detection */ + { 0x402, 0x1800 }, /* Adjusr LD swing */ + { 0x878, 0x2248 }, /* Change PI up/down polarity */ + { 0x10c, 0x0008 }, /* tx filter coefficient */ + { 0x112, 0x1212 }, /* tx filter scaling factor */ + { 0x809, 0x5c80 }, /* AGC retrain */ + { 0x803, 0x1529 }, /* Master Ph1 Back-off */ + { 0x804, 0x1a33 }, /* Master Ph1 Back-off */ + { 0x805, 0x1f3d }, /* Master Ph1 Back-off */ + { 0x850, 0x045b }, /* hybrid gain & delay */ + { 0x874, 0x6967 }, /* kp step 0 for master */ + { 0x852, 0x7800 }, /* FAGC init gain */ + { 0x806, 0x1e1e }, /* Master/Slave Ph2 Back-off */ + { 0x807, 0x2525 }, /* Master/Slave Ph2 Back-off */ + { 0x808, 0x2c2c }, /* Master/Slave Ph2 Back-off */ + { 0x850, 0x0590 }, /* Hybrid Gain/Delay Code */ + { 0x827, 0x4000 }, /* Echo Fixed Delay */ + { 0x849, 0x0fe4 }, /* Hybrid Cal enable */ + { 0x84b, 0x04b5 }, /* Echo Score Sel */ + { 0x018, 0x0043 }, /* Set CRS/RX_DV pin as RX_DV for RMII repeater mode */ +}; + +static int dp83td510_ack_interrupt(struct phy_device *phydev) +{ + int ret; + + ret = phy_read(phydev, DP83TD510_INT_REG1); + if (ret < 0) + return ret; + + ret = phy_read(phydev, DP83TD510_INT_REG2); + if (ret < 0) + return ret; + + return 0; +} + +static int dp83td510_config_intr(struct phy_device *phydev) +{ + int int_status; + int gen_cfg_val; + int ret; + + if (phydev->interrupts == PHY_INTERRUPT_ENABLED) { + int_status = phy_read(phydev, DP83TD510_INT_REG1); + if (int_status < 0) + return int_status; + + int_status = (DP83TD510_INT1_ESD_EN | DP83TD510_INT1_LINK_EN | + DP83TD510_INT1_RHF_EN); + + ret = phy_write(phydev, DP83TD510_INT_REG1, int_status); + if (ret) + return ret; + + int_status = phy_read(phydev, DP83TD510_INT_REG2); + if (int_status < 0) + return int_status; + + int_status = (DP83TD510_INT2_POR | DP83TD510_INT2_POL | + DP83TD510_INT2_PAGE); + + ret = phy_write(phydev, DP83TD510_INT_REG2, int_status); + if (ret) + return ret; + + gen_cfg_val = phy_read(phydev, DP83TD510_GEN_CFG); + if (gen_cfg_val < 0) + return gen_cfg_val; + + gen_cfg_val |= DP83TD510_INT_OE | DP83TD510_INT_EN; + + } else { + ret = phy_write(phydev, DP83TD510_INT_REG1, 0); + if (ret) + return ret; + + ret = phy_write(phydev, DP83TD510_INT_REG2, 0); + if (ret) + return ret; + + gen_cfg_val = phy_read(phydev, DP83TD510_GEN_CFG); + if (gen_cfg_val < 0) + return gen_cfg_val; + + gen_cfg_val &= ~DP83TD510_INT_EN; + } + + return phy_write(phydev, DP83TD510_GEN_CFG, gen_cfg_val); +} + +static int dp83td510_configure_mode(struct phy_device *phydev) +{ + struct dp83td510_private *dp83td510 = phydev->priv; + struct dp83td510_init_reg *init_data; + int size; + int ret; + int i; + + ret = phy_set_bits(phydev, DP83TD510_MII_REG, DP83TD510_HW_RESET); + if (ret < 0) + return ret; + + switch (dp83td510->forced_mode) { + case DP83TD510_MASTER_1: + size = ARRAY_SIZE(dp83td510_master_1_0); + init_data = dp83td510_master_1_0; + break; + case DP83TD510_MASTER_24: + size = ARRAY_SIZE(dp83td510_master_2_4); + init_data = dp83td510_master_2_4; + break; + case DP83TD510_SLAVE_1: + size = ARRAY_SIZE(dp83td510_slave_1_0); + init_data = dp83td510_slave_1_0; + break; + case DP83TD510_SLAVE_24: + size = ARRAY_SIZE(dp83td510_slave_2_4); + init_data = dp83td510_slave_2_4; + break; + case DP83TD510_AUTO_NEG: + default: + size = ARRAY_SIZE(dp83td510_auto_neg); + init_data = dp83td510_auto_neg; + break; + }; + + for (i = 0; i < size; i++) { + ret = phy_write_mmd(phydev, DP83TD510_DEVADDR, init_data[i].reg, + init_data[i].val); + if (ret) + return ret; + } + + return phy_set_bits(phydev, DP83TD510_MII_REG, DP83TD510_SW_RESET); +} + +static int dp83td510_config_init(struct phy_device *phydev) +{ + struct dp83td510_private *dp83td510 = phydev->priv; + int ret = 0; + + if (phy_interface_is_rgmii(phydev)) { + if (dp83td510->rgmii_delay) { + ret = phy_set_bits_mmd(phydev, DP83TD510_DEVADDR, + DP83TD510_MAC_CFG_1, dp83td510->rgmii_delay); + if (ret) + return ret; + } + } + + if (phydev->interface == PHY_INTERFACE_MODE_RMII) + ret = phy_modify(phydev, DP83TD510_GEN_CFG, + DP83TD510_FIFO_DEPTH_MASK, + dp83td510->tx_fifo_depth); + + linkmode_clear_bit(ETHTOOL_LINK_MODE_100baseT_Half_BIT, phydev->advertising); + linkmode_clear_bit(ETHTOOL_LINK_MODE_100baseT_Half_BIT, phydev->supported); + linkmode_clear_bit(ETHTOOL_LINK_MODE_100baseT_Full_BIT, phydev->advertising); + linkmode_clear_bit(ETHTOOL_LINK_MODE_100baseT_Full_BIT, phydev->supported); + linkmode_clear_bit(ETHTOOL_LINK_MODE_MII_BIT, phydev->advertising); + linkmode_clear_bit(ETHTOOL_LINK_MODE_MII_BIT, phydev->supported); + + return dp83td510_configure_mode(phydev); +} + +static int dp83td510_phy_reset(struct phy_device *phydev) +{ + int ret; + + ret = phy_set_bits(phydev, DP83TD510_MII_REG, DP83TD510_SW_RESET); + if (ret < 0) + return ret; + + usleep_range(10, 20); + + return dp83td510_config_init(phydev); +} + +static int dp83td510_read_straps(struct phy_device *phydev) +{ + struct dp83td510_private *dp83td510 = phydev->priv; + int strap; + + strap = phy_read_mmd(phydev, DP83TD510_DEVADDR, DP83TD510_SOR_1); + if (strap < 0) + return strap; + + if (strap & DP83TD510_RGMII) + dp83td510->is_rgmii = true; + + return 0; +}; + +#if IS_ENABLED(CONFIG_OF_MDIO) +static int dp83td510_of_init(struct phy_device *phydev) +{ + struct dp83td510_private *dp83td510 = phydev->priv; + struct device *dev = &phydev->mdio.dev; + struct device_node *of_node = dev->of_node; + s32 rx_int_delay; + s32 tx_int_delay; + int ret; + + if (!of_node) + return -ENODEV; + + ret = dp83td510_read_straps(phydev); + if (ret) + return ret; + + ret = device_property_read_u32(&phydev->mdio.dev, "ti,master-slave-mode", + &dp83td510->forced_mode); + if (ret) + dp83td510->forced_mode = DP83TD510_AUTO_NEG; + + if (dp83td510->forced_mode < DP83TD510_AUTO_NEG || + dp83td510->forced_mode > DP83TD510_SLAVE_24) + return -EINVAL; + + if (device_property_read_u32(&phydev->mdio.dev, "tx-fifo-depth", + &dp83td510->tx_fifo_depth)) + dp83td510->tx_fifo_depth = DP83TD510_FIFO_DEPTH_5_B_NIB; + + switch (dp83td510->tx_fifo_depth) { + case 4: + dp83td510->tx_fifo_depth = DP83TD510_FIFO_DEPTH_4_B_NIB; + break; + case 5: + dp83td510->tx_fifo_depth = DP83TD510_FIFO_DEPTH_5_B_NIB; + break; + case 6: + dp83td510->tx_fifo_depth = DP83TD510_FIFO_DEPTH_6_B_NIB; + break; + case 8: + dp83td510->tx_fifo_depth = DP83TD510_FIFO_DEPTH_8_B_NIB; + break; + default: + return -EINVAL; + } + + rx_int_delay = phy_get_internal_delay(phydev, dev, NULL, 0, true); + if (rx_int_delay <= 0) + dp83td510->rgmii_delay = DP83TD510_RX_CLK_SHIFT; + else + dp83td510->rgmii_delay = 0; + + tx_int_delay = phy_get_internal_delay(phydev, dev, NULL, 0, false); + if (tx_int_delay <= 0) + dp83td510->rgmii_delay |= DP83TD510_TX_CLK_SHIFT; + else + dp83td510->rgmii_delay &= ~DP83TD510_TX_CLK_SHIFT; + + return ret; +} +#else +static int dp83869_of_init(struct phy_device *phydev) +{ + return dp83td510_read_straps(phydev); +} +#endif /* CONFIG_OF_MDIO */ + +static int dp83td510_probe(struct phy_device *phydev) +{ + struct dp83td510_private *dp83td510; + int ret; + + dp83td510 = devm_kzalloc(&phydev->mdio.dev, sizeof(*dp83td510), GFP_KERNEL); + if (!dp83td510) + return -ENOMEM; + + phydev->priv = dp83td510; + + ret = dp83td510_of_init(phydev); + if (ret) + return ret; + + return dp83td510_config_init(phydev); +} + +static struct phy_driver dp83td510_driver[] = { + { + PHY_ID_MATCH_MODEL(DP83TD510E_PHY_ID), + .name = "TI DP83TD510E", + .probe = dp83td510_probe, + .config_init = dp83td510_config_init, + .soft_reset = dp83td510_phy_reset, + .features = PHY_BASIC_FEATURES, + + /* IRQ related */ + .ack_interrupt = dp83td510_ack_interrupt, + .config_intr = dp83td510_config_intr, + + .suspend = genphy_suspend, + .resume = genphy_resume, + }, +}; +module_phy_driver(dp83td510_driver); + +static struct mdio_device_id __maybe_unused dp83td510_tbl[] = { + { PHY_ID_MATCH_MODEL(DP83TD510E_PHY_ID) }, + { } +}; +MODULE_DEVICE_TABLE(mdio, dp83td510_tbl); + +MODULE_DESCRIPTION("Texas Instruments DP83TD510E PHY driver"); +MODULE_AUTHOR("Dan Murphy <dmurphy@ti.com"); +MODULE_LICENSE("GPL v2"); + -- 2.28.0.585.ge1cfff676549 ^ permalink raw reply related [flat|nested] 13+ messages in thread
* Re: [PATCH net-next 2/2] net: phy: dp83td510: Add support for the DP83TD510 Ethernet PHY 2020-10-08 16:23 ` [PATCH net-next 2/2] net: phy: dp83td510: Add support for the " Dan Murphy @ 2020-10-08 16:51 ` Heiner Kallweit 2020-10-08 17:00 ` Dan Murphy 2020-10-16 22:02 ` Andrew Lunn 1 sibling, 1 reply; 13+ messages in thread From: Heiner Kallweit @ 2020-10-08 16:51 UTC (permalink / raw) To: Dan Murphy, davem, andrew, f.fainelli; +Cc: netdev, linux-kernel On 08.10.2020 18:23, Dan Murphy wrote: > The DP83TD510E is an ultra-low power Ethernet physical layer transceiver > that supports 10M single pair cable. > > The device supports both 2.4-V p2p and 1-V p2p output voltage as defined > by IEEE 802.3cg 10Base-T1L specfications. These modes can be forced via > the device tree or the device is defaulted to auto negotiation to > determine the proper p2p voltage. > > Signed-off-by: Dan Murphy <dmurphy@ti.com> > --- > drivers/net/phy/Kconfig | 6 + > drivers/net/phy/Makefile | 1 + > drivers/net/phy/dp83td510.c | 583 ++++++++++++++++++++++++++++++++++++ > 3 files changed, 590 insertions(+) > create mode 100644 drivers/net/phy/dp83td510.c > > diff --git a/drivers/net/phy/Kconfig b/drivers/net/phy/Kconfig > index 698bea312adc..017252e1504c 100644 > --- a/drivers/net/phy/Kconfig > +++ b/drivers/net/phy/Kconfig > @@ -302,6 +302,12 @@ config DP83869_PHY > Currently supports the DP83869 PHY. This PHY supports copper and > fiber connections. > > +config DP83TD510_PHY > + tristate "Texas Instruments DP83TD510 10M Single Pair Ethernet PHY" > + help > + Support for the DP83TD510 Ethernet PHY. This PHY supports a 10M single > + pair Ethernet connection. > + > config VITESSE_PHY > tristate "Vitesse PHYs" > help > diff --git a/drivers/net/phy/Makefile b/drivers/net/phy/Makefile > index a13e402074cf..bf62ce211eb4 100644 > --- a/drivers/net/phy/Makefile > +++ b/drivers/net/phy/Makefile > @@ -56,6 +56,7 @@ obj-$(CONFIG_DP83848_PHY) += dp83848.o > obj-$(CONFIG_DP83867_PHY) += dp83867.o > obj-$(CONFIG_DP83869_PHY) += dp83869.o > obj-$(CONFIG_DP83TC811_PHY) += dp83tc811.o > +obj-$(CONFIG_DP83TD510_PHY) += dp83td510.o > obj-$(CONFIG_FIXED_PHY) += fixed_phy.o > obj-$(CONFIG_ICPLUS_PHY) += icplus.o > obj-$(CONFIG_INTEL_XWAY_PHY) += intel-xway.o > diff --git a/drivers/net/phy/dp83td510.c b/drivers/net/phy/dp83td510.c > new file mode 100644 > index 000000000000..0cce9063fd1c > --- /dev/null > +++ b/drivers/net/phy/dp83td510.c > @@ -0,0 +1,583 @@ > +// SPDX-License-Identifier: GPL-2.0 > +/* Driver for the Texas Instruments DP83TD510 PHY > + * Copyright (C) 2020 Texas Instruments Incorporated - https://www.ti.com/ > + */ > + > +#include <linux/ethtool.h> > +#include <linux/etherdevice.h> > +#include <linux/kernel.h> > +#include <linux/mii.h> > +#include <linux/module.h> > +#include <linux/of.h> > +#include <linux/phy.h> > +#include <linux/netdevice.h> > + > +#define DP83TD510E_PHY_ID 0x20000180 > +#define DP83TD510_DEVADDR_AN 0x7 > +#define DP83TD510_DEVADDR 0x1f > + > +#define DP83TD510_MII_REG 0x0 > +#define DP83TD510_PHY_STAT 0x10 > +#define DP83TD510_GEN_CFG 0x11 > +#define DP83TD510_INT_REG1 0x12 > +#define DP83TD510_INT_REG2 0x13 > +#define DP83TD510_MAC_CFG_1 0x17 > + > +#define DP83TD510_SOR_1 0x467 > + > +#define DP83TD510_HW_RESET BIT(15) > +#define DP83TD510_SW_RESET BIT(14) > + > +/* GEN CFG bits */ > +#define DP83TD510_INT_OE BIT(0) > +#define DP83TD510_INT_EN BIT(1) > + > +/* INT REG 1 bits */ > +#define DP83TD510_INT1_ESD_EN BIT(3) > +#define DP83TD510_INT1_LINK_EN BIT(5) > +#define DP83TD510_INT1_RHF_EN BIT(7) > +#define DP83TD510_INT1_ESD BIT(11) > +#define DP83TD510_INT1_LINK BIT(13) > +#define DP83TD510_INT1_RHF BIT(15) > + > +/* INT REG 2 bits */ > +#define DP83TD510_INT2_POR_EN BIT(0) > +#define DP83TD510_INT2_POL_EN BIT(1) > +#define DP83TD510_INT2_PAGE_EN BIT(5) > +#define DP83TD510_INT2_POR BIT(8) > +#define DP83TD510_INT2_POL BIT(9) > +#define DP83TD510_INT2_PAGE BIT(13) > + > +/* MAC CFG bits */ > +#define DP83TD510_RX_CLK_SHIFT BIT(12) > +#define DP83TD510_TX_CLK_SHIFT BIT(11) > + > +#define DP83TD510_MASTER_MODE BIT(2) > +#define DP83TD510_2_4V BIT(7) > +#define DP83TD510_RGMII BIT(8) > + > +#define DP83TD510_FIFO_DEPTH_MASK GENMASK(6, 5) > +#define DP83TD510_FIFO_DEPTH_4_B_NIB 0 > +#define DP83TD510_FIFO_DEPTH_5_B_NIB BIT(5) > +#define DP83TD510_FIFO_DEPTH_6_B_NIB BIT(6) > +#define DP83TD510_FIFO_DEPTH_8_B_NIB (BIT(5) | BIT(6)) > + > +enum dp83td510_mode_config { > + DP83TD510_AUTO_NEG = 0, > + DP83TD510_MASTER_1 = 1, > + DP83TD510_MASTER_24 = 2, > + DP83TD510_SLAVE_1 = 3, > + DP83TD510_SLAVE_24 = 4, > +}; > + > +struct dp83td510_private { > + u32 forced_mode; > + u32 tx_fifo_depth; > + u32 rgmii_delay; > + bool is_rgmii; > +}; > + > +struct dp83td510_init_reg { > + int reg; > + int val; > +}; > + > +static struct dp83td510_init_reg dp83td510_master_1_0[] = { > + { 0x000d, 0x0007 }, /* disable auto-neg */ > + { 0x000e, 0x0200 }, > + { 0x000d, 0x4007 }, > + { 0x000e, 0x0000 }, > + { 0x000d, 0x0001 }, /* force master mode */ > + { 0x000e, 0x0834 }, > + { 0x000d, 0x4001 }, > + { 0x000e, 0x4000 }, > + { 0x000d, 0x0001 }, /* force 1.0v swing */ > + { 0x000e, 0x08f6 }, > + { 0x000d, 0x4001 }, > + { 0x000e, 0x0000 }, > + { 0x0608, 0x003b }, /* disable_0_transition */ > + { 0x0862, 0x39f8 }, /* AGC Gain during Autoneg */ > + { 0x081a, 0x67c0 }, /* deq offset for 1V swing */ > + { 0x081c, 0xfb62 }, /* deq offset for 2.4V swing */ > + { 0x0830, 0x05a3 }, /* Enable energy lost fallback */ > + { 0x0855, 0x1b55 }, /* MSE Threshold change */ > + { 0x0831, 0x0403 }, /* energy detect threshold */ > + { 0x0856, 0x1800 }, /* good1 MSE threshold change */ > + { 0x0857, 0x8fa0 }, /* Enable fallback to phase 1 on watchdog trigger */ > + { 0x0871, 0x000c }, /* TED input changed to slicer_in without FFE */ > + { 0x0883, 0x022e }, /* Enable Rx Filter, Change PGA threshold for Short Cable detection */ > + { 0x0402, 0x1800 }, /* Adjusr LD swing */ > + { 0x0878, 0x2248 }, /* Change PI up/down polarity */ > + { 0x010c, 0x0008 }, /* tx filter coefficient */ > + { 0x0112, 0x1212 }, /* tx filter scaling factor */ > + { 0x0809, 0x5c80 }, /* AGC retrain */ > + { 0x0803, 0x1529 }, /* Master Ph1 Back-off */ > + { 0x0804, 0x1a33 }, /* Master Ph1 Back-off */ > + { 0x0805, 0x1f3d }, /* Master Ph1 Back-off */ > + { 0x0850, 0x045b }, /* hybrid gain & delay */ > + { 0x0874, 0x6967 }, /* kp step 0 for master */ > + { 0x0852, 0x7800 }, /* FAGC init gain */ > + { 0x0806, 0x1e1e }, /* Master/Slave Ph2 Back-off */ > + { 0x0807, 0x2525 }, /* Master/Slave Ph2 Back-off */ > + { 0x0808, 0x2c2c }, /* Master/Slave Ph2 Back-off */ > + { 0x0850, 0x0590 }, /* Hybrid Gain/Delay Code */ > + { 0x0827, 0x4000 }, /* Echo Fixed Delay */ > + { 0x0849, 0x0fe4 }, /* Hybrid Cal enable */ > + { 0x084b, 0x04b5 }, /* Echo Score Sel */ > + { 0x0018, 0x0043 }, /* Set CRS/RX_DV pin as RX_DV for RMII repeater mode */ > +}; > + > +static struct dp83td510_init_reg dp83td510_slave_1_0[] = { > + { 0x000d, 0x0007 }, /* disable auto-neg */ > + { 0x000e, 0x0200 }, > + { 0x000d, 0x4007 }, > + { 0x000e, 0x0000 }, > + { 0x000d, 0x0001 }, /* force slave mode */ > + { 0x000e, 0x0834 }, > + { 0x000d, 0x4001 }, > + { 0x000e, 0x0000 }, > + { 0x000d, 0x0001 }, /* force 1.0v swing */ > + { 0x000d, 0x4001 }, > + { 0x000e, 0x0000 }, > + { 0x0608, 0x003b }, /* disable_0_transition */ > + { 0x0862, 0x39f8 }, /* AGC Gain during Autoneg */ > + { 0x081a, 0x67c0 }, /* deq offset for 1V swing */ > + { 0x081c, 0xfb62 }, /* deq offset for 2.4V swing */ > + { 0x0830, 0x05a3 }, /* Enable energy lost fallback */ > + { 0x0855, 0x1b55 }, /* MSE Threshold change */ > + { 0x0831, 0x0403 }, /* energy detect threshold */ > + { 0x0856, 0x1800 }, /* good1 MSE threshold change */ > + { 0x0857, 0x8fa0 }, /* Enable fallback to phase 1 on watchdog trigger */ > + { 0x0871, 0x000c }, /* TED input changed to slicer_in without FFE */ > + { 0x0883, 0x022e }, /* Enable Rx Filter, PGA threshold for Short Cable detection */ > + { 0x0402, 0x1800 }, /* Adjusr LD swing */ > + { 0x0878, 0x2248 }, /* Change PI up/down polarity */ > + { 0x010c, 0x0008 }, /* tx filter coefficient */ > + { 0x0112, 0x1212 }, /* tx filter scaling factor */ > + { 0x0809, 0x5c80 }, /* AGC retrain */ > + { 0x0803, 0x1529 }, /* Master Ph1 Back-off */ > + { 0x0804, 0x1a33 }, /* Master Ph1 Back-off */ > + { 0x0805, 0x1f3d }, /* Master Ph1 Back-off */ > + { 0x0850, 0x045b }, /* hybrid gain & delay */ > + { 0x0874, 0x6967 }, /* kp step 0 for master */ > + { 0x0852, 0x7800 }, /* FAGC init gain */ > + { 0x0806, 0x1e1e }, /* Master/Slave Ph2 Back-off */ > + { 0x0807, 0x2525 }, /* Master/Slave Ph2 Back-off */ > + { 0x0808, 0x2c2c }, /* Master/Slave Ph2 Back-off */ > + { 0x0850, 0x0590 }, /* Hybrid Gain/Delay Code */ > + { 0x0827, 0x4000 }, /* Echo Fixed Delay */ > + { 0x0849, 0x0fe4 }, /* Hybrid Cal enable */ > + { 0x084b, 0x04b5 }, /* Echo Score Sel */ > + { 0x0018, 0x0043 }, /* Set CRS/RX_DV pin as RX_DV for RMII repeater mode */ > +}; > + > +static struct dp83td510_init_reg dp83td510_master_2_4[] = { > + { 0x000d, 0x0007 }, /* disable auto-neg */ > + { 0x000e, 0x0200 }, > + { 0x000d, 0x4007 }, > + { 0x000e, 0x0000 }, > + { 0x000d, 0x0001 }, /* force Master mode */ > + { 0x000e, 0x0834 }, > + { 0x000d, 0x4001 }, > + { 0x000e, 0x4000 }, > + { 0x000d, 0x0001 }, /* force 2.4v swing */ > + { 0x000e, 0x08f6 }, > + { 0x000d, 0x4001 }, > + { 0x000e, 0x1000 }, > + { 0x0608, 0x003b }, /* disable_0_transition */ > + { 0x0862, 0x39f8 }, /* AGC Gain during Autoneg */ > + { 0x081a, 0x67c0 }, /* deq offset for 1V swing */ > + { 0x081c, 0xfb62 }, /* deq offset for 2.4V swing */ > + { 0x0830, 0x05a3 }, /* Enable energy lost fallback */ > + { 0x0855, 0x1b55 }, /* MSE Threshold change */ > + { 0x0831, 0x0403 }, /* energy detect threshold */ > + { 0x0856, 0x1800 }, /* good1 MSE threshold change */ > + { 0x0857, 0x8fa0 }, /* Enable fallback to phase 1 on watchdog trigger */ > + { 0x0871, 0x000c }, /* TED input changed to slicer_in without FFE */ > + { 0x0883, 0x022e }, /* Enable Rx Filter, Change PGA threshold for Short Cable detection */ > + { 0x0402, 0x1800 }, /* Adjusr LD swing */ > + { 0x0878, 0x2248 }, /* Change PI up/down polarity */ > + { 0x010c, 0x0008 }, /* tx filter coefficient */ > + { 0x0112, 0x1212 }, /* tx filter scaling factor */ > + { 0x0809, 0x5c80 }, /* AGC retrain */ > + { 0x0803, 0x1529 }, /* Master Ph1 Back-off */ > + { 0x0804, 0x1a33 }, /* Master Ph1 Back-off */ > + { 0x0805, 0x1f3d }, /* Master Ph1 Back-off */ > + { 0x0850, 0x045b }, /* hybrid gain & delay */ > + { 0x0874, 0x6967 }, /* kp step 0 for master */ > + { 0x0852, 0x7800 }, /* FAGC init gain */ > + { 0x0806, 0x1e1e }, /* Master/Slave Ph2 Back-off */ > + { 0x0807, 0x2525 }, /* Master/Slave Ph2 Back-off */ > + { 0x0808, 0x2c2c }, /* Master/Slave Ph2 Back-off */ > + { 0x0850, 0x0590 }, /* Hybrid Gain/Delay Code */ > + { 0x0827, 0x4000 }, /* Echo Fixed Delay */ > + { 0x0849, 0x0fe4 }, /* Hybrid Cal enable */ > + { 0x084b, 0x04b5 }, /* Echo Score Sel */ > + { 0x0018, 0x0043 }, /* Set CRS/RX_DV pin as RX_DV for RMII repeater mode */ > +}; > + > +static struct dp83td510_init_reg dp83td510_slave_2_4[] = { > + { 0x000d, 0x0007}, /* disable auto-neg */ > + { 0x000e, 0x0200}, > + { 0x000d, 0x4007}, > + { 0x000e, 0x0000}, > + { 0x000d, 0x0001}, /* force slave mode */ > + { 0x000e, 0x0834}, > + { 0x000d, 0x4001}, > + { 0x000e, 0x0000}, > + { 0x000d, 0x0001}, /* force 2.4v swing */ > + { 0x000e, 0x08f6}, > + { 0x000d, 0x4001}, > + { 0x000e, 0x1000}, > + { 0x0608, 0x003b}, /* disable_0_transition */ > + { 0x0862, 0x39f8}, /* AGC Gain during Autoneg */ > + { 0x081a, 0x67c0}, /* deq offset for 1V swing */ > + { 0x081c, 0xfb62}, /* deq offset for 2.4V swing */ > + { 0x0830, 0x05a3}, /* Enable energy lost fallback */ > + { 0x0855, 0x1b55}, /* MSE Threshold change */ > + { 0x0831, 0x0403}, /* energy detect threshold */ > + { 0x0856, 0x1800}, /* good1 MSE threshold change */ > + { 0x0857, 0x8fa0}, /* Enable fallback to phase 1 on watchdog trigger */ > + { 0x0871, 0x000c}, /* TED input changed to slicer_in without FFE */ > + { 0x0883, 0x022e}, /* Enable Rx Filter, Change PGA threshold for Short Cable detection */ > + { 0x0402, 0x1800}, /* Adjusr LD swing */ > + { 0x0878, 0x2248}, /* Change PI up/down polarity */ > + { 0x010c, 0x0008}, /* tx filter coefficient */ > + { 0x0112, 0x1212}, /* tx filter scaling factor */ > + { 0x0809, 0x5c80}, /* AGC retrain */ > + { 0x0803, 0x1529}, /* Master Ph1 Back-off */ > + { 0x0804, 0x1a33}, /* Master Ph1 Back-off */ > + { 0x0805, 0x1f3d}, /* Master Ph1 Back-off */ > + { 0x0850, 0x045b}, /* hybrid gain & delay */ > + { 0x0874, 0x6967}, /* kp step 0 for master */ > + { 0x0852, 0x7800}, /* FAGC init gain */ > + { 0x0806, 0x1e1e}, /* Master/Slave Ph2 Back-off */ > + { 0x0807, 0x2525}, /* Master/Slave Ph2 Back-off */ > + { 0x0808, 0x2c2c}, /* Master/Slave Ph2 Back-off */ > + { 0x0850, 0x0590}, /* Hybrid Gain/Delay Code */ > + { 0x0827, 0x4000}, /* Echo Fixed Delay */ > + { 0x0849, 0x0fe4}, /* Hybrid Cal enable */ > + { 0x084b, 0x04b5}, /* Echo Score Sel */ > + { 0x0018, 0x0043}, /* Set CRS/RX_DV pin as RX_DV for RMII repeater mode */ > +}; > + > +static struct dp83td510_init_reg dp83td510_auto_neg[] = { > + { 0x608, 0x003b }, /* disable_0_transition */ > + { 0x862, 0x39f8 }, /* AGC Gain during Autoneg */ > + { 0x81a, 0x67c0 }, /* deq offset for 1V swing */ > + { 0x81c, 0xfb62 }, /* deq offset for 2.4V swing */ > + { 0x830, 0x05a3 }, /* Enable energy lost fallback */ > + { 0x855, 0x1b55 }, /* MSE Threshold change */ > + { 0x831, 0x0403 }, /* energy detect threshold */ > + { 0x856, 0x1800 }, /* good1 MSE threshold change */ > + { 0x857, 0x8fa0 }, /* Enable fallback to phase 1 on watchdog trigger */ > + { 0x871, 0x000c }, /* TED input changed to slicer_in without FFE */ > + { 0x883, 0x022e }, /* Enable Rx Filter Change PGA threshold for Short Cable detection */ > + { 0x402, 0x1800 }, /* Adjusr LD swing */ > + { 0x878, 0x2248 }, /* Change PI up/down polarity */ > + { 0x10c, 0x0008 }, /* tx filter coefficient */ > + { 0x112, 0x1212 }, /* tx filter scaling factor */ > + { 0x809, 0x5c80 }, /* AGC retrain */ > + { 0x803, 0x1529 }, /* Master Ph1 Back-off */ > + { 0x804, 0x1a33 }, /* Master Ph1 Back-off */ > + { 0x805, 0x1f3d }, /* Master Ph1 Back-off */ > + { 0x850, 0x045b }, /* hybrid gain & delay */ > + { 0x874, 0x6967 }, /* kp step 0 for master */ > + { 0x852, 0x7800 }, /* FAGC init gain */ > + { 0x806, 0x1e1e }, /* Master/Slave Ph2 Back-off */ > + { 0x807, 0x2525 }, /* Master/Slave Ph2 Back-off */ > + { 0x808, 0x2c2c }, /* Master/Slave Ph2 Back-off */ > + { 0x850, 0x0590 }, /* Hybrid Gain/Delay Code */ > + { 0x827, 0x4000 }, /* Echo Fixed Delay */ > + { 0x849, 0x0fe4 }, /* Hybrid Cal enable */ > + { 0x84b, 0x04b5 }, /* Echo Score Sel */ > + { 0x018, 0x0043 }, /* Set CRS/RX_DV pin as RX_DV for RMII repeater mode */ > +}; > + > +static int dp83td510_ack_interrupt(struct phy_device *phydev) > +{ > + int ret; > + > + ret = phy_read(phydev, DP83TD510_INT_REG1); > + if (ret < 0) > + return ret; > + > + ret = phy_read(phydev, DP83TD510_INT_REG2); > + if (ret < 0) > + return ret; > + > + return 0; > +} > + > +static int dp83td510_config_intr(struct phy_device *phydev) > +{ > + int int_status; > + int gen_cfg_val; > + int ret; > + > + if (phydev->interrupts == PHY_INTERRUPT_ENABLED) { > + int_status = phy_read(phydev, DP83TD510_INT_REG1); > + if (int_status < 0) > + return int_status; > + > + int_status = (DP83TD510_INT1_ESD_EN | DP83TD510_INT1_LINK_EN | > + DP83TD510_INT1_RHF_EN); > + > + ret = phy_write(phydev, DP83TD510_INT_REG1, int_status); > + if (ret) > + return ret; > + > + int_status = phy_read(phydev, DP83TD510_INT_REG2); > + if (int_status < 0) > + return int_status; > + > + int_status = (DP83TD510_INT2_POR | DP83TD510_INT2_POL | > + DP83TD510_INT2_PAGE); > + > + ret = phy_write(phydev, DP83TD510_INT_REG2, int_status); > + if (ret) > + return ret; > + > + gen_cfg_val = phy_read(phydev, DP83TD510_GEN_CFG); > + if (gen_cfg_val < 0) > + return gen_cfg_val; > + > + gen_cfg_val |= DP83TD510_INT_OE | DP83TD510_INT_EN; > + > + } else { > + ret = phy_write(phydev, DP83TD510_INT_REG1, 0); > + if (ret) > + return ret; > + > + ret = phy_write(phydev, DP83TD510_INT_REG2, 0); > + if (ret) > + return ret; > + > + gen_cfg_val = phy_read(phydev, DP83TD510_GEN_CFG); > + if (gen_cfg_val < 0) > + return gen_cfg_val; > + > + gen_cfg_val &= ~DP83TD510_INT_EN; > + } > + > + return phy_write(phydev, DP83TD510_GEN_CFG, gen_cfg_val); > +} > + > +static int dp83td510_configure_mode(struct phy_device *phydev) > +{ > + struct dp83td510_private *dp83td510 = phydev->priv; > + struct dp83td510_init_reg *init_data; > + int size; > + int ret; > + int i; > + > + ret = phy_set_bits(phydev, DP83TD510_MII_REG, DP83TD510_HW_RESET); > + if (ret < 0) > + return ret; > + > + switch (dp83td510->forced_mode) { > + case DP83TD510_MASTER_1: > + size = ARRAY_SIZE(dp83td510_master_1_0); > + init_data = dp83td510_master_1_0; > + break; > + case DP83TD510_MASTER_24: > + size = ARRAY_SIZE(dp83td510_master_2_4); > + init_data = dp83td510_master_2_4; > + break; > + case DP83TD510_SLAVE_1: > + size = ARRAY_SIZE(dp83td510_slave_1_0); > + init_data = dp83td510_slave_1_0; > + break; > + case DP83TD510_SLAVE_24: > + size = ARRAY_SIZE(dp83td510_slave_2_4); > + init_data = dp83td510_slave_2_4; > + break; > + case DP83TD510_AUTO_NEG: > + default: > + size = ARRAY_SIZE(dp83td510_auto_neg); > + init_data = dp83td510_auto_neg; > + break; > + }; > + > + for (i = 0; i < size; i++) { > + ret = phy_write_mmd(phydev, DP83TD510_DEVADDR, init_data[i].reg, > + init_data[i].val); > + if (ret) > + return ret; > + } > + > + return phy_set_bits(phydev, DP83TD510_MII_REG, DP83TD510_SW_RESET); > +} > + > +static int dp83td510_config_init(struct phy_device *phydev) > +{ > + struct dp83td510_private *dp83td510 = phydev->priv; > + int ret = 0; > + > + if (phy_interface_is_rgmii(phydev)) { > + if (dp83td510->rgmii_delay) { > + ret = phy_set_bits_mmd(phydev, DP83TD510_DEVADDR, > + DP83TD510_MAC_CFG_1, dp83td510->rgmii_delay); > + if (ret) > + return ret; > + } > + } > + > + if (phydev->interface == PHY_INTERFACE_MODE_RMII) > + ret = phy_modify(phydev, DP83TD510_GEN_CFG, > + DP83TD510_FIFO_DEPTH_MASK, > + dp83td510->tx_fifo_depth); > + > + linkmode_clear_bit(ETHTOOL_LINK_MODE_100baseT_Half_BIT, phydev->advertising); > + linkmode_clear_bit(ETHTOOL_LINK_MODE_100baseT_Half_BIT, phydev->supported); > + linkmode_clear_bit(ETHTOOL_LINK_MODE_100baseT_Full_BIT, phydev->advertising); > + linkmode_clear_bit(ETHTOOL_LINK_MODE_100baseT_Full_BIT, phydev->supported); > + linkmode_clear_bit(ETHTOOL_LINK_MODE_MII_BIT, phydev->advertising); > + linkmode_clear_bit(ETHTOOL_LINK_MODE_MII_BIT, phydev->supported); > + > + return dp83td510_configure_mode(phydev); > +} > + > +static int dp83td510_phy_reset(struct phy_device *phydev) > +{ > + int ret; > + > + ret = phy_set_bits(phydev, DP83TD510_MII_REG, DP83TD510_SW_RESET); > + if (ret < 0) > + return ret; > + > + usleep_range(10, 20); > + > + return dp83td510_config_init(phydev); > +} > + > +static int dp83td510_read_straps(struct phy_device *phydev) > +{ > + struct dp83td510_private *dp83td510 = phydev->priv; > + int strap; > + > + strap = phy_read_mmd(phydev, DP83TD510_DEVADDR, DP83TD510_SOR_1); > + if (strap < 0) > + return strap; > + > + if (strap & DP83TD510_RGMII) > + dp83td510->is_rgmii = true; > + > + return 0; > +}; > + > +#if IS_ENABLED(CONFIG_OF_MDIO) > +static int dp83td510_of_init(struct phy_device *phydev) > +{ > + struct dp83td510_private *dp83td510 = phydev->priv; > + struct device *dev = &phydev->mdio.dev; > + struct device_node *of_node = dev->of_node; > + s32 rx_int_delay; > + s32 tx_int_delay; > + int ret; > + > + if (!of_node) > + return -ENODEV; > + > + ret = dp83td510_read_straps(phydev); > + if (ret) > + return ret; > + > + ret = device_property_read_u32(&phydev->mdio.dev, "ti,master-slave-mode", > + &dp83td510->forced_mode); > + if (ret) > + dp83td510->forced_mode = DP83TD510_AUTO_NEG; > + > + if (dp83td510->forced_mode < DP83TD510_AUTO_NEG || > + dp83td510->forced_mode > DP83TD510_SLAVE_24) > + return -EINVAL; > + > + if (device_property_read_u32(&phydev->mdio.dev, "tx-fifo-depth", > + &dp83td510->tx_fifo_depth)) > + dp83td510->tx_fifo_depth = DP83TD510_FIFO_DEPTH_5_B_NIB; > + > + switch (dp83td510->tx_fifo_depth) { > + case 4: > + dp83td510->tx_fifo_depth = DP83TD510_FIFO_DEPTH_4_B_NIB; > + break; > + case 5: > + dp83td510->tx_fifo_depth = DP83TD510_FIFO_DEPTH_5_B_NIB; > + break; > + case 6: > + dp83td510->tx_fifo_depth = DP83TD510_FIFO_DEPTH_6_B_NIB; > + break; > + case 8: > + dp83td510->tx_fifo_depth = DP83TD510_FIFO_DEPTH_8_B_NIB; > + break; > + default: > + return -EINVAL; > + } > + > + rx_int_delay = phy_get_internal_delay(phydev, dev, NULL, 0, true); > + if (rx_int_delay <= 0) > + dp83td510->rgmii_delay = DP83TD510_RX_CLK_SHIFT; > + else > + dp83td510->rgmii_delay = 0; > + > + tx_int_delay = phy_get_internal_delay(phydev, dev, NULL, 0, false); > + if (tx_int_delay <= 0) > + dp83td510->rgmii_delay |= DP83TD510_TX_CLK_SHIFT; > + else > + dp83td510->rgmii_delay &= ~DP83TD510_TX_CLK_SHIFT; > + > + return ret; > +} > +#else > +static int dp83869_of_init(struct phy_device *phydev) > +{ > + return dp83td510_read_straps(phydev); > +} > +#endif /* CONFIG_OF_MDIO */ > + > +static int dp83td510_probe(struct phy_device *phydev) > +{ > + struct dp83td510_private *dp83td510; > + int ret; > + > + dp83td510 = devm_kzalloc(&phydev->mdio.dev, sizeof(*dp83td510), GFP_KERNEL); > + if (!dp83td510) > + return -ENOMEM; > + > + phydev->priv = dp83td510; > + > + ret = dp83td510_of_init(phydev); > + if (ret) > + return ret; > + > + return dp83td510_config_init(phydev); > +} > + > +static struct phy_driver dp83td510_driver[] = { > + { > + PHY_ID_MATCH_MODEL(DP83TD510E_PHY_ID), > + .name = "TI DP83TD510E", > + .probe = dp83td510_probe, > + .config_init = dp83td510_config_init, > + .soft_reset = dp83td510_phy_reset, > + .features = PHY_BASIC_FEATURES, Per default phylib uses genphy_read_abilities() to auto-detect the features. Doesn't your PHY support the needed clause 22 standard registers? Or are they incorrectly populated? Maybe better than setting PHY_BASIC_FEATURES and then removing unsuported features in dp83td510_config_init() would be to implement phy_driver callback get_features. Then you can set the supported fatures directly. > + > + /* IRQ related */ > + .ack_interrupt = dp83td510_ack_interrupt, > + .config_intr = dp83td510_config_intr, > + > + .suspend = genphy_suspend, > + .resume = genphy_resume, > + }, > +}; > +module_phy_driver(dp83td510_driver); > + > +static struct mdio_device_id __maybe_unused dp83td510_tbl[] = { > + { PHY_ID_MATCH_MODEL(DP83TD510E_PHY_ID) }, > + { } > +}; > +MODULE_DEVICE_TABLE(mdio, dp83td510_tbl); > + > +MODULE_DESCRIPTION("Texas Instruments DP83TD510E PHY driver"); > +MODULE_AUTHOR("Dan Murphy <dmurphy@ti.com"); > +MODULE_LICENSE("GPL v2"); > + > ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH net-next 2/2] net: phy: dp83td510: Add support for the DP83TD510 Ethernet PHY 2020-10-08 16:51 ` Heiner Kallweit @ 2020-10-08 17:00 ` Dan Murphy 0 siblings, 0 replies; 13+ messages in thread From: Dan Murphy @ 2020-10-08 17:00 UTC (permalink / raw) To: Heiner Kallweit, davem, andrew, f.fainelli; +Cc: netdev, linux-kernel Heiner On 10/8/20 11:51 AM, Heiner Kallweit wrote: > On 08.10.2020 18:23, Dan Murphy wrote: >> The DP83TD510E is an ultra-low power Ethernet physical layer transceiver >> that supports 10M single pair cable. >> >> The device supports both 2.4-V p2p and 1-V p2p output voltage as defined >> by IEEE 802.3cg 10Base-T1L specfications. These modes can be forced via >> the device tree or the device is defaulted to auto negotiation to >> determine the proper p2p voltage. >> >> Signed-off-by: Dan Murphy <dmurphy@ti.com> >> --- >> drivers/net/phy/Kconfig | 6 + <snip> >> +static struct phy_driver dp83td510_driver[] = { >> + { >> + PHY_ID_MATCH_MODEL(DP83TD510E_PHY_ID), >> + .name = "TI DP83TD510E", >> + .probe = dp83td510_probe, >> + .config_init = dp83td510_config_init, >> + .soft_reset = dp83td510_phy_reset, >> + .features = PHY_BASIC_FEATURES, > Per default phylib uses genphy_read_abilities() to auto-detect > the features. Doesn't your PHY support the needed clause 22 > standard registers? Or are they incorrectly populated? > > Maybe better than setting PHY_BASIC_FEATURES and then removing > unsuported features in dp83td510_config_init() would be to > implement phy_driver callback get_features. Then you can set > the supported fatures directly. > Thanks for the review. I will have to look at this maybe the call back will be better. I will probably post v2 after net-next opens for 5.10. I wanted to get some reviews on this patch before net-next closes for the merge window. Dan ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH net-next 2/2] net: phy: dp83td510: Add support for the DP83TD510 Ethernet PHY 2020-10-08 16:23 ` [PATCH net-next 2/2] net: phy: dp83td510: Add support for the " Dan Murphy 2020-10-08 16:51 ` Heiner Kallweit @ 2020-10-16 22:02 ` Andrew Lunn 2020-10-19 21:33 ` Dan Murphy 1 sibling, 1 reply; 13+ messages in thread From: Andrew Lunn @ 2020-10-16 22:02 UTC (permalink / raw) To: Dan Murphy; +Cc: davem, f.fainelli, hkallweit1, netdev, linux-kernel On Thu, Oct 08, 2020 at 11:23:47AM -0500, Dan Murphy wrote: > The DP83TD510E is an ultra-low power Ethernet physical layer transceiver > that supports 10M single pair cable. Hi Dan I think you are going to have to add ETHTOOL_LINK_MODE_10baseT1_Full_BIT? We already have 100T1 and 1000T1, but not 10T1 :-( Andrew ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH net-next 2/2] net: phy: dp83td510: Add support for the DP83TD510 Ethernet PHY 2020-10-16 22:02 ` Andrew Lunn @ 2020-10-19 21:33 ` Dan Murphy 2020-10-19 21:55 ` Andrew Lunn 0 siblings, 1 reply; 13+ messages in thread From: Dan Murphy @ 2020-10-19 21:33 UTC (permalink / raw) To: Andrew Lunn; +Cc: davem, f.fainelli, hkallweit1, netdev, linux-kernel Andrew On 10/16/20 5:02 PM, Andrew Lunn wrote: > On Thu, Oct 08, 2020 at 11:23:47AM -0500, Dan Murphy wrote: >> The DP83TD510E is an ultra-low power Ethernet physical layer transceiver >> that supports 10M single pair cable. > Hi Dan > > I think you are going to have to add > ETHTOOL_LINK_MODE_10baseT1_Full_BIT? We already have 100T1 and 1000T1, > but not 10T1 :-( The data sheet says 10baseT1L. Which is not there either and seems to be the latest 802.3cg spec and has a greater max distance and used for IoT and Automotive. Dan > Andrew ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH net-next 2/2] net: phy: dp83td510: Add support for the DP83TD510 Ethernet PHY 2020-10-19 21:33 ` Dan Murphy @ 2020-10-19 21:55 ` Andrew Lunn 2020-10-19 22:04 ` Dan Murphy 0 siblings, 1 reply; 13+ messages in thread From: Andrew Lunn @ 2020-10-19 21:55 UTC (permalink / raw) To: Dan Murphy; +Cc: davem, f.fainelli, hkallweit1, netdev, linux-kernel On Mon, Oct 19, 2020 at 04:33:18PM -0500, Dan Murphy wrote: > Andrew > > On 10/16/20 5:02 PM, Andrew Lunn wrote: > > On Thu, Oct 08, 2020 at 11:23:47AM -0500, Dan Murphy wrote: > > > The DP83TD510E is an ultra-low power Ethernet physical layer transceiver > > > that supports 10M single pair cable. > > Hi Dan > > > > I think you are going to have to add > > ETHTOOL_LINK_MODE_10baseT1_Full_BIT? We already have 100T1 and 1000T1, > > but not 10T1 :-( > > The data sheet says 10baseT1L. Which is not there either and seems to be > the latest 802.3cg spec and has a greater max distance and used for IoT and > Automotive. Hi Dan Do you know anything about interropibility? Can a T1 and a T1L talk to each other, if suitably close? I'm wondering if this device should say it is both T1 and T1L? Or just T1L? Andrew ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH net-next 2/2] net: phy: dp83td510: Add support for the DP83TD510 Ethernet PHY 2020-10-19 21:55 ` Andrew Lunn @ 2020-10-19 22:04 ` Dan Murphy 2020-10-20 12:00 ` Dan Murphy 0 siblings, 1 reply; 13+ messages in thread From: Dan Murphy @ 2020-10-19 22:04 UTC (permalink / raw) To: Andrew Lunn; +Cc: davem, f.fainelli, hkallweit1, netdev, linux-kernel Andrew On 10/19/20 4:55 PM, Andrew Lunn wrote: > On Mon, Oct 19, 2020 at 04:33:18PM -0500, Dan Murphy wrote: >> Andrew >> >> On 10/16/20 5:02 PM, Andrew Lunn wrote: >>> On Thu, Oct 08, 2020 at 11:23:47AM -0500, Dan Murphy wrote: >>>> The DP83TD510E is an ultra-low power Ethernet physical layer transceiver >>>> that supports 10M single pair cable. >>> Hi Dan >>> >>> I think you are going to have to add >>> ETHTOOL_LINK_MODE_10baseT1_Full_BIT? We already have 100T1 and 1000T1, >>> but not 10T1 :-( >> The data sheet says 10baseT1L. Which is not there either and seems to be >> the latest 802.3cg spec and has a greater max distance and used for IoT and >> Automotive. > Hi Dan > > Do you know anything about interropibility? Can a T1 and a T1L talk to > each other, if suitably close? I'm wondering if this device should say > it is both T1 and T1L? Or just T1L? I personally don't know about interoperability my tests were T1L to T1L but I forwarded these questions to my HW PHY team. Dan > Andrew ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH net-next 2/2] net: phy: dp83td510: Add support for the DP83TD510 Ethernet PHY 2020-10-19 22:04 ` Dan Murphy @ 2020-10-20 12:00 ` Dan Murphy 2020-10-20 13:46 ` Andrew Lunn 0 siblings, 1 reply; 13+ messages in thread From: Dan Murphy @ 2020-10-20 12:00 UTC (permalink / raw) To: Andrew Lunn; +Cc: davem, f.fainelli, hkallweit1, netdev, linux-kernel Andrew On 10/19/20 5:04 PM, Dan Murphy wrote: > Andrew > > On 10/19/20 4:55 PM, Andrew Lunn wrote: >> On Mon, Oct 19, 2020 at 04:33:18PM -0500, Dan Murphy wrote: >>> Andrew >>> >>> On 10/16/20 5:02 PM, Andrew Lunn wrote: >>>> On Thu, Oct 08, 2020 at 11:23:47AM -0500, Dan Murphy wrote: >>>>> The DP83TD510E is an ultra-low power Ethernet physical layer >>>>> transceiver >>>>> that supports 10M single pair cable. >>>> Hi Dan >>>> >>>> I think you are going to have to add >>>> ETHTOOL_LINK_MODE_10baseT1_Full_BIT? We already have 100T1 and 1000T1, >>>> but not 10T1 :-( >>> The data sheet says 10baseT1L. Which is not there either and seems >>> to be >>> the latest 802.3cg spec and has a greater max distance and used for >>> IoT and >>> Automotive. >> Hi Dan >> >> Do you know anything about interropibility? Can a T1 and a T1L talk to >> each other, if suitably close? I'm wondering if this device should say >> it is both T1 and T1L? Or just T1L? > > I personally don't know about interoperability my tests were T1L to > T1L but I forwarded these questions to my HW PHY team. > HW team says no to interoperability. So we can just add T1L and advertise that type. The DP83TD510L capability is limited to 10Base-T1L only. Dan ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH net-next 2/2] net: phy: dp83td510: Add support for the DP83TD510 Ethernet PHY 2020-10-20 12:00 ` Dan Murphy @ 2020-10-20 13:46 ` Andrew Lunn 0 siblings, 0 replies; 13+ messages in thread From: Andrew Lunn @ 2020-10-20 13:46 UTC (permalink / raw) To: Dan Murphy; +Cc: davem, f.fainelli, hkallweit1, netdev, linux-kernel > HW team says no to interoperability. So we can just add T1L and advertise > that type. > > The DP83TD510L capability is limited to 10Base-T1L only. Thanks for checking. We will avoid issues getting this correct from the start, rather than adding it later, like we did for other T1 PHYs. Andrew ^ permalink raw reply [flat|nested] 13+ messages in thread
end of thread, other threads:[~2020-10-20 13:46 UTC | newest] Thread overview: 13+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2020-10-08 16:23 [PATCH net-next 0/2] DP83TD510 Single Pair 10Mbps Ethernet PHY Dan Murphy 2020-10-08 16:23 ` [PATCH net-next 1/2] dt-bindings: dp83td510: Add binding for DP83TD510 " Dan Murphy 2020-10-08 17:11 ` Florian Fainelli 2020-10-08 18:18 ` Dan Murphy 2020-10-08 16:23 ` [PATCH net-next 2/2] net: phy: dp83td510: Add support for the " Dan Murphy 2020-10-08 16:51 ` Heiner Kallweit 2020-10-08 17:00 ` Dan Murphy 2020-10-16 22:02 ` Andrew Lunn 2020-10-19 21:33 ` Dan Murphy 2020-10-19 21:55 ` Andrew Lunn 2020-10-19 22:04 ` Dan Murphy 2020-10-20 12:00 ` Dan Murphy 2020-10-20 13:46 ` Andrew Lunn
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).