net: renesas: rswitch: do not deinit disabled ports

nikita-yoush · kuba-moo · commit 070927427d82 · 2024-12-09T18:27:29.000-08:00
In rswitch_ether_port_init_all(), only enabled ports are initialized. Then, rswitch_ether_port_deinit_all() shall also only deinitialize enabled ports. Signed-off-by: Nikita Yushchenko <nikita.yoush@cogentembedded.com> Reviewed-by: Michal Swiatkowski <michal.swiatkowski@linux.intel.com> Link: https://patch.msgid.link/20241206192140.1714-1-nikita.yoush@cogentembedded.com Signed-off-by: Jakub Kicinski <kuba@kernel.org>
diff --git a/drivers/net/ethernet/renesas/rswitch.c b/drivers/net/ethernet/renesas/rswitch.c
@@ -1527,7 +1527,7 @@ static void rswitch_ether_port_deinit_all(struct rswitch_private *priv)
 {
 	unsigned int i;
 
-	for (i = 0; i < RSWITCH_NUM_PORTS; i++) {
+	rswitch_for_each_enabled_port(priv, i) {
 		phy_exit(priv->rdev[i]->serdes);
 		rswitch_ether_port_deinit_one(priv->rdev[i]);
 	}

Original file line number	Diff line number	Diff line change
`@@ -1527,7 +1527,7 @@ static void rswitch_ether_port_deinit_all(struct rswitch_private *priv)`
`1527`	`1527`	`{`
`1528`	`1528`	`unsigned int i;`
`1529`	`1529`
`1530`		`- for (i = 0; i < RSWITCH_NUM_PORTS; i++) {`
	`1530`	`+ rswitch_for_each_enabled_port(priv, i) {`
`1531`	`1531`	`phy_exit(priv->rdev[i]->serdes);`
`1532`	`1532`	`rswitch_ether_port_deinit_one(priv->rdev[i]);`
`1533`	`1533`	`}`