12345678910111213141516171819202122232425262728293031323334353637 |
- From d2b13233879ca1268a1c027d4573109e5a777811 Mon Sep 17 00:00:00 2001
- From: Florian Fainelli <f.fainelli@gmail.com>
- Date: Thu, 23 Jun 2016 14:23:12 -0700
- Subject: [PATCH 1/3] net: bgmac: Fix SOF bit checking
- We are checking for the Start of Frame bit in the ctl1 word, while this
- bit is set in the ctl0 word instead. Read the ctl0 word and update the
- check to verify that.
- Fixes: 9cde94506eac ("bgmac: implement scatter/gather support")
- Signed-off-by: Florian Fainelli <f.fainelli@gmail.com>
- Signed-off-by: David S. Miller <davem@davemloft.net>
- ---
- drivers/net/ethernet/broadcom/bgmac.c | 5 +++--
- 1 file changed, 3 insertions(+), 2 deletions(-)
- --- a/drivers/net/ethernet/broadcom/bgmac.c
- +++ b/drivers/net/ethernet/broadcom/bgmac.c
- @@ -269,15 +269,16 @@ static void bgmac_dma_tx_free(struct bgm
- while (ring->start != ring->end) {
- int slot_idx = ring->start % BGMAC_TX_RING_SLOTS;
- struct bgmac_slot_info *slot = &ring->slots[slot_idx];
- - u32 ctl1;
- + u32 ctl0, ctl1;
- int len;
-
- if (slot_idx == empty_slot)
- break;
-
- + ctl0 = le32_to_cpu(ring->cpu_base[slot_idx].ctl0);
- ctl1 = le32_to_cpu(ring->cpu_base[slot_idx].ctl1);
- len = ctl1 & BGMAC_DESC_CTL1_LEN;
- - if (ctl1 & BGMAC_DESC_CTL0_SOF)
- + if (ctl0 & BGMAC_DESC_CTL0_SOF)
- /* Unmap no longer used buffer */
- dma_unmap_single(dma_dev, slot->dma_addr, len,
- DMA_TO_DEVICE);
|