RDMA/i40iw: Remove unused code and fix warning

Remove unused code and fix warning.

Signed-off-by: Mustafa Ismail <mustafa.ismail@intel.com>
Signed-off-by: Faisal Latif <faisal.latif@intel.com>
Signed-off-by: Doug Ledford <dledford@redhat.com>

diff --git a/drivers/infiniband/hw/i40iw/i40iw.h b/drivers/infiniband/hw/i40iw/i40iw.h
index bf3627fea222c8f0cace594e50a5a6186ef64b5c..f299001edbc25e1891bd368a95058a522d9a6c16 100644 (file)
--- a/drivers/infiniband/hw/i40iw/i40iw.h
+++ b/drivers/infiniband/hw/i40iw/i40iw.h
@@ -50,8 +50,6 @@
 #include <rdma/ib_pack.h>
 #include <rdma/rdma_cm.h>
 #include <rdma/iw_cm.h>
-#include <rdma/iw_portmap.h>
-#include <rdma/rdma_netlink.h>
 #include <crypto/hash.h>
 
 #include "i40iw_status.h"

diff --git a/drivers/infiniband/hw/i40iw/i40iw_cm.c b/drivers/infiniband/hw/i40iw/i40iw_cm.c
index bdd4104db40d0ce226e70037a6ca33790d62ac11..ab6eb0bc020eb0c400a0f13e4242dd7c69802655 100644 (file)
--- a/drivers/infiniband/hw/i40iw/i40iw_cm.c
+++ b/drivers/infiniband/hw/i40iw/i40iw_cm.c
@@ -2107,7 +2107,7 @@ static bool i40iw_ipv6_is_loopback(u32 *loc_addr, u32 *rem_addr)
        struct in6_addr raddr6;
 
        i40iw_copy_ip_htonl(raddr6.in6_u.u6_addr32, rem_addr);
-       return (!memcmp(loc_addr, rem_addr, 16) || ipv6_addr_loopback(&raddr6));
+       return !memcmp(loc_addr, rem_addr, 16) || ipv6_addr_loopback(&raddr6);
 }
 
 /**

diff --git a/drivers/infiniband/hw/i40iw/i40iw_cm.h b/drivers/infiniband/hw/i40iw/i40iw_cm.h
index 5f8ceb4a8e847923689f367a6ed7aed9796dabd0..e9046d9f96456c847e1df069b6a5dff9e0f0cfe2 100644 (file)
--- a/drivers/infiniband/hw/i40iw/i40iw_cm.h
+++ b/drivers/infiniband/hw/i40iw/i40iw_cm.h
@@ -1,6 +1,6 @@
 /*******************************************************************************
 *
-* Copyright (c) 2015 Intel Corporation.  All rights reserved.
+* Copyright (c) 2015-2016 Intel Corporation.  All rights reserved.
 *
 * This software is available to you under a choice of one of two
 * licenses.  You may choose to be licensed under the terms of the GNU
@@ -291,8 +291,6 @@ struct i40iw_cm_listener {
        u8 loc_mac[ETH_ALEN];
        u32 loc_addr[4];
        u16 loc_port;
-       u32 map_loc_addr[4];
-       u16 map_loc_port;
        struct iw_cm_id *cm_id;
        atomic_t ref_count;
        struct i40iw_device *iwdev;
@@ -317,8 +315,6 @@ struct i40iw_kmem_info {
 struct i40iw_cm_node {
        u32 loc_addr[4], rem_addr[4];
        u16 loc_port, rem_port;
-       u32 map_loc_addr[4], map_rem_addr[4];
-       u16 map_loc_port, map_rem_port;
        u16 vlan_id;
        enum i40iw_cm_node_state state;
        u8 loc_mac[ETH_ALEN];
@@ -370,10 +366,6 @@ struct i40iw_cm_info {
        u16 rem_port;
        u32 loc_addr[4];
        u32 rem_addr[4];
-       u16 map_loc_port;
-       u16 map_rem_port;
-       u32 map_loc_addr[4];
-       u32 map_rem_addr[4];
        u16 vlan_id;
        int backlog;
        u16 user_pri;

diff --git a/drivers/infiniband/hw/i40iw/i40iw_main.c b/drivers/infiniband/hw/i40iw/i40iw_main.c
index 90e5af21737e0017902b1bf72f7e77cecc195b54..f49aea1cedff0a3c1f8c3cc5239ff98915f5a404 100644 (file)
--- a/drivers/infiniband/hw/i40iw/i40iw_main.c
+++ b/drivers/infiniband/hw/i40iw/i40iw_main.c
@@ -1147,10 +1147,7 @@ static enum i40iw_status_code i40iw_alloc_set_mac_ipaddr(struct i40iw_device *iw
        if (!status) {
                status = i40iw_add_mac_ipaddr_entry(iwdev, macaddr,
                                                    (u8)iwdev->mac_ip_table_idx);
-               if (!status)
-                       status = i40iw_add_mac_ipaddr_entry(iwdev, macaddr,
-                                                           (u8)iwdev->mac_ip_table_idx);
-               else
+               if (status)
                        i40iw_del_macip_entry(iwdev, (u8)iwdev->mac_ip_table_idx);
        }
        return status;