diff options
author | Colin Ian King <colin.king@canonical.com> | 2018-07-04 03:30:43 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2018-07-04 09:39:11 -0400 |
commit | 15cdd5764a547caf7c9ab14306829135c616845d (patch) | |
tree | cadd3fab8097be04fec46c71832e4b8f5bd2f15f | |
parent | 5b9f78ecdf590131aa1411b37151acda4f2ffc17 (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.c | 2 |
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) |