Message ID | 20170914160125.11007-1-colin.king@canonical.com |
---|---|
State | Accepted, archived |
Delegated to: | David Miller |
Headers | show |
Series | tg3: clean up redundant initialization of tnapi | expand |
From: Colin King <colin.king@canonical.com> Date: Thu, 14 Sep 2017 17:01:25 +0100 > From: Colin Ian King <colin.king@canonical.com> > > tnapi is being initialized and then immediately updated and > hence the initialiation is redundant. Clean up the warning > by moving the declaration and initialization to the inside > of the for-loop. > > Cleans up clang scan-build warning: > warning: Value stored to 'tnapi' during its initialization is never read > > Signed-off-by: Colin Ian King <colin.king@canonical.com> Applied.
diff --git a/drivers/net/ethernet/broadcom/tg3.c b/drivers/net/ethernet/broadcom/tg3.c index af33dc15c55f..54588809b867 100644 --- a/drivers/net/ethernet/broadcom/tg3.c +++ b/drivers/net/ethernet/broadcom/tg3.c @@ -11536,11 +11536,11 @@ static int tg3_start(struct tg3 *tp, bool reset_phy, bool test_irq, tg3_napi_enable(tp); for (i = 0; i < tp->irq_cnt; i++) { - struct tg3_napi *tnapi = &tp->napi[i]; err = tg3_request_irq(tp, i); if (err) { for (i--; i >= 0; i--) { - tnapi = &tp->napi[i]; + struct tg3_napi *tnapi = &tp->napi[i]; + free_irq(tnapi->irq_vec, tnapi); } goto out_napi_fini;