From: "Burman Yan" <yan_952@hotmail.com>
To: linux-kernel@vger.kernel.org
Cc: trivial@kernel.org, gregkh@suse.de
Subject: [PATCH 2.6.19-rc5] USB serial: replace kmalloc+memset with kzalloc
Date: Wed, 22 Nov 2006 20:54:38 +0200 [thread overview]
Message-ID: <BAY20-F24285FCF29BD150A986723D8E30@phx.gbl> (raw)
[-- Attachment #1: Type: text/plain, Size: 322 bytes --]
Hi.
This patch replaces kmalloc+memset with kzalloc
Sorry for the previous post (forgot to attach the actual patch)
Regards
Yan Burman
_________________________________________________________________
Don't just search. Find. Check out the new MSN Search!
http://search.msn.click-url.com/go/onm00200636ave/direct/01/
[-- Attachment #2: kzalloc_usb_serial.patch --]
[-- Type: application/octet-stream, Size: 4506 bytes --]
Replace kmalloc+memset with kzalloc
Signed-off-by: Yan Burman <yan_952@hotmail.com>
diff -rubp linux-2.6.19-rc5_orig/drivers/usb/serial/ark3116.c linux-2.6.19-rc5_kzalloc/drivers/usb/serial/ark3116.c
--- linux-2.6.19-rc5_orig/drivers/usb/serial/ark3116.c 2006-11-09 12:16:21.000000000 +0200
+++ linux-2.6.19-rc5_kzalloc/drivers/usb/serial/ark3116.c 2006-11-11 22:44:04.000000000 +0200
@@ -85,10 +85,9 @@ static int ark3116_attach(struct usb_ser
int i;
for (i = 0; i < serial->num_ports; ++i) {
- priv = kmalloc(sizeof (struct ark3116_private), GFP_KERNEL);
+ priv = kzalloc(sizeof (struct ark3116_private), GFP_KERNEL);
if (!priv)
goto cleanup;
- memset(priv, 0x00, sizeof (struct ark3116_private));
spin_lock_init(&priv->lock);
usb_set_serial_port_data(serial->port[i], priv);
diff -rubp linux-2.6.19-rc5_orig/drivers/usb/serial/console.c linux-2.6.19-rc5_kzalloc/drivers/usb/serial/console.c
--- linux-2.6.19-rc5_orig/drivers/usb/serial/console.c 2006-11-09 12:16:21.000000000 +0200
+++ linux-2.6.19-rc5_kzalloc/drivers/usb/serial/console.c 2006-11-11 22:44:04.000000000 +0200
@@ -166,19 +166,17 @@ static int usb_console_setup(struct cons
if (serial->type->set_termios) {
/* build up a fake tty structure so that the open call has something
* to look at to get the cflag value */
- tty = kmalloc (sizeof (*tty), GFP_KERNEL);
+ tty = kzalloc (sizeof (*tty), GFP_KERNEL);
if (!tty) {
err ("no more memory");
return -ENOMEM;
}
- termios = kmalloc (sizeof (*termios), GFP_KERNEL);
+ termios = kzalloc (sizeof (*termios), GFP_KERNEL);
if (!termios) {
err ("no more memory");
kfree (tty);
return -ENOMEM;
}
- memset (tty, 0x00, sizeof(*tty));
- memset (termios, 0x00, sizeof(*termios));
termios->c_cflag = cflag;
tty->termios = termios;
port->tty = tty;
diff -rubp linux-2.6.19-rc5_orig/drivers/usb/serial/garmin_gps.c linux-2.6.19-rc5_kzalloc/drivers/usb/serial/garmin_gps.c
--- linux-2.6.19-rc5_orig/drivers/usb/serial/garmin_gps.c 2006-11-09 12:16:21.000000000 +0200
+++ linux-2.6.19-rc5_kzalloc/drivers/usb/serial/garmin_gps.c 2006-11-11 22:44:04.000000000 +0200
@@ -1523,12 +1523,11 @@ static int garmin_attach (struct usb_ser
dbg("%s", __FUNCTION__);
- garmin_data_p = kmalloc (sizeof(struct garmin_data), GFP_KERNEL);
+ garmin_data_p = kzalloc (sizeof(struct garmin_data), GFP_KERNEL);
if (garmin_data_p == NULL) {
dev_err(&port->dev, "%s - Out of memory\n", __FUNCTION__);
return -ENOMEM;
}
- memset (garmin_data_p, 0, sizeof(struct garmin_data));
init_timer(&garmin_data_p->timer);
spin_lock_init(&garmin_data_p->lock);
INIT_LIST_HEAD(&garmin_data_p->pktlist);
diff -rubp linux-2.6.19-rc5_orig/drivers/usb/serial/mos7840.c linux-2.6.19-rc5_kzalloc/drivers/usb/serial/mos7840.c
--- linux-2.6.19-rc5_orig/drivers/usb/serial/mos7840.c 2006-11-09 12:16:21.000000000 +0200
+++ linux-2.6.19-rc5_kzalloc/drivers/usb/serial/mos7840.c 2006-11-11 22:44:04.000000000 +0200
@@ -2596,12 +2596,11 @@ static int mos7840_startup(struct usb_se
/* set up port private structures */
for (i = 0; i < serial->num_ports; ++i) {
- mos7840_port = kmalloc(sizeof(struct moschip_port), GFP_KERNEL);
+ mos7840_port = kzalloc(sizeof(struct moschip_port), GFP_KERNEL);
if (mos7840_port == NULL) {
err("%s - Out of memory", __FUNCTION__);
return -ENOMEM;
}
- memset(mos7840_port, 0, sizeof(struct moschip_port));
/* Initialize all port interrupt end point to port 0 int endpoint *
* Our device has only one interrupt end point comman to all port */
diff -rubp linux-2.6.19-rc5_orig/drivers/usb/serial/ti_usb_3410_5052.c linux-2.6.19-rc5_kzalloc/drivers/usb/serial/ti_usb_3410_5052.c
--- linux-2.6.19-rc5_orig/drivers/usb/serial/ti_usb_3410_5052.c 2006-11-09 12:16:21.000000000 +0200
+++ linux-2.6.19-rc5_kzalloc/drivers/usb/serial/ti_usb_3410_5052.c 2006-11-11 22:44:04.000000000 +0200
@@ -459,13 +459,12 @@ static int ti_startup(struct usb_serial
/* set up port structures */
for (i = 0; i < serial->num_ports; ++i) {
- tport = kmalloc(sizeof(struct ti_port), GFP_KERNEL);
+ tport = kzalloc(sizeof(struct ti_port), GFP_KERNEL);
if (tport == NULL) {
dev_err(&dev->dev, "%s - out of memory\n", __FUNCTION__);
status = -ENOMEM;
goto free_tports;
}
- memset(tport, 0, sizeof(struct ti_port));
spin_lock_init(&tport->tp_lock);
tport->tp_uart_base_addr = (i == 0 ? TI_UART1_BASE_ADDR : TI_UART2_BASE_ADDR);
tport->tp_flags = low_latency ? ASYNC_LOW_LATENCY : 0;
next reply other threads:[~2006-11-22 18:54 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-11-22 18:54 Burman Yan [this message]
-- strict thread matches above, loose matches on Subject: below --
2006-11-22 18:52 Burman Yan
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=BAY20-F24285FCF29BD150A986723D8E30@phx.gbl \
--to=yan_952@hotmail.com \
--cc=gregkh@suse.de \
--cc=linux-kernel@vger.kernel.org \
--cc=trivial@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®