Style cleanups

[cp210x.git] / src / cp210x.c.karmic

diff --git a/src/cp210x.c.karmic b/src/cp210x.c.karmic
index 707665782103b02ad8597c86c77c839a45b32e91..267ff4c8b50401b97d1ecc9592ccfc636ebe2d9c 100644 (file)
--- a/src/cp210x.c.karmic
+++ b/src/cp210x.c.karmic
@@ -269,10 +269,11 @@ static int make_usb_string(char *usbstr, size_t usblen, char *src,
  * is the size of the buffer at kbuf.
  * Returns the number of bytes used in kbuf.
  */
-static size_t cp210x_usbstr_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)
 {
        struct cp210x_buffer t;
-       char* str;
+       char *str;
        size_t slen;
 
        if (!kbuf || !ubuf || !klen)
@@ -314,7 +315,8 @@ static int cp210x_ctlmsg(struct usb_serial_port *port, u8 request,
        u8 *tbuf;
        int ret;
 
-       if (!(tbuf = kmalloc(size, GFP_KERNEL)))
+       tbuf = kmalloc(size, GFP_KERNEL);
+       if (!tbuf)
                return -ENOMEM;
        if (requestype & 0x80) {
                ret = usb_control_msg(dev, usb_rcvctrlpipe(dev, 0), request,
@@ -329,8 +331,9 @@ static int cp210x_ctlmsg(struct usb_serial_port *port, u8 request,
        }
        kfree(tbuf);
        if (ret < 0 && ret != -EPIPE) {
-               dev_printk(KERN_DEBUG, &dev->dev, "cp210x: control failed cmd rqt %u "
-                               "rq %u len %u ret %d\n", requestype, request, size, ret);
+               dev_printk(KERN_DEBUG, &dev->dev,
+                       "cp210x: ctl failed cmd rqt %u rq %u len %u ret %d\n",
+                       requestype, request, size, ret);
        }
        return ret;
 }
@@ -340,10 +343,10 @@ static int cp210x_reset(struct usb_serial_port *port)
        dbg("%s", __func__);
 
 #if 1
-       /* 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...
+       /* 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, NULL, 0);
 #else
@@ -458,7 +461,7 @@ static int cp210x_portconfset(struct usb_serial_port *port,
 
        memcpy(&lconfig, config, sizeof(lconfig));
 
-       /* Words from cp2103 are MSB */
+       /* Words from cp210x are MSB */
        lconfig.reset.mode = cpu_to_be16(config->reset.mode);
        lconfig.reset.latch = cpu_to_be16(config->reset.latch);
        lconfig.suspend.mode = cpu_to_be16(config->suspend.mode);
@@ -491,7 +494,7 @@ static int cp210x_portconfget(struct usb_serial_port *port,
        ret = cp210x_ctlmsg(port, 0xff, 0xc0, 0x370c, 0, config,
                        sizeof(struct cp210x_port_config));
        if (ret == sizeof(struct cp210x_port_config)) {
-               /* Words from cp2103 are MSB */
+               /* Words from cp210x are MSB */
                config->reset.mode = be16_to_cpu(config->reset.mode);
                config->reset.lowPower = be16_to_cpu(config->reset.lowPower);
                config->reset.latch = be16_to_cpu(config->reset.latch);
@@ -757,8 +760,8 @@ static int cp210x_ioctl(struct tty_struct *tty, struct file *file,
                        return -EFAULT;
 
                /* this looks wrong: TIOCMSET isn't going to work right */
-               if (cp210x_tiocmset(tty, file, cmd==TIOCMBIC ? 0 : val,
-                                       cmd==TIOCMBIC ? val : 0))
+               if (cp210x_tiocmset(tty, file, (cmd == TIOCMBIC) ? 0 : val,
+                                       (cmd == TIOCMBIC) ? val : 0))
                        return -EFAULT;
                return 0;
        }
@@ -785,8 +788,8 @@ static int cp210x_ioctl(struct tty_struct *tty, struct file *file,
        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))
+                               (cmd == CP210x_IOCTL_GPIOBIC) ?  0 : arg,
+                               (cmd == CP210x_IOCTL_GPIOBIC) ? arg : 0))
                return 0;
        return -EFAULT;
        break;