From: Zaiyu Wang <zaiyuwang@trustnetic.com>
To: dev@dpdk.org
Cc: Zaiyu Wang <zaiyuwang@trustnetic.com>,
Jiawen Wu <jiawenwu@trustnetic.com>,
Jian Wang <jianwang@trustnetic.com>
Subject: [PATCH v2 05/15] net/txgbe: add support for PHY configuration via SW-FW mailbox
Date: Wed, 25 Jun 2025 20:50:37 +0800 [thread overview]
Message-ID: <20250625125047.18072-6-zaiyuwang@trustnetic.com> (raw)
In-Reply-To: <20250625125047.18072-1-zaiyuwang@trustnetic.com>
Migrate Amber-Lite PHY configuration to firmware due to complexity.
Driver now sends mailbox commands for link state changes, handled
by firmware’s intricate PHY setup process
Signed-off-by: Zaiyu Wang <zaiyuwang@trustnetic.com>
---
drivers/net/txgbe/base/txgbe_aml.c | 52 ++++++++++++++++------------
drivers/net/txgbe/base/txgbe_aml40.c | 37 +++++++++++++++++++-
drivers/net/txgbe/base/txgbe_hw.c | 1 +
drivers/net/txgbe/base/txgbe_mng.c | 36 +++++++++++++++++++
drivers/net/txgbe/base/txgbe_mng.h | 17 +++++++++
drivers/net/txgbe/base/txgbe_type.h | 7 ++++
6 files changed, 126 insertions(+), 24 deletions(-)
diff --git a/drivers/net/txgbe/base/txgbe_aml.c b/drivers/net/txgbe/base/txgbe_aml.c
index 1b5a3783a9..510a539d38 100644
--- a/drivers/net/txgbe/base/txgbe_aml.c
+++ b/drivers/net/txgbe/base/txgbe_aml.c
@@ -131,21 +131,37 @@ u32 txgbe_get_media_type_aml(struct txgbe_hw *hw)
return media_type;
}
+static void txgbe_wait_for_link_up_aml(struct txgbe_hw *hw, u32 speed)
+{
+ u32 link_speed = TXGBE_LINK_SPEED_UNKNOWN;
+ bool link_up = false;
+ int cnt = 0;
+ int i;
+
+ if (speed == TXGBE_LINK_SPEED_25GB_FULL)
+ cnt = 4;
+ else
+ cnt = 1;
+
+ for (i = 0; i < (4 * cnt); i++) {
+ hw->mac.check_link(hw, &link_speed, &link_up, false);
+ if (link_up)
+ break;
+ msleep(250);
+ }
+}
+
s32 txgbe_setup_mac_link_aml(struct txgbe_hw *hw,
u32 speed,
bool autoneg_wait_to_complete)
{
bool autoneg = false;
s32 status = 0;
- s32 ret_status = 0;
u32 link_speed = TXGBE_LINK_SPEED_UNKNOWN;
bool link_up = false;
- int i;
u32 link_capabilities = TXGBE_LINK_SPEED_UNKNOWN;
u32 value = 0;
- UNREFERENCED_PARAMETER(autoneg_wait_to_complete);
-
if (hw->phy.sfp_type == txgbe_sfp_type_not_present) {
DEBUGOUT("SFP not detected, skip setup mac link");
return 0;
@@ -165,30 +181,20 @@ s32 txgbe_setup_mac_link_aml(struct txgbe_hw *hw,
if (value & (TXGBE_SFP1_MOD_ABS_LS | TXGBE_SFP1_RX_LOS_LS))
return status;
- for (i = 0; i < 4; i++) {
- txgbe_e56_check_phy_link(hw, &link_speed, &link_up);
- if (link_up)
- break;
- msleep(250);
- }
+ status = hw->mac.check_link(hw, &link_speed, &link_up,
+ autoneg_wait_to_complete);
- if (link_speed == speed && link_up &&
- !(speed == TXGBE_LINK_SPEED_25GB_FULL))
+ if (link_speed == speed && link_up)
return status;
- rte_spinlock_lock(&hw->phy_lock);
- ret_status = 0;
- rte_spinlock_unlock(&hw->phy_lock);
+ if (speed & TXGBE_LINK_SPEED_25GB_FULL)
+ speed = 0x10;
+ else if (speed & TXGBE_LINK_SPEED_10GB_FULL)
+ speed = 0x08;
- if (ret_status == TXGBE_ERR_PHY_INIT_NOT_DONE)
- return status;
+ status = hw->phy.set_link_hostif(hw, (u8)speed, autoneg, true);
- for (i = 0; i < 4; i++) {
- txgbe_e56_check_phy_link(hw, &link_speed, &link_up);
- if (link_up)
- return status;
- msleep(250);
- }
+ txgbe_wait_for_link_up_aml(hw, speed);
return status;
}
diff --git a/drivers/net/txgbe/base/txgbe_aml40.c b/drivers/net/txgbe/base/txgbe_aml40.c
index 2bad990aa8..88408550d2 100644
--- a/drivers/net/txgbe/base/txgbe_aml40.c
+++ b/drivers/net/txgbe/base/txgbe_aml40.c
@@ -106,7 +106,42 @@ s32 txgbe_setup_mac_link_aml40(struct txgbe_hw *hw,
u32 speed,
bool autoneg_wait_to_complete)
{
- return 0;
+ bool autoneg = false;
+ s32 status = 0;
+ u32 link_speed = TXGBE_LINK_SPEED_UNKNOWN;
+ bool link_up = false;
+ u32 link_capabilities = TXGBE_LINK_SPEED_UNKNOWN;
+ u32 value = 0;
+
+ if (hw->phy.sfp_type == txgbe_sfp_type_not_present) {
+ DEBUGOUT("SFP not detected, skip setup mac link");
+ return 0;
+ }
+
+ /* Check to see if speed passed in is supported. */
+ status = hw->mac.get_link_capabilities(hw,
+ &link_capabilities, &autoneg);
+ if (status)
+ return status;
+
+ speed &= link_capabilities;
+ if (speed == TXGBE_LINK_SPEED_UNKNOWN)
+ return TXGBE_ERR_LINK_SETUP;
+
+ status = hw->mac.check_link(hw, &link_speed, &link_up,
+ autoneg_wait_to_complete);
+
+ if (link_speed == speed && link_up)
+ return status;
+
+ if (speed & TXGBE_LINK_SPEED_40GB_FULL)
+ speed = 0x20;
+
+ status = hw->phy.set_link_hostif(hw, (u8)speed, autoneg, true);
+
+ txgbe_wait_for_link_up_aml(hw, speed);
+
+ return status;
}
void txgbe_init_mac_link_ops_aml40(struct txgbe_hw *hw)
diff --git a/drivers/net/txgbe/base/txgbe_hw.c b/drivers/net/txgbe/base/txgbe_hw.c
index b14ab90466..1e30d235fd 100644
--- a/drivers/net/txgbe/base/txgbe_hw.c
+++ b/drivers/net/txgbe/base/txgbe_hw.c
@@ -2819,6 +2819,7 @@ s32 txgbe_init_ops_generic(struct txgbe_hw *hw)
phy->write_i2c_byte_unlocked = txgbe_write_i2c_byte_unlocked;
phy->check_overtemp = txgbe_check_overtemp;
phy->reset = txgbe_reset_phy;
+ phy->set_link_hostif = txgbe_hic_ephy_set_link;
/* MAC */
mac->init_hw = txgbe_init_hw;
diff --git a/drivers/net/txgbe/base/txgbe_mng.c b/drivers/net/txgbe/base/txgbe_mng.c
index 8327c5fa01..7af43c7b44 100644
--- a/drivers/net/txgbe/base/txgbe_mng.c
+++ b/drivers/net/txgbe/base/txgbe_mng.c
@@ -590,3 +590,39 @@ s32 txgbe_hic_set_lldp(struct txgbe_hw *hw, bool on)
return txgbe_host_interface_command(hw, (u32 *)&buffer, sizeof(buffer),
TXGBE_HI_COMMAND_TIMEOUT, false);
}
+
+s32 txgbe_hic_ephy_set_link(struct txgbe_hw *hw, u8 speed, u8 autoneg, u8 duplex)
+{
+ struct txgbe_hic_ephy_setlink buffer;
+ s32 status;
+ int i;
+
+ buffer.hdr.cmd = FW_PHY_CONFIG_LINK_CMD;
+ buffer.hdr.buf_len = sizeof(struct txgbe_hic_ephy_setlink) - sizeof(struct txgbe_hic_hdr);
+ buffer.hdr.cmd_or_resp.cmd_resv = FW_CEM_CMD_RESERVED;
+
+ buffer.fec_mode = TXGBE_PHY_FEC_AUTO;
+ buffer.speed = speed;
+ buffer.autoneg = autoneg;
+ buffer.duplex = duplex;
+
+ for (i = 0; i <= FW_CEM_MAX_RETRIES; i++) {
+ status = txgbe_host_interface_command(hw, (u32 *)&buffer,
+ sizeof(buffer),
+ TXGBE_HI_COMMAND_TIMEOUT_SHORT, true);
+ if (status != 0) {
+ msleep(1);
+ continue;
+ }
+
+ if (buffer.hdr.cmd_or_resp.ret_status ==
+ FW_CEM_RESP_STATUS_SUCCESS)
+ status = 0;
+ else
+ status = TXGBE_ERR_HOST_INTERFACE_COMMAND;
+
+ break;
+ }
+
+ return status;
+}
diff --git a/drivers/net/txgbe/base/txgbe_mng.h b/drivers/net/txgbe/base/txgbe_mng.h
index 5300970912..72f01b6229 100644
--- a/drivers/net/txgbe/base/txgbe_mng.h
+++ b/drivers/net/txgbe/base/txgbe_mng.h
@@ -13,6 +13,7 @@
#define TXGBE_PMMBX_BSIZE (TXGBE_PMMBX_QSIZE * 4)
#define TXGBE_PMMBX_DATA_SIZE (TXGBE_PMMBX_BSIZE - FW_NVM_DATA_OFFSET * 4)
#define TXGBE_HI_COMMAND_TIMEOUT 5000 /* Process HI command limit */
+#define TXGBE_HI_COMMAND_TIMEOUT_SHORT 500 /* Process HI command limit */
#define TXGBE_HI_FLASH_ERASE_TIMEOUT 5000 /* Process Erase command limit */
#define TXGBE_HI_FLASH_UPDATE_TIMEOUT 5000 /* Process Update command limit */
#define TXGBE_HI_FLASH_VERIFY_TIMEOUT 60000 /* Process Apply command limit */
@@ -56,6 +57,12 @@
#define FW_LLDP_GET_CMD 0xF2
#define FW_LLDP_SET_CMD_OFF 0xF1
#define FW_LLDP_SET_CMD_ON 0xF0
+#define FW_PHY_CONFIG_READ_CMD 0xc0
+#define FW_PHY_CONFIG_LINK_CMD 0xc1
+#define FW_PHY_CONFIG_FC_CMD 0xc2
+#define FW_PHY_CONFIG_POWER_CMD 0xc3
+#define FW_PHY_CONFIG_RESET_CMD 0xc4
+#define FW_READ_SFP_INFO_CMD 0xc5
#define TXGBE_CHECKSUM_CAP_ST_PASS 0x80658383
#define TXGBE_CHECKSUM_CAP_ST_FAIL 0x70657376
@@ -101,6 +108,15 @@ union txgbe_hic_hdr2 {
struct txgbe_hic_hdr2_rsp rsp;
};
+struct txgbe_hic_ephy_setlink {
+ struct txgbe_hic_hdr hdr;
+ u8 speed;
+ u8 duplex;
+ u8 autoneg;
+ u8 fec_mode;
+ u8 resv[4];
+};
+
struct txgbe_hic_drv_info {
struct txgbe_hic_hdr hdr;
u8 port_num;
@@ -204,5 +220,6 @@ bool txgbe_mng_present(struct txgbe_hw *hw);
bool txgbe_mng_enabled(struct txgbe_hw *hw);
s32 txgbe_hic_get_lldp(struct txgbe_hw *hw);
s32 txgbe_hic_set_lldp(struct txgbe_hw *hw, bool on);
+s32 txgbe_hic_ephy_set_link(struct txgbe_hw *hw, u8 speed, u8 autoneg, u8 duplex);
#endif /* _TXGBE_MNG_H_ */
diff --git a/drivers/net/txgbe/base/txgbe_type.h b/drivers/net/txgbe/base/txgbe_type.h
index 8eeea54f98..8005283a26 100644
--- a/drivers/net/txgbe/base/txgbe_type.h
+++ b/drivers/net/txgbe/base/txgbe_type.h
@@ -688,6 +688,7 @@ struct txgbe_phy_info {
u8 *value);
s32 (*write_i2c_byte_unlocked)(struct txgbe_hw *hw, u8 offset, u8 addr,
u8 value);
+ s32 (*set_link_hostif)(struct txgbe_hw *hw, u8 speed, u8 autoneg, u8 duplex);
enum txgbe_phy_type type;
u32 addr;
@@ -771,6 +772,12 @@ enum txgbe_isb_idx {
TXGBE_ISB_MAX
};
+#define TXGBE_PHY_FEC_RS MS(0, 0x1)
+#define TXGBE_PHY_FEC_BASER MS(1, 0x1)
+#define TXGBE_PHY_FEC_OFF MS(2, 0x1)
+#define TXGBE_PHY_FEC_AUTO (TXGBE_PHY_FEC_OFF | TXGBE_PHY_FEC_BASER |\
+ TXGBE_PHY_FEC_RS)
+
struct txgbe_devargs {
u16 auto_neg;
u16 poll;
--
2.21.0.windows.1
next prev parent reply other threads:[~2025-06-25 12:52 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-04-18 9:41 [PATCH 0/2] *** Wangxun new NIC support *** Zaiyu Wang
2025-04-18 9:41 ` [PATCH 1/2] net/txgbe: add support for Wangxun new NIC Amber-Lite 25g/40g Zaiyu Wang
2025-04-18 9:41 ` [PATCH 2/2] net/txgbe: add basic code for Amber-Liter NIC configuration Zaiyu Wang
2025-06-25 12:50 ` [PATCH v2 00/15] Wangxun new NIC support Zaiyu Wang
2025-06-25 12:50 ` [PATCH v2 01/15] net/txgbe: add basic information for Amber-Lite 25G/40G NICs Zaiyu Wang
2025-06-25 12:50 ` [PATCH v2 02/15] net/txgbe: add new SW-FW mailbox interface Zaiyu Wang
2025-06-25 12:50 ` [PATCH v2 03/15] net/txgbe: add identification support for new SFP/QSFP modules Zaiyu Wang
2025-06-25 12:50 ` [PATCH v2 04/15] net/txgbe: add basic link configuration for Amber-Lite NICs Zaiyu Wang
2025-06-25 12:50 ` Zaiyu Wang [this message]
2025-06-25 12:50 ` [PATCH v2 06/15] net/txgbe: add RX&TX support " Zaiyu Wang
2025-06-25 12:50 ` [PATCH v2 07/15] net/txgbe: add hardware reset change " Zaiyu Wang
2025-06-25 12:50 ` [PATCH v2 08/15] net/txgbe: add MAC reconfiguration to avoid packet loss Zaiyu Wang
2025-06-25 12:50 ` [PATCH v2 09/15] net/txgbe: add TX head Write-Back mode for Amber-Lite NICs Zaiyu Wang
2025-06-25 12:50 ` [PATCH v2 10/15] net/txgbe: add RX desc merge " Zaiyu Wang
2025-06-25 12:50 ` [PATCH v2 11/15] net/txgbe: add FEC support for Amber-Lite 25G NICs Zaiyu Wang
2025-06-25 12:50 ` [PATCH v2 12/15] net/txgbe: add GPIO configuration Zaiyu Wang
2025-06-25 12:50 ` [PATCH v2 13/15] net/txgbe: disable unstable features Zaiyu Wang
2025-06-25 12:50 ` [PATCH v2 14/15] net/txgbe: add other hardware-related changes Zaiyu Wang
2025-06-25 12:50 ` [PATCH v2 15/15] doc: update for txgbe Zaiyu Wang
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20250625125047.18072-6-zaiyuwang@trustnetic.com \
--to=zaiyuwang@trustnetic.com \
--cc=dev@dpdk.org \
--cc=jianwang@trustnetic.com \
--cc=jiawenwu@trustnetic.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).