summaryrefslogtreecommitdiffstats
path: root/drivers/net/ethernet/aurora/nb8800.c
diff options
context:
space:
mode:
authorArnd Bergmann <arnd@arndb.de>2016-01-29 12:39:14 +0100
committerDavid S. Miller <davem@davemloft.net>2016-01-29 20:33:39 -0800
commit8bdb290896d2ff22942c9c9f5bf19909329f3664 (patch)
tree6e29b42ed69a5248d89a33a54844c7c25b16cd8b /drivers/net/ethernet/aurora/nb8800.c
parent57e7c8cef224af166b8ec932b5e383641418c005 (diff)
downloadop-kernel-dev-8bdb290896d2ff22942c9c9f5bf19909329f3664.zip
op-kernel-dev-8bdb290896d2ff22942c9c9f5bf19909329f3664.tar.gz
net: nb8800: avoid uninitialized variable warning
The nb8800_poll() function initializes the 'next' variable in the loop looking for new input data. We know this will be called at least once because 'budget' is a guaranteed to be a positive number when we enter the function, but the compiler doesn't know that and warns when the variable is used later: drivers/net/ethernet/aurora/nb8800.c: In function 'nb8800_poll': drivers/net/ethernet/aurora/nb8800.c:350:21: warning: 'next' may be used uninitialized in this function [-Wmaybe-uninitialized] Changing the 'while() {}' loop to 'do {} while()' makes it obvious to the compiler what is going on so it no longer warns. Signed-off-by: Arnd Bergmann <arnd@arndb.de> Acked-by: Mans Rullgard <mans@mansr.com> Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/ethernet/aurora/nb8800.c')
-rw-r--r--drivers/net/ethernet/aurora/nb8800.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/net/ethernet/aurora/nb8800.c b/drivers/net/ethernet/aurora/nb8800.c
index ecc4a33..f71ab26 100644
--- a/drivers/net/ethernet/aurora/nb8800.c
+++ b/drivers/net/ethernet/aurora/nb8800.c
@@ -302,7 +302,7 @@ static int nb8800_poll(struct napi_struct *napi, int budget)
nb8800_tx_done(dev);
again:
- while (work < budget) {
+ do {
struct nb8800_rx_buf *rxb;
unsigned int len;
@@ -330,7 +330,7 @@ again:
rxd->report = 0;
last = next;
work++;
- }
+ } while (work < budget);
if (work) {
priv->rx_descs[last].desc.config |= DESC_EOC;
OpenPOWER on IntegriCloud