aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorColin Ian King <colin.king@canonical.com>2018-01-31 11:14:25 -0500
committerDavid S. Miller <davem@davemloft.net>2018-02-01 09:43:00 -0500
commit2e85283dabc22f4715b136e8a7426bd9bef4ce69 (patch)
treedd10dc4044ae58e734d4c13a46fe342d192d684d
parent26c26ab02c49e64dbac3a9f56806005bc9e8b914 (diff)
be2net: remove redundant initialization of 'head' and pointer txq
Variable head is initialized to a value that is never read and is being updated to a new value a few lines later, hence this initialization is redundant and can be safely removed as well as the now unused pointer txq. Cleans up clang warning: drivers/net/ethernet/emulex/benet/be_main.c:996:6: warning: Value stored to 'head' during its initialization is never read 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/emulex/benet/be_main.c3
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/net/ethernet/emulex/benet/be_main.c b/drivers/net/ethernet/emulex/benet/be_main.c
index c36c81959198..d81e2d37bc3d 100644
--- a/drivers/net/ethernet/emulex/benet/be_main.c
+++ b/drivers/net/ethernet/emulex/benet/be_main.c
@@ -991,9 +991,8 @@ static u32 be_xmit_enqueue(struct be_adapter *adapter, struct be_tx_obj *txo,
991{ 991{
992 u32 i, copied = 0, wrb_cnt = skb_wrb_cnt(skb); 992 u32 i, copied = 0, wrb_cnt = skb_wrb_cnt(skb);
993 struct device *dev = &adapter->pdev->dev; 993 struct device *dev = &adapter->pdev->dev;
994 struct be_queue_info *txq = &txo->q;
995 bool map_single = false; 994 bool map_single = false;
996 u32 head = txq->head; 995 u32 head;
997 dma_addr_t busaddr; 996 dma_addr_t busaddr;
998 int len; 997 int len;
999 998