]> oss.titaniummirror.com Git - cp210x.git/blobdiff - src/cp210x.c.karmic
Several fixes.
[cp210x.git] / src / cp210x.c.karmic
index 7843672c9dc0ac98e62dcca24305703fcd25bd1e..707665782103b02ad8597c86c77c839a45b32e91 100644 (file)
  * thanks to Karl Hiramoto karl@hiramoto.org. RTSCTS hardware flow
  * control thanks to Munir Nassar nassarmu@real-time.com
  *
- * Silicon Labs RJM: updated to 0.11 for 2.6.15 and later kernels
- *
- * vxxPKGVERxx by R. Steve McKown, Titanium Mirror, Inc., rsmckown@gmail.com.
- * Added port configuration, usb descriptor and gpio management.
- *
  */
 
 #include <linux/kernel.h>
@@ -28,6 +23,7 @@
 #include <linux/usb.h>
 #include <linux/uaccess.h>
 #include <linux/usb/serial.h>
+#include "cp210x.h"
 
 /*
  * Version Information
@@ -61,7 +57,7 @@ static int cp210x_ioctl(struct tty_struct *, struct file *,
 
 static int debug;
 
-static struct usb_device_id id_table [] = {
+static const struct usb_device_id id_table[] = {
        { USB_DEVICE(0x0471, 0x066A) }, /* AKTAKOM ACE-1001 cable */
        { USB_DEVICE(0x0489, 0xE000) }, /* Pirelli Broadband S.p.A, DP-L10 SIP/GSM Mobile */
        { USB_DEVICE(0x0745, 0x1000) }, /* CipherLab USB CCD Barcode Scanner 1000 */
@@ -97,11 +93,12 @@ static struct usb_device_id id_table [] = {
        { USB_DEVICE(0x10C4, 0x81C8) }, /* Lipowsky Industrie Elektronik GmbH, Baby-JTAG */
        { USB_DEVICE(0x10C4, 0x81E2) }, /* Lipowsky Industrie Elektronik GmbH, Baby-LIN */
        { USB_DEVICE(0x10C4, 0x81E7) }, /* Aerocomm Radio */
+       { USB_DEVICE(0x10C4, 0x81E8) }, /* Zephyr Bioharness */
        { USB_DEVICE(0x10C4, 0x81F2) }, /* C1007 HF band RFID controller */
        { USB_DEVICE(0x10C4, 0x8218) }, /* Lipowsky Industrie Elektronik GmbH, HARP-1 */
        { USB_DEVICE(0x10C4, 0x822B) }, /* Modem EDGE(GSM) Comander 2 */
        { USB_DEVICE(0x10C4, 0x826B) }, /* Cygnal Integrated Products, Inc., Fasttrax GPS demostration module */
-       { USB_DEVICE(0x10c4, 0x8293) }, /* Telegesys ETRX2USB */
+       { USB_DEVICE(0x10C4, 0x8293) }, /* Telegesys ETRX2USB */
        { USB_DEVICE(0x10C4, 0x82F9) }, /* Procyon AVS */
        { USB_DEVICE(0x10C4, 0x8341) }, /* Siemens MC35PU GPRS Modem */
        { USB_DEVICE(0x10C4, 0x8382) }, /* Cygnal Integrated Products, Inc. */
@@ -121,6 +118,7 @@ static struct usb_device_id id_table [] = {
        { USB_DEVICE(0x166A, 0x0303) }, /* Clipsal 5500PCU C-Bus USB interface */
        { USB_DEVICE(0x16D6, 0x0001) }, /* Jablotron serial interface */
        { USB_DEVICE(0x18EF, 0xE00F) }, /* ELV USB-I2C-Interface */
+       { USB_DEVICE(0x413C, 0x9500) }, /* DW700 GPS USB interface */
        { } /* Terminating Entry */
 };
 
@@ -224,92 +222,83 @@ static struct usb_serial_driver cp210x_device = {
 #define CONTROL_WRITE_DTR      0x0100
 #define CONTROL_WRITE_RTS      0x0200
 
-/* CP2103 ioctls */
-#define IOCTL_GPIOGET          0x8000  /* Get gpio bits */
-#define IOCTL_GPIOSET          0x8001  /* Set gpio bits */
-#define IOCTL_GPIOBIC          0x8002  /* Clear specific gpio bit(s) */
-#define IOCTL_GPIOBIS          0x8003  /* Set specific gpio bit(s) */
-
-/* CP210x ioctls principally used during initial device configuration */
-#define IOCTL_DEVICERESET      0x8004  /* Reset the cp210x */
-#define IOCTL_PORTCONFGET      0x8005  /* Get port configuration */
-#define IOCTL_PORTCONFSET      0x8006  /* Set port configuration */
-#define IOCTL_SETVID           0x8007  /* Set vendor id */
-#define IOCTL_SETPID           0x8008  /* Set product id */
-#define IOCTL_SETMFG           0x8009  /* Set manufacturer string */
-#define IOCTL_SETPRODUCT       0x800a  /* Set product string */
-#define IOCTL_SETSERIAL                0x800b  /* Set serial number string */
-#define IOCTL_SETDEVVER                0x800c  /* set device version id */
-/* FIXME: where is IOCTL_SETMFG? */
-
-/* CP2103 GPIO */
-#define GPIO_0                 0x01
-#define GPIO_1                 0x02
-#define GPIO_2                 0x04
-#define GPIO_3                 0x08
-#define GPIO_MASK              (GPIO_0|GPIO_1|GPIO_2|GPIO_3)
-
-/* GetDeviceVersion() return codes */
-#define CP210x_CP2101_VERSION  0x01
-#define        CP210x_CP2102_VERSION   0x02
-#define        CP210x_CP2103_VERSION   0x03
-
-/* Return codes */
-#define        CP210x_SUCCESS                  0x00
-#define        CP210x_DEVICE_NOT_FOUND         0xFF
-#define        CP210x_INVALID_HANDLE           0x01
-#define        CP210x_INVALID_PARAMETER        0x02
-#define        CP210x_DEVICE_IO_FAILED         0x03
-#define        CP210x_FUNCTION_NOT_SUPPORTED   0x04
-#define        CP210x_GLOBAL_DATA_ERROR        0x05
-#define        CP210x_FILE_ERROR               0x06
-#define        CP210x_COMMAND_FAILED           0x08
-#define        CP210x_INVALID_ACCESS_TYPE      0x09
-
-/* USB descriptor sizes */
-#define        CP210x_MAX_DEVICE_STRLEN        256
-#define        CP210x_MAX_PRODUCT_STRLEN       126
-#define        CP210x_MAX_SERIAL_STRLEN        63
-#define        CP210x_MAX_MAXPOWER             250
-
-/* Used to pass variable sized buffers between user and kernel space (ioctls) */
-typedef struct {
-       char *buf;
-       size_t len;
-} cp210x_buffer_t;
-
-/* Port config definitions */
-typedef struct {
-       uint16_t mode;          /* Push-pull = 1, Open-drain = 0 */
-       uint16_t lowPower;
-       uint16_t latch;         /* Logic high = 1, Logic low = 0 */
-} cp210x_port_state_t;
-
-typedef struct {
-       cp210x_port_state_t reset;
-       cp210x_port_state_t suspend;
-       uint8_t enhancedFxn;
-} cp210x_port_config_t;
-
-#define PORT_CONFIG_LEN        13      /* Because sizeof() will pad to full words */
+/* cp210x_get_partnum() return codes */
+#define CP210x_PART_UNKNOWN    0x00
+#define CP210x_PART_CP2101     0x01
+#define CP210x_PART_CP2102     0x02
+#define CP210x_PART_CP2103     0x03
+
+/* Helper to make usb string size */
+#define USBSTRLEN(x)           (x * 2 + 2)
+
+/* Populates usbstr with: (len) + (0x03) + unicode(str).  Each char in str
+ * takes two bytes in unicode format.
+ * Returns the resulting length of the string in usbstr.
+ * This function can accept overlapping usbstr and str as long as the overlap
+ * does not cause data written to usbstr to overwrite data not yet read from
+ * str.
+ */
+static int make_usb_string(char *usbstr, size_t usblen, char *src,
+               size_t srclen)
+{
+       int len = 0;
+
+       if (usbstr && usblen >= 2 && src && *src && srclen) {
+               char *p;
+
+               if (usblen > 255)
+                       usblen = 255;
+
+               p = usbstr + 1;
+               *p++ = 0x03;
+               len = 2;
+               while (srclen && len < usblen) {
+                       *p++ = *src++;
+                       *p++ = 0;
+                       len += 2;
+                       srclen--;
+               }
+               *usbstr = (char)len;
+       }
+       return len;
+}
 
 /*
- * cp210x_buf_from_user
- * Copy a buffer from user space, returning the number of bytes copied
- * from ubuf.buf into kbuf.  klen is the size of the buffer at kbuf.
+ * cp210x_usbstr_from_user
+ * Populate kbuf with a usb string derived from a user space variable.  klen
+ * is the size of the buffer at kbuf.
+ * Returns the number of bytes used in kbuf.
  */
-static size_t copy_buf_from_user(char *kbuf, unsigned long ubuf, size_t klen)
+static size_t cp210x_usbstr_from_user(char *kbuf, unsigned long ubuf, size_t klen)
 {
-       cp210x_buffer_t t;
+       struct cp210x_buffer t;
+       char* str;
+       size_t slen;
 
-       if (!kbuf || !ubuf || !klen ||
-                       copy_from_user(&t, (cp210x_buffer_t *)ubuf, sizeof(t)))
+       if (!kbuf || !ubuf || !klen)
+               return 0;
+       if (copy_from_user(&t, (struct cp210x_buffer __user *)ubuf, sizeof(t)))
                return 0;
-       if (t.len < klen)
-               klen = t.len;
-       if (!t.buf || !t.len || copy_from_user(kbuf, t.buf, klen))
+       if (!t.buf || !t.len || USBSTRLEN(t.len) > klen)
                return 0;
-       return klen;
+       slen = (klen - 2) / 2;
+       if (t.len < slen)
+               slen = t.len;
+       str = kbuf + klen - slen;
+       if (copy_from_user(str, (u8 __user *)t.buf, slen))
+               return 0;
+       return make_usb_string(kbuf, klen, str, slen);
+}
+
+/* cp210x_has_setmfg
+ * Returns 1 if the CP210X part includes firmware that allows setting the
+ * USB MFG descriptor, else 0.  As of this writing, no CP210X firmware allows
+ * this.  SiLabs has suggested this may change in future firmware versions or
+ * parts.
+ */
+static inline int cp210x_has_setmfg(void)
+{
+       return 0;
 }
 
 /*
@@ -348,15 +337,15 @@ static int cp210x_ctlmsg(struct usb_serial_port *port, u8 request,
 
 static int cp210x_reset(struct usb_serial_port *port)
 {
-       dbg("%s", __FUNCTION__);
+       dbg("%s", __func__);
 
 #if 1
-    /* Is this better than usb_device_reset?  It may be.  Once a client issues
+       /* Is this better than usb_device_reset?  It may be.  Once a client issues
         * the reset ioctl, it must disconnect and reconnect, since the USB
         * connections are torn down.  We also ignore the error return, since
         * the part resets and doesn't send one...
         */
-       cp210x_ctlmsg(port, 0xff, 0x40, 0x0008, 0x00, 0, 0);
+       cp210x_ctlmsg(port, 0xff, 0x40, 0x0008, 0x00, NULL, 0);
 #else
        usb_reset_device(port->serial->dev);
 #endif
@@ -365,66 +354,31 @@ static int cp210x_reset(struct usb_serial_port *port)
 
 static int cp210x_get_partnum(struct usb_serial_port *port)
 {
-       static u8 _partnum = 0;
-       int ret = CP210x_SUCCESS;
-       if (!_partnum) {
+       static u8 _partnum = CP210x_PART_UNKNOWN;
+
+       if (_partnum == CP210x_PART_UNKNOWN) {
                u8 addr = port->serial->dev->actconfig->interface[0]->
                        cur_altsetting->endpoint[0].desc.bEndpointAddress &
                        USB_ENDPOINT_NUMBER_MASK;
 
-               if (addr == 0x03 || addr == 0x02) /* The part is a cp2101 */
-                       _partnum = 0x01;
+               if (addr == 0x03 || addr == 0x02)
+                       _partnum = CP210x_PART_CP2101;
                else if (addr == 0x01) {
                        /* Must query part to determine part number */
                        if (cp210x_ctlmsg(port, 0xff, 0xc0, 0x370b, 0x00,
                                        &_partnum, 1) != 1)
-                               ret = CP210x_DEVICE_IO_FAILED;
-               } else
-                       ret = CP210x_DEVICE_IO_FAILED;
+                               _partnum = CP210x_PART_UNKNOWN;
+               }
        }
-       dbg("%s - partnum %u err %d", __FUNCTION__, _partnum, ret);
-       return (ret == CP210x_SUCCESS) ? _partnum : 0;
+       dbg("%s - partnum %u", __func__, _partnum);
+       return _partnum;
 }
 
 static inline int cp210x_setu16(struct usb_serial_port *port, int cmd,
                unsigned int value)
 {
        return cp210x_ctlmsg(port, 0xff, 0x40, 0x3700 | (cmd & 0xff),
-                       value, 0, 0);
-}
-
-/* Populates usbstr with: (len) + (0x03) + unicode(str).  Each char in str
- * takes up two bytes in unicode format, so the resulting len(usbstr) is
- * 2 * len(str) + 2.
- * Returns the resulting length of the string in usbstr.
- * This function can accept overlapping usbstr and str as long as the overlap
- * does not cause data written to usbstr to overwrite data not yet read from
- * str.
- */
-static int make_usb_string(char *usbstr, size_t usblen, char *src,
-               size_t srclen)
-{
-       int len = 0;
-
-       if (usbstr && usblen >= 2 && src && *src && srclen) {
-               char *p;
-
-               /* The usb string format uses first byte as length */
-               if (usblen > 256)
-                       usblen = 256;
-
-               p = usbstr + 1;
-               *p++ = 0x03;
-               len = 2;
-               while (srclen && len < usblen) {
-                       *p++ = *src++;
-                       *p++ = 0;
-                       len += 2;
-                       srclen--;
-               }
-               *usbstr = (char)len;
-       }
-       return len;
+                       value, NULL, 0);
 }
 
 /*
@@ -438,17 +392,17 @@ static int cp210x_setstr(struct usb_serial_port *port, int cmd, char *usbstr)
        unsigned len = usbstr[0];
        int ret = cp210x_ctlmsg(port, 0xff, 0x40, 0x3700 | (cmd & 0xff), 0,
                        usbstr, len);
-       dbg("%s - cmd 0x%02x len %d ret %d", __FUNCTION__, cmd, len, ret);
+       dbg("%s - cmd 0x%02x len %d ret %d", __func__, cmd, len, ret);
        return ret;
 }
 
 /* Set all gpio simultaneously */
 static int cp210x_gpioset(struct usb_serial_port *port, u8 gpio)
 {
-       dbg("%s - port %d, gpio = 0x%.2x", __FUNCTION__, port->number, gpio);
+       dbg("%s - port %d, gpio = 0x%.2x", __func__, port->number, gpio);
 
        return cp210x_ctlmsg(port, 0xff, 0x40, 0x37e1,
-                       ((uint16_t)gpio << 8) | GPIO_MASK, 0, 0);
+                       ((uint16_t)gpio << 8) | GPIO_MASK, NULL, 0);
 }
 
 /* Set select gpio bits */
@@ -474,51 +428,52 @@ static int cp210x_gpiosetb(struct usb_serial_port *port, u8 set, u8 clear)
        if (clear & GPIO_3)
                gpio = (gpio & ~(GPIO_3 << 8))|GPIO_3;
 
-       dbg("%s - port %d, gpiob = 0x%.4x", __FUNCTION__, port->number, gpio);
+       dbg("%s - port %d, gpiob = 0x%.4x", __func__, port->number, gpio);
 
        /* FIXME: how about REQTYPE_HOST_TO_DEVICE instead of 0x40? */
-       return cp210x_ctlmsg(port, 0xff, 0x40, 0x37e1, gpio, 0, 0);
+       return cp210x_ctlmsg(port, 0xff, 0x40, 0x37e1, gpio, NULL, 0);
 }
 
 static int cp210x_gpioget(struct usb_serial_port *port, u8 *gpio)
 {
        int ret;
 
-       dbg("%s - port %d", __FUNCTION__, port->number);
+       dbg("%s - port %d", __func__, port->number);
 
        /* FIXME: how about REQTYPE_DEVICE_TO_HOST instead of 0xc0? */
        ret = cp210x_ctlmsg(port, 0xff, 0xc0, 0x00c2, 0, gpio, 1);
 
-       dbg("%s - gpio = 0x%.2x (%d)", __FUNCTION__, *gpio, ret);
+       dbg("%s - gpio = 0x%.2x (%d)", __func__, *gpio, ret);
 
        return (ret == 1) ? 0 : -1;
 }
 
 static int cp210x_portconfset(struct usb_serial_port *port,
-               cp210x_port_config_t *config)
+               struct cp210x_port_config *config)
 {
-       cp210x_port_config_t lconfig;
+       struct cp210x_port_config lconfig;
        int ret;
 
-       dbg("%s", __FUNCTION__);
+       dbg("%s", __func__);
 
        memcpy(&lconfig, config, sizeof(lconfig));
 
-       /* apparently not implemented yet */
-       lconfig.suspend.lowPower = 0;
-       lconfig.reset.lowPower = 0;
-
        /* Words from cp2103 are MSB */
        lconfig.reset.mode = cpu_to_be16(config->reset.mode);
-       /* lconfig.reset.lowPower = cpu_to_be16(config->reset.lowPower); */
        lconfig.reset.latch = cpu_to_be16(config->reset.latch);
        lconfig.suspend.mode = cpu_to_be16(config->suspend.mode);
-       /* lconfig.suspend.lowPower = cpu_to_be16(config->suspend.lowPower); */
        lconfig.suspend.latch = cpu_to_be16(config->suspend.latch);
+#if 0 /* apparently not implemented by CP210X firmware */
+       lconfig.reset.lowPower = cpu_to_be16(config->reset.lowPower);
+       lconfig.suspend.lowPower = cpu_to_be16(config->suspend.lowPower);
+#else
+       lconfig.suspend.lowPower = 0;
+       lconfig.reset.lowPower = 0;
+#endif
 
        ret = cp210x_ctlmsg(port, 0xff, 0x40, 0x370c, 0, &lconfig,
-                       PORT_CONFIG_LEN);
-       if (ret == PORT_CONFIG_LEN)
+                       sizeof(struct cp210x_port_config));
+       if (ret == sizeof(struct cp210x_port_config))
                return 0;
        else if (ret >= 0)
                return -1;
@@ -527,15 +482,15 @@ static int cp210x_portconfset(struct usb_serial_port *port,
 }
 
 static int cp210x_portconfget(struct usb_serial_port *port,
-               cp210x_port_config_t *config)
+               struct cp210x_port_config *config)
 {
        int ret;
 
-       dbg("%s", __FUNCTION__);
+       dbg("%s", __func__);
 
        ret = cp210x_ctlmsg(port, 0xff, 0xc0, 0x370c, 0, config,
-                       PORT_CONFIG_LEN);
-       if (ret == PORT_CONFIG_LEN) {
+                       sizeof(struct cp210x_port_config));
+       if (ret == sizeof(struct cp210x_port_config)) {
                /* Words from cp2103 are MSB */
                config->reset.mode = be16_to_cpu(config->reset.mode);
                config->reset.lowPower = be16_to_cpu(config->reset.lowPower);
@@ -649,11 +604,6 @@ static int cp210x_set_config(struct usb_serial_port *port, u8 request,
                return -EPROTO;
        }
 
-       /* Single data value */
-       result = usb_control_msg(serial->dev,
-                       usb_sndctrlpipe(serial->dev, 0),
-                       request, REQTYPE_HOST_TO_DEVICE, data[0],
-                       0, NULL, 0, 300);
        return 0;
 }
 
@@ -784,27 +734,26 @@ static int cp210x_ioctl(struct tty_struct *tty, struct file *file,
 {
        struct usb_serial_port *port = tty->driver_data;
 
-       dbg("%s (%d) cmd = 0x%04x", __FUNCTION__, port->number, cmd);
+       dbg("%s (%d) cmd = 0x%04x", __func__, port->number, cmd);
 
        switch (cmd) {
-               
+
        case TIOCMGET:
        {
                int result = cp210x_tiocmget(tty, file);
-               if (copy_to_user(&arg, &result, sizeof(int)))
-                       return -EFAULT; 
+               if (copy_to_user((int __user *)&arg, &result, sizeof(int)))
+                       return -EFAULT;
                return 0;
        }
        break;
 
        case TIOCMSET:
        case TIOCMBIS:
-
        case TIOCMBIC:
        {
                int val = 0;
 
-               if (copy_from_user(&val, &arg, sizeof(int)))
+               if (copy_from_user(&val, (int __user *)&arg, sizeof(int)))
                        return -EFAULT;
 
                /* this looks wrong: TIOCMSET isn't going to work right */
@@ -815,51 +764,52 @@ static int cp210x_ioctl(struct tty_struct *tty, struct file *file,
        }
        break;
 
-       case IOCTL_GPIOGET:
-       if (cp210x_get_partnum(port) == CP210x_CP2103_VERSION) {
+       case CP210x_IOCTL_GPIOGET:
+       if (cp210x_get_partnum(port) == CP210x_PART_CP2103) {
                u8 gpio = 0;
-               if (!cp210x_gpioget(port, &gpio) && !copy_to_user((u8*)arg,
-                                       &gpio, sizeof(gpio)))
+               if (!cp210x_gpioget(port, &gpio) && !copy_to_user(
+                                       (u8 __user *)arg, &gpio, sizeof(gpio)))
                        return 0;
        }
        return -EFAULT;
        break;
 
-       case IOCTL_GPIOSET:
-       if (cp210x_get_partnum(port) == CP210x_CP2103_VERSION &&
+       case CP210x_IOCTL_GPIOSET:
+       if (cp210x_get_partnum(port) == CP210x_PART_CP2103 &&
                        !cp210x_gpioset(port, arg))
                return 0;
        return -EFAULT;
        break;
 
-       case IOCTL_GPIOBIC:
-       case IOCTL_GPIOBIS:
-       if (cp210x_get_partnum(port) == CP210x_CP2103_VERSION &&
-                       !cp210x_gpiosetb(port, (cmd==IOCTL_GPIOBIC) ?  0 : arg,
-                       (cmd==IOCTL_GPIOBIC) ? arg : 0))
+       case CP210x_IOCTL_GPIOBIC:
+       case CP210x_IOCTL_GPIOBIS:
+       if (cp210x_get_partnum(port) == CP210x_PART_CP2103 &&
+                       !cp210x_gpiosetb(port,
+                               (cmd==CP210x_IOCTL_GPIOBIC) ?  0 : arg,
+                               (cmd==CP210x_IOCTL_GPIOBIC) ? arg : 0))
                return 0;
        return -EFAULT;
        break;
 
-       case IOCTL_DEVICERESET:
+       case CP210x_IOCTL_DEVICERESET:
        return cp210x_reset(port);
        break;
 
-       case IOCTL_PORTCONFGET:
+       case CP210x_IOCTL_PORTCONFGET:
        {
-               cp210x_port_config_t config;
+               struct cp210x_port_config config;
                if (!cp210x_portconfget(port, &config) && !copy_to_user(
-                                       (cp210x_port_config_t*)arg, &config,
-                                       sizeof(config)))
+                                       (struct cp210x_port_config __user *)arg,
+                                       &config, sizeof(config)))
                        return 0;
        }
        return -EFAULT;
        break;
 
-       case IOCTL_PORTCONFSET:
+       case CP210x_IOCTL_PORTCONFSET:
        {
-       cp210x_port_config_t config;
-       if (!copy_from_user(&config, (cp210x_port_config_t*)arg,
+       struct cp210x_port_config config;
+       if (!copy_from_user(&config, (struct cp210x_port_config __user *)arg,
                        sizeof(config)) &&
                        !cp210x_portconfset(port, &config))
                return 0;
@@ -867,10 +817,10 @@ static int cp210x_ioctl(struct tty_struct *tty, struct file *file,
        }
        break;
 
-       case IOCTL_SETVID:
+       case CP210x_IOCTL_SETVID:
        {
                unsigned int vid;
-               if (!copy_from_user(&vid, (unsigned int *)arg,
+               if (!copy_from_user(&vid, (unsigned int __user *)arg,
                                sizeof(unsigned int)) &&
                                !cp210x_setu16(port, 0x01, vid))
                        return 0;
@@ -878,10 +828,10 @@ static int cp210x_ioctl(struct tty_struct *tty, struct file *file,
        }
        break;
 
-       case IOCTL_SETPID:
+       case CP210x_IOCTL_SETPID:
        {
                unsigned int pid;
-               if (!copy_from_user(&pid, (unsigned int *)arg,
+               if (!copy_from_user(&pid, (unsigned int __user *)arg,
                                sizeof(unsigned int)) &&
                                !cp210x_setu16(port, 0x02, pid))
                        return 0;
@@ -889,51 +839,43 @@ static int cp210x_ioctl(struct tty_struct *tty, struct file *file,
        }
        break;
 
-       case IOCTL_SETMFG:
-#if 0 /* Silicon Labs apparently doesn't provide for setting of mfg desc */
-       {
-               char usbstr[CP210x_MAX_MFG_STRLEN * 2 + 2];
-               size_t len = copy_buf_from_user(usbstr + sizeof(usbstr) -
-                               CP210x_MAX_MFG_STRLEN, arg,
-                               CP210x_MAX_MFG_STRLEN);
-               len = make_usb_string(usbstr, sizeof(usbstr), str, len);
+       case CP210x_IOCTL_SETMFG:
+       if (cp210x_has_setmfg()) {
+               char usbstr[USBSTRLEN(CP210x_MAX_MFG_STRLEN)];
+               size_t len = cp210x_usbstr_from_user(usbstr, arg,
+                               sizeof(usbstr));
                if (len && cp210x_setstr(port, 0x00, usbstr) == len)
                        return 0;
-               return -EFAULT;
        }
-#endif
+       return -EFAULT;
        break;
 
-       case IOCTL_SETPRODUCT:
+       case CP210x_IOCTL_SETPRODUCT:
        {
-               char usbstr[CP210x_MAX_PRODUCT_STRLEN * 2 + 2];
-               char *str = usbstr + sizeof(usbstr) - CP210x_MAX_PRODUCT_STRLEN;
-               size_t len = copy_buf_from_user(str, arg,
-                               CP210x_MAX_PRODUCT_STRLEN);
-               len = make_usb_string(usbstr, sizeof(usbstr), str, len);
+               char usbstr[USBSTRLEN(CP210x_MAX_PRODUCT_STRLEN)];
+               size_t len = cp210x_usbstr_from_user(usbstr, arg,
+                               sizeof(usbstr));
                if (len && cp210x_setstr(port, 0x03, usbstr) == len)
                        return 0;
                return -EFAULT;
        }
        break;
 
-       case IOCTL_SETSERIAL:
+       case CP210x_IOCTL_SETSERIAL:
        {
-               char usbstr[CP210x_MAX_SERIAL_STRLEN * 2 + 2];
-               char *str = usbstr + sizeof(usbstr) - CP210x_MAX_SERIAL_STRLEN;
-               size_t len = copy_buf_from_user(str, arg,
-                               CP210x_MAX_SERIAL_STRLEN);
-               len = make_usb_string(usbstr, sizeof(usbstr), str, len);
+               char usbstr[USBSTRLEN(CP210x_MAX_SERIAL_STRLEN)];
+               size_t len = cp210x_usbstr_from_user(usbstr, arg,
+                               sizeof(usbstr));
                if (len && cp210x_setstr(port, 0x04, usbstr) == len)
                        return 0;
                return -EFAULT;
        }
        break;
 
-       case IOCTL_SETDEVVER:
+       case CP210x_IOCTL_SETDEVVER:
        {
                unsigned int ver;
-               if (!copy_from_user(&ver, (unsigned int *)arg,
+               if (!copy_from_user(&ver, (unsigned int __user *)arg,
                                sizeof(unsigned int)) &&
                                !cp210x_setu16(port, 0x07, ver))
                        return 0;
@@ -942,7 +884,7 @@ static int cp210x_ioctl(struct tty_struct *tty, struct file *file,
        break;
 
        default:
-       dbg("%s not supported = 0x%04x", __FUNCTION__, cmd);
+       dbg("%s not supported = 0x%04x", __func__, cmd);
        break;
        }
 
@@ -1126,7 +1068,7 @@ static void cp210x_set_termios(struct tty_struct *tty,
                                baud);
                if (cp210x_set_config_single(port, CP210X_SET_BAUDDIV,
                                        ((BAUD_RATE_GEN_FREQ + baud/2) / baud))) {
-                       dbg("Baud rate requested not supported by device\n");
+                       dbg("Baud rate requested not supported by device");
                        baud = tty_termios_baud_rate(old_termios);
                }
        }