serial: exar: added a exar_get_nr_ports function
authorParker Newman <pnewman@connecttech.com>
Wed, 17 Apr 2024 20:31:24 +0000 (16:31 -0400)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 18 Apr 2024 06:23:50 +0000 (08:23 +0200)
Moved code for getting number of ports from exar_pci_probe() to a
separate exar_get_nr_ports() function. CTI specific code will be added
in another patch in this series.

Signed-off-by: Parker Newman <pnewman@connecttech.com>
Link: https://lore.kernel.org/r/33f2bf66bc334573c10cf670a299ecef0b7264bc.1713382717.git.pnewman@connecttech.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/tty/serial/8250/8250_exar.c

index 04ce5e8ddb24dbe53d03cb3ae16a733fb1f6eeac..72385c7d2eda88ca8d61d4c4446818d820485bcf 100644 (file)
@@ -704,6 +704,21 @@ static irqreturn_t exar_misc_handler(int irq, void *data)
        return IRQ_HANDLED;
 }
 
+static unsigned int exar_get_nr_ports(struct exar8250_board *board,
+                                       struct pci_dev *pcidev)
+{
+       unsigned int nr_ports = 0;
+
+       if (pcidev->vendor == PCI_VENDOR_ID_ACCESSIO)
+               nr_ports = BIT(((pcidev->device & 0x38) >> 3) - 1);
+       else if (board->num_ports)
+               nr_ports = board->num_ports;
+       else
+               nr_ports = pcidev->device & 0x0f;
+
+       return nr_ports;
+}
+
 static int
 exar_pci_probe(struct pci_dev *pcidev, const struct pci_device_id *ent)
 {
@@ -723,12 +738,12 @@ exar_pci_probe(struct pci_dev *pcidev, const struct pci_device_id *ent)
 
        maxnr = pci_resource_len(pcidev, bar) >> (board->reg_shift + 3);
 
-       if (pcidev->vendor == PCI_VENDOR_ID_ACCESSIO)
-               nr_ports = BIT(((pcidev->device & 0x38) >> 3) - 1);
-       else if (board->num_ports)
-               nr_ports = board->num_ports;
-       else
-               nr_ports = pcidev->device & 0x0f;
+       nr_ports = exar_get_nr_ports(board, pcidev);
+       if (nr_ports == 0) {
+               dev_err_probe(&pcidev->dev, -ENODEV,
+                               "failed to get number of ports\n");
+               return -ENODEV;
+       }
 
        priv = devm_kzalloc(&pcidev->dev, struct_size(priv, line, nr_ports), GFP_KERNEL);
        if (!priv)