drivers: net: xgene: Fix MSS programming
[cascardo/linux.git] / drivers / net / ethernet / apm / xgene / xgene_enet_main.c
index d1d6b5e..429f18f 100644 (file)
@@ -19,6 +19,7 @@
  * along with this program.  If not, see <http://www.gnu.org/licenses/>.
  */
 
+#include <linux/gpio.h>
 #include "xgene_enet_main.h"
 #include "xgene_enet_hw.h"
 #include "xgene_enet_sgmac.h"
@@ -72,7 +73,6 @@ static int xgene_enet_refill_bufpool(struct xgene_enet_desc_ring *buf_pool,
                skb = netdev_alloc_skb_ip_align(ndev, len);
                if (unlikely(!skb))
                        return -ENOMEM;
-               buf_pool->rx_skb[tail] = skb;
 
                dma_addr = dma_map_single(dev, skb->data, len, DMA_FROM_DEVICE);
                if (dma_mapping_error(dev, dma_addr)) {
@@ -81,6 +81,8 @@ static int xgene_enet_refill_bufpool(struct xgene_enet_desc_ring *buf_pool,
                        return -EINVAL;
                }
 
+               buf_pool->rx_skb[tail] = skb;
+
                raw_desc->m1 = cpu_to_le64(SET_VAL(DATAADDR, dma_addr) |
                                           SET_VAL(BUFDATALEN, bufdatalen) |
                                           SET_BIT(COHERENT));
@@ -102,12 +104,21 @@ static u8 xgene_enet_hdr_len(const void *data)
 
 static void xgene_enet_delete_bufpool(struct xgene_enet_desc_ring *buf_pool)
 {
+       struct device *dev = ndev_to_dev(buf_pool->ndev);
+       struct xgene_enet_raw_desc16 *raw_desc;
+       dma_addr_t dma_addr;
        int i;
 
        /* Free up the buffers held by hardware */
        for (i = 0; i < buf_pool->slots; i++) {
-               if (buf_pool->rx_skb[i])
+               if (buf_pool->rx_skb[i]) {
                        dev_kfree_skb_any(buf_pool->rx_skb[i]);
+
+                       raw_desc = &buf_pool->raw_desc16[i];
+                       dma_addr = GET_VAL(DATAADDR, le64_to_cpu(raw_desc->m1));
+                       dma_unmap_single(dev, dma_addr, XGENE_ENET_MAX_MTU,
+                                        DMA_FROM_DEVICE);
+               }
        }
 }
 
@@ -126,6 +137,7 @@ static irqreturn_t xgene_enet_rx_irq(const int irq, void *data)
 static int xgene_enet_tx_completion(struct xgene_enet_desc_ring *cp_ring,
                                    struct xgene_enet_raw_desc *raw_desc)
 {
+       struct xgene_enet_pdata *pdata = netdev_priv(cp_ring->ndev);
        struct sk_buff *skb;
        struct device *dev;
        skb_frag_t *frag;
@@ -133,6 +145,7 @@ static int xgene_enet_tx_completion(struct xgene_enet_desc_ring *cp_ring,
        u16 skb_index;
        u8 status;
        int i, ret = 0;
+       u8 mss_index;
 
        skb_index = GET_VAL(USERINFO, le64_to_cpu(raw_desc->m0));
        skb = cp_ring->cp_skb[skb_index];
@@ -149,6 +162,13 @@ static int xgene_enet_tx_completion(struct xgene_enet_desc_ring *cp_ring,
                               DMA_TO_DEVICE);
        }
 
+       if (GET_BIT(ET, le64_to_cpu(raw_desc->m3))) {
+               mss_index = GET_VAL(MSS, le64_to_cpu(raw_desc->m3));
+               spin_lock(&pdata->mss_lock);
+               pdata->mss_refcnt[mss_index]--;
+               spin_unlock(&pdata->mss_lock);
+       }
+
        /* Checking for error */
        status = GET_VAL(LERR, le64_to_cpu(raw_desc->m0));
        if (unlikely(status > 2)) {
@@ -167,15 +187,53 @@ static int xgene_enet_tx_completion(struct xgene_enet_desc_ring *cp_ring,
        return ret;
 }
 
-static u64 xgene_enet_work_msg(struct sk_buff *skb)
+static int xgene_enet_setup_mss(struct net_device *ndev, u32 mss)
+{
+       struct xgene_enet_pdata *pdata = netdev_priv(ndev);
+       bool mss_index_found = false;
+       int mss_index;
+       int i;
+
+       spin_lock(&pdata->mss_lock);
+
+       /* Reuse the slot if MSS matches */
+       for (i = 0; !mss_index_found && i < NUM_MSS_REG; i++) {
+               if (pdata->mss[i] == mss) {
+                       pdata->mss_refcnt[i]++;
+                       mss_index = i;
+                       mss_index_found = true;
+               }
+       }
+
+       /* Overwrite the slot with ref_count = 0 */
+       for (i = 0; !mss_index_found && i < NUM_MSS_REG; i++) {
+               if (!pdata->mss_refcnt[i]) {
+                       pdata->mss_refcnt[i]++;
+                       pdata->mac_ops->set_mss(pdata, mss, i);
+                       pdata->mss[i] = mss;
+                       mss_index = i;
+                       mss_index_found = true;
+               }
+       }
+
+       spin_unlock(&pdata->mss_lock);
+
+       /* No slots with ref_count = 0 available, return busy */
+       if (!mss_index_found)
+               return -EBUSY;
+
+       return mss_index;
+}
+
+static int xgene_enet_work_msg(struct sk_buff *skb, u64 *hopinfo)
 {
        struct net_device *ndev = skb->dev;
        struct iphdr *iph;
        u8 l3hlen = 0, l4hlen = 0;
        u8 ethhdr, proto = 0, csum_enable = 0;
-       u64 hopinfo = 0;
        u32 hdr_len, mss = 0;
        u32 i, len, nr_frags;
+       int mss_index;
 
        ethhdr = xgene_enet_hdr_len(skb->data);
 
@@ -215,7 +273,11 @@ static u64 xgene_enet_work_msg(struct sk_buff *skb)
                        if (!mss || ((skb->len - hdr_len) <= mss))
                                goto out;
 
-                       hopinfo |= SET_BIT(ET);
+                       mss_index = xgene_enet_setup_mss(ndev, mss);
+                       if (unlikely(mss_index < 0))
+                               return -EBUSY;
+
+                       *hopinfo |= SET_BIT(ET) | SET_VAL(MSS, mss_index);
                }
        } else if (iph->protocol == IPPROTO_UDP) {
                l4hlen = UDP_HDR_SIZE;
@@ -223,15 +285,15 @@ static u64 xgene_enet_work_msg(struct sk_buff *skb)
        }
 out:
        l3hlen = ip_hdrlen(skb) >> 2;
-       hopinfo |= SET_VAL(TCPHDR, l4hlen) |
-                 SET_VAL(IPHDR, l3hlen) |
-                 SET_VAL(ETHHDR, ethhdr) |
-                 SET_VAL(EC, csum_enable) |
-                 SET_VAL(IS, proto) |
-                 SET_BIT(IC) |
-                 SET_BIT(TYPE_ETH_WORK_MESSAGE);
-
-       return hopinfo;
+       *hopinfo |= SET_VAL(TCPHDR, l4hlen) |
+                   SET_VAL(IPHDR, l3hlen) |
+                   SET_VAL(ETHHDR, ethhdr) |
+                   SET_VAL(EC, csum_enable) |
+                   SET_VAL(IS, proto) |
+                   SET_BIT(IC) |
+                   SET_BIT(TYPE_ETH_WORK_MESSAGE);
+
+       return 0;
 }
 
 static u16 xgene_enet_encode_len(u16 len)
@@ -271,20 +333,22 @@ static int xgene_enet_setup_tx_desc(struct xgene_enet_desc_ring *tx_ring,
        dma_addr_t dma_addr, pbuf_addr, *frag_dma_addr;
        skb_frag_t *frag;
        u16 tail = tx_ring->tail;
-       u64 hopinfo;
+       u64 hopinfo = 0;
        u32 len, hw_len;
        u8 ll = 0, nv = 0, idx = 0;
        bool split = false;
        u32 size, offset, ell_bytes = 0;
        u32 i, fidx, nr_frags, count = 1;
+       int ret;
 
        raw_desc = &tx_ring->raw_desc[tail];
        tail = (tail + 1) & (tx_ring->slots - 1);
        memset(raw_desc, 0, sizeof(struct xgene_enet_raw_desc));
 
-       hopinfo = xgene_enet_work_msg(skb);
-       if (!hopinfo)
-               return -EINVAL;
+       ret = xgene_enet_work_msg(skb, &hopinfo);
+       if (ret)
+               return ret;
+
        raw_desc->m3 = cpu_to_le64(SET_VAL(HENQNUM, tx_ring->dst_ring_num) |
                                   hopinfo);
 
@@ -424,6 +488,9 @@ static netdev_tx_t xgene_enet_start_xmit(struct sk_buff *skb,
                return NETDEV_TX_OK;
 
        count = xgene_enet_setup_tx_desc(tx_ring, skb);
+       if (count == -EBUSY)
+               return NETDEV_TX_BUSY;
+
        if (count <= 0) {
                dev_kfree_skb_any(skb);
                return NETDEV_TX_OK;
@@ -452,7 +519,6 @@ static int xgene_enet_rx_frame(struct xgene_enet_desc_ring *rx_ring,
                               struct xgene_enet_raw_desc *raw_desc)
 {
        struct net_device *ndev;
-       struct xgene_enet_pdata *pdata;
        struct device *dev;
        struct xgene_enet_desc_ring *buf_pool;
        u32 datalen, skb_index;
@@ -461,7 +527,6 @@ static int xgene_enet_rx_frame(struct xgene_enet_desc_ring *rx_ring,
        int ret = 0;
 
        ndev = rx_ring->ndev;
-       pdata = netdev_priv(ndev);
        dev = ndev_to_dev(rx_ring->ndev);
        buf_pool = rx_ring->buf_pool;
 
@@ -739,8 +804,8 @@ static int xgene_enet_open(struct net_device *ndev)
        if (ret)
                return ret;
 
-       if (pdata->phy_dev) {
-               phy_start(pdata->phy_dev);
+       if (ndev->phydev) {
+               phy_start(ndev->phydev);
        } else {
                schedule_delayed_work(&pdata->link_work, PHY_POLL_LINK_OFF);
                netif_carrier_off(ndev);
@@ -763,8 +828,8 @@ static int xgene_enet_close(struct net_device *ndev)
        mac_ops->tx_disable(pdata);
        mac_ops->rx_disable(pdata);
 
-       if (pdata->phy_dev)
-               phy_stop(pdata->phy_dev);
+       if (ndev->phydev)
+               phy_stop(ndev->phydev);
        else
                cancel_delayed_work_sync(&pdata->link_work);
 
@@ -1312,6 +1377,18 @@ static int xgene_enet_check_phy_handle(struct xgene_enet_pdata *pdata)
        return 0;
 }
 
+static void xgene_enet_gpiod_get(struct xgene_enet_pdata *pdata)
+{
+       struct device *dev = &pdata->pdev->dev;
+
+       if (pdata->phy_mode != PHY_INTERFACE_MODE_XGMII)
+               return;
+
+       pdata->sfp_rdy = gpiod_get(dev, "rxlos", GPIOD_IN);
+       if (IS_ERR(pdata->sfp_rdy))
+               pdata->sfp_rdy = gpiod_get(dev, "sfp", GPIOD_IN);
+}
+
 static int xgene_enet_get_resources(struct xgene_enet_pdata *pdata)
 {
        struct platform_device *pdev;
@@ -1401,6 +1478,8 @@ static int xgene_enet_get_resources(struct xgene_enet_pdata *pdata)
        if (ret)
                return ret;
 
+       xgene_enet_gpiod_get(pdata);
+
        pdata->clk = devm_clk_get(&pdev->dev, NULL);
        if (IS_ERR(pdata->clk)) {
                /* Firmware may have set up the clock already. */
@@ -1425,6 +1504,7 @@ static int xgene_enet_get_resources(struct xgene_enet_pdata *pdata)
        } else {
                pdata->mcx_mac_addr = base_addr + BLOCK_AXG_MAC_OFFSET;
                pdata->mcx_mac_csr_addr = base_addr + BLOCK_AXG_MAC_CSR_OFFSET;
+               pdata->pcs_addr = base_addr + BLOCK_PCS_OFFSET;
        }
        pdata->rx_buff_cnt = NUM_PKT_BUF;
 
@@ -1454,10 +1534,8 @@ static int xgene_enet_init_hw(struct xgene_enet_pdata *pdata)
                buf_pool = pdata->rx_ring[i]->buf_pool;
                xgene_enet_init_bufpool(buf_pool);
                ret = xgene_enet_refill_bufpool(buf_pool, pdata->rx_buff_cnt);
-               if (ret) {
-                       xgene_enet_delete_desc_rings(pdata);
-                       return ret;
-               }
+               if (ret)
+                       goto err;
        }
 
        dst_ring_num = xgene_enet_dst_ring_num(pdata->rx_ring[0]);
@@ -1474,7 +1552,7 @@ static int xgene_enet_init_hw(struct xgene_enet_pdata *pdata)
                ret = pdata->cle_ops->cle_init(pdata);
                if (ret) {
                        netdev_err(ndev, "Preclass Tree init error\n");
-                       return ret;
+                       goto err;
                }
        } else {
                pdata->port_ops->cle_bypass(pdata, dst_ring_num, buf_pool->id);
@@ -1484,6 +1562,10 @@ static int xgene_enet_init_hw(struct xgene_enet_pdata *pdata)
        pdata->mac_ops->init(pdata);
 
        return ret;
+
+err:
+       xgene_enet_delete_desc_rings(pdata);
+       return ret;
 }
 
 static void xgene_enet_setup_ops(struct xgene_enet_pdata *pdata)
@@ -1631,8 +1713,8 @@ static int xgene_enet_probe(struct platform_device *pdev)
        }
 #endif
        if (!pdata->enet_id) {
-               free_netdev(ndev);
-               return -ENODEV;
+               ret = -ENODEV;
+               goto err;
        }
 
        ret = xgene_enet_get_resources(pdata);
@@ -1643,7 +1725,7 @@ static int xgene_enet_probe(struct platform_device *pdev)
 
        if (pdata->phy_mode == PHY_INTERFACE_MODE_XGMII) {
                ndev->features |= NETIF_F_TSO;
-               pdata->mss = XGENE_ENET_MSS;
+               spin_lock_init(&pdata->mss_lock);
        }
        ndev->hw_features = ndev->features;
 
@@ -1655,7 +1737,7 @@ static int xgene_enet_probe(struct platform_device *pdev)
 
        ret = xgene_enet_init_hw(pdata);
        if (ret)
-               goto err_netdev;
+               goto err;
 
        link_state = pdata->mac_ops->link_state;
        if (pdata->phy_mode == PHY_INTERFACE_MODE_XGMII) {
@@ -1665,21 +1747,32 @@ static int xgene_enet_probe(struct platform_device *pdev)
                        ret = xgene_enet_mdio_config(pdata);
                else
                        INIT_DELAYED_WORK(&pdata->link_work, link_state);
+
+               if (ret)
+                       goto err1;
        }
-       if (ret)
-               goto err;
 
        xgene_enet_napi_add(pdata);
        ret = register_netdev(ndev);
        if (ret) {
                netdev_err(ndev, "Failed to register netdev\n");
-               goto err;
+               goto err2;
        }
 
        return 0;
 
-err_netdev:
-       unregister_netdev(ndev);
+err2:
+       /*
+        * If necessary, free_netdev() will call netif_napi_del() and undo
+        * the effects of xgene_enet_napi_add()'s calls to netif_napi_add().
+        */
+
+       if (pdata->mdio_driver)
+               xgene_enet_phy_disconnect(pdata);
+       else if (pdata->phy_mode == PHY_INTERFACE_MODE_RGMII)
+               xgene_enet_mdio_remove(pdata);
+err1:
+       xgene_enet_delete_desc_rings(pdata);
 err:
        free_netdev(ndev);
        return ret;
@@ -1688,11 +1781,9 @@ err:
 static int xgene_enet_remove(struct platform_device *pdev)
 {
        struct xgene_enet_pdata *pdata;
-       const struct xgene_mac_ops *mac_ops;
        struct net_device *ndev;
 
        pdata = platform_get_drvdata(pdev);
-       mac_ops = pdata->mac_ops;
        ndev = pdata->ndev;
 
        rtnl_lock();