Skip to content

Commit

Permalink
ionic: extract common bits from ionic_probe
Browse files Browse the repository at this point in the history
Pull out some chunks of code from ionic_probe() that will
be common in rebuild paths.

Signed-off-by: Shannon Nelson <shannon.nelson@amd.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
  • Loading branch information
Shannon Nelson authored and David S. Miller committed Jul 24, 2023
1 parent 87d7a9f commit 0de38d9
Showing 1 changed file with 49 additions and 35 deletions.
84 changes: 49 additions & 35 deletions drivers/net/ethernet/pensando/ionic/ionic_bus_pci.c
Original file line number Diff line number Diff line change
Expand Up @@ -220,30 +220,12 @@ static void ionic_clear_pci(struct ionic *ionic)
pci_disable_device(ionic->pdev);
}

static int ionic_probe(struct pci_dev *pdev, const struct pci_device_id *ent)
static int ionic_setup_one(struct ionic *ionic)
{
struct device *dev = &pdev->dev;
struct ionic *ionic;
int num_vfs;
struct pci_dev *pdev = ionic->pdev;
struct device *dev = ionic->dev;
int err;

ionic = ionic_devlink_alloc(dev);
if (!ionic)
return -ENOMEM;

ionic->pdev = pdev;
ionic->dev = dev;
pci_set_drvdata(pdev, ionic);
mutex_init(&ionic->dev_cmd_lock);

/* Query system for DMA addressing limitation for the device. */
err = dma_set_mask_and_coherent(dev, DMA_BIT_MASK(IONIC_ADDR_LEN));
if (err) {
dev_err(dev, "Unable to obtain 64-bit DMA for consistent allocations, aborting. err=%d\n",
err);
goto err_out_clear_drvdata;
}

ionic_debugfs_add_dev(ionic);

/* Setup PCI device */
Expand All @@ -258,7 +240,6 @@ static int ionic_probe(struct pci_dev *pdev, const struct pci_device_id *ent)
dev_err(dev, "Cannot request PCI regions: %d, aborting\n", err);
goto err_out_clear_pci;
}

pcie_print_link_status(pdev);

err = ionic_map_bars(ionic);
Expand Down Expand Up @@ -286,24 +267,64 @@ static int ionic_probe(struct pci_dev *pdev, const struct pci_device_id *ent)
goto err_out_teardown;
}

/* Configure the ports */
/* Configure the port */
err = ionic_port_identify(ionic);
if (err) {
dev_err(dev, "Cannot identify port: %d, aborting\n", err);
goto err_out_reset;
goto err_out_teardown;
}

err = ionic_port_init(ionic);
if (err) {
dev_err(dev, "Cannot init port: %d, aborting\n", err);
goto err_out_reset;
goto err_out_teardown;
}

return 0;

err_out_teardown:
ionic_dev_teardown(ionic);
err_out_clear_pci:
ionic_clear_pci(ionic);
err_out_debugfs_del_dev:
ionic_debugfs_del_dev(ionic);

return err;
}

static int ionic_probe(struct pci_dev *pdev, const struct pci_device_id *ent)
{
struct device *dev = &pdev->dev;
struct ionic *ionic;
int num_vfs;
int err;

ionic = ionic_devlink_alloc(dev);
if (!ionic)
return -ENOMEM;

ionic->pdev = pdev;
ionic->dev = dev;
pci_set_drvdata(pdev, ionic);
mutex_init(&ionic->dev_cmd_lock);

/* Query system for DMA addressing limitation for the device. */
err = dma_set_mask_and_coherent(dev, DMA_BIT_MASK(IONIC_ADDR_LEN));
if (err) {
dev_err(dev, "Unable to obtain 64-bit DMA for consistent allocations, aborting. err=%d\n",
err);
goto err_out;
}

err = ionic_setup_one(ionic);
if (err)
goto err_out;

/* Allocate and init the LIF */
err = ionic_lif_size(ionic);
if (err) {
dev_err(dev, "Cannot size LIF: %d, aborting\n", err);
goto err_out_port_reset;
goto err_out_pci;
}

err = ionic_lif_alloc(ionic);
Expand Down Expand Up @@ -354,17 +375,10 @@ static int ionic_probe(struct pci_dev *pdev, const struct pci_device_id *ent)
ionic->lif = NULL;
err_out_free_irqs:
ionic_bus_free_irq_vectors(ionic);
err_out_port_reset:
ionic_port_reset(ionic);
err_out_reset:
ionic_reset(ionic);
err_out_teardown:
err_out_pci:
ionic_dev_teardown(ionic);
err_out_clear_pci:
ionic_clear_pci(ionic);
err_out_debugfs_del_dev:
ionic_debugfs_del_dev(ionic);
err_out_clear_drvdata:
err_out:
mutex_destroy(&ionic->dev_cmd_lock);
ionic_devlink_free(ionic);

Expand Down

0 comments on commit 0de38d9

Please sign in to comment.