Re: Linux v2.6.18-rc5

From: Takashi Iwai
Date: Thu Aug 31 2006 - 04:19:47 EST


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 <torvalds@xxxxxxxx> wrote:
> > >
> > >> Linux 2.6.18-rc5 is out there now
> > >
> > > (Reporters Bcc'ed: please provide updates)
> > >
> > > Serious-looking regressions include:
> > >
> > >
> > > From: Andrew Benton <b3nt@xxxxxxxxxxxxxx>
> > > 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 <tiwai@xxxxxxx>

---
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 majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/