Commit 26a1b0fe authored by Martin Jocic's avatar Martin Jocic Committed by Marc Kleine-Budde

can: kvaser_pciefd: Change name of return code variable

Replace the variable name err used for return codes with the more
generic name ret. An upcoming patch series for adding MSI interrupts
will introduce code which also returns values other than return codes.
Renaming the variable to ret enables using it for both purposes.
This is applied to the whole file to make it consistent.
Signed-off-by: default avatarMartin Jocic <martin.jocic@kvaser.com>
Link: https://lore.kernel.org/all/20240614151524.2718287-8-martin.jocic@kvaser.comSigned-off-by: default avatarMarc Kleine-Budde <mkl@pengutronix.de>
parent cbf88a6b
...@@ -711,17 +711,17 @@ static void kvaser_pciefd_pwm_start(struct kvaser_pciefd_can *can) ...@@ -711,17 +711,17 @@ static void kvaser_pciefd_pwm_start(struct kvaser_pciefd_can *can)
static int kvaser_pciefd_open(struct net_device *netdev) static int kvaser_pciefd_open(struct net_device *netdev)
{ {
int err; int ret;
struct kvaser_pciefd_can *can = netdev_priv(netdev); struct kvaser_pciefd_can *can = netdev_priv(netdev);
err = open_candev(netdev); ret = open_candev(netdev);
if (err) if (ret)
return err; return ret;
err = kvaser_pciefd_bus_on(can); ret = kvaser_pciefd_bus_on(can);
if (err) { if (ret) {
close_candev(netdev); close_candev(netdev);
return err; return ret;
} }
return 0; return 0;
...@@ -1032,15 +1032,15 @@ static int kvaser_pciefd_reg_candev(struct kvaser_pciefd *pcie) ...@@ -1032,15 +1032,15 @@ static int kvaser_pciefd_reg_candev(struct kvaser_pciefd *pcie)
int i; int i;
for (i = 0; i < pcie->nr_channels; i++) { for (i = 0; i < pcie->nr_channels; i++) {
int err = register_candev(pcie->can[i]->can.dev); int ret = register_candev(pcie->can[i]->can.dev);
if (err) { if (ret) {
int j; int j;
/* Unregister all successfully registered devices. */ /* Unregister all successfully registered devices. */
for (j = 0; j < i; j++) for (j = 0; j < i; j++)
unregister_candev(pcie->can[j]->can.dev); unregister_candev(pcie->can[j]->can.dev);
return err; return ret;
} }
} }
...@@ -1726,7 +1726,7 @@ static void kvaser_pciefd_teardown_can_ctrls(struct kvaser_pciefd *pcie) ...@@ -1726,7 +1726,7 @@ static void kvaser_pciefd_teardown_can_ctrls(struct kvaser_pciefd *pcie)
static int kvaser_pciefd_probe(struct pci_dev *pdev, static int kvaser_pciefd_probe(struct pci_dev *pdev,
const struct pci_device_id *id) const struct pci_device_id *id)
{ {
int err; int ret;
struct kvaser_pciefd *pcie; struct kvaser_pciefd *pcie;
const struct kvaser_pciefd_irq_mask *irq_mask; const struct kvaser_pciefd_irq_mask *irq_mask;
void __iomem *irq_en_base; void __iomem *irq_en_base;
...@@ -1740,37 +1740,37 @@ static int kvaser_pciefd_probe(struct pci_dev *pdev, ...@@ -1740,37 +1740,37 @@ static int kvaser_pciefd_probe(struct pci_dev *pdev,
pcie->driver_data = (const struct kvaser_pciefd_driver_data *)id->driver_data; pcie->driver_data = (const struct kvaser_pciefd_driver_data *)id->driver_data;
irq_mask = pcie->driver_data->irq_mask; irq_mask = pcie->driver_data->irq_mask;
err = pci_enable_device(pdev); ret = pci_enable_device(pdev);
if (err) if (ret)
return err; return ret;
err = pci_request_regions(pdev, KVASER_PCIEFD_DRV_NAME); ret = pci_request_regions(pdev, KVASER_PCIEFD_DRV_NAME);
if (err) if (ret)
goto err_disable_pci; goto err_disable_pci;
pcie->reg_base = pci_iomap(pdev, 0, 0); pcie->reg_base = pci_iomap(pdev, 0, 0);
if (!pcie->reg_base) { if (!pcie->reg_base) {
err = -ENOMEM; ret = -ENOMEM;
goto err_release_regions; goto err_release_regions;
} }
err = kvaser_pciefd_setup_board(pcie); ret = kvaser_pciefd_setup_board(pcie);
if (err) if (ret)
goto err_pci_iounmap; goto err_pci_iounmap;
err = kvaser_pciefd_setup_dma(pcie); ret = kvaser_pciefd_setup_dma(pcie);
if (err) if (ret)
goto err_pci_iounmap; goto err_pci_iounmap;
pci_set_master(pdev); pci_set_master(pdev);
err = kvaser_pciefd_setup_can_ctrls(pcie); ret = kvaser_pciefd_setup_can_ctrls(pcie);
if (err) if (ret)
goto err_teardown_can_ctrls; goto err_teardown_can_ctrls;
err = request_irq(pcie->pci->irq, kvaser_pciefd_irq_handler, ret = request_irq(pcie->pci->irq, kvaser_pciefd_irq_handler,
IRQF_SHARED, KVASER_PCIEFD_DRV_NAME, pcie); IRQF_SHARED, KVASER_PCIEFD_DRV_NAME, pcie);
if (err) if (ret)
goto err_teardown_can_ctrls; goto err_teardown_can_ctrls;
iowrite32(KVASER_PCIEFD_SRB_IRQ_DPD0 | KVASER_PCIEFD_SRB_IRQ_DPD1, iowrite32(KVASER_PCIEFD_SRB_IRQ_DPD0 | KVASER_PCIEFD_SRB_IRQ_DPD1,
...@@ -1790,8 +1790,8 @@ static int kvaser_pciefd_probe(struct pci_dev *pdev, ...@@ -1790,8 +1790,8 @@ static int kvaser_pciefd_probe(struct pci_dev *pdev,
iowrite32(KVASER_PCIEFD_SRB_CMD_RDB1, iowrite32(KVASER_PCIEFD_SRB_CMD_RDB1,
KVASER_PCIEFD_SRB_ADDR(pcie) + KVASER_PCIEFD_SRB_CMD_REG); KVASER_PCIEFD_SRB_ADDR(pcie) + KVASER_PCIEFD_SRB_CMD_REG);
err = kvaser_pciefd_reg_candev(pcie); ret = kvaser_pciefd_reg_candev(pcie);
if (err) if (ret)
goto err_free_irq; goto err_free_irq;
return 0; return 0;
...@@ -1815,7 +1815,7 @@ static int kvaser_pciefd_probe(struct pci_dev *pdev, ...@@ -1815,7 +1815,7 @@ static int kvaser_pciefd_probe(struct pci_dev *pdev,
err_disable_pci: err_disable_pci:
pci_disable_device(pdev); pci_disable_device(pdev);
return err; return ret;
} }
static void kvaser_pciefd_remove_all_ctrls(struct kvaser_pciefd *pcie) static void kvaser_pciefd_remove_all_ctrls(struct kvaser_pciefd *pcie)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment