Re: [PATCH v6 2/3] serial: exar: split out the exar code from 8250_pci

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Jan 5, 2017 at 12:20 PM, Sudip Mukherjee
<sudipm.mukherjee@xxxxxxxxx> wrote:
> Add the serial driver for the exar chips. And also register the
> platform device for the exar gpio.

> +++ b/drivers/tty/serial/8250/8250_exar.c
> @@ -0,0 +1,706 @@
> +/*

> + *  Probe module for 8250/16550-type PCI serial ports.
> + *
> + *  Based on drivers/char/serial.c, by Linus Torvalds, Theodore Ts'o.
> + *
> + *  Copyright (C) 2001 Russell King, All Rights Reserved.

I'm not sure this is a right portion of the top.
This one is based on 8250_pci.c apparently.

> + *
> + * This program is free software; you can redistribute it and/or modify
> + * it under the terms of the GNU General Public License as published by
> + * the Free Software Foundation; either version 2 of the License.
> + */

> +#undef DEBUG

> +#include <linux/module.h>
> +#include <linux/pci.h>
> +#include <linux/string.h>
> +#include <linux/kernel.h>
> +#include <linux/slab.h>
> +#include <linux/delay.h>
> +#include <linux/tty.h>
> +#include <linux/serial_reg.h>
> +#include <linux/serial_core.h>
> +#include <linux/8250_pci.h>
> +#include <linux/bitops.h>
> +#include <linux/io.h>

Alphabetical order?

> +
> +#include <asm/byteorder.h>
> +
> +#include "8250.h"
> +
> +/*
> + * init function returns:
> + *  > 0 - number of ports
> + *  = 0 - use board->num_ports
> + *  < 0 - error
> + */
> +struct pci_serial_quirk {
> +       u32     vendor;
> +       u32     device;
> +       u32     subvendor;
> +       u32     subdevice;
> +       int     (*probe)(struct pci_dev *dev);
> +       int     (*init)(struct pci_dev *dev);
> +       int     (*setup)(struct serial_private *,
> +                        const struct pciserial_board *,
> +                        struct uart_8250_port *, int);
> +       void    (*exit)(struct pci_dev *dev);
> +};
> +
> +#define PCI_NUM_BAR_RESOURCES  6
> +
> +struct serial_private {
> +       struct pci_dev          *dev;
> +       unsigned int            nr;
> +       struct pci_serial_quirk *quirk;
> +       int                     line[0];
> +};
> +
> +static int
> +setup_port(struct serial_private *priv, struct uart_8250_port *port,
> +          int bar, int offset, int regshift)
> +{
> +       struct pci_dev *dev = priv->dev;
> +
> +       if (bar >= PCI_NUM_BAR_RESOURCES)
> +               return -EINVAL;
> +
> +       if (pci_resource_flags(dev, bar) & IORESOURCE_MEM) {
> +               if (!pcim_iomap(dev, bar, 0) && !pcim_iomap_table(dev))
> +                       return -ENOMEM;
> +
> +               port->port.iotype = UPIO_MEM;
> +               port->port.iobase = 0;
> +               port->port.mapbase = pci_resource_start(dev, bar) + offset;
> +               port->port.membase = pcim_iomap_table(dev)[bar] + offset;
> +               port->port.regshift = regshift;
> +       } else {
> +               port->port.iotype = UPIO_PORT;
> +               port->port.iobase = pci_resource_start(dev, bar) + offset;
> +               port->port.mapbase = 0;
> +               port->port.membase = NULL;
> +               port->port.regshift = 0;
> +       }
> +       return 0;

The point of splitting is to get rid of code that is not used. Do you
have UPIO_PORT case?

> +}
> +
> +static int pci_default_setup(struct serial_private *priv,
> +                            const struct pciserial_board *board,
> +                            struct uart_8250_port *port, int idx)
> +{
> +       unsigned int bar, offset = board->first_offset, maxnr;
> +
> +       bar = FL_GET_BASE(board->flags);
> +       if (board->flags & FL_BASE_BARS)
> +               bar += idx;
> +       else
> +               offset += idx * board->uart_offset;
> +
> +       maxnr = (pci_resource_len(priv->dev, bar) - board->first_offset) >>
> +               (board->reg_shift + 3);
> +
> +       if (board->flags & FL_REGION_SZ_CAP && idx >= maxnr)
> +               return 1;
> +
> +       return setup_port(priv, port, bar, offset, board->reg_shift);
> +}
> +
> +#define PCI_DEVICE_ID_EXAR_XR17V4358   0x4358
> +#define PCI_DEVICE_ID_EXAR_XR17V8358   0x8358
> +
> +#define UART_EXAR_MPIOINT_7_0  0x8f    /* MPIOINT[7:0] */
> +#define UART_EXAR_MPIOLVL_7_0  0x90    /* MPIOLVL[7:0] */
> +#define UART_EXAR_MPIO3T_7_0   0x91    /* MPIO3T[7:0] */
> +#define UART_EXAR_MPIOINV_7_0  0x92    /* MPIOINV[7:0] */
> +#define UART_EXAR_MPIOSEL_7_0  0x93    /* MPIOSEL[7:0] */
> +#define UART_EXAR_MPIOOD_7_0   0x94    /* MPIOOD[7:0] */
> +#define UART_EXAR_MPIOINT_15_8 0x95    /* MPIOINT[15:8] */
> +#define UART_EXAR_MPIOLVL_15_8 0x96    /* MPIOLVL[15:8] */
> +#define UART_EXAR_MPIO3T_15_8  0x97    /* MPIO3T[15:8] */
> +#define UART_EXAR_MPIOINV_15_8 0x98    /* MPIOINV[15:8] */
> +#define UART_EXAR_MPIOSEL_15_8 0x99    /* MPIOSEL[15:8] */
> +#define UART_EXAR_MPIOOD_15_8  0x9a    /* MPIOOD[15:8] */
> +
> +static int
> +pci_xr17c154_setup(struct serial_private *priv,
> +                  const struct pciserial_board *board,
> +                 struct uart_8250_port *port, int idx)
> +{
> +       port->port.flags |= UPF_EXAR_EFR;
> +       return pci_default_setup(priv, board, port, idx);
> +}
> +
> +static inline int
> +xr17v35x_has_slave(struct serial_private *priv)
> +{
> +       const int dev_id = priv->dev->device;
> +
> +       return ((dev_id == PCI_DEVICE_ID_EXAR_XR17V4358) ||
> +               (dev_id == PCI_DEVICE_ID_EXAR_XR17V8358));
> +}
> +
> +static int
> +pci_xr17v35x_setup(struct serial_private *priv,
> +                  const struct pciserial_board *board,
> +                 struct uart_8250_port *port, int idx)
> +{
> +       u8 __iomem *p;
> +       int ret;
> +
> +       p = pci_ioremap_bar(priv->dev, 0);
> +       if (!p)
> +               return -ENOMEM;
> +
> +       port->port.flags |= UPF_EXAR_EFR;
> +
> +       /*
> +        * Setup the uart clock for the devices on expansion slot to
> +        * half the clock speed of the main chip (which is 125MHz)
> +        */
> +       if (xr17v35x_has_slave(priv) && idx >= 8)
> +               port->port.uartclk = (7812500 * 16 / 2);
> +
> +       /*
> +        * Setup Multipurpose Input/Output pins.
> +        */
> +       if (idx == 0) {
> +               writeb(0x00, p + UART_EXAR_MPIOINT_7_0);
> +               writeb(0x00, p + UART_EXAR_MPIOLVL_7_0);
> +               writeb(0x00, p + UART_EXAR_MPIO3T_7_0);
> +               writeb(0x00, p + UART_EXAR_MPIOINV_7_0);
> +               writeb(0x00, p + UART_EXAR_MPIOSEL_7_0);
> +               writeb(0x00, p + UART_EXAR_MPIOOD_7_0);
> +               writeb(0x00, p + UART_EXAR_MPIOINT_15_8);
> +               writeb(0x00, p + UART_EXAR_MPIOLVL_15_8);
> +               writeb(0x00, p + UART_EXAR_MPIO3T_15_8);
> +               writeb(0x00, p + UART_EXAR_MPIOINV_15_8);
> +               writeb(0x00, p + UART_EXAR_MPIOSEL_15_8);
> +               writeb(0x00, p + UART_EXAR_MPIOOD_15_8);
> +       }
> +       writeb(0x00, p + UART_EXAR_8XMODE);
> +       writeb(UART_FCTR_EXAR_TRGD, p + UART_EXAR_FCTR);
> +       writeb(128, p + UART_EXAR_TXTRG);
> +       writeb(128, p + UART_EXAR_RXTRG);
> +       iounmap(p);
> +
> +       ret = pci_default_setup(priv, board, port, idx);
> +       if (ret)
> +               return ret;
> +
> +       if (idx == 0) {
> +               struct platform_device *device;
> +
> +               device = platform_device_alloc("gpio_exar",
> +                                              PLATFORM_DEVID_AUTO);
> +               if (!device)
> +                       return -ENOMEM;
> +
> +               if (platform_device_add(device) < 0) {
> +                       platform_device_put(device);
> +                       return -ENODEV;
> +               }
> +
> +               port->port.private_data = device;
> +               platform_set_drvdata(device, priv->dev);
> +       }
> +
> +       return 0;
> +}
> +
> +static void pci_xr17v35x_exit(struct pci_dev *dev)
> +{
> +       struct serial_private *priv = pci_get_drvdata(dev);
> +       struct uart_8250_port *port = serial8250_get_port(priv->line[0]);
> +       struct platform_device *pdev = port->port.private_data;
> +
> +       if (pdev) {
> +               platform_device_unregister(pdev);
> +               port->port.private_data = NULL;
> +       }
> +}
> +
> +#define PCI_DEVICE_ID_COMMTECH_4224PCIE        0x0020
> +#define PCI_DEVICE_ID_COMMTECH_4228PCIE        0x0021
> +#define PCI_DEVICE_ID_COMMTECH_4222PCIE        0x0022
> +
> +static struct pci_serial_quirk pci_serial_quirks[] __refdata = {
> +       /*
> +        * Exar cards
> +        */
> +       {
> +               .vendor = PCI_VENDOR_ID_EXAR,
> +               .device = PCI_DEVICE_ID_EXAR_XR17C152,
> +               .subvendor      = PCI_ANY_ID,
> +               .subdevice      = PCI_ANY_ID,
> +               .setup          = pci_xr17c154_setup,
> +       },

Those are not needed anymore. This is the point of split. Check how it
was done for 8250_mid.c and/or 8250_lpss.c.

> +       {
> +               .vendor = PCI_VENDOR_ID_EXAR,
> +               .device = PCI_DEVICE_ID_EXAR_XR17C154,
> +               .subvendor      = PCI_ANY_ID,
> +               .subdevice      = PCI_ANY_ID,
> +               .setup          = pci_xr17c154_setup,
> +       },
> +       {
> +               .vendor = PCI_VENDOR_ID_EXAR,
> +               .device = PCI_DEVICE_ID_EXAR_XR17C158,
> +               .subvendor      = PCI_ANY_ID,
> +               .subdevice      = PCI_ANY_ID,
> +               .setup          = pci_xr17c154_setup,
> +       },
> +       {
> +               .vendor = PCI_VENDOR_ID_EXAR,
> +               .device = PCI_DEVICE_ID_EXAR_XR17V352,
> +               .subvendor      = PCI_ANY_ID,
> +               .subdevice      = PCI_ANY_ID,
> +               .setup          = pci_xr17v35x_setup,
> +               .exit           = pci_xr17v35x_exit,
> +       },
> +       {
> +               .vendor = PCI_VENDOR_ID_EXAR,
> +               .device = PCI_DEVICE_ID_EXAR_XR17V354,
> +               .subvendor      = PCI_ANY_ID,
> +               .subdevice      = PCI_ANY_ID,
> +               .setup          = pci_xr17v35x_setup,
> +               .exit           = pci_xr17v35x_exit,
> +       },
> +       {
> +               .vendor = PCI_VENDOR_ID_EXAR,
> +               .device = PCI_DEVICE_ID_EXAR_XR17V358,
> +               .subvendor      = PCI_ANY_ID,
> +               .subdevice      = PCI_ANY_ID,
> +               .setup          = pci_xr17v35x_setup,
> +               .exit           = pci_xr17v35x_exit,
> +       },
> +       {
> +               .vendor = PCI_VENDOR_ID_EXAR,
> +               .device = PCI_DEVICE_ID_EXAR_XR17V4358,
> +               .subvendor      = PCI_ANY_ID,
> +               .subdevice      = PCI_ANY_ID,
> +               .setup          = pci_xr17v35x_setup,
> +               .exit           = pci_xr17v35x_exit,
> +       },
> +       {
> +               .vendor = PCI_VENDOR_ID_EXAR,
> +               .device = PCI_DEVICE_ID_EXAR_XR17V8358,
> +               .subvendor      = PCI_ANY_ID,
> +               .subdevice      = PCI_ANY_ID,
> +               .setup          = pci_xr17v35x_setup,
> +               .exit           = pci_xr17v35x_exit,
> +       },
> +};
> +
> +static inline int quirk_id_matches(u32 quirk_id, u32 dev_id)
> +{
> +       return quirk_id == PCI_ANY_ID || quirk_id == dev_id;
> +}
> +
> +static struct pci_serial_quirk *find_quirk(struct pci_dev *dev)
> +{
> +       struct pci_serial_quirk *quirk;
> +
> +       for (quirk = pci_serial_quirks; ; quirk++)
> +               if (quirk_id_matches(quirk->vendor, dev->vendor) &&
> +                   quirk_id_matches(quirk->device, dev->device) &&
> +                   quirk_id_matches(quirk->subvendor, dev->subsystem_vendor) &&
> +                   quirk_id_matches(quirk->subdevice, dev->subsystem_device))
> +                       break;
> +       return quirk;
> +}
> +
> +static inline int get_pci_irq(struct pci_dev *dev,
> +                             const struct pciserial_board *board)
> +{
> +       if (board->flags & FL_NOIRQ)
> +               return 0;
> +       else
> +               return dev->irq;
> +}
> +
> +/*
> + * This is the configuration table for all of the PCI serial boards
> + * which we support.  It is directly indexed by the pci_board_num_t enum
> + * value, which is encoded in the pci_device_id PCI probe table's
> + * driver_data member.
> + *
> + * The makeup of these names are:
> + *  pbn_bn{_bt}_n_baud{_offsetinhex}
> + *
> + *  bn         = PCI BAR number
> + *  bt         = Index using PCI BARs
> + *  n          = number of serial ports
> + *  baud       = baud rate
> + *  offsetinhex        = offset for each sequential port (in hex)
> + *
> + * This table is sorted by (in order): bn, bt, baud, offsetindex, n.
> + *
> + * Please note: in theory if n = 1, _bt infix should make no difference.
> + * ie, pbn_b0_1_115200 is the same as pbn_b0_bt_1_115200
> + */
> +enum pci_board_num_t {
> +       pbn_b0_2_1843200_200 = 0,
> +       pbn_b0_4_1843200_200,
> +       pbn_b0_8_1843200_200,
> +
> +       /*
> +        * Board-specific versions.
> +        */
> +       pbn_exar_XR17C152,
> +       pbn_exar_XR17C154,
> +       pbn_exar_XR17C158,
> +       pbn_exar_XR17V352,
> +       pbn_exar_XR17V354,
> +       pbn_exar_XR17V358,
> +       pbn_exar_XR17V4358,
> +       pbn_exar_XR17V8358,
> +       pbn_exar_ibm_saturn,
> +};
> +
> +/*
> + * uart_offset - the space between channels
> + * reg_shift   - describes how the UART registers are mapped
> + *               to PCI memory by the card.
> + * For example IER register on SBS, Inc. PMC-OctPro is located at
> + * offset 0x10 from the UART base, while UART_IER is defined as 1
> + * in include/linux/serial_reg.h,
> + * see first lines of serial_in() and serial_out() in 8250.c
> + */
> +
> +static struct pciserial_board pci_boards[] = {
> +       [pbn_b0_2_1843200_200] = {
> +               .flags          = FL_BASE0,
> +               .num_ports      = 2,
> +               .base_baud      = 1843200,
> +               .uart_offset    = 0x200,
> +       },
> +       [pbn_b0_4_1843200_200] = {
> +               .flags          = FL_BASE0,
> +               .num_ports      = 4,
> +               .base_baud      = 1843200,
> +               .uart_offset    = 0x200,
> +       },
> +       [pbn_b0_8_1843200_200] = {
> +               .flags          = FL_BASE0,
> +               .num_ports      = 8,
> +               .base_baud      = 1843200,
> +               .uart_offset    = 0x200,
> +       },
> +       /*
> +        * Exar Corp. XR17C15[248] Dual/Quad/Octal UART
> +        *  Only basic 16550A support.
> +        *  XR17C15[24] are not tested, but they should work.
> +        */
> +       [pbn_exar_XR17C152] = {
> +               .flags          = FL_BASE0,
> +               .num_ports      = 2,
> +               .base_baud      = 921600,
> +               .uart_offset    = 0x200,
> +       },
> +       [pbn_exar_XR17C154] = {
> +               .flags          = FL_BASE0,
> +               .num_ports      = 4,
> +               .base_baud      = 921600,
> +               .uart_offset    = 0x200,
> +       },
> +       [pbn_exar_XR17C158] = {
> +               .flags          = FL_BASE0,
> +               .num_ports      = 8,
> +               .base_baud      = 921600,
> +               .uart_offset    = 0x200,
> +       },
> +       [pbn_exar_XR17V352] = {
> +               .flags          = FL_BASE0,
> +               .num_ports      = 2,
> +               .base_baud      = 7812500,
> +               .uart_offset    = 0x400,
> +               .reg_shift      = 0,
> +               .first_offset   = 0,
> +       },
> +       [pbn_exar_XR17V354] = {
> +               .flags          = FL_BASE0,
> +               .num_ports      = 4,
> +               .base_baud      = 7812500,
> +               .uart_offset    = 0x400,
> +               .reg_shift      = 0,
> +               .first_offset   = 0,
> +       },
> +       [pbn_exar_XR17V358] = {
> +               .flags          = FL_BASE0,
> +               .num_ports      = 8,
> +               .base_baud      = 7812500,
> +               .uart_offset    = 0x400,
> +               .reg_shift      = 0,
> +               .first_offset   = 0,
> +       },
> +       [pbn_exar_XR17V4358] = {
> +               .flags          = FL_BASE0,
> +               .num_ports      = 12,
> +               .base_baud      = 7812500,
> +               .uart_offset    = 0x400,
> +               .reg_shift      = 0,
> +               .first_offset   = 0,
> +       },
> +       [pbn_exar_XR17V8358] = {
> +               .flags          = FL_BASE0,
> +               .num_ports      = 16,
> +               .base_baud      = 7812500,
> +               .uart_offset    = 0x400,
> +               .reg_shift      = 0,
> +               .first_offset   = 0,
> +       },
> +       [pbn_exar_ibm_saturn] = {
> +               .flags          = FL_BASE0,
> +               .num_ports      = 1,
> +               .base_baud      = 921600,
> +               .uart_offset    = 0x200,
> +       },

Similar here. It might be reduced.

> +};
> +
> +static struct serial_private *
> +init_ports(struct pci_dev *dev, const struct pciserial_board *board)
> +{
> +       struct uart_8250_port uart;
> +       struct serial_private *priv;
> +       struct pci_serial_quirk *quirk;
> +       int nr_ports, i;
> +
> +       nr_ports = board->num_ports;
> +
> +       /*
> +        * Find an init and setup quirks.
> +        */
> +       quirk = find_quirk(dev);
> +
> +       priv = kzalloc(sizeof(*priv) + sizeof(unsigned int) * nr_ports,
> +                      GFP_KERNEL);
> +       if (!priv) {
> +               priv = ERR_PTR(-ENOMEM);
> +               goto err_deinit;
> +       }
> +
> +       priv->dev = dev;
> +       priv->quirk = quirk;
> +
> +       memset(&uart, 0, sizeof(uart));
> +       uart.port.flags = UPF_SKIP_TEST | UPF_BOOT_AUTOCONF | UPF_SHARE_IRQ;
> +       uart.port.uartclk = board->base_baud * 16;
> +       uart.port.irq = dev->irq;
> +       uart.port.dev = &dev->dev;
> +
> +       for (i = 0; i < nr_ports; i++) {
> +               if (quirk->setup(priv, board, &uart, i))
> +                       break;
> +
> +               dev_dbg(&dev->dev, "Setup PCI port: port %lx, irq %d, type %d\n",
> +                       uart.port.iobase, uart.port.irq, uart.port.iotype);
> +
> +               priv->line[i] = serial8250_register_8250_port(&uart);
> +               if (priv->line[i] < 0) {
> +                       dev_err(&dev->dev,
> +                               "Couldn't register serial port %lx, irq %d, type %d, error %d\n",
> +                               uart.port.iobase, uart.port.irq,
> +                               uart.port.iotype, priv->line[i]);
> +                       break;
> +               }
> +       }
> +       priv->nr = i;
> +       return priv;
> +
> +err_deinit:
> +       if (quirk->exit)
> +               quirk->exit(dev);
> +       return priv;
> +}
> +
> +/*
> + * Probe one serial board.  Unfortunately, there is no rhyme nor reason
> + * to the arrangement of serial ports on a PCI card.
> + */
> +static int
> +exar_pci_init(struct pci_dev *dev, const struct pci_device_id *ent)
> +{
> +       struct serial_private *priv;
> +       const struct pciserial_board *board;
> +       int rc;
> +
> +       if (ent->driver_data >= ARRAY_SIZE(pci_boards)) {
> +               dev_err(&dev->dev, "invalid driver_data: %ld\n",
> +                       ent->driver_data);
> +               return -EINVAL;
> +       }
> +
> +       board = &pci_boards[ent->driver_data];
> +
> +       rc = pcim_enable_device(dev);
> +       pci_save_state(dev);
> +       if (rc)
> +               return rc;
> +
> +       priv = init_ports(dev, board);
> +       if (IS_ERR(priv))
> +               return PTR_ERR(priv);
> +
> +       pci_set_drvdata(dev, priv);
> +       return 0;

No need to copy'n'paste everything. Just minimize your stuff to do
exactly what is needed. I'm pretty sure you copied a lot of legacy
from 8250_pci.c.
Consider to reduce this module approximately twice by LOC.

> +}
> +
> +static void exar_pci_remove(struct pci_dev *dev)
> +{
> +       struct serial_private *priv = pci_get_drvdata(dev);
> +
> +       pciserial_remove_ports(priv);
> +}
> +

> +#ifdef CONFIG_PM_SLEEP
> +static int exar_suspend(struct device *dev)
> +{
> +       struct pci_dev *pdev = to_pci_dev(dev);
> +       struct serial_private *priv = pci_get_drvdata(pdev);
> +
> +       if (priv)
> +               pciserial_suspend_ports(priv);
> +
> +       return 0;
> +}
> +
> +static int exar_resume(struct device *dev)
> +{
> +       struct pci_dev *pdev = to_pci_dev(dev);
> +       struct serial_private *priv = pci_get_drvdata(pdev);
> +       int err;
> +
> +       if (priv) {
> +               /*
> +                * The device may have been disabled.  Re-enable it.
> +                */
> +               err = pci_enable_device(pdev);
> +               /* FIXME: We cannot simply error out here */
> +               if (err)
> +                       dev_err(dev, "Unable to re-enable ports, trying to continue.\n");
> +               pciserial_resume_ports(priv);
> +       }
> +       return 0;
> +}
> +#endif


> +
> +static SIMPLE_DEV_PM_OPS(exar_pci_pm, exar_suspend, exar_resume);
> +
> +static struct pci_device_id exar_pci_tbl[] = {
> +       {       PCI_VENDOR_ID_EXAR, PCI_DEVICE_ID_EXAR_XR17C152,
> +               PCI_SUBVENDOR_ID_CONNECT_TECH,
> +               PCI_SUBDEVICE_ID_CONNECT_TECH_PCI_UART_2_232, 0, 0,
> +               pbn_b0_2_1843200_200 },

You can do a common macro that helps for this. Check 8250_mid.c for example.

> +       {       PCI_VENDOR_ID_EXAR, PCI_DEVICE_ID_EXAR_XR17C154,
> +               PCI_SUBVENDOR_ID_CONNECT_TECH,
> +               PCI_SUBDEVICE_ID_CONNECT_TECH_PCI_UART_4_232, 0, 0,
> +               pbn_b0_4_1843200_200 },
> +       {       PCI_VENDOR_ID_EXAR, PCI_DEVICE_ID_EXAR_XR17C158,
> +               PCI_SUBVENDOR_ID_CONNECT_TECH,
> +               PCI_SUBDEVICE_ID_CONNECT_TECH_PCI_UART_8_232, 0, 0,
> +               pbn_b0_8_1843200_200 },
> +       {       PCI_VENDOR_ID_EXAR, PCI_DEVICE_ID_EXAR_XR17C152,
> +               PCI_SUBVENDOR_ID_CONNECT_TECH,
> +               PCI_SUBDEVICE_ID_CONNECT_TECH_PCI_UART_1_1, 0, 0,
> +               pbn_b0_2_1843200_200 },
> +       {       PCI_VENDOR_ID_EXAR, PCI_DEVICE_ID_EXAR_XR17C154,
> +               PCI_SUBVENDOR_ID_CONNECT_TECH,
> +               PCI_SUBDEVICE_ID_CONNECT_TECH_PCI_UART_2_2, 0, 0,
> +               pbn_b0_4_1843200_200 },
> +       {       PCI_VENDOR_ID_EXAR, PCI_DEVICE_ID_EXAR_XR17C158,
> +               PCI_SUBVENDOR_ID_CONNECT_TECH,
> +               PCI_SUBDEVICE_ID_CONNECT_TECH_PCI_UART_4_4, 0, 0,
> +               pbn_b0_8_1843200_200 },
> +       {       PCI_VENDOR_ID_EXAR, PCI_DEVICE_ID_EXAR_XR17C152,
> +               PCI_SUBVENDOR_ID_CONNECT_TECH,
> +               PCI_SUBDEVICE_ID_CONNECT_TECH_PCI_UART_2, 0, 0,
> +               pbn_b0_2_1843200_200 },
> +       {       PCI_VENDOR_ID_EXAR, PCI_DEVICE_ID_EXAR_XR17C154,
> +               PCI_SUBVENDOR_ID_CONNECT_TECH,
> +               PCI_SUBDEVICE_ID_CONNECT_TECH_PCI_UART_4, 0, 0,
> +               pbn_b0_4_1843200_200 },
> +       {       PCI_VENDOR_ID_EXAR, PCI_DEVICE_ID_EXAR_XR17C158,
> +               PCI_SUBVENDOR_ID_CONNECT_TECH,
> +               PCI_SUBDEVICE_ID_CONNECT_TECH_PCI_UART_8, 0, 0,
> +               pbn_b0_8_1843200_200 },
> +       {       PCI_VENDOR_ID_EXAR, PCI_DEVICE_ID_EXAR_XR17C152,
> +               PCI_SUBVENDOR_ID_CONNECT_TECH,
> +               PCI_SUBDEVICE_ID_CONNECT_TECH_PCI_UART_2_485, 0, 0,
> +               pbn_b0_2_1843200_200 },
> +       {       PCI_VENDOR_ID_EXAR, PCI_DEVICE_ID_EXAR_XR17C154,
> +               PCI_SUBVENDOR_ID_CONNECT_TECH,
> +               PCI_SUBDEVICE_ID_CONNECT_TECH_PCI_UART_4_485, 0, 0,
> +               pbn_b0_4_1843200_200 },
> +       {       PCI_VENDOR_ID_EXAR, PCI_DEVICE_ID_EXAR_XR17C158,
> +               PCI_SUBVENDOR_ID_CONNECT_TECH,
> +               PCI_SUBDEVICE_ID_CONNECT_TECH_PCI_UART_8_485, 0, 0,
> +               pbn_b0_8_1843200_200 },
> +       {       PCI_VENDOR_ID_EXAR, PCI_DEVICE_ID_EXAR_XR17C152,
> +               PCI_VENDOR_ID_IBM, PCI_SUBDEVICE_ID_IBM_SATURN_SERIAL_ONE_PORT,
> +               0, 0, pbn_exar_ibm_saturn },
> +       /*
> +        * Exar Corp. XR17C15[248] Dual/Quad/Octal UART
> +        */
> +       {       PCI_VENDOR_ID_EXAR, PCI_DEVICE_ID_EXAR_XR17C152,
> +               PCI_ANY_ID, PCI_ANY_ID,
> +               0,
> +               0, pbn_exar_XR17C152 },
> +       {       PCI_VENDOR_ID_EXAR, PCI_DEVICE_ID_EXAR_XR17C154,
> +               PCI_ANY_ID, PCI_ANY_ID,
> +               0,
> +               0, pbn_exar_XR17C154 },
> +       {       PCI_VENDOR_ID_EXAR, PCI_DEVICE_ID_EXAR_XR17C158,
> +               PCI_ANY_ID, PCI_ANY_ID,
> +               0,
> +               0, pbn_exar_XR17C158 },
> +       /*
> +        * Exar Corp. XR17V[48]35[248] Dual/Quad/Octal/Hexa PCIe UARTs
> +        */
> +       {       PCI_VENDOR_ID_EXAR, PCI_DEVICE_ID_EXAR_XR17V352,
> +               PCI_ANY_ID, PCI_ANY_ID,
> +               0,
> +               0, pbn_exar_XR17V352 },
> +       {       PCI_VENDOR_ID_EXAR, PCI_DEVICE_ID_EXAR_XR17V354,
> +               PCI_ANY_ID, PCI_ANY_ID,
> +               0,
> +               0, pbn_exar_XR17V354 },
> +       {       PCI_VENDOR_ID_EXAR, PCI_DEVICE_ID_EXAR_XR17V358,
> +               PCI_ANY_ID, PCI_ANY_ID,
> +               0,
> +               0, pbn_exar_XR17V358 },
> +       {       PCI_VENDOR_ID_EXAR, PCI_DEVICE_ID_EXAR_XR17V4358,
> +               PCI_ANY_ID, PCI_ANY_ID,
> +               0,
> +               0, pbn_exar_XR17V4358 },
> +       {       PCI_VENDOR_ID_EXAR, PCI_DEVICE_ID_EXAR_XR17V8358,
> +               PCI_ANY_ID, PCI_ANY_ID,
> +               0,
> +               0, pbn_exar_XR17V8358 },
> +       {       PCI_VENDOR_ID_COMMTECH, PCI_DEVICE_ID_COMMTECH_4222PCIE,
> +               PCI_ANY_ID, PCI_ANY_ID,
> +               0,
> +               0, pbn_exar_XR17V352 },
> +       {       PCI_VENDOR_ID_COMMTECH, PCI_DEVICE_ID_COMMTECH_4224PCIE,
> +               PCI_ANY_ID, PCI_ANY_ID,
> +               0,
> +               0, pbn_exar_XR17V354 },
> +       {       PCI_VENDOR_ID_COMMTECH, PCI_DEVICE_ID_COMMTECH_4228PCIE,
> +               PCI_ANY_ID, PCI_ANY_ID,
> +               0,
> +               0, pbn_exar_XR17V358 },
> +       { 0, }
> +};

-- 
With Best Regards,
Andy Shevchenko
--
To unsubscribe from this list: send the line "unsubscribe linux-serial" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Kernel Newbies]     [Security]     [Netfilter]     [Bugtraq]     [Linux PPP]     [Linux FS]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Linmodem]     [Device Mapper]     [Linux Kernel for ARM]

  Powered by Linux