Commit e7026f15 authored by Colin Foster's avatar Colin Foster Committed by David S. Miller
Browse files

net: phy: lynx: refactor Lynx PCS module to use generic phylink_pcs 



Remove references to lynx_pcs structures so drivers like the Felix DSA
can reference alternate PCS drivers.

Signed-off-by: default avatarColin Foster <colin.foster@in-advantage.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 60c33202

drivers/net/dsa/ocelot/felix.c

+1 −2
Original line number Diff line number Diff line
@@ -21,7 +21,6 @@ 
#include <linux/of_net.h>

#include <linux/pci.h>

#include <linux/of.h>

#include <linux/pcs-lynx.h>

#include <net/pkt_sched.h>

#include <net/dsa.h>

#include "felix.h"
@@ -832,7 +831,7 @@ static void felix_phylink_mac_config(struct dsa_switch *ds, int port, 
	struct dsa_port *dp = dsa_to_port(ds, port);



	if (felix->pcs && felix->pcs[port])

		phylink_set_pcs(dp->pl, &felix->pcs[port]->pcs);

		phylink_set_pcs(dp->pl, felix->pcs[port]);

}



static void felix_phylink_mac_link_down(struct dsa_switch *ds, int port,

drivers/net/dsa/ocelot/felix.h

+1 −1
Original line number Diff line number Diff line
@@ -62,7 +62,7 @@ struct felix { 
	const struct felix_info		*info;

	struct ocelot			ocelot;

	struct mii_bus			*imdio;

	struct lynx_pcs			**pcs;

	struct phylink_pcs		**pcs;

	resource_size_t			switch_base;

	resource_size_t			imdio_base;

	enum dsa_tag_protocol		tag_proto;

drivers/net/dsa/ocelot/felix_vsc9959.c

+11 −9
Original line number Diff line number Diff line
@@ -1039,7 +1039,7 @@ static int vsc9959_mdio_bus_alloc(struct ocelot *ocelot) 
	int rc;



	felix->pcs = devm_kcalloc(dev, felix->info->num_ports,

				  sizeof(struct lynx_pcs *),

				  sizeof(struct phylink_pcs *),

				  GFP_KERNEL);

	if (!felix->pcs) {

		dev_err(dev, "failed to allocate array for PCS PHYs\n");
@@ -1088,8 +1088,8 @@ static int vsc9959_mdio_bus_alloc(struct ocelot *ocelot) 


	for (port = 0; port < felix->info->num_ports; port++) {

		struct ocelot_port *ocelot_port = ocelot->ports[port];

		struct phylink_pcs *phylink_pcs;

		struct mdio_device *pcs;

		struct lynx_pcs *lynx;



		if (dsa_is_unused_port(felix->ds, port))

			continue;
@@ -1101,13 +1101,13 @@ static int vsc9959_mdio_bus_alloc(struct ocelot *ocelot) 
		if (IS_ERR(pcs))

			continue;



		lynx = lynx_pcs_create(pcs);

		if (!lynx) {

		phylink_pcs = lynx_pcs_create(pcs);

		if (!phylink_pcs) {

			mdio_device_free(pcs);

			continue;

		}



		felix->pcs[port] = lynx;

		felix->pcs[port] = phylink_pcs;



		dev_info(dev, "Found PCS at internal MDIO address %d\n", port);

	}
@@ -1121,13 +1121,15 @@ static void vsc9959_mdio_bus_free(struct ocelot *ocelot) 
	int port;



	for (port = 0; port < ocelot->num_phys_ports; port++) {

		struct lynx_pcs *pcs = felix->pcs[port];

		struct phylink_pcs *phylink_pcs = felix->pcs[port];

		struct mdio_device *mdio_device;



		if (!pcs)

		if (!phylink_pcs)

			continue;



		mdio_device_free(pcs->mdio);

		lynx_pcs_destroy(pcs);

		mdio_device = lynx_get_mdio_device(phylink_pcs);

		mdio_device_free(mdio_device);

		lynx_pcs_destroy(phylink_pcs);

	}

	mdiobus_unregister(felix->imdio);

}

drivers/net/dsa/ocelot/seville_vsc9953.c

+12 −10
Original line number Diff line number Diff line
@@ -1012,7 +1012,7 @@ static int vsc9953_mdio_bus_alloc(struct ocelot *ocelot) 
	int rc;



	felix->pcs = devm_kcalloc(dev, felix->info->num_ports,

				  sizeof(struct phy_device *),

				  sizeof(struct phylink_pcs *),

				  GFP_KERNEL);

	if (!felix->pcs) {

		dev_err(dev, "failed to allocate array for PCS PHYs\n");
@@ -1039,9 +1039,9 @@ static int vsc9953_mdio_bus_alloc(struct ocelot *ocelot) 


	for (port = 0; port < felix->info->num_ports; port++) {

		struct ocelot_port *ocelot_port = ocelot->ports[port];

		int addr = port + 4;

		struct phylink_pcs *phylink_pcs;

		struct mdio_device *pcs;

		struct lynx_pcs *lynx;

		int addr = port + 4;



		if (dsa_is_unused_port(felix->ds, port))

			continue;
@@ -1053,13 +1053,13 @@ static int vsc9953_mdio_bus_alloc(struct ocelot *ocelot) 
		if (IS_ERR(pcs))

			continue;



		lynx = lynx_pcs_create(pcs);

		if (!lynx) {

		phylink_pcs = lynx_pcs_create(pcs);

		if (!phylink_pcs) {

			mdio_device_free(pcs);

			continue;

		}



		felix->pcs[port] = lynx;

		felix->pcs[port] = phylink_pcs;



		dev_info(dev, "Found PCS at internal MDIO address %d\n", addr);

	}
@@ -1073,13 +1073,15 @@ static void vsc9953_mdio_bus_free(struct ocelot *ocelot) 
	int port;



	for (port = 0; port < ocelot->num_phys_ports; port++) {

		struct lynx_pcs *pcs = felix->pcs[port];

		struct phylink_pcs *phylink_pcs = felix->pcs[port];

		struct mdio_device *mdio_device;



		if (!pcs)

		if (!phylink_pcs)

			continue;



		mdio_device_free(pcs->mdio);

		lynx_pcs_destroy(pcs);

		mdio_device = lynx_get_mdio_device(phylink_pcs);

		mdio_device_free(mdio_device);

		lynx_pcs_destroy(phylink_pcs);

	}

	mdiobus_unregister(felix->imdio);

}

drivers/net/ethernet/freescale/dpaa2/dpaa2-mac.c

+8 −5
Original line number Diff line number Diff line
@@ -2,6 +2,7 @@ 
/* Copyright 2019 NXP */



#include <linux/acpi.h>

#include <linux/pcs-lynx.h>

#include <linux/property.h>



#include "dpaa2-eth.h"
@@ -204,11 +205,13 @@ static int dpaa2_pcs_create(struct dpaa2_mac *mac, 


static void dpaa2_pcs_destroy(struct dpaa2_mac *mac)

{

	struct lynx_pcs *pcs = mac->pcs;

	struct phylink_pcs *phylink_pcs = mac->pcs;



	if (pcs) {

		struct device *dev = &pcs->mdio->dev;

		lynx_pcs_destroy(pcs);

	if (phylink_pcs) {

		struct mdio_device *mdio = lynx_get_mdio_device(phylink_pcs);

		struct device *dev = &mdio->dev;



		lynx_pcs_destroy(phylink_pcs);

		put_device(dev);

		mac->pcs = NULL;

	}
@@ -292,7 +295,7 @@ int dpaa2_mac_connect(struct dpaa2_mac *mac) 
	mac->phylink = phylink;



	if (mac->pcs)

		phylink_set_pcs(mac->phylink, &mac->pcs->pcs);

		phylink_set_pcs(mac->phylink, mac->pcs);



	err = phylink_fwnode_phy_connect(mac->phylink, dpmac_node, 0);

	if (err) {