USB: serial: ports: add minor and port number
authorGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 6 Jun 2013 17:32:00 +0000 (10:32 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 10 Jun 2013 21:46:40 +0000 (14:46 -0700)
The usb_serial_port structure had the number field, which was the minor
number for the port, which almost no one really cared about.  They
really wanted the number of the port within the device, which you had to
subtract from the minor of the parent usb_serial_device structure.  To
clean this up, provide the real minor number of the port, and the number
of the port within the serial device separately, as these numbers might
not be related in the future.

Bonus is that this cleans up a lot of logic in the drivers, and saves
lines overall.

Tested-by: Tobias Winter <tobias@linuxdingsda.de>
Reviewed-by: Johan Hovold <jhovold@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
--
 drivers/staging/serqt_usb2/serqt_usb2.c |   21 +++--------
 drivers/usb/serial/ark3116.c            |    2 -
 drivers/usb/serial/bus.c                |    6 +--
 drivers/usb/serial/console.c            |    2 -
 drivers/usb/serial/cp210x.c             |    2 -
 drivers/usb/serial/cypress_m8.c         |    4 +-
 drivers/usb/serial/digi_acceleport.c    |    6 ---
 drivers/usb/serial/f81232.c             |    5 +-
 drivers/usb/serial/garmin_gps.c         |    6 +--
 drivers/usb/serial/io_edgeport.c        |   58 ++++++++++++--------------------
 drivers/usb/serial/io_ti.c              |   21 ++++-------
 drivers/usb/serial/keyspan.c            |   29 +++++++---------
 drivers/usb/serial/metro-usb.c          |    4 +-
 drivers/usb/serial/mos7720.c            |   37 +++++++++-----------
 drivers/usb/serial/mos7840.c            |   52 +++++++++-------------------
 drivers/usb/serial/opticon.c            |    2 -
 drivers/usb/serial/pl2303.c             |    2 -
 drivers/usb/serial/quatech2.c           |    7 +--
 drivers/usb/serial/sierra.c             |    2 -
 drivers/usb/serial/ti_usb_3410_5052.c   |   10 ++---
 drivers/usb/serial/usb-serial.c         |    7 ++-
 drivers/usb/serial/usb_wwan.c           |    2 -
 drivers/usb/serial/whiteheat.c          |   20 +++++------
 include/linux/usb/serial.h              |    6 ++-
 24 files changed, 133 insertions(+), 180 deletions(-)

24 files changed:
drivers/staging/serqt_usb2/serqt_usb2.c
drivers/usb/serial/ark3116.c
drivers/usb/serial/bus.c
drivers/usb/serial/console.c
drivers/usb/serial/cp210x.c
drivers/usb/serial/cypress_m8.c
drivers/usb/serial/digi_acceleport.c
drivers/usb/serial/f81232.c
drivers/usb/serial/garmin_gps.c
drivers/usb/serial/io_edgeport.c
drivers/usb/serial/io_ti.c
drivers/usb/serial/keyspan.c
drivers/usb/serial/metro-usb.c
drivers/usb/serial/mos7720.c
drivers/usb/serial/mos7840.c
drivers/usb/serial/opticon.c
drivers/usb/serial/pl2303.c
drivers/usb/serial/quatech2.c
drivers/usb/serial/sierra.c
drivers/usb/serial/ti_usb_3410_5052.c
drivers/usb/serial/usb-serial.c
drivers/usb/serial/usb_wwan.c
drivers/usb/serial/whiteheat.c
include/linux/usb/serial.h

index 8a6e5ea476e1cf59f123742372bcb6a8a10425eb..4f891dba2faa6858b48caeb77c526fdfb1d85988 100644 (file)
@@ -873,7 +873,7 @@ static int qt_open(struct tty_struct *tty,
        result = qt_get_device(serial, &port0->DeviceData);
 
        /* Port specific setups */
-       result = qt_open_channel(serial, port->number, &ChannelData);
+       result = qt_open_channel(serial, port->port_number, &ChannelData);
        if (result < 0) {
                dev_dbg(&port->dev, "qt_open_channel failed\n");
                return result;
@@ -888,7 +888,7 @@ static int qt_open(struct tty_struct *tty,
            (SERIAL_MSR_CTS | SERIAL_MSR_DSR | SERIAL_MSR_RI | SERIAL_MSR_CD);
 
        /* Set Baud rate to default and turn off (default)flow control here */
-       result = qt_setuart(serial, port->number, DEFAULT_DIVISOR, DEFAULT_LCR);
+       result = qt_setuart(serial, port->port_number, DEFAULT_DIVISOR, DEFAULT_LCR);
        if (result < 0) {
                dev_dbg(&port->dev, "qt_setuart failed\n");
                return result;
@@ -906,7 +906,6 @@ static int qt_open(struct tty_struct *tty,
                        qt_submit_urb_from_open(serial, port);
        }
 
-       dev_dbg(&port->dev, "port number is %d\n", port->number);
        dev_dbg(&port->dev, "serial number is %d\n", port->serial->minor);
        dev_dbg(&port->dev,
                "Bulkin endpoint is %d\n", port->bulk_in_endpointAddress);
@@ -1022,14 +1021,11 @@ static void qt_close(struct usb_serial_port *port)
        /* Close uart channel */
        status = qt_close_channel(serial, index);
        if (status < 0)
-               dev_dbg(&port->dev,
-                       "%s - port %d qt_close_channel failed.\n",
-                       __func__, port->number);
+               dev_dbg(&port->dev, "%s - qt_close_channel failed.\n", __func__);
 
        port0->open_ports--;
 
-       dev_dbg(&port->dev, "qt_num_open_ports in close%d:in port%d\n",
-               port0->open_ports, port->number);
+       dev_dbg(&port->dev, "qt_num_open_ports in close%d\n", port0->open_ports);
 
        if (port0->open_ports == 0) {
                if (serial->port[0]->interrupt_in_urb) {
@@ -1169,8 +1165,7 @@ static int qt_ioctl(struct tty_struct *tty,
                return 0;
        }
 
-       dev_dbg(&port->dev, "%s -No ioctl for that one.  port = %d\n",
-               __func__, port->number);
+       dev_dbg(&port->dev, "%s -No ioctl for that one.\n", __func__);
        return -ENOIOCTLCMD;
 }
 
@@ -1245,8 +1240,7 @@ static void qt_set_termios(struct tty_struct *tty,
 
        /* Now determine flow control */
        if (cflag & CRTSCTS) {
-               dev_dbg(&port->dev, "%s - Enabling HW flow control port %d\n",
-                       __func__, port->number);
+               dev_dbg(&port->dev, "%s - Enabling HW flow control\n", __func__);
 
                /* Enable RTS/CTS flow control */
                status = BoxSetHW_FlowCtrl(port->serial, index, 1);
@@ -1258,8 +1252,7 @@ static void qt_set_termios(struct tty_struct *tty,
        } else {
                /* Disable RTS/CTS flow control */
                dev_dbg(&port->dev,
-                       "%s - disabling HW flow control port %d\n",
-                       __func__, port->number);
+                       "%s - disabling HW flow control\n", __func__);
 
                status = BoxSetHW_FlowCtrl(port->serial, index, 0);
                if (status < 0) {
index 40e7fd94646f4bfbad247c486bc8659dad6d8f0a..293a7706ba3f1fb307181a4798a216130de801ef 100644 (file)
@@ -414,7 +414,7 @@ static int ark3116_ioctl(struct tty_struct *tty,
                memset(&serstruct, 0, sizeof(serstruct));
                serstruct.type = PORT_16654;
                serstruct.line = port->serial->minor;
-               serstruct.port = port->number;
+               serstruct.port = port->port_number;
                serstruct.custom_divisor = 0;
                serstruct.baud_base = 460800;
 
index 3c4db6d196c630257611883e9f8f7ec46f5c20b6..f053b302a00d543e83fe6562a995ff9960c4e9af 100644 (file)
@@ -43,7 +43,7 @@ static ssize_t show_port_number(struct device *dev,
 {
        struct usb_serial_port *port = to_usb_serial_port(dev);
 
-       return sprintf(buf, "%d\n", port->number - port->serial->minor);
+       return sprintf(buf, "%d\n", port->port_number);
 }
 
 static DEVICE_ATTR(port_number, S_IRUGO, show_port_number, NULL);
@@ -80,7 +80,7 @@ static int usb_serial_device_probe(struct device *dev)
                goto exit_with_autopm;
        }
 
-       minor = port->number;
+       minor = port->minor;
        tty_register_device(usb_serial_tty_driver, minor, dev);
        dev_info(&port->serial->dev->dev,
                 "%s converter now attached to ttyUSB%d\n",
@@ -106,7 +106,7 @@ static int usb_serial_device_remove(struct device *dev)
        /* make sure suspend/resume doesn't race against port_remove */
        usb_autopm_get_interface(port->serial->interface);
 
-       minor = port->number;
+       minor = port->minor;
        tty_unregister_device(usb_serial_tty_driver, minor);
 
        device_remove_file(&port->dev, &dev_attr_port_number);
index 5f3bcd31e2045b89a821e1bfe10814d6089534ca..1b811022f1a199c1722276185f4022491e1c9f90 100644 (file)
@@ -210,7 +210,7 @@ static void usb_console_write(struct console *co,
        if (count == 0)
                return;
 
-       pr_debug("%s - port %d, %d byte(s)\n", __func__, port->number, count);
+       pr_debug("%s - minor %d, %d byte(s)\n", __func__, port->minor, count);
 
        if (!port->port.console) {
                pr_debug("%s - port not opened\n", __func__);
index 2c659553c07c01df5dc1a65be8341809939fc6f4..d6ef2f8da37dca2f7206f26db7d6269bdf0d13e6 100644 (file)
@@ -666,8 +666,6 @@ static void cp210x_set_termios(struct tty_struct *tty,
        unsigned int bits;
        unsigned int modem_ctl[4];
 
-       dev_dbg(dev, "%s - port %d\n", __func__, port->number);
-
        if (!tty)
                return;
 
index 082120198f870b2104d5230a5f1274803bc1074f..e948dc02795d4ede34c7c132d9fb03a1142cf0f5 100644 (file)
@@ -435,7 +435,7 @@ static void cypress_set_dead(struct usb_serial_port *port)
        spin_unlock_irqrestore(&priv->lock, flags);
 
        dev_err(&port->dev, "cypress_m8 suspending failing port %d - "
-               "interval might be too short\n", port->number);
+               "interval might be too short\n", port->port_number);
 }
 
 
@@ -667,7 +667,7 @@ static int cypress_write(struct tty_struct *tty, struct usb_serial_port *port,
 {
        struct cypress_private *priv = usb_get_serial_port_data(port);
 
-       dev_dbg(&port->dev, "%s - port %d, %d bytes\n", __func__, port->number, count);
+       dev_dbg(&port->dev, "%s - %d bytes\n", __func__, count);
 
        /* line control commands, which need to be executed immediately,
           are not put into the buffer for obvious reasons.
index 7b807d38952783b43854370590e0cc9db628ae49..19b467fe03886ec5a16d1a069982857bf3074507 100644 (file)
@@ -1304,11 +1304,7 @@ static void digi_release(struct usb_serial *serial)
 
 static int digi_port_probe(struct usb_serial_port *port)
 {
-       unsigned port_num;
-
-       port_num = port->number - port->serial->minor;
-
-       return digi_port_init(port, port_num);
+       return digi_port_init(port, port->port_number);
 }
 
 static int digi_port_remove(struct usb_serial_port *port)
index 090b411d893f8b957d816eb6b45bd599948ea055..350dfcb5fa62c8c8946ffbcdf4bf4686e13b6148 100644 (file)
@@ -288,15 +288,14 @@ static int f81232_ioctl(struct tty_struct *tty,
        struct serial_struct ser;
        struct usb_serial_port *port = tty->driver_data;
 
-       dev_dbg(&port->dev, "%s (%d) cmd = 0x%04x\n", __func__,
-               port->number, cmd);
+       dev_dbg(&port->dev, "%s cmd = 0x%04x\n", __func__, cmd);
 
        switch (cmd) {
        case TIOCGSERIAL:
                memset(&ser, 0, sizeof ser);
                ser.type = PORT_16654;
                ser.line = port->serial->minor;
-               ser.port = port->number;
+               ser.port = port->port_number;
                ser.baud_base = 460800;
 
                if (copy_to_user((void __user *)arg, &ser, sizeof ser))
index b110c573ea85319490922d8f3c538d507913c060..04b5ed90ffb2ded9739407e081e4dd4f52794e69 100644 (file)
@@ -948,9 +948,9 @@ static void garmin_close(struct usb_serial_port *port)
 {
        struct garmin_data *garmin_data_p = usb_get_serial_port_data(port);
 
-       dev_dbg(&port->dev, "%s - port %d - mode=%d state=%d flags=0x%X\n",
-               __func__, port->number, garmin_data_p->mode,
-               garmin_data_p->state, garmin_data_p->flags);
+       dev_dbg(&port->dev, "%s - mode=%d state=%d flags=0x%X\n",
+               __func__, garmin_data_p->mode, garmin_data_p->state,
+               garmin_data_p->flags);
 
        garmin_clear(garmin_data_p);
 
index 1477e8593476deb54a831e3169c00c42e705c50a..0c27ff3d2e8f9d3bfab7a249b146145f5f06efa5 100644 (file)
@@ -915,8 +915,8 @@ static int edge_open(struct tty_struct *tty, struct usb_serial_port *port)
                return -ENOMEM;
        }
 
-       dev_dbg(dev, "%s(%d) - Initialize TX fifo to %d bytes\n",
-               __func__, port->number, edge_port->maxTxCredits);
+       dev_dbg(dev, "%s - Initialize TX fifo to %d bytes\n",
+               __func__, edge_port->maxTxCredits);
 
        return 0;
 }
@@ -1122,9 +1122,8 @@ static int edge_write(struct tty_struct *tty, struct usb_serial_port *port,
        copySize = min((unsigned int)count,
                                (edge_port->txCredits - fifo->count));
 
-       dev_dbg(&port->dev, "%s(%d) of %d byte(s) Fifo room  %d -- will copy %d bytes\n",
-               __func__, port->number, count,
-                       edge_port->txCredits - fifo->count, copySize);
+       dev_dbg(&port->dev, "%s of %d byte(s) Fifo room  %d -- will copy %d bytes\n",
+               __func__, count, edge_port->txCredits - fifo->count, copySize);
 
        /* catch writes of 0 bytes which the tty driver likes to give us,
           and when txCredits is empty */
@@ -1216,9 +1215,8 @@ static void send_more_port_data(struct edgeport_serial *edge_serial,
        if (edge_port->write_in_progress ||
            !edge_port->open             ||
            (fifo->count == 0)) {
-               dev_dbg(dev, "%s(%d) EXIT - fifo %d, PendingWrite = %d\n",
-                       __func__, edge_port->port->number,
-                       fifo->count, edge_port->write_in_progress);
+               dev_dbg(dev, "%s EXIT - fifo %d, PendingWrite = %d\n",
+                       __func__, fifo->count, edge_port->write_in_progress);
                goto exit_send;
        }
 
@@ -1230,9 +1228,8 @@ static void send_more_port_data(struct edgeport_serial *edge_serial,
         * it's better to wait for more credits so we can do a larger write.
         */
        if (edge_port->txCredits < EDGE_FW_GET_TX_CREDITS_SEND_THRESHOLD(edge_port->maxTxCredits, EDGE_FW_BULK_MAX_PACKET_SIZE)) {
-               dev_dbg(dev, "%s(%d) Not enough credit - fifo %d TxCredit %d\n",
-                       __func__, edge_port->port->number, fifo->count,
-                       edge_port->txCredits);
+               dev_dbg(dev, "%s Not enough credit - fifo %d TxCredit %d\n",
+                       __func__, fifo->count, edge_port->txCredits);
                goto exit_send;
        }
 
@@ -1256,10 +1253,8 @@ static void send_more_port_data(struct edgeport_serial *edge_serial,
                edge_port->write_in_progress = false;
                goto exit_send;
        }
-       buffer[0] = IOSP_BUILD_DATA_HDR1(edge_port->port->number
-                               - edge_port->port->serial->minor, count);
-       buffer[1] = IOSP_BUILD_DATA_HDR2(edge_port->port->number
-                               - edge_port->port->serial->minor, count);
+       buffer[0] = IOSP_BUILD_DATA_HDR1(edge_port->port->port_number, count);
+       buffer[1] = IOSP_BUILD_DATA_HDR2(edge_port->port->port_number, count);
 
        /* now copy our data */
        bytesleft =  fifo->size - fifo->tail;
@@ -1377,8 +1372,7 @@ static int edge_chars_in_buffer(struct tty_struct *tty)
                                                edge_port->txfifo.count;
        spin_unlock_irqrestore(&edge_port->ep_lock, flags);
        if (num_chars) {
-               dev_dbg(&port->dev, "%s(port %d) - returns %d\n", __func__,
-                       port->number, num_chars);
+               dev_dbg(&port->dev, "%s - returns %d\n", __func__, num_chars);
        }
 
        return num_chars;
@@ -1576,7 +1570,7 @@ static int get_serial_info(struct edgeport_port *edge_port,
 
        tmp.type                = PORT_16550A;
        tmp.line                = edge_port->port->serial->minor;
-       tmp.port                = edge_port->port->number;
+       tmp.port                = edge_port->port->port_number;
        tmp.irq                 = 0;
        tmp.flags               = ASYNC_SKIP_TEST | ASYNC_AUTO_IRQ;
        tmp.xmit_fifo_size      = edge_port->maxTxCredits;
@@ -1601,15 +1595,15 @@ static int edge_ioctl(struct tty_struct *tty,
        DEFINE_WAIT(wait);
        struct edgeport_port *edge_port = usb_get_serial_port_data(port);
 
-       dev_dbg(&port->dev, "%s - port %d, cmd = 0x%x\n", __func__, port->number, cmd);
+       dev_dbg(&port->dev, "%s - cmd = 0x%x\n", __func__, cmd);
 
        switch (cmd) {
        case TIOCSERGETLSR:
-               dev_dbg(&port->dev, "%s (%d) TIOCSERGETLSR\n", __func__,  port->number);
+               dev_dbg(&port->dev, "%s TIOCSERGETLSR\n", __func__);
                return get_lsr_info(edge_port, (unsigned int __user *) arg);
 
        case TIOCGSERIAL:
-               dev_dbg(&port->dev, "%s (%d) TIOCGSERIAL\n", __func__,  port->number);
+               dev_dbg(&port->dev, "%s TIOCGSERIAL\n", __func__);
                return get_serial_info(edge_port, (struct serial_struct __user *) arg);
        }
        return -ENOIOCTLCMD;
@@ -2181,9 +2175,8 @@ static int send_iosp_ext_cmd(struct edgeport_port *edge_port,
 
        currentCommand = buffer;
 
-       MAKE_CMD_EXT_CMD(&currentCommand, &length,
-               edge_port->port->number - edge_port->port->serial->minor,
-               command, param);
+       MAKE_CMD_EXT_CMD(&currentCommand, &length, edge_port->port->port_number,
+                        command, param);
 
        status = write_cmd_usb(edge_port, buffer, length);
        if (status) {
@@ -2266,18 +2259,16 @@ static int send_cmd_write_baud_rate(struct edgeport_port *edge_port,
        int cmdLen = 0;
        int divisor;
        int status;
-       unsigned char number =
-               edge_port->port->number - edge_port->port->serial->minor;
+       u32 number = edge_port->port->port_number;
 
        if (edge_serial->is_epic &&
            !edge_serial->epic_descriptor.Supports.IOSPSetBaudRate) {
-               dev_dbg(dev, "SendCmdWriteBaudRate - NOT Setting baud rate for port = %d, baud = %d\n",
-                       edge_port->port->number, baudRate);
+               dev_dbg(dev, "SendCmdWriteBaudRate - NOT Setting baud rate for port, baud = %d\n",
+                       baudRate);
                return 0;
        }
 
-       dev_dbg(dev, "%s - port = %d, baud = %d\n", __func__,
-               edge_port->port->number, baudRate);
+       dev_dbg(dev, "%s - baud = %d\n", __func__, baudRate);
 
        status = calc_baud_rate_divisor(dev, baudRate, &divisor);
        if (status) {
@@ -2388,9 +2379,8 @@ static int send_cmd_write_uart_register(struct edgeport_port *edge_port,
        currCmd = cmdBuffer;
 
        /* Build a cmd in the buffer to write the given register */
-       MAKE_CMD_WRITE_REG(&currCmd, &cmdLen,
-               edge_port->port->number - edge_port->port->serial->minor,
-               regNum, regValue);
+       MAKE_CMD_WRITE_REG(&currCmd, &cmdLen, edge_port->port->port_number,
+                          regNum, regValue);
 
        status = write_cmd_usb(edge_port, cmdBuffer, cmdLen);
        if (status) {
@@ -2424,8 +2414,6 @@ static void change_port_settings(struct tty_struct *tty,
        __u8 txFlow;
        int status;
 
-       dev_dbg(dev, "%s - port %d\n", __func__, edge_port->port->number);
-
        if (!edge_port->open &&
            !edge_port->openPending) {
                dev_dbg(dev, "%s - port not opened\n", __func__);
index 1be6ba7bee27452ac55c08a06d98ff2394a2f7fb..d32bf2b7f988bdea38fc04e0e27fab1ccf57c9a3 100644 (file)
@@ -259,7 +259,7 @@ static int send_cmd(struct usb_device *dev, __u8 command,
 /* clear tx/rx buffers and fifo in TI UMP */
 static int purge_port(struct usb_serial_port *port, __u16 mask)
 {
-       int port_number = port->number - port->serial->minor;
+       int port_number = port->port_number;
 
        dev_dbg(&port->dev, "%s - port %d, mask %x\n", __func__, port_number, mask);
 
@@ -1392,7 +1392,8 @@ stayinbootmode:
 
 static int ti_do_config(struct edgeport_port *port, int feature, int on)
 {
-       int port_number = port->port->number - port->port->serial->minor;
+       int port_number = port->port->port_number;
+
        on = !!on;      /* 1 or 0 not bitmask */
        return send_cmd(port->port->serial->dev,
                        feature, (__u8)(UMPM_UART1_PORT + port_number),
@@ -1637,7 +1638,7 @@ static void edge_bulk_in_callback(struct urb *urb)
                return;
        }
 
-       port_number = edge_port->port->number - edge_port->port->serial->minor;
+       port_number = edge_port->port->port_number;
 
        if (edge_port->lsr_event) {
                edge_port->lsr_event = 0;
@@ -1730,7 +1731,7 @@ static int edge_open(struct tty_struct *tty, struct usb_serial_port *port)
        if (edge_port == NULL)
                return -ENODEV;
 
-       port_number = port->number - port->serial->minor;
+       port_number = port->port_number;
        switch (port_number) {
        case 0:
                edge_port->uart_base = UMPMEM_BASE_UART1;
@@ -1908,7 +1909,7 @@ static void edge_close(struct usb_serial_port *port)
        spin_unlock_irqrestore(&edge_port->ep_lock, flags);
 
        dev_dbg(&port->dev, "%s - send umpc_close_port\n", __func__);
-       port_number = port->number - port->serial->minor;
+       port_number = port->port_number;
        send_cmd(serial->dev, UMPC_CLOSE_PORT,
                     (__u8)(UMPM_UART1_PORT + port_number), 0, NULL, 0);
 
@@ -2137,10 +2138,7 @@ static void change_port_settings(struct tty_struct *tty,
        int baud;
        unsigned cflag;
        int status;
-       int port_number = edge_port->port->number -
-                                       edge_port->port->serial->minor;
-
-       dev_dbg(dev, "%s - port %d\n", __func__, edge_port->port->number);
+       int port_number = edge_port->port->port_number;
 
        config = kmalloc (sizeof (*config), GFP_KERNEL);
        if (!config) {
@@ -2284,7 +2282,6 @@ static void edge_set_termios(struct tty_struct *tty,
                tty->termios.c_cflag, tty->termios.c_iflag);
        dev_dbg(&port->dev, "%s - old clfag %08x old iflag %08x\n", __func__,
                old_termios->c_cflag, old_termios->c_iflag);
-       dev_dbg(&port->dev, "%s - port %d\n", __func__, port->number);
 
        if (edge_port == NULL)
                return;
@@ -2367,7 +2364,7 @@ static int get_serial_info(struct edgeport_port *edge_port,
 
        tmp.type                = PORT_16550A;
        tmp.line                = edge_port->port->serial->minor;
-       tmp.port                = edge_port->port->number;
+       tmp.port                = edge_port->port->port_number;
        tmp.irq                 = 0;
        tmp.flags               = ASYNC_SKIP_TEST | ASYNC_AUTO_IRQ;
        tmp.xmit_fifo_size      = edge_port->port->bulk_out_size;
@@ -2386,7 +2383,7 @@ static int edge_ioctl(struct tty_struct *tty,
        struct usb_serial_port *port = tty->driver_data;
        struct edgeport_port *edge_port = usb_get_serial_port_data(port);
 
-       dev_dbg(&port->dev, "%s - port %d, cmd = 0x%x\n", __func__, port->number, cmd);
+       dev_dbg(&port->dev, "%s - cmd = 0x%x\n", __func__, cmd);
 
        switch (cmd) {
        case TIOCGSERIAL:
index 28365e647168e73372175ec338be281ca9f3accb..5a979729f8ec4695a1fa4e4395b86a430184e1c9 100644 (file)
@@ -152,7 +152,7 @@ static void keyspan_set_termios(struct tty_struct *tty,
        p_priv = usb_get_serial_port_data(port);
        d_details = p_priv->device_details;
        cflag = tty->termios.c_cflag;
-       device_port = port->number - port->serial->minor;
+       device_port = port->port_number;
 
        /* Baud rate calculation takes baud rate as an integer
           so other rates can be generated if desired. */
@@ -234,8 +234,8 @@ static int keyspan_write(struct tty_struct *tty,
                dataOffset = 1;
        }
 
-       dev_dbg(&port->dev, "%s - for port %d (%d chars), flip=%d\n",
-               __func__, port->number, count, p_priv->out_flip);
+       dev_dbg(&port->dev, "%s - %d chars, flip=%d\n", __func__, count,
+               p_priv->out_flip);
 
        for (left = count; left > 0; left -= todo) {
                todo = left;
@@ -1041,7 +1041,7 @@ static int keyspan_open(struct tty_struct *tty, struct usb_serial_port *port)
        /* get the terminal config for the setup message now so we don't
         * need to send 2 of them */
 
-       device_port = port->number - port->serial->minor;
+       device_port = port->port_number;
        if (tty) {
                cflag = tty->termios.c_cflag;
                /* Baud rate calculation takes baud rate as an integer
@@ -1547,7 +1547,7 @@ static int keyspan_usa26_send_setup(struct usb_serial *serial,
        s_priv = usb_get_serial_data(serial);
        p_priv = usb_get_serial_port_data(port);
        d_details = s_priv->device_details;
-       device_port = port->number - port->serial->minor;
+       device_port = port->port_number;
 
        this_urb = p_priv->outcont_urb;
 
@@ -1691,7 +1691,7 @@ static int keyspan_usa28_send_setup(struct usb_serial *serial,
        s_priv = usb_get_serial_data(serial);
        p_priv = usb_get_serial_port_data(port);
        d_details = s_priv->device_details;
-       device_port = port->number - port->serial->minor;
+       device_port = port->port_number;
 
        /* only do something if we have a bulk out endpoint */
        this_urb = p_priv->outcont_urb;
@@ -1821,17 +1821,16 @@ static int keyspan_usa49_send_setup(struct usb_serial *serial,
        this_urb = s_priv->glocont_urb;
 
        /* Work out which port within the device is being setup */
-       device_port = port->number - port->serial->minor;
+       device_port = port->port_number;
 
        /* Make sure we have an urb then send the message */
        if (this_urb == NULL) {
-               dev_dbg(&port->dev, "%s - oops no urb for port %d.\n", __func__, port->number);
+               dev_dbg(&port->dev, "%s - oops no urb for port.\n", __func__);
                return -1;
        }
 
-       dev_dbg(&port->dev, "%s - endpoint %d port %d (%d)\n",
-               __func__, usb_pipeendpoint(this_urb->pipe),
-               port->number, device_port);
+       dev_dbg(&port->dev, "%s - endpoint %d (%d)\n",
+               __func__, usb_pipeendpoint(this_urb->pipe), device_port);
 
        /* Save reset port val for resend.
           Don't overwrite resend for open/close condition. */
@@ -1846,7 +1845,6 @@ static int keyspan_usa49_send_setup(struct usb_serial *serial,
 
        memset(&msg, 0, sizeof(struct keyspan_usa49_portControlMessage));
 
-       /*msg.portNumber = port->number;*/
        msg.portNumber = device_port;
 
        /* Only set baud rate if it's changed */
@@ -2136,12 +2134,11 @@ static int keyspan_usa67_send_setup(struct usb_serial *serial,
        this_urb = s_priv->glocont_urb;
 
        /* Work out which port within the device is being setup */
-       device_port = port->number - port->serial->minor;
+       device_port = port->port_number;
 
        /* Make sure we have an urb then send the message */
        if (this_urb == NULL) {
-               dev_dbg(&port->dev, "%s - oops no urb for port %d.\n", __func__,
-                       port->number);
+               dev_dbg(&port->dev, "%s - oops no urb for port.\n", __func__);
                return -1;
        }
 
@@ -2382,7 +2379,7 @@ static int keyspan_port_probe(struct usb_serial_port *port)
        /* Setup values for the various callback routines */
        cback = &keyspan_callbacks[d_details->msg_format];
 
-       port_num = port->number - port->serial->minor;
+       port_num = port->port_number;
 
        /* Do indat endpoints first, once for each flip */
        endp = d_details->indat_endpoints[port_num];
index 47e247759eb0ed975ddcca6482b34080525397d3..40ccf6e5e318bb8be43054d643af4376e4aa5a06 100644 (file)
@@ -224,8 +224,8 @@ static int metrousb_open(struct tty_struct *tty, struct usb_serial_port *port)
        result = metrousb_send_unidirectional_cmd(UNI_CMD_OPEN, port);
        if (result) {
                dev_err(&port->dev,
-                       "%s - failed to configure device for port number=%d, error code=%d\n",
-                       __func__, port->number, result);
+                       "%s - failed to configure device, error code=%d\n",
+                       __func__, result);
                goto exit;
        }
 
index f27c621a9297f3c896b84c9ea700d6285ef6b94f..f79ae7fe37ffaf6f93a5b49ddab493eeca7078ea 100644 (file)
@@ -1047,7 +1047,7 @@ static int mos7720_open(struct tty_struct *tty, struct usb_serial_port *port)
          *
          * 0x08 : SP1/2 Control Reg
          */
-       port_number = port->number - port->serial->minor;
+       port_number = port->port_number;
        read_mos_reg(serial, port_number, LSR, &data);
 
        dev_dbg(&port->dev, "SS::%p LSR:%x\n", mos7720_port, data);
@@ -1066,7 +1066,7 @@ static int mos7720_open(struct tty_struct *tty, struct usb_serial_port *port)
 
        write_mos_reg(serial, port_number, SP_CONTROL_REG, 0x00);
        read_mos_reg(serial, dummy, SP_CONTROL_REG, &data);
-       data = data | (port->number - port->serial->minor + 1);
+       data = data | (port->port_number + 1);
        write_mos_reg(serial, dummy, SP_CONTROL_REG, data);
        mos7720_port->shadowLCR = 0x83;
        write_mos_reg(serial, port_number, LCR, mos7720_port->shadowLCR);
@@ -1147,8 +1147,8 @@ static void mos7720_close(struct usb_serial_port *port)
        usb_kill_urb(port->write_urb);
        usb_kill_urb(port->read_urb);
 
-       write_mos_reg(serial, port->number - port->serial->minor, MCR, 0x00);
-       write_mos_reg(serial, port->number - port->serial->minor, IER, 0x00);
+       write_mos_reg(serial, port->port_number, MCR, 0x00);
+       write_mos_reg(serial, port->port_number, IER, 0x00);
 
        mos7720_port->open = 0;
 }
@@ -1172,8 +1172,7 @@ static void mos7720_break(struct tty_struct *tty, int break_state)
                data = mos7720_port->shadowLCR & ~UART_LCR_SBC;
 
        mos7720_port->shadowLCR  = data;
-       write_mos_reg(serial, port->number - port->serial->minor,
-                     LCR, mos7720_port->shadowLCR);
+       write_mos_reg(serial, port->port_number, LCR, mos7720_port->shadowLCR);
 }
 
 /*
@@ -1304,8 +1303,8 @@ static void mos7720_throttle(struct tty_struct *tty)
        /* if we are implementing RTS/CTS, toggle that line */
        if (tty->termios.c_cflag & CRTSCTS) {
                mos7720_port->shadowMCR &= ~UART_MCR_RTS;
-               write_mos_reg(port->serial, port->number - port->serial->minor,
-                             MCR, mos7720_port->shadowMCR);
+               write_mos_reg(port->serial, port->port_number, MCR,
+                             mos7720_port->shadowMCR);
                if (status != 0)
                        return;
        }
@@ -1336,8 +1335,8 @@ static void mos7720_unthrottle(struct tty_struct *tty)
        /* if we are implementing RTS/CTS, toggle that line */
        if (tty->termios.c_cflag & CRTSCTS) {
                mos7720_port->shadowMCR |= UART_MCR_RTS;
-               write_mos_reg(port->serial, port->number - port->serial->minor,
-                             MCR, mos7720_port->shadowMCR);
+               write_mos_reg(port->serial, port->port_number, MCR,
+                             mos7720_port->shadowMCR);
                if (status != 0)
                        return;
        }
@@ -1361,7 +1360,7 @@ static int set_higher_rates(struct moschip_port *mos7720_port,
         *      Init Sequence for higher rates
         ***********************************************/
        dev_dbg(&port->dev, "Sending Setting Commands ..........\n");
-       port_number = port->number - port->serial->minor;
+       port_number = port->port_number;
 
        write_mos_reg(serial, port_number, IER, 0x00);
        write_mos_reg(serial, port_number, FCR, 0x00);
@@ -1487,7 +1486,7 @@ static int send_cmd_write_baud_rate(struct moschip_port *mos7720_port,
        port = mos7720_port->port;
        serial = port->serial;
 
-       number = port->number - port->serial->minor;
+       number = port->port_number;
        dev_dbg(&port->dev, "%s - baud = %d\n", __func__, baudrate);
 
        /* Calculate the Divisor */
@@ -1538,7 +1537,7 @@ static void change_port_settings(struct tty_struct *tty,
 
        port = mos7720_port->port;
        serial = port->serial;
-       port_number = port->number - port->serial->minor;
+       port_number = port->port_number;
 
        if (!mos7720_port->open) {
                dev_dbg(&port->dev, "%s - port not opened\n", __func__);
@@ -1731,7 +1730,7 @@ static int get_lsr_info(struct tty_struct *tty,
        struct usb_serial_port *port = tty->driver_data;
        unsigned int result = 0;
        unsigned char data = 0;
-       int port_number = port->number - port->serial->minor;
+       int port_number = port->port_number;
        int count;
 
        count = mos7720_chars_in_buffer(tty);
@@ -1793,8 +1792,8 @@ static int mos7720_tiocmset(struct tty_struct *tty,
                mcr &= ~UART_MCR_LOOP;
 
        mos7720_port->shadowMCR = mcr;
-       write_mos_reg(port->serial, port->number - port->serial->minor,
-                     MCR, mos7720_port->shadowMCR);
+       write_mos_reg(port->serial, port->port_number, MCR,
+                     mos7720_port->shadowMCR);
 
        return 0;
 }
@@ -1838,8 +1837,8 @@ static int set_modem_info(struct moschip_port *mos7720_port, unsigned int cmd,
        }
 
        mos7720_port->shadowMCR = mcr;
-       write_mos_reg(port->serial, port->number - port->serial->minor,
-                     MCR, mos7720_port->shadowMCR);
+       write_mos_reg(port->serial, port->port_number, MCR,
+                     mos7720_port->shadowMCR);
 
        return 0;
 }
@@ -1856,7 +1855,7 @@ static int get_serial_info(struct moschip_port *mos7720_port,
 
        tmp.type                = PORT_16550A;
        tmp.line                = mos7720_port->port->serial->minor;
-       tmp.port                = mos7720_port->port->number;
+       tmp.port                = mos7720_port->port->port_number;
        tmp.irq                 = 0;
        tmp.flags               = ASYNC_SKIP_TEST | ASYNC_AUTO_IRQ;
        tmp.xmit_fifo_size      = NUM_URBS * URB_TRANSFER_BUFFER_SIZE;
index 7e998081e1cd9b42651143e7f01e63286410393d..f981b08ff32f209bf2ede4b7382ace3e939d1c1f 100644 (file)
@@ -303,15 +303,12 @@ static int mos7840_set_uart_reg(struct usb_serial_port *port, __u16 reg,
        /* For the UART control registers, the application number need
           to be Or'ed */
        if (port->serial->num_ports == 4) {
-               val |= (((__u16) port->number -
-                               (__u16) (port->serial->minor)) + 1) << 8;
+               val |= ((__u16)port->port_number + 1) << 8;
        } else {
-               if (((__u16) port->number - (__u16) (port->serial->minor)) == 0) {
-                       val |= (((__u16) port->number -
-                             (__u16) (port->serial->minor)) + 1) << 8;
+               if (port->port_number == 0) {
+                       val |= ((__u16)port->port_number + 1) << 8;
                } else {
-                       val |= (((__u16) port->number -
-                             (__u16) (port->serial->minor)) + 2) << 8;
+                       val |= ((__u16)port->port_number + 2) << 8;
                }
        }
        dev_dbg(&port->dev, "%s application number is %x\n", __func__, val);
@@ -340,16 +337,12 @@ static int mos7840_get_uart_reg(struct usb_serial_port *port, __u16 reg,
 
        /* Wval  is same as application number */
        if (port->serial->num_ports == 4) {
-               Wval =
-                   (((__u16) port->number - (__u16) (port->serial->minor)) +
-                    1) << 8;
+               Wval = ((__u16)port->port_number + 1) << 8;
        } else {
-               if (((__u16) port->number - (__u16) (port->serial->minor)) == 0) {
-                       Wval = (((__u16) port->number -
-                             (__u16) (port->serial->minor)) + 1) << 8;
+               if (port->port_number == 0) {
+                       Wval = ((__u16)port->port_number + 1) << 8;
                } else {
-                       Wval = (((__u16) port->number -
-                             (__u16) (port->serial->minor)) + 2) << 8;
+                       Wval = ((__u16)port->port_number + 2) << 8;
                }
        }
        dev_dbg(&port->dev, "%s application number is %x\n", __func__, Wval);
@@ -631,9 +624,7 @@ static void mos7840_interrupt_callback(struct urb *urb)
 
        for (i = 0; i < serial->num_ports; i++) {
                mos7840_port = mos7840_get_port_private(serial->port[i]);
-               wval =
-                   (((__u16) serial->port[i]->number -
-                     (__u16) (serial->minor)) + 1) << 8;
+               wval = ((__u16)serial->port[i]->port_number + 1) << 8;
                if (mos7840_port->open) {
                        if (sp[i] & 0x01) {
                                dev_dbg(&urb->dev->dev, "SP%d No Interrupt !!!\n", i);
@@ -1065,8 +1056,8 @@ static int mos7840_open(struct tty_struct *tty, struct usb_serial_port *port)
         * (can't set it up in mos7840_startup as the  *
         * structures were not set up at that time.)   */
 
-       dev_dbg(&port->dev, "port number is %d\n", port->number);
-       dev_dbg(&port->dev, "serial number is %d\n", port->serial->minor);
+       dev_dbg(&port->dev, "port number is %d\n", port->port_number);
+       dev_dbg(&port->dev, "minor number is %d\n", port->serial->minor);
        dev_dbg(&port->dev, "Bulkin endpoint is %d\n", port->bulk_in_endpointAddress);
        dev_dbg(&port->dev, "BulkOut endpoint is %d\n", port->bulk_out_endpointAddress);
        dev_dbg(&port->dev, "Interrupt endpoint is %d\n", port->interrupt_in_endpointAddress);
@@ -1074,9 +1065,7 @@ static int mos7840_open(struct tty_struct *tty, struct usb_serial_port *port)
        mos7840_port->read_urb = port->read_urb;
 
        /* set up our bulk in urb */
-       if ((serial->num_ports == 2)
-               && ((((__u16)port->number -
-                       (__u16)(port->serial->minor)) % 2) != 0)) {
+       if ((serial->num_ports == 2) && (((__u16)port->port_number % 2) != 0)) {
                usb_fill_bulk_urb(mos7840_port->read_urb,
                        serial->dev,
                        usb_rcvbulkpipe(serial->dev,
@@ -1199,7 +1188,7 @@ static void mos7840_close(struct usb_serial_port *port)
        mos7840_port->read_urb_busy = false;
 
        port0->open_ports--;
-       dev_dbg(&port->dev, "%s in close%d:in port%d\n", __func__, port0->open_ports, port->number);
+       dev_dbg(&port->dev, "%s in close%d\n", __func__, port0->open_ports);
        if (port0->open_ports == 0) {
                if (serial->port[0]->interrupt_in_urb) {
                        dev_dbg(&port->dev, "Shutdown interrupt_in_urb\n");
@@ -1435,9 +1424,7 @@ static int mos7840_write(struct tty_struct *tty, struct usb_serial_port *port,
        memcpy(urb->transfer_buffer, current_position, transfer_size);
 
        /* fill urb with data and submit  */
-       if ((serial->num_ports == 2)
-               && ((((__u16)port->number -
-                       (__u16)(port->serial->minor)) % 2) != 0)) {
+       if ((serial->num_ports == 2) && (((__u16)port->port_number % 2) != 0)) {
                usb_fill_bulk_urb(urb,
                        serial->dev,
                        usb_sndbulkpipe(serial->dev,
@@ -1732,10 +1719,9 @@ static int mos7840_send_cmd_write_baud_rate(struct moschip_port *mos7840_port,
        if (mos7840_serial_paranoia_check(port->serial, __func__))
                return -1;
 
-       number = mos7840_port->port->number - mos7840_port->port->serial->minor;
+       number = mos7840_port->port->port_number;
 
-       dev_dbg(&port->dev, "%s - port = %d, baud = %d\n", __func__,
-               mos7840_port->port->number, baudRate);
+       dev_dbg(&port->dev, "%s - baud = %d\n", __func__, baudRate);
        /* reset clk_uart_sel in spregOffset */
        if (baudRate > 115200) {
 #ifdef HW_flow_control
@@ -2016,7 +2002,6 @@ static void mos7840_set_termios(struct tty_struct *tty,
                tty->termios.c_cflag, RELEVANT_IFLAG(tty->termios.c_iflag));
        dev_dbg(&port->dev, "%s - old clfag %08x old iflag %08x\n", __func__,
                old_termios->c_cflag, RELEVANT_IFLAG(old_termios->c_iflag));
-       dev_dbg(&port->dev, "%s - port %d\n", __func__, port->number);
 
        /* change the port settings to the new ones specified */
 
@@ -2084,7 +2069,7 @@ static int mos7840_get_serial_info(struct moschip_port *mos7840_port,
 
        tmp.type = PORT_16550A;
        tmp.line = mos7840_port->port->serial->minor;
-       tmp.port = mos7840_port->port->number;
+       tmp.port = mos7840_port->port->port_number;
        tmp.irq = 0;
        tmp.flags = ASYNC_SKIP_TEST | ASYNC_AUTO_IRQ;
        tmp.xmit_fifo_size = NUM_URBS * URB_TRANSFER_BUFFER_SIZE;
@@ -2240,7 +2225,7 @@ static int mos7840_port_probe(struct usb_serial_port *port)
        /* we set up the pointers to the endpoints in the mos7840_open *
         * function, as the structures aren't created yet.             */
 
-       pnum = port->number - serial->minor;
+       pnum = port->port_number;
 
        dev_dbg(&port->dev, "mos7840_startup: configuring port %d\n", pnum);
        mos7840_port = kzalloc(sizeof(struct moschip_port), GFP_KERNEL);
@@ -2261,7 +2246,6 @@ static int mos7840_port_probe(struct usb_serial_port *port)
         * usb-serial.c:get_free_serial() and cannot therefore be used
         * to index device instances */
        mos7840_port->port_num = pnum + 1;
-       dev_dbg(&port->dev, "port->number = %d\n", port->number);
        dev_dbg(&port->dev, "port->serial->minor = %d\n", port->serial->minor);
        dev_dbg(&port->dev, "mos7840_port->port_num = %d\n", mos7840_port->port_num);
        dev_dbg(&port->dev, "serial->minor = %d\n", serial->minor);
index 5f4b0cd0f6e9734193dac84ba23373f13e301d65..6e1ee85e44f26f9e01b06912ead0775f5592615e 100644 (file)
@@ -367,7 +367,7 @@ static int opticon_ioctl(struct tty_struct *tty,
 {
        struct usb_serial_port *port = tty->driver_data;
 
-       dev_dbg(&port->dev, "%s - port %d, cmd = 0x%x\n", __func__, port->number, cmd);
+       dev_dbg(&port->dev, "%s - cmd = 0x%x\n", __func__, cmd);
 
        switch (cmd) {
        case TIOCGSERIAL:
index 97c1e6a9dd4f8961f28a6fa61c9d152d9cacf219..966c20c9ced4f34d4769eba16435f39e1209eebd 100644 (file)
@@ -640,7 +640,7 @@ static int pl2303_ioctl(struct tty_struct *tty,
                memset(&ser, 0, sizeof ser);
                ser.type = PORT_16654;
                ser.line = port->serial->minor;
-               ser.port = port->number;
+               ser.port = port->port_number;
                ser.baud_base = 460800;
 
                if (copy_to_user((void __user *)arg, &ser, sizeof ser))
index b0a9478a8c9e408bb4b9a1dba59d9bd730a1c4b2..f2ca7d80c8a0301126c2b358e564647da3ec2439 100644 (file)
@@ -343,7 +343,7 @@ static int qt2_open(struct tty_struct *tty, struct usb_serial_port *port)
        int status;
        unsigned long flags;
 
-       device_port = (u16) (port->number - port->serial->minor);
+       device_port = port->port_number;
 
        serial = port->serial;
 
@@ -388,9 +388,8 @@ static int qt2_open(struct tty_struct *tty, struct usb_serial_port *port)
        status = qt2_set_port_config(serial->dev, device_port,
                                     DEFAULT_BAUD_RATE, UART_LCR_WLEN8);
        if (status < 0) {
-               dev_err(&port->dev,
-                       "%s - initial setup failed for port %i (%i)\n",
-                       __func__, port->number, device_port);
+               dev_err(&port->dev, "%s - initial setup failed (%i)\n",
+                       __func__, device_port);
                return status;
        }
 
index 8894665cd6102cfe756ea2a86f46176010483444..de958c5b52e33ea5b7c3ff23efe42d33322bdeb4 100644 (file)
@@ -914,7 +914,7 @@ static int sierra_port_probe(struct usb_serial_port *port)
                /* This is really the usb-serial port number of the interface
                 * rather than the interface number.
                 */
-               ifnum = port->number - serial->minor;
+               ifnum = port->port_number;
                himemoryp = &typeA_interface_list;
        }
 
index c92c5ed4e580ec6761d0acabc379cfe667faaeb1..07e5c9bec48a1c57674de41f46df7bdc9c51f45f 100644 (file)
@@ -476,7 +476,7 @@ static int ti_open(struct tty_struct *tty, struct usb_serial_port *port)
        if (mutex_lock_interruptible(&tdev->td_open_close_lock))
                return -ERESTARTSYS;
 
-       port_number = port->number - port->serial->minor;
+       port_number = port->port_number;
 
        tport->tp_msr = 0;
        tport->tp_shadow_mcr |= (TI_MCR_RTS | TI_MCR_DTR);
@@ -618,7 +618,7 @@ static void ti_close(struct usb_serial_port *port)
        kfifo_reset_out(&tport->write_fifo);
        spin_unlock_irqrestore(&tport->tp_lock, flags);
 
-       port_number = port->number - port->serial->minor;
+       port_number = port->port_number;
 
        dev_dbg(&port->dev, "%s - sending TI_CLOSE_PORT\n", __func__);
        status = ti_command_out_sync(tdev, TI_CLOSE_PORT,
@@ -776,7 +776,7 @@ static void ti_set_termios(struct tty_struct *tty,
        tcflag_t cflag, iflag;
        int baud;
        int status;
-       int port_number = port->number - port->serial->minor;
+       int port_number = port->port_number;
        unsigned int mcr;
 
        cflag = tty->termios.c_cflag;
@@ -1262,7 +1262,7 @@ static int ti_get_lsr(struct ti_port *tport, u8 *lsr)
        int size, status;
        struct ti_device *tdev = tport->tp_tdev;
        struct usb_serial_port *port = tport->tp_port;
-       int port_number = port->number - port->serial->minor;
+       int port_number = port->port_number;
        struct ti_port_status *data;
 
        size = sizeof(struct ti_port_status);
@@ -1309,7 +1309,7 @@ static int ti_get_serial_info(struct ti_port *tport,
 
        ret_serial.type = PORT_16550A;
        ret_serial.line = port->serial->minor;
-       ret_serial.port = port->number - port->serial->minor;
+       ret_serial.port = port->port_number;
        ret_serial.flags = tport->tp_flags;
        ret_serial.xmit_fifo_size = TI_WRITE_BUF_SIZE;
        ret_serial.baud_base = tport->tp_tdev->td_is_3410 ? 921600 : 460800;
index 5f6b1ff9d29e6c4166213d04263b83aca0a51a52..a47fa715aaba786adf38ea1edb5399698e3abbef 100644 (file)
@@ -105,9 +105,10 @@ static struct usb_serial *get_free_serial(struct usb_serial *serial,
                *minor = i;
                j = 0;
                dev_dbg(&serial->interface->dev, "%s - minor base = %d\n", __func__, *minor);
-               for (i = *minor; (i < (*minor + num_ports)) && (i < SERIAL_TTY_MINORS); ++i) {
+               for (i = *minor; (i < (*minor + num_ports)) && (i < SERIAL_TTY_MINORS); ++i, ++j) {
                        serial_table[i] = serial;
-                       serial->port[j++]->number = i;
+                       serial->port[j]->minor = i;
+                       serial->port[j]->port_number = i - *minor;
                }
                mutex_unlock(&table_lock);
                return serial;
@@ -1048,7 +1049,7 @@ static int usb_serial_probe(struct usb_interface *interface,
        /* register all of the individual ports with the driver core */
        for (i = 0; i < num_ports; ++i) {
                port = serial->port[i];
-               dev_set_name(&port->dev, "ttyUSB%d", port->number);
+               dev_set_name(&port->dev, "ttyUSB%d", port->minor);
                dev_dbg(ddev, "registering %s", dev_name(&port->dev));
                device_enable_async_suspend(&port->dev);
 
index ece326ef63a0b64c831e5f541b7171b95a9761f6..eacc27dc465788eb456cc7496207d58efe730cc2 100644 (file)
@@ -125,7 +125,7 @@ static int get_serial_info(struct usb_serial_port *port,
 
        memset(&tmp, 0, sizeof(tmp));
        tmp.line            = port->serial->minor;
-       tmp.port            = port->number;
+       tmp.port            = port->port_number;
        tmp.baud_base       = tty_get_baud_rate(port->port.tty);
        tmp.close_delay     = port->port.close_delay / 10;
        tmp.closing_wait    = port->port.closing_wait == ASYNC_CLOSING_WAIT_NONE ?
index 347caad47a121d3f7a26ec7f3246a6960114ffc2..7eb34cd6b5791afd3e68ba9e508ea378857cd952 100644 (file)
@@ -462,7 +462,7 @@ static int whiteheat_ioctl(struct tty_struct *tty,
                memset(&serstruct, 0, sizeof(serstruct));
                serstruct.type = PORT_16654;
                serstruct.line = port->serial->minor;
-               serstruct.port = port->number;
+               serstruct.port = port->port_number;
                serstruct.flags = ASYNC_SKIP_TEST | ASYNC_AUTO_IRQ;
                serstruct.xmit_fifo_size = kfifo_size(&port->write_fifo);
                serstruct.custom_divisor = 0;
@@ -626,7 +626,7 @@ static int firm_open(struct usb_serial_port *port)
 {
        struct whiteheat_simple open_command;
 
-       open_command.port = port->number - port->serial->minor + 1;
+       open_command.port = port->port_number + 1;
        return firm_send_command(port, WHITEHEAT_OPEN,
                (__u8 *)&open_command, sizeof(open_command));
 }
@@ -636,7 +636,7 @@ static int firm_close(struct usb_serial_port *port)
 {
        struct whiteheat_simple close_command;
 
-       close_command.port = port->number - port->serial->minor + 1;
+       close_command.port = port->port_number + 1;
        return firm_send_command(port, WHITEHEAT_CLOSE,
                        (__u8 *)&close_command, sizeof(close_command));
 }
@@ -649,7 +649,7 @@ static void firm_setup_port(struct tty_struct *tty)
        struct whiteheat_port_settings port_settings;
        unsigned int cflag = tty->termios.c_cflag;
 
-       port_settings.port = port->number - port->serial->minor + 1;
+       port_settings.port = port->port_number + 1;
 
        /* get the byte size */
        switch (cflag & CSIZE) {
@@ -726,7 +726,7 @@ static int firm_set_rts(struct usb_serial_port *port, __u8 onoff)
 {
        struct whiteheat_set_rdb rts_command;
 
-       rts_command.port = port->number - port->serial->minor + 1;
+       rts_command.port = port->port_number + 1;
        rts_command.state = onoff;
        return firm_send_command(port, WHITEHEAT_SET_RTS,
                        (__u8 *)&rts_command, sizeof(rts_command));
@@ -737,7 +737,7 @@ static int firm_set_dtr(struct usb_serial_port *port, __u8 onoff)
 {
        struct whiteheat_set_rdb dtr_command;
 
-       dtr_command.port = port->number - port->serial->minor + 1;
+       dtr_command.port = port->port_number + 1;
        dtr_command.state = onoff;
        return firm_send_command(port, WHITEHEAT_SET_DTR,
                        (__u8 *)&dtr_command, sizeof(dtr_command));
@@ -748,7 +748,7 @@ static int firm_set_break(struct usb_serial_port *port, __u8 onoff)
 {
        struct whiteheat_set_rdb break_command;
 
-       break_command.port = port->number - port->serial->minor + 1;
+       break_command.port = port->port_number + 1;
        break_command.state = onoff;
        return firm_send_command(port, WHITEHEAT_SET_BREAK,
                        (__u8 *)&break_command, sizeof(break_command));
@@ -759,7 +759,7 @@ static int firm_purge(struct usb_serial_port *port, __u8 rxtx)
 {
        struct whiteheat_purge purge_command;
 
-       purge_command.port = port->number - port->serial->minor + 1;
+       purge_command.port = port->port_number + 1;
        purge_command.what = rxtx;
        return firm_send_command(port, WHITEHEAT_PURGE,
                        (__u8 *)&purge_command, sizeof(purge_command));
@@ -770,7 +770,7 @@ static int firm_get_dtr_rts(struct usb_serial_port *port)
 {
        struct whiteheat_simple get_dr_command;
 
-       get_dr_command.port = port->number - port->serial->minor + 1;
+       get_dr_command.port = port->port_number + 1;
        return firm_send_command(port, WHITEHEAT_GET_DTR_RTS,
                        (__u8 *)&get_dr_command, sizeof(get_dr_command));
 }
@@ -780,7 +780,7 @@ static int firm_report_tx_done(struct usb_serial_port *port)
 {
        struct whiteheat_simple close_command;
 
-       close_command.port = port->number - port->serial->minor + 1;
+       close_command.port = port->port_number + 1;
        return firm_send_command(port, WHITEHEAT_REPORT_TX_DONE,
                        (__u8 *)&close_command, sizeof(close_command));
 }
index 302ddf55d2daca5d74291b1eb9aaa2b4127fe7d2..3fa68b615ac1ec3be7ba14c1b6af53fc60c206e5 100644 (file)
@@ -37,7 +37,8 @@
  * @serial: pointer back to the struct usb_serial owner of this port.
  * @port: pointer to the corresponding tty_port for this port.
  * @lock: spinlock to grab when updating portions of this structure.
- * @number: the number of the port (the minor number).
+ * @minor: the minor number of the port
+ * @port_number: the struct usb_serial port number of this port (starts at 0)
  * @interrupt_in_buffer: pointer to the interrupt in buffer for this port.
  * @interrupt_in_urb: pointer to the interrupt in struct urb for this port.
  * @interrupt_in_endpointAddress: endpoint address for the interrupt in pipe
@@ -80,7 +81,8 @@ struct usb_serial_port {
        struct usb_serial       *serial;
        struct tty_port         port;
        spinlock_t              lock;
-       unsigned char           number;
+       u32                     minor;
+       u8                      port_number;
 
        unsigned char           *interrupt_in_buffer;
        struct urb              *interrupt_in_urb;