From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933489AbaKMPWt (ORCPT ); Thu, 13 Nov 2014 10:22:49 -0500 Received: from cantor2.suse.de ([195.135.220.15]:39995 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933088AbaKMPWr (ORCPT ); Thu, 13 Nov 2014 10:22:47 -0500 Date: Thu, 13 Nov 2014 16:22:45 +0100 Message-ID: From: Takashi Iwai To: Sudip Mukherjee Cc: alsa-devel@alsa-project.org, linux-kernel@vger.kernel.org Subject: Re: [alsa-devel] [PATCH 2/2] ALSA: ice1712: remove unused variable In-Reply-To: <1415891657-7244-2-git-send-email-sudipm.mukherjee@gmail.com> References: <1415891657-7244-1-git-send-email-sudipm.mukherjee@gmail.com> <1415891657-7244-2-git-send-email-sudipm.mukherjee@gmail.com> User-Agent: Wanderlust/2.15.9 (Almost Unreal) SEMI/1.14.6 (Maruoka) FLIM/1.14.9 (=?UTF-8?B?R29qxY0=?=) APEL/10.8 Emacs/24.4 (x86_64-suse-linux-gnu) MULE/6.0 (HANACHIRUSATO) MIME-Version: 1.0 (generated by SEMI 1.14.6 - "Maruoka") Content-Type: text/plain; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org At Thu, 13 Nov 2014 20:44:17 +0530, Sudip Mukherjee wrote: > > these variable were initialized with some value, but never used. > and so they are safe to be deleted. > > Signed-off-by: Sudip Mukherjee > --- > > i have not given Jaroslav Kysela in the To list, > as in my last mail, all mails bounced with the error: > "cant create user output file" > > sound/pci/ice1712/ice1712.c | 3 +-- > sound/pci/ice1712/revo.c | 9 +++------ > 2 files changed, 4 insertions(+), 8 deletions(-) > > diff --git a/sound/pci/ice1712/ice1712.c b/sound/pci/ice1712/ice1712.c > index b97a0f9..8f27357 100644 > --- a/sound/pci/ice1712/ice1712.c > +++ b/sound/pci/ice1712/ice1712.c > @@ -620,10 +620,9 @@ static int snd_ice1712_playback_ds_prepare(struct snd_pcm_substream *substream) > { > struct snd_ice1712 *ice = snd_pcm_substream_chip(substream); > struct snd_pcm_runtime *runtime = substream->runtime; > - u32 period_size, buf_size, rate, tmp, chn; > + u32 period_size, rate, tmp, chn; > > period_size = snd_pcm_lib_period_bytes(substream) - 1; > - buf_size = snd_pcm_lib_buffer_bytes(substream) - 1; > tmp = 0x0064; > if (snd_pcm_format_width(runtime->format) == 16) > tmp &= ~0x04; > diff --git a/sound/pci/ice1712/revo.c b/sound/pci/ice1712/revo.c > index 1112ec1..1f40dab 100644 > --- a/sound/pci/ice1712/revo.c > +++ b/sound/pci/ice1712/revo.c > @@ -481,7 +481,6 @@ static int ap192_ak4114_init(struct snd_ice1712 *ice) > static const unsigned char ak4114_init_txcsb[] = { > 0x41, 0x02, 0x2c, 0x00, 0x00 > }; > - int err; > > struct revo51_spec *spec; > spec = kzalloc(sizeof(*spec), GFP_KERNEL); > @@ -489,11 +488,9 @@ static int ap192_ak4114_init(struct snd_ice1712 *ice) > return -ENOMEM; > ice->spec = spec; > > - err = snd_ak4114_create(ice->card, > - ap192_ak4114_read, > - ap192_ak4114_write, > - ak4114_init_vals, ak4114_init_txcsb, > - ice, &spec->ak4114); > + snd_ak4114_create(ice->card, ap192_ak4114_read, ap192_ak4114_write, > + ak4114_init_vals, ak4114_init_txcsb, ice, > + &spec->ak4114); IMO, this is rather a bug. It should return the error (with a proper error handling). thanks, Takashi > /* AK4114 in Revo cannot detect external rate correctly. > * No reason to stop capture stream due to incorrect checks */ > spec->ak4114->check_flags = AK4114_CHECK_NO_RATE; > -- > 1.8.1.2 > > _______________________________________________ > Alsa-devel mailing list > Alsa-devel@alsa-project.org > http://mailman.alsa-project.org/mailman/listinfo/alsa-devel >