diff options
author | Jeff Garzik <jeff@garzik.org> | 2007-07-17 02:03:49 -0400 |
---|---|---|
committer | Jeff Garzik <jeff@garzik.org> | 2007-07-17 16:18:00 -0400 |
commit | 9db48926208562df3c778682e064990170ab8971 (patch) | |
tree | ee9ab885460433b9cdac27b87e863bd0c0d174f2 /drivers/infiniband | |
parent | e5fb4f42268654ca41ab50b1406fb7da97559db5 (diff) |
drivers/infiniband/hw/mthca/mthca_qp: kill uninit'd var warning
drivers/infiniband/hw/mthca/mthca_qp.c: In function
‘mthca_tavor_post_send’:
drivers/infiniband/hw/mthca/mthca_qp.c:1594: warning: ‘f0’ may be used
uninitialized in this function
drivers/infiniband/hw/mthca/mthca_qp.c: In function
‘mthca_arbel_post_send’:
drivers/infiniband/hw/mthca/mthca_qp.c:1949: warning: ‘f0’ may be used
uninitialized in this function
Initializing 'f0' is not strictly necessary in either case, AFAICS.
I was considering use of uninitialized_var(), but looking at the
complex flow of control in each function, I feel it is wiser and
safer to simply zero the var and be certain of ourselves.
Signed-off-by: Jeff Garzik <jeff@garzik.org>
Diffstat (limited to 'drivers/infiniband')
-rw-r--r-- | drivers/infiniband/hw/mthca/mthca_qp.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/infiniband/hw/mthca/mthca_qp.c b/drivers/infiniband/hw/mthca/mthca_qp.c index eef415b12b2e..11f1d99db40b 100644 --- a/drivers/infiniband/hw/mthca/mthca_qp.c +++ b/drivers/infiniband/hw/mthca/mthca_qp.c | |||
@@ -1591,7 +1591,7 @@ int mthca_tavor_post_send(struct ib_qp *ibqp, struct ib_send_wr *wr, | |||
1591 | int i; | 1591 | int i; |
1592 | int size; | 1592 | int size; |
1593 | int size0 = 0; | 1593 | int size0 = 0; |
1594 | u32 f0; | 1594 | u32 f0 = 0; |
1595 | int ind; | 1595 | int ind; |
1596 | u8 op0 = 0; | 1596 | u8 op0 = 0; |
1597 | 1597 | ||
@@ -1946,7 +1946,7 @@ int mthca_arbel_post_send(struct ib_qp *ibqp, struct ib_send_wr *wr, | |||
1946 | int i; | 1946 | int i; |
1947 | int size; | 1947 | int size; |
1948 | int size0 = 0; | 1948 | int size0 = 0; |
1949 | u32 f0; | 1949 | u32 f0 = 0; |
1950 | int ind; | 1950 | int ind; |
1951 | u8 op0 = 0; | 1951 | u8 op0 = 0; |
1952 | 1952 | ||