Re: Linux v2.6.18-rc5

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



At Wed, 30 Aug 2006 18:15:09 +0200,
I wrote:
> 
> At Wed, 30 Aug 2006 16:54:38 +0100,
> Andrew Benton wrote:
> > 
> > Andrew Morton wrote:
> > > On Sun, 27 Aug 2006 21:30:50 -0700 (PDT)
> > > Linus Torvalds <[email protected]> wrote:
> > > 
> > >> Linux 2.6.18-rc5 is out there now
> > > 
> > > (Reporters Bcc'ed: please provide updates)
> > > 
> > > Serious-looking regressions include:
> > > 
> > > 
> > > From: Andrew Benton <[email protected]>
> > > Subject: ALSA problems with 2.6.18-rc3
> > 
> > The problem remains in 2.6.18-rc5.
> > The workaround people have suggested (using alsactl -F restore) works if 
> > I have a working /etc/asound.state created with a 2.6.17 kernel. If I 
> > was starting from scratch with 2.6.18-rc5 I would have no way to set the 
> > sound level for the digital output. But maybe the bug is in alsamixer 
> > and alsactl?
> 
> No, it doesn't sound like a bug of alsamixer or alsactl if "alsactl
> -F" works.
> 
> What did you exactly do and what doesn't work right now?
> A detailed explanation for reproducing the bug is needed.

AnOther bugreport suggest that the similar name mismatch appears in
ac97.  IMO, it's no real breakage, but surely safer to avoid such
a thing.

Could you try the patch below (same found in bugzilla #7080)?


Thanks,

Takashi

====

[PATCH] ALSA: ac97 - Correct some Mic mixer elements

Revert the mixer element names of some Mic controls to the state of
2.6.17.  This should fix the name mismatch in alsactl.

Signed-off-by: Takashi Iwai <[email protected]>

---
diff --git a/sound/pci/ac97/ac97_codec.c b/sound/pci/ac97/ac97_codec.c
index 0abf280..51e83d7 100644
--- a/sound/pci/ac97/ac97_codec.c
+++ b/sound/pci/ac97/ac97_codec.c
@@ -573,7 +573,7 @@ AC97_SINGLE("PC Speaker Playback Volume"
 };
 
 static const struct snd_kcontrol_new snd_ac97_controls_mic_boost =
-	AC97_SINGLE("Mic Boost (+20dB) Switch", AC97_MIC, 6, 1, 0);
+	AC97_SINGLE("Mic Boost (+20dB)", AC97_MIC, 6, 1, 0);
 
 
 static const char* std_rec_sel[] = {"Mic", "CD", "Video", "Aux", "Line", "Mix", "Mix Mono", "Phone"};
@@ -615,7 +615,7 @@ AC97_SINGLE("Simulated Stereo Enhancemen
 AC97_SINGLE("3D Control - Switch", AC97_GENERAL_PURPOSE, 13, 1, 0),
 AC97_SINGLE("Loudness (bass boost)", AC97_GENERAL_PURPOSE, 12, 1, 0),
 AC97_ENUM("Mono Output Select", std_enum[2]),
-AC97_ENUM("Mic Select Capture Switch", std_enum[3]),
+AC97_ENUM("Mic Select", std_enum[3]),
 AC97_SINGLE("ADC/DAC Loopback", AC97_GENERAL_PURPOSE, 7, 1, 0)
 };
 
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux