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

Reply via email to