diff options
author | Arthur Jones <arthur.jones@qlogic.com> | 2008-04-17 00:01:11 -0400 |
---|---|---|
committer | Roland Dreier <rolandd@cisco.com> | 2008-04-17 00:01:11 -0400 |
commit | 3dd59e226e01ddb5b041eb0b2e7c7f28b1f730c9 (patch) | |
tree | 934a3b45a346af134bc80ca121208d4f4a33551e /drivers/infiniband/hw/ipath/ipath_intr.c | |
parent | 680b575f6d1ae8aa39c4d7ee7e40b749d277fa9f (diff) |
IB/ipath: Misc sparse warning cleanup
Recent sparse versions and kernel cleanups knock down the false positive
rate of the ipath driver code to a point where having it be sparse clean
is worthwhile. Here we fixup the sparse warnings. Some of these warnings
(and the impetus to run sparse again) are due to work by Roland Dreier.
Signed-off-by: Arthur Jones <arthur.jones@qlogic.com>
Signed-off-by: Roland Dreier <rolandd@cisco.com>
Diffstat (limited to 'drivers/infiniband/hw/ipath/ipath_intr.c')
-rw-r--r-- | drivers/infiniband/hw/ipath/ipath_intr.c | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/drivers/infiniband/hw/ipath/ipath_intr.c b/drivers/infiniband/hw/ipath/ipath_intr.c index 92e58c921522..3b8995206657 100644 --- a/drivers/infiniband/hw/ipath/ipath_intr.c +++ b/drivers/infiniband/hw/ipath/ipath_intr.c | |||
@@ -59,9 +59,11 @@ static void ipath_clrpiobuf(struct ipath_devdata *dd, u32 pnum) | |||
59 | dev_info(&dd->pcidev->dev, | 59 | dev_info(&dd->pcidev->dev, |
60 | "Rewrite PIO buffer %u, to recover from parity error\n", | 60 | "Rewrite PIO buffer %u, to recover from parity error\n", |
61 | pnum); | 61 | pnum); |
62 | *pbuf = dwcnt+1; /* no flush required, since already in freeze */ | 62 | |
63 | while(--dwcnt) | 63 | /* no flush required, since already in freeze */ |
64 | *pbuf++ = 0; | 64 | writel(dwcnt + 1, pbuf); |
65 | while (--dwcnt) | ||
66 | writel(0, pbuf++); | ||
65 | } | 67 | } |
66 | 68 | ||
67 | /* | 69 | /* |