diff options
author | Roel Kluin <roel.kluin@gmail.com> | 2008-12-30 02:42:33 (GMT) |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2008-12-30 02:42:33 (GMT) |
commit | 684f4a4c4a69f7226d8c7559c0cdfc7bd388335a (patch) | |
tree | 7388e9ff9e1bdc60526731b242dc10f8f1a294bb /net/ethernet | |
parent | 18cc42a3a17d19774b332e933cf34c71b0d3903c (diff) | |
download | linux-fsl-qoriq-684f4a4c4a69f7226d8c7559c0cdfc7bd388335a.tar.xz |
EtherExpress16: fix printing timed out status
in drivers/net/eexpress.c:558, function unstick_cu()
while (!SCB_complete(rsst=scb_status(dev))) {
...
if (...)
printk(KERN_WARNING "%s: Reset timed out status %04x, retrying...\n",
dev->name,rsst);
}
but this will become
while (!((rsst = scb_status(dev) & 0x8000) != 0) ...
because of the macro:
#define SCB_complete(s) ((s&0x8000)!=0)
so rsst can only become either 0x8000 or 0, but in the latter case the
loop ends, I think the wrong timed out status is printed. This also
cleans up similar macros.
Signed-off-by: Roel Kluin <roel.kluin@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/ethernet')
0 files changed, 0 insertions, 0 deletions