Skip to content

Commit 896de44

Browse files
emuslndavem330
authored andcommitted
ionic: remove unnecessary void casts
Minor Code cleanup details. Signed-off-by: Shannon Nelson <[email protected]> Reviewed-by: Leon Romanovsky <[email protected]> Signed-off-by: David S. Miller <[email protected]>
1 parent 7bb9900 commit 896de44

File tree

4 files changed

+7
-7
lines changed

4 files changed

+7
-7
lines changed

drivers/net/ethernet/pensando/ionic/ionic_bus_pci.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -121,7 +121,7 @@ static void ionic_vf_dealloc_locked(struct ionic *ionic)
121121

122122
if (v->stats_pa) {
123123
vfc.stats_pa = 0;
124-
(void)ionic_set_vf_config(ionic, i, &vfc);
124+
ionic_set_vf_config(ionic, i, &vfc);
125125
dma_unmap_single(ionic->dev, v->stats_pa,
126126
sizeof(v->stats), DMA_FROM_DEVICE);
127127
v->stats_pa = 0;
@@ -169,7 +169,7 @@ static int ionic_vf_alloc(struct ionic *ionic, int num_vfs)
169169

170170
/* ignore failures from older FW, we just won't get stats */
171171
vfc.stats_pa = cpu_to_le64(v->stats_pa);
172-
(void)ionic_set_vf_config(ionic, i, &vfc);
172+
ionic_set_vf_config(ionic, i, &vfc);
173173
}
174174

175175
out:

drivers/net/ethernet/pensando/ionic/ionic_main.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -388,7 +388,7 @@ int ionic_adminq_wait(struct ionic_lif *lif, struct ionic_admin_ctx *ctx,
388388
break;
389389

390390
/* force a check of FW status and break out if FW reset */
391-
(void)ionic_heartbeat_check(lif->ionic);
391+
ionic_heartbeat_check(lif->ionic);
392392
if ((test_bit(IONIC_LIF_F_FW_RESET, lif->state) &&
393393
!lif->ionic->idev.fw_status_ready) ||
394394
test_bit(IONIC_LIF_F_FW_STOPPING, lif->state)) {
@@ -676,7 +676,7 @@ int ionic_port_init(struct ionic *ionic)
676676
err = ionic_dev_cmd_wait(ionic, DEVCMD_TIMEOUT);
677677

678678
ionic_dev_cmd_port_state(&ionic->idev, IONIC_PORT_ADMIN_STATE_UP);
679-
(void)ionic_dev_cmd_wait(ionic, DEVCMD_TIMEOUT);
679+
ionic_dev_cmd_wait(ionic, DEVCMD_TIMEOUT);
680680

681681
mutex_unlock(&ionic->dev_cmd_lock);
682682
if (err) {

drivers/net/ethernet/pensando/ionic/ionic_phc.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -268,7 +268,7 @@ static u64 ionic_hwstamp_read(struct ionic *ionic,
268268
u32 tick_high_before, tick_high, tick_low;
269269

270270
/* read and discard low part to defeat hw staging of high part */
271-
(void)ioread32(&ionic->idev.hwstamp_regs->tick_low);
271+
ioread32(&ionic->idev.hwstamp_regs->tick_low);
272272

273273
tick_high_before = ioread32(&ionic->idev.hwstamp_regs->tick_high);
274274

drivers/net/ethernet/pensando/ionic/ionic_rx_filter.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -604,14 +604,14 @@ void ionic_rx_filter_sync(struct ionic_lif *lif)
604604
* they can clear room for some new filters
605605
*/
606606
list_for_each_entry_safe(sync_item, spos, &sync_del_list, list) {
607-
(void)ionic_lif_filter_del(lif, &sync_item->f.cmd);
607+
ionic_lif_filter_del(lif, &sync_item->f.cmd);
608608

609609
list_del(&sync_item->list);
610610
devm_kfree(dev, sync_item);
611611
}
612612

613613
list_for_each_entry_safe(sync_item, spos, &sync_add_list, list) {
614-
(void)ionic_lif_filter_add(lif, &sync_item->f.cmd);
614+
ionic_lif_filter_add(lif, &sync_item->f.cmd);
615615

616616
list_del(&sync_item->list);
617617
devm_kfree(dev, sync_item);

0 commit comments

Comments
 (0)