realtek: rtl838x: fix lan9 and lan10 on Netgear GS110TUP
Some checks failed
Build Kernel / Build all affected Kernels (push) Waiting to run
Build all core packages / Build all core packages for selected target (push) Waiting to run
Build host tools / Build host tools for linux and macos based systems (push) Has been cancelled

The GS110TUP's lan9 port is connected via a QSGMII PHY to SERDES 2, and
therefore should use the SWITCH_PORT_SDS macro instead of SWITCH_PORT. This
was missed in e956adfe because the GS110TUP is not particularly well
documented and the old code was confusing.

lan10 is an SFP and doesn't have an onboard PHY, so also remove its
associated PHY references and update it to match other devices' SFP ports.

Fixes: https://github.com/openwrt/openwrt/issues/21324
Signed-off-by: Jacob Potter <jacob@j4cbo.com>
Link: https://github.com/openwrt/openwrt/pull/21346
Signed-off-by: Hauke Mehrtens <hauke@hauke-m.de>
This commit is contained in:
Jacob Potter 2025-12-31 14:45:31 -05:00 committed by Hauke Mehrtens
parent 96ad29a0f3
commit b2c0182f26

View file

@ -47,24 +47,19 @@
&mdio_bus0 {
EXTERNAL_PHY(16)
EXTERNAL_PHY(24)
};
&switch0 {
ports {
SWITCH_PORT(16, 9, qsgmii)
SWITCH_PORT_SDS(16, 9, 2, qsgmii)
/* TODO: fixed link SFP is not right */
port24: port@24 {
port@24 {
reg = <24>;
label = SWITCH_PORT_LABEL(10);
label = "lan10";
pcs-handle = <&serdes4>;
phy-handle = <&phy24>;
phy-mode = "1000base-x";
fixed-link {
speed = <1000>;
full-duplex;
};
managed = "in-band-status";
/* i2c and gpios not yet identified */
};
};
};