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: Martin Jocic <martin.jocic@kvaser.com>
Link: https://lore.kernel.org/all/20240614151524.2718287-8-martin.jocic@kvaser.com
Signed-off-by: Marc Kleine-Budde <mkl@pengutronix.de>

diff --git a/drivers/net/can/kvaser_pciefd.c b/drivers/net/can/kvaser_pciefd.c
index 8b2c18f2f23b19d5b3c9f42794c5a54511cff251..24871c276b3150ccef9b4f9ec2b46429f95b761e 100644 (file)
--- a/drivers/net/can/kvaser_pciefd.c
+++ b/drivers/net/can/kvaser_pciefd.c
@@ -711,17 +711,17 @@ static void kvaser_pciefd_pwm_start(struct kvaser_pciefd_can *can)
 
 static int kvaser_pciefd_open(struct net_device *netdev)
 {
-       int err;
+       int ret;
        struct kvaser_pciefd_can *can = netdev_priv(netdev);
 
-       err = open_candev(netdev);
-       if (err)
-               return err;
+       ret = open_candev(netdev);
+       if (ret)
+               return ret;
 
-       err = kvaser_pciefd_bus_on(can);
-       if (err) {
+       ret = kvaser_pciefd_bus_on(can);
+       if (ret) {
                close_candev(netdev);
-               return err;
+               return ret;
        }
 
        return 0;
@@ -1032,15 +1032,15 @@ static int kvaser_pciefd_reg_candev(struct kvaser_pciefd *pcie)
        int 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;
 
                        /* Unregister all successfully registered devices. */
                        for (j = 0; j < i; j++)
                                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)
 static int kvaser_pciefd_probe(struct pci_dev *pdev,
                               const struct pci_device_id *id)
 {
-       int err;
+       int ret;
        struct kvaser_pciefd *pcie;
        const struct kvaser_pciefd_irq_mask *irq_mask;
        void __iomem *irq_en_base;
@@ -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;
        irq_mask = pcie->driver_data->irq_mask;
 
-       err = pci_enable_device(pdev);
-       if (err)
-               return err;
+       ret = pci_enable_device(pdev);
+       if (ret)
+               return ret;
 
-       err = pci_request_regions(pdev, KVASER_PCIEFD_DRV_NAME);
-       if (err)
+       ret = pci_request_regions(pdev, KVASER_PCIEFD_DRV_NAME);
+       if (ret)
                goto err_disable_pci;
 
        pcie->reg_base = pci_iomap(pdev, 0, 0);
        if (!pcie->reg_base) {
-               err = -ENOMEM;
+               ret = -ENOMEM;
                goto err_release_regions;
        }
 
-       err = kvaser_pciefd_setup_board(pcie);
-       if (err)
+       ret = kvaser_pciefd_setup_board(pcie);
+       if (ret)
                goto err_pci_iounmap;
 
-       err = kvaser_pciefd_setup_dma(pcie);
-       if (err)
+       ret = kvaser_pciefd_setup_dma(pcie);
+       if (ret)
                goto err_pci_iounmap;
 
        pci_set_master(pdev);
 
-       err = kvaser_pciefd_setup_can_ctrls(pcie);
-       if (err)
+       ret = kvaser_pciefd_setup_can_ctrls(pcie);
+       if (ret)
                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);
-       if (err)
+       if (ret)
                goto err_teardown_can_ctrls;
 
        iowrite32(KVASER_PCIEFD_SRB_IRQ_DPD0 | KVASER_PCIEFD_SRB_IRQ_DPD1,
@@ -1790,8 +1790,8 @@ static int kvaser_pciefd_probe(struct pci_dev *pdev,
        iowrite32(KVASER_PCIEFD_SRB_CMD_RDB1,
                  KVASER_PCIEFD_SRB_ADDR(pcie) + KVASER_PCIEFD_SRB_CMD_REG);
 
-       err = kvaser_pciefd_reg_candev(pcie);
-       if (err)
+       ret = kvaser_pciefd_reg_candev(pcie);
+       if (ret)
                goto err_free_irq;
 
        return 0;
@@ -1815,7 +1815,7 @@ err_release_regions:
 err_disable_pci:
        pci_disable_device(pdev);
 
-       return err;
+       return ret;
 }
 
 static void kvaser_pciefd_remove_all_ctrls(struct kvaser_pciefd *pcie)