net: bgmac: remove redundant variable 'freed' (15cdd576) · Commits · 戴 / test

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>

drivers/net/ethernet/broadcom/bgmac.c

+0 −2

Original line number	Diff line number	Diff line
		@@ -236,7 +236,6 @@ static void bgmac_dma_tx_free(struct bgmac bgmac, struct bgmac_dma_ring ring)
		{
		struct device *dma_dev = bgmac->dma_dev;
		int empty_slot;
		bool freed = false;
		unsigned bytes_compl = 0, pkts_compl = 0;

		/* 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)

		slot->dma_addr = 0;
		ring->start++;
		freed = true;
		}

		if (!pkts_compl)

Admin message