Commit 9c5625f5 authored by Maxime Chevallier's avatar Maxime Chevallier Committed by David S. Miller
Browse files

net: sfp: pass the phy_device when disconnecting an sfp module's PHY



Pass the phy_device as a parameter to the sfp upstream .disconnect_phy
operation. This is preparatory work to help track phy devices across
a net_device's link.

Signed-off-by: default avatarMaxime Chevallier <maxime.chevallier@bootlin.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 02018c54
Loading
Loading
Loading
Loading
+8 −0
Original line number Diff line number Diff line
@@ -266,6 +266,14 @@ static void phy_mdio_device_remove(struct mdio_device *mdiodev)

static struct phy_driver genphy_driver;

static struct phy_link_topology *phy_get_link_topology(struct phy_device *phydev)
{
	if (phydev->attached_dev)
		return &phydev->attached_dev->link_topo;

	return NULL;
}

static LIST_HEAD(phy_fixup_list);
static DEFINE_MUTEX(phy_fixup_lock);

+2 −1
Original line number Diff line number Diff line
@@ -3319,7 +3319,8 @@ static int phylink_sfp_connect_phy(void *upstream, struct phy_device *phy)
	return ret;
}

static void phylink_sfp_disconnect_phy(void *upstream)
static void phylink_sfp_disconnect_phy(void *upstream,
				       struct phy_device *phydev)
{
	phylink_disconnect_phy(upstream);
}
+2 −2
Original line number Diff line number Diff line
@@ -486,7 +486,7 @@ static void sfp_unregister_bus(struct sfp_bus *bus)
			bus->socket_ops->stop(bus->sfp);
		bus->socket_ops->detach(bus->sfp);
		if (bus->phydev && ops && ops->disconnect_phy)
			ops->disconnect_phy(bus->upstream);
			ops->disconnect_phy(bus->upstream, bus->phydev);
	}
	bus->registered = false;
}
@@ -742,7 +742,7 @@ void sfp_remove_phy(struct sfp_bus *bus)
	const struct sfp_upstream_ops *ops = sfp_get_upstream_ops(bus);

	if (ops && ops->disconnect_phy)
		ops->disconnect_phy(bus->upstream);
		ops->disconnect_phy(bus->upstream, bus->phydev);
	bus->phydev = NULL;
}
EXPORT_SYMBOL_GPL(sfp_remove_phy);
+1 −1
Original line number Diff line number Diff line
@@ -544,7 +544,7 @@ struct sfp_upstream_ops {
	void (*link_down)(void *priv);
	void (*link_up)(void *priv);
	int (*connect_phy)(void *priv, struct phy_device *);
	void (*disconnect_phy)(void *priv);
	void (*disconnect_phy)(void *priv, struct phy_device *);
};

#if IS_ENABLED(CONFIG_SFP)