Просмотр исходного кода

rtl93xx: fix condition intended to only select internal serdes ports

This condition was introduced in commit 51c8f7661244 ("realtek: Improve
MAC config handling for all SoCs") to correctly report the speed of the
internal serdes ports as 10G, but instead makes all ports read 10G
because the or-operator should have been an and-operator.

Fixes: #9953
Fixes: 51c8f7661244 ("realtek: Improve MAC config handling for all SoCs")
Signed-off-by: Peter Körner <[email protected]>
[ wrap comment to 72 column and improve commit ref ]
Signed-off-by: Christian Marangi <[email protected]>
Peter Körner 2 лет назад
Родитель
Сommit
9fb5082e25
1 измененных файлов с 1 добавлено и 1 удалено
  1. 1 1
      target/linux/realtek/files-5.15/drivers/net/dsa/rtl83xx/dsa.c

+ 1 - 1
target/linux/realtek/files-5.15/drivers/net/dsa/rtl83xx/dsa.c

@@ -559,7 +559,7 @@ static int rtl93xx_phylink_mac_link_state(struct dsa_switch *ds, int port,
 	}
 
 	if (priv->family_id == RTL9310_FAMILY_ID
-		&& (port >= 52 || port <= 55)) { /* Internal serdes */
+		&& (port >= 52 && port <= 55)) { /* Internal serdes */
 			state->speed = SPEED_10000;
 			state->link = 1;
 			state->duplex = 1;