diff options
author | Parav Pandit <parav@mellanox.com> | 2017-02-02 00:14:08 -0500 |
---|---|---|
committer | Doug Ledford <dledford@redhat.com> | 2017-02-15 09:51:28 -0500 |
commit | 590396084039b6504097b371cda6474dfcb83648 (patch) | |
tree | 026b5c3b15ebaffb2e6fdfb9061e857ca33dc2be | |
parent | 2b0841766a898aba84630fb723989a77a9d3b4e6 (diff) |
IB/core: Remove pointer casting from void to net_device
This patch avoids unnecessary type casting from void to net_device.
CC: Yuval Shaia <yuval.shaia@oracle.com>
Signed-off-by: Parav Pandit <parav@mellanox.com>
Signed-off-by: Leon Romanovsky <leon@kernel.org>
Reviewed-by: Yuval Shaia <yuval.shaia@oracle.com>
Signed-off-by: Doug Ledford <dledford@redhat.com>
-rw-r--r-- | drivers/infiniband/core/roce_gid_mgmt.c | 28 |
1 files changed, 10 insertions, 18 deletions
diff --git a/drivers/infiniband/core/roce_gid_mgmt.c b/drivers/infiniband/core/roce_gid_mgmt.c index 0621f4455732..db958d3207ef 100644 --- a/drivers/infiniband/core/roce_gid_mgmt.c +++ b/drivers/infiniband/core/roce_gid_mgmt.c | |||
@@ -144,7 +144,6 @@ static enum bonding_slave_state is_eth_active_slave_of_bonding_rcu(struct net_de | |||
144 | static int is_eth_port_of_netdev(struct ib_device *ib_dev, u8 port, | 144 | static int is_eth_port_of_netdev(struct ib_device *ib_dev, u8 port, |
145 | struct net_device *rdma_ndev, void *cookie) | 145 | struct net_device *rdma_ndev, void *cookie) |
146 | { | 146 | { |
147 | struct net_device *event_ndev = (struct net_device *)cookie; | ||
148 | struct net_device *real_dev; | 147 | struct net_device *real_dev; |
149 | int res; | 148 | int res; |
150 | 149 | ||
@@ -152,11 +151,11 @@ static int is_eth_port_of_netdev(struct ib_device *ib_dev, u8 port, | |||
152 | return 0; | 151 | return 0; |
153 | 152 | ||
154 | rcu_read_lock(); | 153 | rcu_read_lock(); |
155 | real_dev = rdma_vlan_dev_real_dev(event_ndev); | 154 | real_dev = rdma_vlan_dev_real_dev(cookie); |
156 | if (!real_dev) | 155 | if (!real_dev) |
157 | real_dev = event_ndev; | 156 | real_dev = cookie; |
158 | 157 | ||
159 | res = ((rdma_is_upper_dev_rcu(rdma_ndev, event_ndev) && | 158 | res = ((rdma_is_upper_dev_rcu(rdma_ndev, cookie) && |
160 | (is_eth_active_slave_of_bonding_rcu(rdma_ndev, real_dev) & | 159 | (is_eth_active_slave_of_bonding_rcu(rdma_ndev, real_dev) & |
161 | REQUIRED_BOND_STATES)) || | 160 | REQUIRED_BOND_STATES)) || |
162 | real_dev == rdma_ndev); | 161 | real_dev == rdma_ndev); |
@@ -192,17 +191,16 @@ static int pass_all_filter(struct ib_device *ib_dev, u8 port, | |||
192 | static int upper_device_filter(struct ib_device *ib_dev, u8 port, | 191 | static int upper_device_filter(struct ib_device *ib_dev, u8 port, |
193 | struct net_device *rdma_ndev, void *cookie) | 192 | struct net_device *rdma_ndev, void *cookie) |
194 | { | 193 | { |
195 | struct net_device *event_ndev = (struct net_device *)cookie; | ||
196 | int res; | 194 | int res; |
197 | 195 | ||
198 | if (!rdma_ndev) | 196 | if (!rdma_ndev) |
199 | return 0; | 197 | return 0; |
200 | 198 | ||
201 | if (rdma_ndev == event_ndev) | 199 | if (rdma_ndev == cookie) |
202 | return 1; | 200 | return 1; |
203 | 201 | ||
204 | rcu_read_lock(); | 202 | rcu_read_lock(); |
205 | res = rdma_is_upper_dev_rcu(rdma_ndev, event_ndev); | 203 | res = rdma_is_upper_dev_rcu(rdma_ndev, cookie); |
206 | rcu_read_unlock(); | 204 | rcu_read_unlock(); |
207 | 205 | ||
208 | return res; | 206 | return res; |
@@ -379,18 +377,14 @@ static void _add_netdev_ips(struct ib_device *ib_dev, u8 port, | |||
379 | static void add_netdev_ips(struct ib_device *ib_dev, u8 port, | 377 | static void add_netdev_ips(struct ib_device *ib_dev, u8 port, |
380 | struct net_device *rdma_ndev, void *cookie) | 378 | struct net_device *rdma_ndev, void *cookie) |
381 | { | 379 | { |
382 | struct net_device *event_ndev = (struct net_device *)cookie; | 380 | enum_netdev_default_gids(ib_dev, port, cookie, rdma_ndev); |
383 | 381 | _add_netdev_ips(ib_dev, port, cookie); | |
384 | enum_netdev_default_gids(ib_dev, port, event_ndev, rdma_ndev); | ||
385 | _add_netdev_ips(ib_dev, port, event_ndev); | ||
386 | } | 382 | } |
387 | 383 | ||
388 | static void del_netdev_ips(struct ib_device *ib_dev, u8 port, | 384 | static void del_netdev_ips(struct ib_device *ib_dev, u8 port, |
389 | struct net_device *rdma_ndev, void *cookie) | 385 | struct net_device *rdma_ndev, void *cookie) |
390 | { | 386 | { |
391 | struct net_device *event_ndev = (struct net_device *)cookie; | 387 | ib_cache_gid_del_all_netdev_gids(ib_dev, port, cookie); |
392 | |||
393 | ib_cache_gid_del_all_netdev_gids(ib_dev, port, event_ndev); | ||
394 | } | 388 | } |
395 | 389 | ||
396 | static void enum_all_gids_of_dev_cb(struct ib_device *ib_dev, | 390 | static void enum_all_gids_of_dev_cb(struct ib_device *ib_dev, |
@@ -460,7 +454,7 @@ static void handle_netdev_upper(struct ib_device *ib_dev, u8 port, | |||
460 | u8 port, | 454 | u8 port, |
461 | struct net_device *ndev)) | 455 | struct net_device *ndev)) |
462 | { | 456 | { |
463 | struct net_device *ndev = (struct net_device *)cookie; | 457 | struct net_device *ndev = cookie; |
464 | struct upper_list *upper_iter; | 458 | struct upper_list *upper_iter; |
465 | struct upper_list *upper_temp; | 459 | struct upper_list *upper_temp; |
466 | LIST_HEAD(upper_list); | 460 | LIST_HEAD(upper_list); |
@@ -519,9 +513,7 @@ static void del_netdev_default_ips_join(struct ib_device *ib_dev, u8 port, | |||
519 | static void del_netdev_default_ips(struct ib_device *ib_dev, u8 port, | 513 | static void del_netdev_default_ips(struct ib_device *ib_dev, u8 port, |
520 | struct net_device *rdma_ndev, void *cookie) | 514 | struct net_device *rdma_ndev, void *cookie) |
521 | { | 515 | { |
522 | struct net_device *event_ndev = (struct net_device *)cookie; | 516 | bond_delete_netdev_default_gids(ib_dev, port, cookie, rdma_ndev); |
523 | |||
524 | bond_delete_netdev_default_gids(ib_dev, port, event_ndev, rdma_ndev); | ||
525 | } | 517 | } |
526 | 518 | ||
527 | /* The following functions operate on all IB devices. netdevice_event and | 519 | /* The following functions operate on all IB devices. netdevice_event and |