From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1758744AbYFVVio (ORCPT ); Sun, 22 Jun 2008 17:38:44 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1758310AbYFVViF (ORCPT ); Sun, 22 Jun 2008 17:38:05 -0400 Received: from smtp21.orange.fr ([80.12.242.47]:13055 "EHLO smtp21.orange.fr" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756592AbYFVViD (ORCPT ); Sun, 22 Jun 2008 17:38:03 -0400 X-ME-UUID: 20080622213800752.B7AD81C00084@mwinf2107.orange.fr Message-ID: <485EC637.2090505@wanadoo.fr> Date: Sun, 22 Jun 2008 23:37:59 +0200 From: JAILLET Christophe User-Agent: Thunderbird 2.0.0.14 (X11/20080505) MIME-Version: 1.0 To: linux , cj , petero2@telia.com Subject: [PATCH 1/1] drivers/block/pktcdvd.c : avoid useless memset Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Christophe Jaillet Hi, here is a patch against linux/drivers/block/pktcdvd.c which : 1) Avoid to 'memset(...,0, ...)' before calling 'init_cdrom_command' because this function already does it. It is the same patch as the one I sent earlier but the formating should be OK this time. Signed-off-by: Christophe Jaillet --- diff --git a/drivers/block/pktcdvd.c b/drivers/block/pktcdvd.c index 3ba1df9..8369ebc 100644 --- a/drivers/block/pktcdvd.c +++ b/drivers/block/pktcdvd.c @@ -2079,7 +2079,6 @@ static noinline_for_stack int pkt_write_caching(struct pktcdvd_device *pd, unsigned char buf[64]; int ret; - memset(buf, 0, sizeof(buf)); init_cdrom_command(&cgc, buf, sizeof(buf), CGC_DATA_READ); cgc.sense = &sense; cgc.buflen = pd->mode_offset + 12; @@ -2126,7 +2125,6 @@ static noinline_for_stack int pkt_get_max_speed(struct pktcdvd_device *pd, unsigned char *cap_buf; int ret, offset; - memset(buf, 0, sizeof(buf)); cap_buf = &buf[sizeof(struct mode_page_header) + pd->mode_offset]; init_cdrom_command(&cgc, buf, sizeof(buf), CGC_DATA_UNKNOWN); cgc.sense = &sense;