netxen_nic: remove redundant check if retries is zero

At the end of the timeout loop, retries will always be zero so
the check for zero is redundant so remove it.  Also replace
printk with pr_err as recommended by checkpatch.

Signed-off-by: Colin Ian King <colin.king@canonical.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
This commit is contained in:
Colin Ian King 2017-03-15 15:31:58 +00:00 committed by David S. Miller
parent a28453c042
commit 5b7696499c
1 changed files with 2 additions and 7 deletions

View File

@ -1375,13 +1375,8 @@ netxen_receive_peg_ready(struct netxen_adapter *adapter)
} while (--retries);
if (!retries) {
printk(KERN_ERR "Receive Peg initialization not "
"complete, state: 0x%x.\n", val);
return -EIO;
}
return 0;
pr_err("Receive Peg initialization not complete, state: 0x%x.\n", val);
return -EIO;
}
int netxen_init_firmware(struct netxen_adapter *adapter)