aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorColin Ian King <colin.king@canonical.com>2018-07-04 03:30:43 -0400
committerDavid S. Miller <davem@davemloft.net>2018-07-04 09:39:11 -0400
commit15cdd5764a547caf7c9ab14306829135c616845d (patch)
treecadd3fab8097be04fec46c71832e4b8f5bd2f15f
parent5b9f78ecdf590131aa1411b37151acda4f2ffc17 (diff)
net: bgmac: remove redundant variable 'freed'
Variable 'freed' is being assigned but is never used hence it is redundant and can be removed. Cleans up clang warning: warning: variable 'freed' set but not used [-Wunused-but-set-variable] Signed-off-by: Colin Ian King <colin.king@canonical.com> Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r--drivers/net/ethernet/broadcom/bgmac.c2
1 files changed, 0 insertions, 2 deletions
diff --git a/drivers/net/ethernet/broadcom/bgmac.c b/drivers/net/ethernet/broadcom/bgmac.c
index e6ea8e61f96d..4c94d9218bba 100644
--- a/drivers/net/ethernet/broadcom/bgmac.c
+++ b/drivers/net/ethernet/broadcom/bgmac.c
@@ -236,7 +236,6 @@ static void bgmac_dma_tx_free(struct bgmac *bgmac, struct bgmac_dma_ring *ring)
236{ 236{
237 struct device *dma_dev = bgmac->dma_dev; 237 struct device *dma_dev = bgmac->dma_dev;
238 int empty_slot; 238 int empty_slot;
239 bool freed = false;
240 unsigned bytes_compl = 0, pkts_compl = 0; 239 unsigned bytes_compl = 0, pkts_compl = 0;
241 240
242 /* The last slot that hardware didn't consume yet */ 241 /* The last slot that hardware didn't consume yet */
@@ -279,7 +278,6 @@ static void bgmac_dma_tx_free(struct bgmac *bgmac, struct bgmac_dma_ring *ring)
279 278
280 slot->dma_addr = 0; 279 slot->dma_addr = 0;
281 ring->start++; 280 ring->start++;
282 freed = true;
283 } 281 }
284 282
285 if (!pkts_compl) 283 if (!pkts_compl)