Multiple assignments should be avoided.
For example:
cm_event.ird = cm_event.ord = 128;
Should be:
cm_event.ird = 128;
cm_event.ord = 128;
Signed-off-by: Weston Silbaugh <weston.silba...@gmail.com>
---
drivers/staging/rdma/amso1100/c2_ae.c | 6 ++++--
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/rdma/amso1100/c2_ae.c
b/drivers/staging/rdma/amso1100/c2_ae.c
index eb7a92b..20d98b7 100644
--- a/drivers/staging/rdma/amso1100/c2_ae.c
+++ b/drivers/staging/rdma/amso1100/c2_ae.c
@@ -173,7 +173,8 @@ void c2_ae_event(struct c2_dev *c2dev, u32 mq_index)
resource_user_context =
(void *) (unsigned long) wr->ae.ae_generic.user_context;
- status = cm_event.status = c2_convert_cm_status(c2_wr_get_result(wr));
+ status = c2_convert_cm_status(c2_wr_get_result(wr));
+ cm_event.status = c2_convert_cm_status(c2_wr_get_result(wr));
pr_debug("event received c2_dev=%p, event_id=%d, "
"resource_indicator=%d, user_context=%p, status = %d\n",
@@ -294,7 +295,8 @@ void c2_ae_event(struct c2_dev *c2dev, u32 mq_index)
* Until ird/ord negotiation via MPAv2 support is added, send
* max supported values
*/
- cm_event.ird = cm_event.ord = 128;
+ cm_event.ird = 128;
+ cm_event.ord = 128;
if (cm_id->event_handler)
cm_id->event_handler(cm_id, &cm_event);
--
2.4.3
_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
