net: phylink: Add PHYLINK_DEV operation type
authorIoana Ciornei <ioana.ciornei@nxp.com>
Tue, 28 May 2019 17:38:13 +0000 (20:38 +0300)
committerDavid S. Miller <davem@davemloft.net>
Thu, 30 May 2019 04:48:53 +0000 (21:48 -0700)
In the PHYLINK_DEV operation type, the PHYLINK infrastructure can work
without an attached net_device. For printing usecases, instead, a struct
device * should be passed to PHYLINK using the phylink_config structure.

Also, netif_carrier_* calls ar guarded by the presence of a valid
net_device. When using the PHYLINK_DEV operation type, we cannot check
link status using the netif_carrier_ok() API so instead, keep an
internal state of the MAC and call mac_link_{down,up} only when the link
changed.

Signed-off-by: Ioana Ciornei <ioana.ciornei@nxp.com>
Signed-off-by: Vladimir Oltean <olteanv@gmail.com>
Reviewed-by: Florian Fainelli <f.fainelli@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/phy/phylink.c
include/linux/phylink.h

index 5a283bf9d4026b5da2877b78c15591b7706d3c19..5f6120f3fa3f332bdaa7535e2a5d84159254bae5 100644 (file)
@@ -42,6 +42,8 @@ struct phylink {
        struct net_device *netdev;
        const struct phylink_mac_ops *ops;
        struct phylink_config *config;
+       struct device *dev;
+       unsigned int old_link_state:1;
 
        unsigned long phylink_disable_state; /* bitmask of disables */
        struct phy_device *phydev;
@@ -404,7 +406,8 @@ static void phylink_mac_link_up(struct phylink *pl,
                             pl->phy_state.interface,
                             pl->phydev);
 
-       netif_carrier_on(ndev);
+       if (ndev)
+               netif_carrier_on(ndev);
 
        netdev_info(ndev,
                    "Link is Up - %s/%s - flow control %s\n",
@@ -417,7 +420,8 @@ static void phylink_mac_link_down(struct phylink *pl)
 {
        struct net_device *ndev = pl->netdev;
 
-       netif_carrier_off(ndev);
+       if (ndev)
+               netif_carrier_off(ndev);
        pl->ops->mac_link_down(pl->config, pl->link_an_mode,
                               pl->phy_state.interface);
        netdev_info(ndev, "Link is Down\n");
@@ -428,6 +432,7 @@ static void phylink_resolve(struct work_struct *w)
        struct phylink *pl = container_of(w, struct phylink, resolve);
        struct phylink_link_state link_state;
        struct net_device *ndev = pl->netdev;
+       int link_changed;
 
        mutex_lock(&pl->state_mutex);
        if (pl->phylink_disable_state) {
@@ -470,7 +475,13 @@ static void phylink_resolve(struct work_struct *w)
                }
        }
 
-       if (link_state.link != netif_carrier_ok(ndev)) {
+       if (pl->netdev)
+               link_changed = (link_state.link != netif_carrier_ok(ndev));
+       else
+               link_changed = (link_state.link != pl->old_link_state);
+
+       if (link_changed) {
+               pl->old_link_state = link_state.link;
                if (!link_state.link)
                        phylink_mac_link_down(pl);
                else
@@ -571,6 +582,8 @@ struct phylink *phylink_create(struct phylink_config *config,
        pl->config = config;
        if (config->type == PHYLINK_NETDEV) {
                pl->netdev = to_net_dev(config->dev);
+       } else if (config->type == PHYLINK_DEV) {
+               pl->dev = config->dev;
        } else {
                kfree(pl);
                return ERR_PTR(-EINVAL);
@@ -910,7 +923,8 @@ void phylink_start(struct phylink *pl)
                    phy_modes(pl->link_config.interface));
 
        /* Always set the carrier off */
-       netif_carrier_off(pl->netdev);
+       if (pl->netdev)
+               netif_carrier_off(pl->netdev);
 
        /* Apply the link configuration to the MAC when starting. This allows
         * a fixed-link to start with the correct parameters, and also
@@ -1255,7 +1269,8 @@ int phylink_ethtool_set_pauseparam(struct phylink *pl,
                switch (pl->link_an_mode) {
                case MLO_AN_PHY:
                        /* Silently mark the carrier down, and then trigger a resolve */
-                       netif_carrier_off(pl->netdev);
+                       if (pl->netdev)
+                               netif_carrier_off(pl->netdev);
                        phylink_run_resolve(pl);
                        break;
 
index 67f35f07ac4bd1f1d390eda938d79ea36fa578e1..0f6f65bb9d44e0b1169fe61d64061d5b106bf9e4 100644 (file)
@@ -56,6 +56,7 @@ struct phylink_link_state {
 
 enum phylink_op_type {
        PHYLINK_NETDEV = 0,
+       PHYLINK_DEV,
 };
 
 /**