diff options
| author |   Claudiu Manoil <claudiu.manoil@freescale.com> | 2013-03-21 03:12:13 +0000 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2013-03-21 12:01:34 -0400 |
| commit | 39c0a0d5be287415ef0dc0dc15813e6fc86487e9 (patch) | |
| tree | a4ad23dbdbcaa812482a402ff3f3f4c1716d6a51 | |
| parent | net: sh-eth: Use pr_err instead of printk (diff) | |
| download | linux-dev-39c0a0d5be287415ef0dc0dc15813e6fc86487e9.tar.xz linux-dev-39c0a0d5be287415ef0dc0dc15813e6fc86487e9.zip | |
gianfar: Remove 'maybe-uninitialized' compile warning
Warning message:
warning: 'budget_per_q' may be used uninitialized in this function
budget_per_q won't be used uninitialized since the only time
it doesn't get initialized is when entering gfar_poll with
num_act_queues == 0, meaning rstat_rxf == 0, in which case
budget_per_q is not utilized (as it has no meaning).
Inititalize budget_per_q to 0 though to suppress this compile
warning.
Signed-off-by: Claudiu Manoil <claudiu.manoil@freescale.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
| -rw-r--r-- | drivers/net/ethernet/freescale/gianfar.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/ethernet/freescale/gianfar.c b/drivers/net/ethernet/freescale/gianfar.c index 49ce83b1088f..37fbf6756297 100644 --- a/drivers/net/ethernet/freescale/gianfar.c +++ b/drivers/net/ethernet/freescale/gianfar.c @@ -2838,7 +2838,7 @@ static int gfar_poll(struct napi_struct *napi, int budget) struct gfar_priv_tx_q *tx_queue = NULL; struct gfar_priv_rx_q *rx_queue = NULL; int work_done = 0, work_done_per_q = 0; - int i, budget_per_q; + int i, budget_per_q = 0; int has_tx_work; unsigned long rstat_rxf; int num_act_queues; |