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

net: stmmac: remove write-only priv->speed 



priv->speed is only ever written to in two locations, but never
read. Therefore, it serves no useful purpose. Remove this unnecessary
struct member.

Signed-off-by: default avatarRussell King (Oracle) <rmk+kernel@armlinux.org.uk>
Reviewed-by: default avatarAndrew Lunn <andrew@lunn.ch>
Link: https://patch.msgid.link/E1tqLJJ-005aQm-Mv@rmk-PC.armlinux.org.uk


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

drivers/net/ethernet/stmicro/stmmac/stmmac.h

+0 −1
Original line number Diff line number Diff line
@@ -282,7 +282,6 @@ struct stmmac_priv { 
	/* Generic channel for NAPI */

	struct stmmac_channel channel[STMMAC_CH_MAX];



	int speed;

	unsigned int pause_time;

	struct mii_bus *mii;

drivers/net/ethernet/stmicro/stmmac/stmmac_main.c

+0 −3
Original line number Diff line number Diff line
@@ -1027,8 +1027,6 @@ static void stmmac_mac_link_up(struct phylink_config *config, 
		}

	}



	priv->speed = speed;



	if (priv->plat->fix_mac_speed)

		priv->plat->fix_mac_speed(priv->plat->bsp_priv, speed, mode);
@@ -7852,7 +7850,6 @@ int stmmac_suspend(struct device *dev) 
	if (stmmac_fpe_supported(priv))

		timer_shutdown_sync(&priv->fpe_cfg.verify_timer);



	priv->speed = SPEED_UNKNOWN;

	return 0;

}

EXPORT_SYMBOL_GPL(stmmac_suspend);