* [PATCH net-next v5] r8169: add support for RTL8125cp
@ 2026-02-24 2:53 javen
2026-02-26 11:33 ` [net-next,v5] " Paolo Abeni
0 siblings, 1 reply; 3+ messages in thread
From: javen @ 2026-02-24 2:53 UTC (permalink / raw)
To: hkallweit1, nic_swsd, andrew+netdev, davem, edumazet, kuba,
pabeni, horms
Cc: netdev, linux-kernel, Javen Xu
From: Javen Xu <javen_xu@realsil.com.cn>
This patch adds support for chip RTL8125cp. Its XID is 0x708. We apply
different and firmware for RTL8125cp.
Signed-off-by: Javen Xu <javen_xu@realsil.com.cn>
---
v2: This patch fix one mistake on phy_modify_paged(phydev, 0xa43, 0x10,
0x0000, 0x1001) which is phy_modify_paged(phydev, 0xa43, 0x00, 0x0000, 0x1001) on patch v1.
v3: Set phy_modify_paged(phydev, 0xa43, 0x10, 0x0000, 0x0003), bit 0
means 'link speed 10m PLL OFF', bit 1 means 'ALDPS PLL OFF', bit 2 means
'ENABLE ALDPS', bit 12 means 'ALDPS XTAL OFF'.
v4: Fix some format mistakes.
v5: Rebase on latest net-next. No changes.
---
drivers/net/ethernet/realtek/r8169.h | 1 +
drivers/net/ethernet/realtek/r8169_main.c | 6 +++++
.../net/ethernet/realtek/r8169_phy_config.c | 22 +++++++++++++++++++
3 files changed, 29 insertions(+)
diff --git a/drivers/net/ethernet/realtek/r8169.h b/drivers/net/ethernet/realtek/r8169.h
index aed4cf852091..0b9c1d4eb48b 100644
--- a/drivers/net/ethernet/realtek/r8169.h
+++ b/drivers/net/ethernet/realtek/r8169.h
@@ -68,6 +68,7 @@ enum mac_version {
RTL_GIGA_MAC_VER_61,
RTL_GIGA_MAC_VER_63,
RTL_GIGA_MAC_VER_64,
+ RTL_GIGA_MAC_VER_65,
RTL_GIGA_MAC_VER_66,
RTL_GIGA_MAC_VER_70,
RTL_GIGA_MAC_VER_80,
diff --git a/drivers/net/ethernet/realtek/r8169_main.c b/drivers/net/ethernet/realtek/r8169_main.c
index 2f7d9809c373..d040f2074db6 100644
--- a/drivers/net/ethernet/realtek/r8169_main.c
+++ b/drivers/net/ethernet/realtek/r8169_main.c
@@ -60,6 +60,7 @@
#define FIRMWARE_8125D_2 "rtl_nic/rtl8125d-2.fw"
#define FIRMWARE_8125K_1 "rtl_nic/rtl8125k-1.fw"
#define FIRMWARE_8125BP_2 "rtl_nic/rtl8125bp-2.fw"
+#define FIRMWARE_8125CP_1 "rtl_nic/rtl8125cp-1.fw"
#define FIRMWARE_9151A_1 "rtl_nic/rtl9151a-1.fw"
#define FIRMWARE_8126A_2 "rtl_nic/rtl8126a-2.fw"
#define FIRMWARE_8126A_3 "rtl_nic/rtl8126a-3.fw"
@@ -112,6 +113,9 @@ static const struct rtl_chip_info {
/* 8125BP family. */
{ 0x7cf, 0x681, RTL_GIGA_MAC_VER_66, "RTL8125BP", FIRMWARE_8125BP_2 },
+ /* 8125CP family*/
+ { 0x7cf, 0x708, RTL_GIGA_MAC_VER_65, "RTL8125CP", FIRMWARE_8125CP_1 },
+
/* 8125D family. */
{ 0x7cf, 0x68b, RTL_GIGA_MAC_VER_64, "RTL9151A", FIRMWARE_9151A_1 },
{ 0x7cf, 0x68a, RTL_GIGA_MAC_VER_64, "RTL8125K", FIRMWARE_8125K_1 },
@@ -802,6 +806,7 @@ MODULE_FIRMWARE(FIRMWARE_8125D_1);
MODULE_FIRMWARE(FIRMWARE_8125D_2);
MODULE_FIRMWARE(FIRMWARE_8125K_1);
MODULE_FIRMWARE(FIRMWARE_8125BP_2);
+MODULE_FIRMWARE(FIRMWARE_8125CP_1);
MODULE_FIRMWARE(FIRMWARE_9151A_1);
MODULE_FIRMWARE(FIRMWARE_8126A_2);
MODULE_FIRMWARE(FIRMWARE_8126A_3);
@@ -4021,6 +4026,7 @@ static void rtl_hw_config(struct rtl8169_private *tp)
[RTL_GIGA_MAC_VER_61] = rtl_hw_start_8125a_2,
[RTL_GIGA_MAC_VER_63] = rtl_hw_start_8125b,
[RTL_GIGA_MAC_VER_64] = rtl_hw_start_8125d,
+ [RTL_GIGA_MAC_VER_65] = rtl_hw_start_8125d,
[RTL_GIGA_MAC_VER_66] = rtl_hw_start_8125d,
[RTL_GIGA_MAC_VER_70] = rtl_hw_start_8126a,
[RTL_GIGA_MAC_VER_80] = rtl_hw_start_8127a,
diff --git a/drivers/net/ethernet/realtek/r8169_phy_config.c b/drivers/net/ethernet/realtek/r8169_phy_config.c
index 032d9d2cfa2a..56ab052135aa 100644
--- a/drivers/net/ethernet/realtek/r8169_phy_config.c
+++ b/drivers/net/ethernet/realtek/r8169_phy_config.c
@@ -1102,6 +1102,27 @@ static void rtl8125d_hw_phy_config(struct rtl8169_private *tp,
rtl8125_config_eee_phy(phydev);
}
+static void rtl8125cp_hw_phy_config(struct rtl8169_private *tp,
+ struct phy_device *phydev)
+{
+ r8169_apply_firmware(tp);
+ rtl8168g_enable_gphy_10m(phydev);
+
+ phy_modify_paged(phydev, 0xad0, 0x17, 0x007f, 0x000b);
+ phy_modify_paged(phydev, 0xad7, 0x14, 0x0000, BIT(4));
+ rtl8125_phy_param(phydev, 0x807f, 0xff00, 0x5300);
+ r8168g_phy_param(phydev, 0x81b8, 0xffff, 0x00b4);
+ r8168g_phy_param(phydev, 0x81ba, 0xffff, 0x00e4);
+ r8168g_phy_param(phydev, 0x81c5, 0xffff, 0x0104);
+ r8168g_phy_param(phydev, 0x81d0, 0xffff, 0x054d);
+ phy_modify_paged(phydev, 0x0a43, 0x10, 0x0000, 0x0003);
+ phy_modify_paged(phydev, 0x0a44, 0x11, 0x0000, BIT(7));
+
+ rtl8125_legacy_force_mode(phydev);
+ rtl8168g_disable_aldps(phydev);
+ rtl8125_config_eee_phy(phydev);
+}
+
static void rtl8125bp_hw_phy_config(struct rtl8169_private *tp,
struct phy_device *phydev)
{
@@ -1344,6 +1365,7 @@ void r8169_hw_phy_config(struct rtl8169_private *tp, struct phy_device *phydev,
[RTL_GIGA_MAC_VER_61] = rtl8125a_2_hw_phy_config,
[RTL_GIGA_MAC_VER_63] = rtl8125b_hw_phy_config,
[RTL_GIGA_MAC_VER_64] = rtl8125d_hw_phy_config,
+ [RTL_GIGA_MAC_VER_65] = rtl8125cp_hw_phy_config,
[RTL_GIGA_MAC_VER_66] = rtl8125bp_hw_phy_config,
[RTL_GIGA_MAC_VER_70] = rtl8126a_hw_phy_config,
[RTL_GIGA_MAC_VER_80] = rtl8127a_1_hw_phy_config,
--
2.43.0
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [net-next,v5] r8169: add support for RTL8125cp
2026-02-24 2:53 [PATCH net-next v5] r8169: add support for RTL8125cp javen
@ 2026-02-26 11:33 ` Paolo Abeni
2026-03-03 9:17 ` 许俊伟
0 siblings, 1 reply; 3+ messages in thread
From: Paolo Abeni @ 2026-02-26 11:33 UTC (permalink / raw)
To: javen_xu
Cc: kuba, davem, andrew+netdev, netdev, horms, edumazet, hkallweit1,
nic_swsd, pabeni, linux-kernel
This is an AI-generated review of your patch. The human sending this
email has considered the AI review valid, or at least plausible.
For local reproduction steps see https://netdev-ai.bots.linux.dev/ai-local.html
---
r8169: add support for RTL8125cp
This patch adds support for chip RTL8125cp with XID 0x708, applying
different firmware (rtl8125cp-1.fw) and a dedicated PHY configuration
function.
> diff --git a/drivers/net/ethernet/realtek/r8169.h b/drivers/net/ethernet/realtek/r8169.h
> index aed4cf852091..0b9c1d4eb48b 100644
> --- a/drivers/net/ethernet/realtek/r8169.h
> +++ b/drivers/net/ethernet/realtek/r8169.h
> @@ -68,6 +68,7 @@ enum mac_version {
> RTL_GIGA_MAC_VER_61,
> RTL_GIGA_MAC_VER_63,
> RTL_GIGA_MAC_VER_64,
> + RTL_GIGA_MAC_VER_65,
> RTL_GIGA_MAC_VER_66,
> RTL_GIGA_MAC_VER_70,
> RTL_GIGA_MAC_VER_80,
[ ... ]
> diff --git a/drivers/net/ethernet/realtek/r8169_main.c b/drivers/net/ethernet/realtek/r8169_main.c
> index 2f7d9809c373..d040f2074db6 100644
> --- a/drivers/net/ethernet/realtek/r8169_main.c
> +++ b/drivers/net/ethernet/realtek/r8169_main.c
[ ... ]
> @@ -4021,6 +4026,7 @@ static void rtl_hw_config(struct rtl8169_private *tp)
> [RTL_GIGA_MAC_VER_61] = rtl_hw_start_8125a_2,
> [RTL_GIGA_MAC_VER_63] = rtl_hw_start_8125b,
> [RTL_GIGA_MAC_VER_64] = rtl_hw_start_8125d,
> + [RTL_GIGA_MAC_VER_65] = rtl_hw_start_8125d,
> [RTL_GIGA_MAC_VER_66] = rtl_hw_start_8125d,
> [RTL_GIGA_MAC_VER_70] = rtl_hw_start_8126a,
> [RTL_GIGA_MAC_VER_80] = rtl_hw_start_8127a,
RTL_GIGA_MAC_VER_65 (RTL8125CP) maps to rtl_hw_start_8125d, which is also
used by VER_64 and VER_66. However, these versions need special handling in
rtl_hw_start_8125():
static void rtl_hw_start_8125(struct rtl8169_private *tp)
{
...
switch (tp->mac_version) {
case RTL_GIGA_MAC_VER_61:
case RTL_GIGA_MAC_VER_64:
case RTL_GIGA_MAC_VER_66:
case RTL_GIGA_MAC_VER_80:
for (i = 0xa00; i < 0xb00; i += 4)
RTL_W32(tp, i, 0);
if (tp->mac_version == RTL_GIGA_MAC_VER_80)
RTL_W16(tp, INT_CFG1_8125, 0x0000);
break;
default:
break;
}
...
}
Should RTL_GIGA_MAC_VER_65 be added to this case group? Both VER_64 and
VER_66, which share the same rtl_hw_start_8125d function as VER_65, are
explicitly listed here to disable interrupt coalescing (clearing registers
0xa00-0xb00). Without this, the RTL8125CP may operate with stale interrupt
coalescing settings, potentially causing incorrect interrupt delivery.
> diff --git a/drivers/net/ethernet/realtek/r8169_phy_config.c b/drivers/net/ethernet/realtek/r8169_phy_config.c
> index 032d9d2cfa2a..f78c3dd36483 100644
> --- a/drivers/net/ethernet/realtek/r8169_phy_config.c
> +++ b/drivers/net/ethernet/realtek/r8169_phy_config.c
> @@ -1102,6 +1102,27 @@ static void rtl8125d_hw_phy_config(struct rtl8169_private *tp,
> rtl8125_config_eee_phy(phydev);
> }
>
> +static void rtl8125cp_hw_phy_config(struct rtl8169_private *tp,
> + struct phy_device *phydev)
> +{
> + r8169_apply_firmware(tp);
> + rtl8168g_enable_gphy_10m(phydev);
> +
> + phy_modify_paged(phydev, 0xad0, 0x17, 0x007f, 0x000b);
> + phy_modify_paged(phydev, 0xad7, 0x14, 0x0000, BIT(4));
> + rtl8125_phy_param(phydev, 0x807f, 0xff00, 0x5300);
> + r8168g_phy_param(phydev, 0x81b8, 0xffff, 0x00b4);
> + r8168g_phy_param(phydev, 0x81ba, 0xffff, 0x00e4);
> + r8168g_phy_param(phydev, 0x81c5, 0xffff, 0x0104);
> + r8168g_phy_param(phydev, 0x81d0, 0xffff, 0x054d);
> + phy_modify_paged(phydev, 0x0a43, 0x10, 0x0000, 0x0003);
^^^^^^
Andrew Lunn suggested in v4 review that magic bit values should be replaced
with named constants. The value 0x0003 sets bits for 'link speed 10m PLL
OFF' (bit 0) and 'ALDPS PLL OFF' (bit 1). This would improve code
readability and maintainability. Andrew suggested defines like
RTL8125CP_LINK_SPEED_10M_PLL_OFF BIT(0), RTL8125CP_ALDPS_PLL_OFF BIT(1),
etc.
Reference: https://lore.kernel.org/netdev/1a36b013-ae9b-4d51-8259-92728644ceab@lunn.ch/
> + phy_modify_paged(phydev, 0x0a44, 0x11, 0x0000, BIT(7));
> +
> + rtl8125_legacy_force_mode(phydev);
> + rtl8168g_disable_aldps(phydev);
> + rtl8125_config_eee_phy(phydev);
> +}
> +
> static void rtl8125bp_hw_phy_config(struct rtl8169_private *tp,
> struct phy_device *phydev)
> {
[ ... ]
^ permalink raw reply [flat|nested] 3+ messages in thread
* RE: [net-next,v5] r8169: add support for RTL8125cp
2026-02-26 11:33 ` [net-next,v5] " Paolo Abeni
@ 2026-03-03 9:17 ` 许俊伟
0 siblings, 0 replies; 3+ messages in thread
From: 许俊伟 @ 2026-03-03 9:17 UTC (permalink / raw)
To: Paolo Abeni
Cc: kuba@kernel.org, davem@davemloft.net, andrew+netdev@lunn.ch,
netdev@vger.kernel.org, horms@kernel.org, edumazet@google.com,
hkallweit1@gmail.com, nic_swsd@realtek.com,
linux-kernel@vger.kernel.org
>External mail : This email originated from outside the organization. Do not
>reply, click links, or open attachments unless you recognize the sender and
>know the content is safe.
>
>
>
>This is an AI-generated review of your patch. The human sending this email
>has considered the AI review valid, or at least plausible.
>
>For local reproduction steps see https://netdev-ai.bots.linux.dev/ai-local.html
>---
>r8169: add support for RTL8125cp
>
>This patch adds support for chip RTL8125cp with XID 0x708, applying different
>firmware (rtl8125cp-1.fw) and a dedicated PHY configuration function.
>
>> diff --git a/drivers/net/ethernet/realtek/r8169.h
>> b/drivers/net/ethernet/realtek/r8169.h
>> index aed4cf852091..0b9c1d4eb48b 100644
>> --- a/drivers/net/ethernet/realtek/r8169.h
>> +++ b/drivers/net/ethernet/realtek/r8169.h
>> @@ -68,6 +68,7 @@ enum mac_version {
>> RTL_GIGA_MAC_VER_61,
>> RTL_GIGA_MAC_VER_63,
>> RTL_GIGA_MAC_VER_64,
>> + RTL_GIGA_MAC_VER_65,
>> RTL_GIGA_MAC_VER_66,
>> RTL_GIGA_MAC_VER_70,
>> RTL_GIGA_MAC_VER_80,
>
>[ ... ]
>
>> diff --git a/drivers/net/ethernet/realtek/r8169_main.c
>> b/drivers/net/ethernet/realtek/r8169_main.c
>> index 2f7d9809c373..d040f2074db6 100644
>> --- a/drivers/net/ethernet/realtek/r8169_main.c
>> +++ b/drivers/net/ethernet/realtek/r8169_main.c
>
>[ ... ]
>
>> @@ -4021,6 +4026,7 @@ static void rtl_hw_config(struct rtl8169_private *tp)
>> [RTL_GIGA_MAC_VER_61] = rtl_hw_start_8125a_2,
>> [RTL_GIGA_MAC_VER_63] = rtl_hw_start_8125b,
>> [RTL_GIGA_MAC_VER_64] = rtl_hw_start_8125d,
>> + [RTL_GIGA_MAC_VER_65] = rtl_hw_start_8125d,
>> [RTL_GIGA_MAC_VER_66] = rtl_hw_start_8125d,
>> [RTL_GIGA_MAC_VER_70] = rtl_hw_start_8126a,
>> [RTL_GIGA_MAC_VER_80] = rtl_hw_start_8127a,
>
>RTL_GIGA_MAC_VER_65 (RTL8125CP) maps to rtl_hw_start_8125d, which is
>also used by VER_64 and VER_66. However, these versions need special
>handling in
>rtl_hw_start_8125():
>
>static void rtl_hw_start_8125(struct rtl8169_private *tp) {
> ...
> switch (tp->mac_version) {
> case RTL_GIGA_MAC_VER_61:
> case RTL_GIGA_MAC_VER_64:
> case RTL_GIGA_MAC_VER_66:
> case RTL_GIGA_MAC_VER_80:
> for (i = 0xa00; i < 0xb00; i += 4)
> RTL_W32(tp, i, 0);
> if (tp->mac_version == RTL_GIGA_MAC_VER_80)
> RTL_W16(tp, INT_CFG1_8125, 0x0000);
> break;
> default:
> break;
> }
> ...
>}
>
>Should RTL_GIGA_MAC_VER_65 be added to this case group? Both VER_64
>and VER_66, which share the same rtl_hw_start_8125d function as VER_65,
>are explicitly listed here to disable interrupt coalescing (clearing registers
>0xa00-0xb00). Without this, the RTL8125CP may operate with stale interrupt
>coalescing settings, potentially causing incorrect interrupt delivery.
>
>> diff --git a/drivers/net/ethernet/realtek/r8169_phy_config.c
>> b/drivers/net/ethernet/realtek/r8169_phy_config.c
>> index 032d9d2cfa2a..f78c3dd36483 100644
>> --- a/drivers/net/ethernet/realtek/r8169_phy_config.c
>> +++ b/drivers/net/ethernet/realtek/r8169_phy_config.c
>> @@ -1102,6 +1102,27 @@ static void rtl8125d_hw_phy_config(struct
>rtl8169_private *tp,
>> rtl8125_config_eee_phy(phydev);
>> }
>>
>> +static void rtl8125cp_hw_phy_config(struct rtl8169_private *tp,
>> + struct phy_device *phydev) {
>> + r8169_apply_firmware(tp);
>> + rtl8168g_enable_gphy_10m(phydev);
>> +
>> + phy_modify_paged(phydev, 0xad0, 0x17, 0x007f, 0x000b);
>> + phy_modify_paged(phydev, 0xad7, 0x14, 0x0000, BIT(4));
>> + rtl8125_phy_param(phydev, 0x807f, 0xff00, 0x5300);
>> + r8168g_phy_param(phydev, 0x81b8, 0xffff, 0x00b4);
>> + r8168g_phy_param(phydev, 0x81ba, 0xffff, 0x00e4);
>> + r8168g_phy_param(phydev, 0x81c5, 0xffff, 0x0104);
>> + r8168g_phy_param(phydev, 0x81d0, 0xffff, 0x054d);
>> + phy_modify_paged(phydev, 0x0a43, 0x10, 0x0000, 0x0003);
> ^^^^^^
>
>Andrew Lunn suggested in v4 review that magic bit values should be replaced
>with named constants. The value 0x0003 sets bits for 'link speed 10m PLL OFF'
>(bit 0) and 'ALDPS PLL OFF' (bit 1). This would improve code readability and
>maintainability. Andrew suggested defines like
>RTL8125CP_LINK_SPEED_10M_PLL_OFF BIT(0), RTL8125CP_ALDPS_PLL_OFF
>BIT(1), etc.
Thanks for your advice. I will consider the two suggestions and resubmit it in v6.
>
>Reference: https://lore.kernel.org/netdev/1a36b013-ae9b-4d51-8259-
>92728644ceab@lunn.ch/
>
>> + phy_modify_paged(phydev, 0x0a44, 0x11, 0x0000, BIT(7));
>> +
>> + rtl8125_legacy_force_mode(phydev);
>> + rtl8168g_disable_aldps(phydev);
>> + rtl8125_config_eee_phy(phydev);
>> +}
>> +
>> static void rtl8125bp_hw_phy_config(struct rtl8169_private *tp,
>> struct phy_device *phydev) {
>
>[ ... ]
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2026-03-03 9:17 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-02-24 2:53 [PATCH net-next v5] r8169: add support for RTL8125cp javen
2026-02-26 11:33 ` [net-next,v5] " Paolo Abeni
2026-03-03 9:17 ` 许俊伟
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox