Commit afd29f36 authored by Vladimir Oltean's avatar Vladimir Oltean Committed by Jakub Kicinski
Browse files

net: pcs: lynx: no need to read LPA in lynx_pcs_get_state_2500basex() 



Nothing useful is done with the LPA variable in lynx_pcs_get_state_2500basex(),
we can just remove the read.

Signed-off-by: default avatarVladimir Oltean <vladimir.oltean@nxp.com>
Reviewed-by: default avatarAndrew Lunn <andrew@lunn.ch>
Link: https://lore.kernel.org/r/20240513115345.2452799-1-vladimir.oltean@nxp.com


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

drivers/net/pcs/pcs-lynx.c

+2 −3
Original line number Diff line number Diff line
@@ -61,11 +61,10 @@ static void lynx_pcs_get_state_usxgmii(struct mdio_device *pcs, 
static void lynx_pcs_get_state_2500basex(struct mdio_device *pcs,

					 struct phylink_link_state *state)

{

	int bmsr, lpa;

	int bmsr;



	bmsr = mdiodev_read(pcs, MII_BMSR);

	lpa = mdiodev_read(pcs, MII_LPA);

	if (bmsr < 0 || lpa < 0) {

	if (bmsr < 0) {

		state->link = false;

		return;

	}