RDMA/bnxt_re: Remove unused vlan_tag variable
authorLeon Romanovsky <leonro@mellanox.com>
Sun, 29 Oct 2017 15:05:22 +0000 (17:05 +0200)
committerDoug Ledford <dledford@redhat.com>
Mon, 13 Nov 2017 18:53:22 +0000 (13:53 -0500)
The Broadcom driver produces the following compilation warning

drivers/infiniband/hw/bnxt_re/ib_verbs.c:
In function ‘bnxt_re_create_ah’:
drivers/infiniband/hw/bnxt_re/ib_verbs.c:668:6:
warning: variable ‘vlan_tag’ set but not used [-Wunused-but-set-variable]
u16 vlan_tag;

Let's remove it till vlan_tag will be implemented properly.

Cc: Selvin Xavier <selvin.xavier@broadcom.com>
Signed-off-by: Leon Romanovsky <leonro@mellanox.com>
Signed-off-by: Doug Ledford <dledford@redhat.com>
drivers/infiniband/hw/bnxt_re/ib_verbs.c

index ebcdfb4..39e49d0 100644 (file)
@@ -665,7 +665,6 @@ struct ib_ah *bnxt_re_create_ah(struct ib_pd *ib_pd,
        struct bnxt_re_ah *ah;
        const struct ib_global_route *grh = rdma_ah_read_grh(ah_attr);
        int rc;
-       u16 vlan_tag;
        u8 nw_type;
 
        struct ib_gid_attr sgid_attr;
@@ -711,11 +710,8 @@ struct ib_ah *bnxt_re_create_ah(struct ib_pd *ib_pd,
                                grh->sgid_index);
                        goto fail;
                }
-               if (sgid_attr.ndev) {
-                       if (is_vlan_dev(sgid_attr.ndev))
-                               vlan_tag = vlan_dev_vlan_id(sgid_attr.ndev);
+               if (sgid_attr.ndev)
                        dev_put(sgid_attr.ndev);
-               }
                /* Get network header type for this GID */
                nw_type = ib_gid_to_network_type(sgid_attr.gid_type, &sgid);
                switch (nw_type) {