You cannot select more than 25 topics
Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.
38 lines
1.3 KiB
Diff
38 lines
1.3 KiB
Diff
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);
|