From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wm1-f47.google.com (mail-wm1-f47.google.com [209.85.128.47]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 33A631A6802 for ; Mon, 2 Mar 2026 22:07:27 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.47 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1772489248; cv=none; b=cpxdSdM7uB5dj5RgyAqE43tLMx0QivS7TrTNVLUQd8KazYDx4Beqre4ZFSQ0Js944eB/lIcYkOtpIfHGwTFbHqlk+BOgY6aHiUqo8uVtTEWVPwEJOamGppc+yHASKvlzgqs7rzOo4/v6g6KqPZULAF06af16AZzdLrzuIVTfWF8= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1772489248; c=relaxed/simple; bh=d0ZZI0cx+iOSwwZnVAt0GjPNuMqSXf6U58PzAda0ew8=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=BglRZvEQE8Fa5QMv3roryBbNKg8SWa6ZmPwFI1KgBvgeyp3TizXl3XvmMknQImhtVeOI8i0v2Aaw28pTfXmirLDjCV9LgdsTM6O9cn44wouar91XTHbeJQelfzKZAGp4pSBoMkrCDOqkPCw8/qZnxJcgPwOvn2zqh4z+kP3qMvw= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=V5KpUz7J; arc=none smtp.client-ip=209.85.128.47 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="V5KpUz7J" Received: by mail-wm1-f47.google.com with SMTP id 5b1f17b1804b1-483770e0b25so42930205e9.0 for ; Mon, 02 Mar 2026 14:07:26 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1772489245; x=1773094045; darn=vger.kernel.org; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=7H/1BDKSXZaBOMmIE8H6F18jbcqQoi44dLneKh6xhwk=; b=V5KpUz7JqWwmNSh+KLCXlkNdeEBuzFQTJPkEx1lu5OSJ/RkjJSF4SBFzCAkAklbAtg MAgTY8nHraFi8AhDrZJ9s+wzqqhQNPsVe19UpcHNcP/YFu8v9MFo+INqY3nE3I8/bRD2 tmtElVM2M8+KnduiEAsEtOE6jSBV+GqfrOXMzNu44zNwksu8hvBQ3KFiCUAPRweYHQhQ g/oxmeUwDUmocWdkAR5GhJc789AVEiXDApT/7bAqnDgq3RfMV5baKuEGsthclMiGnWne cvUVK5ABH6s1OHJFOAgKLfUaR3Vp/DvoP6GJxgJcwjNn6JiB7db52eHKEWFMSYV3LUN6 WCPw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1772489245; x=1773094045; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :x-gm-gg:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=7H/1BDKSXZaBOMmIE8H6F18jbcqQoi44dLneKh6xhwk=; b=R0yG/OcH51ooDVeRscPiMEFa85f0+1QFkM7lb9uw/3HvdnUmWObrhV7Jz0z4Cmedfw e09f55XalOQNzQZyh6PHX1SZ5d1iHmt9m4kahdbKj7xEY2WjNAVFDTgYEZoZrwZkEG3a I1G4RVrkgHQem3XGEZ2GmKzvZkLzyn2fEORM24t40devtX5KyKHmgtvnjh9lH2UpYdWd 8KS6g484nv7oQ7p14b3ilZf5IrRgdpjkW+nHk3v0v5eBmmHEv1mNjtOYBliSSfAsN5tN 3QAsRw9z/QUZhTVEuwni6ANMxEzt5mu3HX56eQZ+mQnaO5IcZOMeSkCapNRM74fpFFyC eRgA== X-Forwarded-Encrypted: i=1; AJvYcCXoOsk1DvVGZ8HiQMhYngiHXVQeeO7j2CbfFLwpcDapGsRufaxRTz8PLLsD9kwBj9o6gd2QLp7z4SpY15c=@vger.kernel.org X-Gm-Message-State: AOJu0YzRRmXyxJfmH4ao8S9cF6+L901qH3O3CC9f/0CooFNbMV8B/+Rs TNEUHujbr+f7/4BIhRSwvw9WvDPr51yQM2uaTuv1C49540N11K6qTNHZ X-Gm-Gg: ATEYQzxin5yJIEKc8NtQX4U7Y76puDG9kZb0IFjBtdt871Q5Y4AI7vz8WcKPM3e6jPl GvQFfgYBw5lbiHBUxVYHCjVcEgmQa292aSO/fsCElABm0ba77GyqkRyIY0uO0XOMKJ7h4ybRY+i U8+aQqmTRb+KeB+DHaUu1heAFHnDSdOgk0lQMBBqfA/XOiCPFAVsqle3YvXmcPN7q5cdFnqBzE+ RhPLs8wyjcipeNO09jC4Ezj23D9reh2JODjKhsArtVsPS7qNxuzBUwUq38IZmuPfjsbuVgKPPFL AdOMOuwqteyfRgk5TbeHnn3vDCrKhcNm5kjAgq62jmGIYrfiYVRtlqMQAEtktk57T2sI4cECBK7 727PYP7obE5K0QMoWB4vIwwktqFzYAFoRkSBLJ9UOkNXX7Ivkp4V/9bB2UCRCKkp0f+U/jHLyrs sr8eUVcithX5pgMcL3YuXr0P+VwDcyUe3X6Hg7OLD9mZ1OKdmQund6PAgvFehxtM8wAy9jC3nOI dkCmIrkqrqHXBL55ErDpGrTBRkqWDo5qAHs3G6vSvUXMuVt7t9ASO//VxDiwU4kjA== X-Received: by 2002:a05:600c:548a:b0:47e:e20e:bb9c with SMTP id 5b1f17b1804b1-483c9bdb16emr220529315e9.8.1772489245425; Mon, 02 Mar 2026 14:07:25 -0800 (PST) Received: from ?IPV6:2003:ea:8f29:3b00:7d0b:dcf1:6608:9853? (p200300ea8f293b007d0bdcf166089853.dip0.t-ipconnect.de. [2003:ea:8f29:3b00:7d0b:dcf1:6608:9853]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-483c3addb3asm384603245e9.0.2026.03.02.14.07.24 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Mon, 02 Mar 2026 14:07:25 -0800 (PST) Message-ID: Date: Mon, 2 Mar 2026 23:07:23 +0100 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH net-next v1 2/4] r8169: add sfp mode for RTL8116af To: javen , nic_swsd@realtek.com, andrew+netdev@lunn.ch, davem@davemloft.net, edumazet@google.com, kuba@kernel.org, pabeni@redhat.com, horms@kernel.org Cc: netdev@vger.kernel.org, linux-kernel@vger.kernel.org References: <20260302063215.1790-1-javen_xu@realsil.com.cn> <20260302063215.1790-3-javen_xu@realsil.com.cn> Content-Language: en-US From: Heiner Kallweit In-Reply-To: <20260302063215.1790-3-javen_xu@realsil.com.cn> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit On 02.03.2026 07:32, javen wrote: > From: Javen Xu > > RTL8116af is a variation of RTL8168fp. It uses SerDes instead of PHY. > But SerDes status will not relect to PHY. So it needs to add sfp mode > for quirk to help reflect SerDes status during PHY read. > Is there any mass market device using this chip version? As far as possible I'd like to avoid adding support for chip versions that never make it to the mass market. This just makes driver maintenance harder. The patch includes support for 100Mbps fiber mode. Is there any use case for such legacy modes? > Signed-off-by: Javen Xu > --- > drivers/net/ethernet/realtek/r8169_main.c | 71 ++++++++++++++++++++--- > 1 file changed, 62 insertions(+), 9 deletions(-) > > diff --git a/drivers/net/ethernet/realtek/r8169_main.c b/drivers/net/ethernet/realtek/r8169_main.c > index fb2247a20c36..a5c0d3995328 100644 > --- a/drivers/net/ethernet/realtek/r8169_main.c > +++ b/drivers/net/ethernet/realtek/r8169_main.c > @@ -726,6 +726,12 @@ enum rtl_dash_type { > RTL_DASH_25_BP, > }; > > +enum rtl_sfp_mode { > + RTL_SFP_NONE, > + RTL_SFP_8168_AF, > + RTL_SFP_8127_ATF, > +}; > + > struct rtl8169_private { > void __iomem *mmio_addr; /* memory map physical address */ > struct pci_dev *pci_dev; > @@ -734,6 +740,7 @@ struct rtl8169_private { > struct napi_struct napi; > enum mac_version mac_version; > enum rtl_dash_type dash_type; > + enum rtl_sfp_mode sfp_mode; > u32 cur_rx; /* Index into the Rx descriptor buffer of next Rx pkt. */ > u32 cur_tx; /* Index into the Tx descriptor buffer of next Rx pkt. */ > u32 dirty_tx; > @@ -760,7 +767,6 @@ struct rtl8169_private { > unsigned supports_gmii:1; > unsigned aspm_manageable:1; > unsigned dash_enabled:1; > - bool sfp_mode:1; > dma_addr_t counters_phys_addr; > struct rtl8169_counters *counters; > struct rtl8169_tc_offsets tc_offset; > @@ -1126,7 +1132,7 @@ static int r8168_phy_ocp_read(struct rtl8169_private *tp, u32 reg) > return 0; > > /* Return dummy MII_PHYSID2 in SFP mode to match SFP PHY driver */ > - if (tp->sfp_mode && reg == (OCP_STD_PHY_BASE + 2 * MII_PHYSID2)) > + if (tp->sfp_mode == RTL_SFP_8127_ATF && reg == (OCP_STD_PHY_BASE + 2 * MII_PHYSID2)) > return PHY_ID_RTL_DUMMY_SFP & 0xffff; > > RTL_W32(tp, GPHY_OCP, reg << 15); > @@ -1270,6 +1276,34 @@ static int r8168g_mdio_read(struct rtl8169_private *tp, int reg) > return r8168_phy_ocp_read(tp, tp->ocp_base + reg * 2); > } > > +/* The quirk reflects RTL8116af SerDes status. */ > +static int r8116af_mdio_read_quirk(struct rtl8169_private *tp, int reg) > +{ > + u8 phyStatus = RTL_R8(tp, PHYstatus); > + > + if (!(phyStatus & LinkStatus)) > + return 0; > + > + /* BMSR */ > + if (tp->ocp_base == OCP_STD_PHY_BASE && reg == MII_BMSR) > + return BMSR_ANEGCOMPLETE | BMSR_LSTATUS; > + > + /* PHYSR */ > + if (tp->ocp_base == 0xa430 && reg == 0x12) { > + if (phyStatus & _1000bpsF) > + return 0x0028; > + else if (phyStatus & _100bps) > + return 0x0018; This is a complete hack. Any means to access the SerDes directly? > + } > + > + return 0; > +} > + > +static int r8116af_mdio_read(struct rtl8169_private *tp, int reg) > +{ > + return r8168g_mdio_read(tp, reg) | r8116af_mdio_read_quirk(tp, reg); > +} > + > static void mac_mcu_write(struct rtl8169_private *tp, int reg, int value) > { > if (reg == 0x1f) { > @@ -1280,6 +1314,13 @@ static void mac_mcu_write(struct rtl8169_private *tp, int reg, int value) > r8168_mac_ocp_write(tp, tp->ocp_base + reg, value); > } > > +static bool rtl_is_8116af(struct rtl8169_private *tp) > +{ > + return tp->mac_version == RTL_GIGA_MAC_VER_52 && > + (r8168_mac_ocp_read(tp, 0xdc00) & 0x0078) == 0x0030 && > + (r8168_mac_ocp_read(tp, 0xd006) & 0x00ff) == 0x0000; > +} > + > static int mac_mcu_read(struct rtl8169_private *tp, int reg) > { > return r8168_mac_ocp_read(tp, tp->ocp_base + reg); > @@ -1386,7 +1427,10 @@ static int rtl_readphy(struct rtl8169_private *tp, int location) > case RTL_GIGA_MAC_VER_31: > return r8168dp_2_mdio_read(tp, location); > case RTL_GIGA_MAC_VER_40 ... RTL_GIGA_MAC_VER_LAST: > - return r8168g_mdio_read(tp, location); > + if (tp->sfp_mode == RTL_SFP_8168_AF) > + return r8116af_mdio_read(tp, location); > + else > + return r8168g_mdio_read(tp, location); > default: > return r8169_mdio_read(tp, location); > } > @@ -1575,6 +1619,20 @@ static bool rtl_dash_is_enabled(struct rtl8169_private *tp) > } > } > > +static enum rtl_sfp_mode rtl_get_sfp_mode(struct rtl8169_private *tp) > +{ > + if (rtl_is_8125(tp)) { > + u16 data = r8168_mac_ocp_read(tp, 0xd006); > + > + if ((data & 0xff) == 0x07) > + return RTL_SFP_8127_ATF; > + } else if (rtl_is_8116af(tp)) { > + return RTL_SFP_8168_AF; > + } > + > + return RTL_SFP_NONE; > +} > + > static enum rtl_dash_type rtl_get_dash_type(struct rtl8169_private *tp) > { > switch (tp->mac_version) { > @@ -5693,12 +5751,7 @@ static int rtl_init_one(struct pci_dev *pdev, const struct pci_device_id *ent) > } > tp->aspm_manageable = !rc; > > - if (rtl_is_8125(tp)) { > - u16 data = r8168_mac_ocp_read(tp, 0xd006); > - > - if ((data & 0xff) == 0x07) > - tp->sfp_mode = true; > - } > + tp->sfp_mode = rtl_get_sfp_mode(tp); > > tp->dash_type = rtl_get_dash_type(tp); > tp->dash_enabled = rtl_dash_is_enabled(tp);