Commit f0f46c2a authored by Russell King (Oracle)'s avatar Russell King (Oracle) Committed by Jakub Kicinski
Browse files

net: phylink: move MLO_AN_PHY resolve handling to if() statement 



The switch() statement doesn't sit very well with the preceeding if()
statements, and results in excessive indentation that spoils code
readability. Continue cleaning this up by converting the MLO_AN_PHY
case to use an if() statmeent.

Signed-off-by: default avatarRussell King (Oracle) <rmk+kernel@armlinux.org.uk>
Link: https://patch.msgid.link/E1t9RQp-002Fet-5W@rmk-PC.armlinux.org.uk


Signed-off-by: default avatarJakub Kicinski <kuba@kernel.org>
parent 92abfcb4

drivers/net/phy/phylink.c

+3 −5
Original line number Diff line number Diff line
@@ -1466,13 +1466,11 @@ static void phylink_resolve(struct work_struct *w) 
	} else if (pl->cur_link_an_mode == MLO_AN_FIXED) {

		phylink_get_fixed_state(pl, &link_state);

		mac_config = link_state.link;

	} else {

		switch (pl->cur_link_an_mode) {

		case MLO_AN_PHY:

	} else if (pl->cur_link_an_mode == MLO_AN_PHY) {

		link_state = pl->phy_state;

		mac_config = link_state.link;

			break;



	} else {

		switch (pl->cur_link_an_mode) {

		case MLO_AN_INBAND:

			phylink_mac_pcs_get_state(pl, &link_state);