realtek: pcs: rtl930x: read correct symbol errors for SGMII/1000Base-X

Both modes were incorrectly assigned to the same branch that 10GBase-R
and USXGMII use for reading the symbol error count. This behavior is not
supported by anything in the SDK. According to that, they fall into the
generic else case. Fix that by dropping those two modes from the branch
condition.

Signed-off-by: Jonas Jelonek <jonas@jonasjelonek.de>
This commit is contained in:
Jonas Jelonek committed 2026-09-29 18:35:50 +00:00
1 parent 1cf469ac9d
commit 83a1dc9101
1 file changed
+2 -4
@@ -2886,10 +2886,8 @@ static u32 rtpcs_930x_sds_sym_err_get(struct rtpcs_serdes *sds,
} else if (hw_mode == RTPCS_SDS_MODE_USXGMII &&
sds->usxgmii_submode == RTPCS_SDS_USXGMII_SM_10GQXGMII) {
/* no known symbol error count for USXGMII QXGMII */
} else if (hw_mode == RTPCS_SDS_MODE_1000BASEX || hw_mode == RTPCS_SDS_MODE_SGMII ||
hw_mode == RTPCS_SDS_MODE_10GBASER ||
(hw_mode == RTPCS_SDS_MODE_USXGMII &&
sds->usxgmii_submode == RTPCS_SDS_USXGMII_SM_10GSXGMII)) {
} else if (hw_mode == RTPCS_SDS_MODE_10GBASER || (hw_mode == RTPCS_SDS_MODE_USXGMII &&
sds->usxgmii_submode == RTPCS_SDS_USXGMII_SM_10GSXGMII)) {
v = rtpcs_sds_read(sds, PAGE_TGR_STD_1, 0x1);
v &= 0xff;
} else {