LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
* [PATCH 01/12] sound: cmipci.c fix shadowed variable warning
@ 2008-02-28  0:56 Harvey Harrison
  2008-02-28 11:07 ` Takashi Iwai
  0 siblings, 1 reply; 2+ messages in thread
From: Harvey Harrison @ 2008-02-28  0:56 UTC (permalink / raw)
  To: Andrew Morton; +Cc: LKML

A temporary variable for each mixer element is used in an initialization
loop, use the name elem_id.

sound/pci/cmipci.c:2747:26: warning: symbol 'id' shadows an earlier one
sound/pci/cmipci.c:56:13: originally declared here

Signed-off-by: Harvey Harrison <harvey.harrison@gmail.com>
---
 sound/pci/cmipci.c |   10 +++++-----
 1 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/sound/pci/cmipci.c b/sound/pci/cmipci.c
index 135f308..3e763b9 100644
--- a/sound/pci/cmipci.c
+++ b/sound/pci/cmipci.c
@@ -2744,12 +2744,12 @@ static int __devinit snd_cmipci_mixer_new(struct cmipci *cm, int pcm_spdif_devic
 	}
 
 	for (idx = 0; idx < CM_SAVED_MIXERS; idx++) {
-		struct snd_ctl_elem_id id;
+		struct snd_ctl_elem_id elem_id;
 		struct snd_kcontrol *ctl;
-		memset(&id, 0, sizeof(id));
-		id.iface = SNDRV_CTL_ELEM_IFACE_MIXER;
-		strcpy(id.name, cm_saved_mixer[idx].name);
-		if ((ctl = snd_ctl_find_id(cm->card, &id)) != NULL)
+		memset(&elem_id, 0, sizeof(elem_id));
+		elem_id.iface = SNDRV_CTL_ELEM_IFACE_MIXER;
+		strcpy(elem_id.name, cm_saved_mixer[idx].name);
+		if ((ctl = snd_ctl_find_id(cm->card, &elem_id)) != NULL)
 			cm->mixer_res_ctl[idx] = ctl;
 	}
 
-- 
1.5.4.3.342.g99e8



^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [PATCH 01/12] sound: cmipci.c fix shadowed variable warning
  2008-02-28  0:56 [PATCH 01/12] sound: cmipci.c fix shadowed variable warning Harvey Harrison
@ 2008-02-28 11:07 ` Takashi Iwai
  0 siblings, 0 replies; 2+ messages in thread
From: Takashi Iwai @ 2008-02-28 11:07 UTC (permalink / raw)
  To: Harvey Harrison; +Cc: Andrew Morton, LKML

At Wed, 27 Feb 2008 16:56:04 -0800,
Harvey Harrison wrote:
> 
> A temporary variable for each mixer element is used in an initialization
> loop, use the name elem_id.
> 
> sound/pci/cmipci.c:2747:26: warning: symbol 'id' shadows an earlier one
> sound/pci/cmipci.c:56:13: originally declared here
> 
> Signed-off-by: Harvey Harrison <harvey.harrison@gmail.com>

Thanks, applied most patches (with coding style fixes if any) to ALSA
tree.


Takashi

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2008-02-28 11:08 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2008-02-28  0:56 [PATCH 01/12] sound: cmipci.c fix shadowed variable warning Harvey Harrison
2008-02-28 11:07 ` Takashi Iwai

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).