USB: serial: metro-usb: remove unneeded cast and function call
authorGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 8 Mar 2012 22:00:11 +0000 (14:00 -0800)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 8 Mar 2012 22:00:11 +0000 (14:00 -0800)
We should use kzalloc() instead of kmalloc() and memset(), and remove an
unneeded void * cast as well.

Cc: Aleksey Babahin <tamerlan311@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/usb/serial/metro-usb.c

index e968d3396813cd09d4744b6154685d894c2f28ee..6b7d2779148c9dfb6311b98f3e13e88c2bc71ad3 100644 (file)
@@ -58,7 +58,7 @@ static bool debug;
 
 static void metrousb_read_int_callback(struct urb *urb)
 {
-       struct usb_serial_port *port = (struct usb_serial_port *)urb->context;
+       struct usb_serial_port *port = urb->context;
        struct metrousb_private *metro_priv = usb_get_serial_port_data(port);
        struct tty_struct *tty;
        unsigned char *data = urb->transfer_buffer;
@@ -253,13 +253,10 @@ static int metrousb_startup(struct usb_serial *serial)
                port = serial->port[i];
 
                /* Declare memory. */
-               metro_priv = kmalloc(sizeof(struct metrousb_private), GFP_KERNEL);
+               metro_priv = kzalloc(sizeof(struct metrousb_private), GFP_KERNEL);
                if (!metro_priv)
                        return -ENOMEM;
 
-               /* Clear memory. */
-               memset(metro_priv, 0x00, sizeof(struct metrousb_private));
-
                /* Initialize memory. */
                spin_lock_init(&metro_priv->lock);
                usb_set_serial_port_data(port, metro_priv);