[ar71xx] ag71xx driver: store descriptor pointers in ag71xx_buf

SVN-Revision: 16824

diff --git a/target/linux/ar71xx/files/drivers/net/ag71xx/ag71xx.h b/target/linux/ar71xx/files/drivers/net/ag71xx/ag71xx.h
index 843fc4cc9cba5882bb5d50fcbd8f2f427cb0b08e..ca328e35d7ca9d2b323b57700f82d994b13ae052 100644 (file)
--- a/target/linux/ar71xx/files/drivers/net/ag71xx/ag71xx.h
+++ b/target/linux/ar71xx/files/drivers/net/ag71xx/ag71xx.h
@@ -38,7 +38,7 @@
 #define ETH_FCS_LEN    4
 
 #define AG71XX_DRV_NAME                "ag71xx"
 #define ETH_FCS_LEN    4
 
 #define AG71XX_DRV_NAME                "ag71xx"

-#define AG71XX_DRV_VERSION     "0.5.21"
+#define AG71XX_DRV_VERSION     "0.5.22"

 
 #define AG71XX_NAPI_WEIGHT     64
 #define AG71XX_OOM_REFILL      (1 + HZ/10)
 
 #define AG71XX_NAPI_WEIGHT     64
 #define AG71XX_OOM_REFILL      (1 + HZ/10)


@@ -89,11 +89,12 @@ struct ag71xx_desc {
 
 struct ag71xx_buf {
        struct sk_buff  *skb;
 
 struct ag71xx_buf {
        struct sk_buff  *skb;

+       struct ag71xx_desc *desc;

 };
 
 struct ag71xx_ring {
        struct ag71xx_buf       *buf;
 };
 
 struct ag71xx_ring {
        struct ag71xx_buf       *buf;

-       struct ag71xx_desc      *descs;
+       struct ag71xx_desc      *descs_cpu;

        dma_addr_t              descs_dma;
        unsigned int            curr;
        unsigned int            dirty;
        dma_addr_t              descs_dma;
        unsigned int            curr;
        unsigned int            dirty;

diff --git a/target/linux/ar71xx/files/drivers/net/ag71xx/ag71xx_main.c b/target/linux/ar71xx/files/drivers/net/ag71xx/ag71xx_main.c
index 9b96f3d14b203291a524e6ea5b4ff53b89228274..6302bf648c60b40ae87ea91859952fd19e470420 100644 (file)
--- a/target/linux/ar71xx/files/drivers/net/ag71xx/ag71xx_main.c
+++ b/target/linux/ar71xx/files/drivers/net/ag71xx/ag71xx_main.c
@@ -85,19 +85,21 @@ static void ag71xx_ring_free(struct ag71xx_ring *ring)
 {
        kfree(ring->buf);
 
 {
        kfree(ring->buf);
 

-       if (ring->descs)
-               dma_free_coherent(NULL, ring->size * sizeof(*ring->descs),
-                                 ring->descs, ring->descs_dma);
+       if (ring->descs_cpu)
+               dma_free_coherent(NULL, ring->size * sizeof(struct ag71xx_desc),
+                                 ring->descs_cpu, ring->descs_dma);

 }
 
 static int ag71xx_ring_alloc(struct ag71xx_ring *ring, unsigned int size)
 {
        int err;
 }
 
 static int ag71xx_ring_alloc(struct ag71xx_ring *ring, unsigned int size)
 {
        int err;

+       int i;

 
 

-       ring->descs = dma_alloc_coherent(NULL, size * sizeof(*ring->descs),
-                                        &ring->descs_dma,
-                                        GFP_ATOMIC);
-       if (!ring->descs) {
+       ring->descs_cpu = dma_alloc_coherent(NULL,
+                                            size * sizeof(struct ag71xx_desc),
+                                            &ring->descs_dma,
+                                            GFP_ATOMIC);
+       if (!ring->descs_cpu) {

                err = -ENOMEM;
                goto err;
        }
                err = -ENOMEM;
                goto err;
        }


@@ -110,6 +112,11 @@ static int ag71xx_ring_alloc(struct ag71xx_ring *ring, unsigned int size)
                goto err;
        }
 
                goto err;
        }
 

+       for (i = 0; i < size; i++) {
+               struct ag71xx_desc *descs = (struct ag71xx_desc *) ring->descs_cpu;
+               ring->buf[i].desc = &descs[i];
+       }
+

        return 0;
 
  err:
        return 0;
 
  err:


@@ -124,8 +131,8 @@ static void ag71xx_ring_tx_clean(struct ag71xx *ag)
        while (ring->curr != ring->dirty) {
                u32 i = ring->dirty % AG71XX_TX_RING_SIZE;
 
        while (ring->curr != ring->dirty) {
                u32 i = ring->dirty % AG71XX_TX_RING_SIZE;
 

-               if (!ag71xx_desc_empty(&ring->descs[i])) {
-                       ring->descs[i].ctrl = 0;
+               if (!ag71xx_desc_empty(ring->buf[i].desc)) {
+                       ring->buf[i].desc->ctrl = 0;

                        dev->stats.tx_errors++;
                }
 
                        dev->stats.tx_errors++;
                }
 


@@ -148,10 +155,10 @@ static void ag71xx_ring_tx_init(struct ag71xx *ag)
        int i;
 
        for (i = 0; i < AG71XX_TX_RING_SIZE; i++) {
        int i;
 
        for (i = 0; i < AG71XX_TX_RING_SIZE; i++) {

-               ring->descs[i].next = (u32) (ring->descs_dma +
-                       sizeof(*ring->descs) * ((i + 1) % AG71XX_TX_RING_SIZE));
+               ring->buf[i].desc->next = (u32) (ring->descs_dma +
+                       sizeof(struct ag71xx_desc) * ((i + 1) % AG71XX_TX_RING_SIZE));

 
 

-               ring->descs[i].ctrl = DESC_EMPTY;
+               ring->buf[i].desc->ctrl = DESC_EMPTY;

                ring->buf[i].skb = NULL;
        }
 
                ring->buf[i].skb = NULL;
        }
 


@@ -184,8 +191,8 @@ static int ag71xx_ring_rx_init(struct ag71xx *ag)
 
        ret = 0;
        for (i = 0; i < AG71XX_RX_RING_SIZE; i++)
 
        ret = 0;
        for (i = 0; i < AG71XX_RX_RING_SIZE; i++)

-               ring->descs[i].next = (u32) (ring->descs_dma +
-                       sizeof(*ring->descs) * ((i + 1) % AG71XX_RX_RING_SIZE));
+               ring->buf[i].desc->next = (u32) (ring->descs_dma +
+                       sizeof(struct ag71xx_desc) * ((i + 1) % AG71XX_RX_RING_SIZE));

 
        for (i = 0; i < AG71XX_RX_RING_SIZE; i++) {
                struct sk_buff *skb;
 
        for (i = 0; i < AG71XX_RX_RING_SIZE; i++) {
                struct sk_buff *skb;


@@ -203,8 +210,8 @@ static int ag71xx_ring_rx_init(struct ag71xx *ag)
                skb_reserve(skb, AG71XX_RX_PKT_RESERVE);
 
                ring->buf[i].skb = skb;
                skb_reserve(skb, AG71XX_RX_PKT_RESERVE);
 
                ring->buf[i].skb = skb;

-               ring->descs[i].data = virt_to_phys(skb->data);
-               ring->descs[i].ctrl = DESC_EMPTY;
+               ring->buf[i].desc->data = virt_to_phys(skb->data);
+               ring->buf[i].desc->ctrl = DESC_EMPTY;

        }
 
        /* flush descriptors */
        }
 
        /* flush descriptors */


@@ -241,10 +248,10 @@ static int ag71xx_ring_rx_refill(struct ag71xx *ag)
                        skb->dev = ag->dev;
 
                        ring->buf[i].skb = skb;
                        skb->dev = ag->dev;
 
                        ring->buf[i].skb = skb;

-                       ring->descs[i].data = virt_to_phys(skb->data);
+                       ring->buf[i].desc->data = virt_to_phys(skb->data);

                }
 
                }
 

-               ring->descs[i].ctrl = DESC_EMPTY;
+               ring->buf[i].desc->ctrl = DESC_EMPTY;

                count++;
        }
 
                count++;
        }
 


@@ -462,7 +469,7 @@ static int ag71xx_hard_start_xmit(struct sk_buff *skb, struct net_device *dev)
        int i;
 
        i = ring->curr % AG71XX_TX_RING_SIZE;
        int i;
 
        i = ring->curr % AG71XX_TX_RING_SIZE;

-       desc = &ring->descs[i];
+       desc = ring->buf[i].desc;

 
        if (!ag71xx_desc_empty(desc))
                goto err_drop;
 
        if (!ag71xx_desc_empty(desc))
                goto err_drop;


@@ -586,7 +593,7 @@ static void ag71xx_tx_packets(struct ag71xx *ag)
        sent = 0;
        while (ring->dirty != ring->curr) {
                unsigned int i = ring->dirty % AG71XX_TX_RING_SIZE;
        sent = 0;
        while (ring->dirty != ring->curr) {
                unsigned int i = ring->dirty % AG71XX_TX_RING_SIZE;

-               struct ag71xx_desc *desc = &ring->descs[i];
+               struct ag71xx_desc *desc = ring->buf[i].desc;

                struct sk_buff *skb = ring->buf[i].skb;
 
                if (!ag71xx_desc_empty(desc))
                struct sk_buff *skb = ring->buf[i].skb;
 
                if (!ag71xx_desc_empty(desc))


@@ -622,7 +629,7 @@ static int ag71xx_rx_packets(struct ag71xx *ag, int limit)
 
        while (done < limit) {
                unsigned int i = ring->curr % AG71XX_RX_RING_SIZE;
 
        while (done < limit) {
                unsigned int i = ring->curr % AG71XX_RX_RING_SIZE;

-               struct ag71xx_desc *desc = &ring->descs[i];
+               struct ag71xx_desc *desc = ring->buf[i].desc;

                struct sk_buff *skb;
                int pktlen;
 
                struct sk_buff *skb;
                int pktlen;