Skip to content

Commit cb85f2b

Browse files
committed
Merge branch 'ionic-minor-code-fixes'
Shannon Nelson says: ==================== ionic: minor code fixes These are a couple of code fixes for the ionic driver. ==================== Link: https://patch.msgid.link/[email protected] Signed-off-by: Jakub Kicinski <[email protected]>
2 parents 663ad74 + b096d62 commit cb85f2b

File tree

3 files changed

+8
-5
lines changed

3 files changed

+8
-5
lines changed

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

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -277,7 +277,10 @@ void ionic_dev_teardown(struct ionic *ionic)
277277
idev->phy_cmb_pages = 0;
278278
idev->cmb_npages = 0;
279279

280-
destroy_workqueue(ionic->wq);
280+
if (ionic->wq) {
281+
destroy_workqueue(ionic->wq);
282+
ionic->wq = NULL;
283+
}
281284
mutex_destroy(&idev->cmb_inuse_lock);
282285
}
283286

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

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -961,8 +961,8 @@ static int ionic_get_module_eeprom(struct net_device *netdev,
961961
len = min_t(u32, sizeof(xcvr->sprom), ee->len);
962962

963963
do {
964-
memcpy(data, xcvr->sprom, len);
965-
memcpy(tbuf, xcvr->sprom, len);
964+
memcpy(data, &xcvr->sprom[ee->offset], len);
965+
memcpy(tbuf, &xcvr->sprom[ee->offset], len);
966966

967967
/* Let's make sure we got a consistent copy */
968968
if (!memcmp(data, tbuf, len))

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

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3869,8 +3869,8 @@ int ionic_lif_register(struct ionic_lif *lif)
38693869
/* only register LIF0 for now */
38703870
err = register_netdev(lif->netdev);
38713871
if (err) {
3872-
dev_err(lif->ionic->dev, "Cannot register net device, aborting\n");
3873-
ionic_lif_unregister_phc(lif);
3872+
dev_err(lif->ionic->dev, "Cannot register net device: %d, aborting\n", err);
3873+
ionic_lif_unregister(lif);
38743874
return err;
38753875
}
38763876

0 commit comments

Comments
 (0)