[alsa-devel] HDA - Add more support for Dell systems.
This patch adds support for more Dell systems with Sigmatel/IDT codecs.
Signed off by Tobin Davis tdavis@dsl-only.net
At Sun, 19 Aug 2007 12:54:39 -0700, Tobin Davis wrote:
This patch adds support for more Dell systems with Sigmatel/IDT codecs.
Signed off by Tobin Davis tdavis@dsl-only.net
Hm, the pin config looks different from the patch in ALSA bug#3319.
https://bugtrack.alsa-project.org/alsa-bug/view.php?id=3319
Which one is correct?
Takashi
[2 Dell.patch <text/x-patch; UTF-8 (7bit)>] diff -r d6314f13dcc7 pci/hda/patch_sigmatel.c --- a/pci/hda/patch_sigmatel.c Fri Aug 17 09:17:36 2007 +0200 +++ b/pci/hda/patch_sigmatel.c Sun Aug 19 09:53:07 2007 -0700 @@ -39,6 +39,7 @@
enum { STAC_REF,
- STAC_XPS, STAC_9200_MODELS
};
@@ -63,6 +64,7 @@ enum { STAC_D945GTP3, STAC_D945GTP5, STAC_922X_DELL,
- STAC_922X_DELL2, STAC_INTEL_MAC_V1, STAC_INTEL_MAC_V2, STAC_INTEL_MAC_V3,
@@ -456,12 +458,19 @@ static unsigned int ref9200_pin_configs[ 0x02a19020, 0x01a19021, 0x90100140, 0x01813122, };
+static unsigned int xps9200_pin_configs[8] = {
- 0x40c003fc, 0x0144131f, 0x04a11020, 0x90170310,
- 0x40f000fa, 0x0421121f, 0x90170310, 0x408003fb,
+};
static unsigned int *stac9200_brd_tbl[STAC_9200_MODELS] = { [STAC_REF] = ref9200_pin_configs,
- [STAC_XPS] = xps9200_pin_configs,
};
static const char *stac9200_models[STAC_9200_MODELS] = { [STAC_REF] = "ref",
- [STAC_XPS] = "xps",
};
static struct snd_pci_quirk stac9200_cfg_tbl[] = { @@ -480,9 +489,9 @@ static struct snd_pci_quirk stac9200_cfg SND_PCI_QUIRK(PCI_VENDOR_ID_DELL, 0x01cd, "Dell Inspiron E1705/9400", STAC_REF), SND_PCI_QUIRK(PCI_VENDOR_ID_DELL, 0x01ce,
"Dell XPS M1710", STAC_REF),
"Dell XPS M1710", STAC_XPS),
"Dell Precision M90", STAC_REF),
"Dell Precision M90", STAC_XPS),@@ -596,11 +605,18 @@ static unsigned int stac922x_dell_pin_co 0x50a003f3, 0x405003f4 };
+static unsigned int stac922x_dell2_pin_configs[10] = {
- 0x02211230, 0x01a1902e, 0x01111212, 0x01114210,
- 0x02a11021, 0x01116211, 0x404003f0, 0x40c003f1,
- 0x01813120, 0x400003f2
+};
static unsigned int *stac922x_brd_tbl[STAC_922X_MODELS] = { [STAC_D945_REF] = ref922x_pin_configs, [STAC_D945GTP3] = d945gtp3_pin_configs, [STAC_D945GTP5] = d945gtp5_pin_configs, [STAC_922X_DELL] = stac922x_dell_pin_configs,
- [STAC_922X_DELL2] = stac922x_dell2_pin_configs, [STAC_INTEL_MAC_V1] = intel_mac_v1_pin_configs, [STAC_INTEL_MAC_V2] = intel_mac_v2_pin_configs, [STAC_INTEL_MAC_V3] = intel_mac_v3_pin_configs,
@@ -620,6 +636,7 @@ static const char *stac922x_models[STAC_ [STAC_D945GTP5] = "5stack", [STAC_D945GTP3] = "3stack", [STAC_922X_DELL] = "dell",
- [STAC_922X_DELL2] = "dell2", [STAC_INTEL_MAC_V1] = "intel-mac-v1", [STAC_INTEL_MAC_V2] = "intel-mac-v2", [STAC_INTEL_MAC_V3] = "intel-mac-v3",
@@ -697,6 +714,7 @@ static struct snd_pci_quirk stac922x_cfg "Mac Mini", STAC_INTEL_MAC_V3), /* Dell */ SND_PCI_QUIRK(0x1028, 0x01d7, "Dell XPS M1210", STAC_922X_DELL),
SND_PCI_QUIRK(0x1028, 0x01d2, "Dell ", STAC_922X_DELL2),
{} /* terminator */
}; diff -r d6314f13dcc7 Documentation/ALSA-Configuration.txt --- a/Documentation/ALSA-Configuration.txt Fri Aug 17 09:17:36 2007 +0200 +++ b/Documentation/ALSA-Configuration.txt Sun Aug 19 12:51:56 2007 -0700 @@ -954,7 +954,11 @@ Prior to version 0.9.0rc4 options had a can be adjusted. Appearing only when compiled with $CONFIG_SND_DEBUG=y
- STAC9200/9205/9254
- STAC9200
ref Reference boardXPS Dell XPS Systems- STAC9205/9254 ref Reference board dell-m43 Dell Precision dell-m44 Dell Inspiron
That's hard to tell. I have two different Windows INI files for the same system with different pin configs. This is from the newer INI file. The difference is the reason I created a new model.
Tobin
On Mon, 2007-08-20 at 11:59 +0200, Takashi Iwai wrote:
At Sun, 19 Aug 2007 12:54:39 -0700, Tobin Davis wrote:
This patch adds support for more Dell systems with Sigmatel/IDT codecs.
Signed off by Tobin Davis tdavis@dsl-only.net
Hm, the pin config looks different from the patch in ALSA bug#3319.
https://bugtrack.alsa-project.org/alsa-bug/view.php?id=3319
Which one is correct?
Takashi
[2 Dell.patch <text/x-patch; UTF-8 (7bit)>] diff -r d6314f13dcc7 pci/hda/patch_sigmatel.c --- a/pci/hda/patch_sigmatel.c Fri Aug 17 09:17:36 2007 +0200 +++ b/pci/hda/patch_sigmatel.c Sun Aug 19 09:53:07 2007 -0700 @@ -39,6 +39,7 @@
enum { STAC_REF,
- STAC_XPS, STAC_9200_MODELS
};
@@ -63,6 +64,7 @@ enum { STAC_D945GTP3, STAC_D945GTP5, STAC_922X_DELL,
- STAC_922X_DELL2, STAC_INTEL_MAC_V1, STAC_INTEL_MAC_V2, STAC_INTEL_MAC_V3,
@@ -456,12 +458,19 @@ static unsigned int ref9200_pin_configs[ 0x02a19020, 0x01a19021, 0x90100140, 0x01813122, };
+static unsigned int xps9200_pin_configs[8] = {
- 0x40c003fc, 0x0144131f, 0x04a11020, 0x90170310,
- 0x40f000fa, 0x0421121f, 0x90170310, 0x408003fb,
+};
static unsigned int *stac9200_brd_tbl[STAC_9200_MODELS] = { [STAC_REF] = ref9200_pin_configs,
- [STAC_XPS] = xps9200_pin_configs,
};
static const char *stac9200_models[STAC_9200_MODELS] = { [STAC_REF] = "ref",
- [STAC_XPS] = "xps",
};
static struct snd_pci_quirk stac9200_cfg_tbl[] = { @@ -480,9 +489,9 @@ static struct snd_pci_quirk stac9200_cfg SND_PCI_QUIRK(PCI_VENDOR_ID_DELL, 0x01cd, "Dell Inspiron E1705/9400", STAC_REF), SND_PCI_QUIRK(PCI_VENDOR_ID_DELL, 0x01ce,
"Dell XPS M1710", STAC_REF),
"Dell XPS M1710", STAC_XPS),
"Dell Precision M90", STAC_REF),
"Dell Precision M90", STAC_XPS),@@ -596,11 +605,18 @@ static unsigned int stac922x_dell_pin_co 0x50a003f3, 0x405003f4 };
+static unsigned int stac922x_dell2_pin_configs[10] = {
- 0x02211230, 0x01a1902e, 0x01111212, 0x01114210,
- 0x02a11021, 0x01116211, 0x404003f0, 0x40c003f1,
- 0x01813120, 0x400003f2
+};
static unsigned int *stac922x_brd_tbl[STAC_922X_MODELS] = { [STAC_D945_REF] = ref922x_pin_configs, [STAC_D945GTP3] = d945gtp3_pin_configs, [STAC_D945GTP5] = d945gtp5_pin_configs, [STAC_922X_DELL] = stac922x_dell_pin_configs,
- [STAC_922X_DELL2] = stac922x_dell2_pin_configs, [STAC_INTEL_MAC_V1] = intel_mac_v1_pin_configs, [STAC_INTEL_MAC_V2] = intel_mac_v2_pin_configs, [STAC_INTEL_MAC_V3] = intel_mac_v3_pin_configs,
@@ -620,6 +636,7 @@ static const char *stac922x_models[STAC_ [STAC_D945GTP5] = "5stack", [STAC_D945GTP3] = "3stack", [STAC_922X_DELL] = "dell",
- [STAC_922X_DELL2] = "dell2", [STAC_INTEL_MAC_V1] = "intel-mac-v1", [STAC_INTEL_MAC_V2] = "intel-mac-v2", [STAC_INTEL_MAC_V3] = "intel-mac-v3",
@@ -697,6 +714,7 @@ static struct snd_pci_quirk stac922x_cfg "Mac Mini", STAC_INTEL_MAC_V3), /* Dell */ SND_PCI_QUIRK(0x1028, 0x01d7, "Dell XPS M1210", STAC_922X_DELL),
SND_PCI_QUIRK(0x1028, 0x01d2, "Dell ", STAC_922X_DELL2),
{} /* terminator */
}; diff -r d6314f13dcc7 Documentation/ALSA-Configuration.txt --- a/Documentation/ALSA-Configuration.txt Fri Aug 17 09:17:36 2007 +0200 +++ b/Documentation/ALSA-Configuration.txt Sun Aug 19 12:51:56 2007 -0700 @@ -954,7 +954,11 @@ Prior to version 0.9.0rc4 options had a can be adjusted. Appearing only when compiled with $CONFIG_SND_DEBUG=y
- STAC9200/9205/9254
- STAC9200
ref Reference boardXPS Dell XPS Systems- STAC9205/9254 ref Reference board dell-m43 Dell Precision dell-m44 Dell Inspiron
At Mon, 20 Aug 2007 06:56:52 -0700, Tobin Davis wrote:
That's hard to tell. I have two different Windows INI files for the same system with different pin configs. This is from the newer INI file. The difference is the reason I created a new model.
Oh, well. Then apparently we need to let people test both cases.
thanks,
Takashi
Tobin
On Mon, 2007-08-20 at 11:59 +0200, Takashi Iwai wrote:
At Sun, 19 Aug 2007 12:54:39 -0700, Tobin Davis wrote: > > This patch adds support for more Dell systems with Sigmatel/IDT codecs. > > Signed off by Tobin Davis <tdavis@dsl-only.net> Hm, the pin config looks different from the patch in ALSA bug#3319. https://bugtrack.alsa-project.org/alsa-bug/view.php?id=3319 Which one is correct? Takashi > > [2 Dell.patch <text/x-patch; UTF-8 (7bit)>] > diff -r d6314f13dcc7 pci/hda/patch_sigmatel.c > --- a/pci/hda/patch_sigmatel.c Fri Aug 17 09:17:36 2007 +0200 > +++ b/pci/hda/patch_sigmatel.c Sun Aug 19 09:53:07 2007 -0700 > @@ -39,6 +39,7 @@ > > enum { > STAC_REF, > + STAC_XPS, > STAC_9200_MODELS > }; > > @@ -63,6 +64,7 @@ enum { > STAC_D945GTP3, > STAC_D945GTP5, > STAC_922X_DELL, > + STAC_922X_DELL2, > STAC_INTEL_MAC_V1, > STAC_INTEL_MAC_V2, > STAC_INTEL_MAC_V3, > @@ -456,12 +458,19 @@ static unsigned int ref9200_pin_configs[ > 0x02a19020, 0x01a19021, 0x90100140, 0x01813122, > }; > > +static unsigned int xps9200_pin_configs[8] = { > + 0x40c003fc, 0x0144131f, 0x04a11020, 0x90170310, > + 0x40f000fa, 0x0421121f, 0x90170310, 0x408003fb, > +}; > + > static unsigned int *stac9200_brd_tbl[STAC_9200_MODELS] = { > [STAC_REF] = ref9200_pin_configs, > + [STAC_XPS] = xps9200_pin_configs, > }; > > static const char *stac9200_models[STAC_9200_MODELS] = { > [STAC_REF] = "ref", > + [STAC_XPS] = "xps", > }; > > static struct snd_pci_quirk stac9200_cfg_tbl[] = { > @@ -480,9 +489,9 @@ static struct snd_pci_quirk stac9200_cfg > SND_PCI_QUIRK(PCI_VENDOR_ID_DELL, 0x01cd, > "Dell Inspiron E1705/9400", STAC_REF), > SND_PCI_QUIRK(PCI_VENDOR_ID_DELL, 0x01ce, > - "Dell XPS M1710", STAC_REF), > + "Dell XPS M1710", STAC_XPS), > SND_PCI_QUIRK(PCI_VENDOR_ID_DELL, 0x01cf, > - "Dell Precision M90", STAC_REF), > + "Dell Precision M90", STAC_XPS), > SND_PCI_QUIRK(PCI_VENDOR_ID_DELL, 0x01d6, > "unknown Dell", STAC_REF), > SND_PCI_QUIRK(PCI_VENDOR_ID_DELL, 0x01d8, > @@ -596,11 +605,18 @@ static unsigned int stac922x_dell_pin_co > 0x50a003f3, 0x405003f4 > }; > > +static unsigned int stac922x_dell2_pin_configs[10] = { > + 0x02211230, 0x01a1902e, 0x01111212, 0x01114210, > + 0x02a11021, 0x01116211, 0x404003f0, 0x40c003f1, > + 0x01813120, 0x400003f2 > +}; > + > static unsigned int *stac922x_brd_tbl[STAC_922X_MODELS] = { > [STAC_D945_REF] = ref922x_pin_configs, > [STAC_D945GTP3] = d945gtp3_pin_configs, > [STAC_D945GTP5] = d945gtp5_pin_configs, > [STAC_922X_DELL] = stac922x_dell_pin_configs, > + [STAC_922X_DELL2] = stac922x_dell2_pin_configs, > [STAC_INTEL_MAC_V1] = intel_mac_v1_pin_configs, > [STAC_INTEL_MAC_V2] = intel_mac_v2_pin_configs, > [STAC_INTEL_MAC_V3] = intel_mac_v3_pin_configs, > @@ -620,6 +636,7 @@ static const char *stac922x_models[STAC_ > [STAC_D945GTP5] = "5stack", > [STAC_D945GTP3] = "3stack", > [STAC_922X_DELL] = "dell", > + [STAC_922X_DELL2] = "dell2", > [STAC_INTEL_MAC_V1] = "intel-mac-v1", > [STAC_INTEL_MAC_V2] = "intel-mac-v2", > [STAC_INTEL_MAC_V3] = "intel-mac-v3", > @@ -697,6 +714,7 @@ static struct snd_pci_quirk stac922x_cfg > "Mac Mini", STAC_INTEL_MAC_V3), > /* Dell */ > SND_PCI_QUIRK(0x1028, 0x01d7, "Dell XPS M1210", STAC_922X_DELL), > + SND_PCI_QUIRK(0x1028, 0x01d2, "Dell ", STAC_922X_DELL2), > > {} /* terminator */ > }; > diff -r d6314f13dcc7 Documentation/ALSA-Configuration.txt > --- a/Documentation/ALSA-Configuration.txt Fri Aug 17 09:17:36 2007 +0200 > +++ b/Documentation/ALSA-Configuration.txt Sun Aug 19 12:51:56 2007 -0700 > @@ -954,7 +954,11 @@ Prior to version 0.9.0rc4 options had a > can be adjusted. Appearing only when compiled with > $CONFIG_SND_DEBUG=y > > - STAC9200/9205/9254 > + STAC9200 > + ref Reference board > + XPS Dell XPS Systems > + > + STAC9205/9254 > ref Reference board > dell-m43 Dell Precision > dell-m44 Dell Inspiron
I did. It worked yesterday according to Michael Gerdau, but apparently it doesn't today.
I guess the thing to do is to change the two pci quirk entries back to STAC_REF, but leave the new model in for testing.
Tobin
On Mon, 2007-08-20 at 16:02 +0200, Takashi Iwai wrote:
At Mon, 20 Aug 2007 06:56:52 -0700, Tobin Davis wrote:
That's hard to tell. I have two different Windows INI files for the same system with different pin configs. This is from the newer INI file. The difference is the reason I created a new model.
Oh, well. Then apparently we need to let people test both cases.
thanks,
Takashi
Tobin
On Mon, 2007-08-20 at 11:59 +0200, Takashi Iwai wrote:
At Sun, 19 Aug 2007 12:54:39 -0700, Tobin Davis wrote: > > This patch adds support for more Dell systems with Sigmatel/IDT codecs. > > Signed off by Tobin Davis <tdavis@dsl-only.net> Hm, the pin config looks different from the patch in ALSA bug#3319. https://bugtrack.alsa-project.org/alsa-bug/view.php?id=3319 Which one is correct? Takashi > > [2 Dell.patch <text/x-patch; UTF-8 (7bit)>] > diff -r d6314f13dcc7 pci/hda/patch_sigmatel.c > --- a/pci/hda/patch_sigmatel.c Fri Aug 17 09:17:36 2007 +0200 > +++ b/pci/hda/patch_sigmatel.c Sun Aug 19 09:53:07 2007 -0700 > @@ -39,6 +39,7 @@ > > enum { > STAC_REF, > + STAC_XPS, > STAC_9200_MODELS > }; > > @@ -63,6 +64,7 @@ enum { > STAC_D945GTP3, > STAC_D945GTP5, > STAC_922X_DELL, > + STAC_922X_DELL2, > STAC_INTEL_MAC_V1, > STAC_INTEL_MAC_V2, > STAC_INTEL_MAC_V3, > @@ -456,12 +458,19 @@ static unsigned int ref9200_pin_configs[ > 0x02a19020, 0x01a19021, 0x90100140, 0x01813122, > }; > > +static unsigned int xps9200_pin_configs[8] = { > + 0x40c003fc, 0x0144131f, 0x04a11020, 0x90170310, > + 0x40f000fa, 0x0421121f, 0x90170310, 0x408003fb, > +}; > + > static unsigned int *stac9200_brd_tbl[STAC_9200_MODELS] = { > [STAC_REF] = ref9200_pin_configs, > + [STAC_XPS] = xps9200_pin_configs, > }; > > static const char *stac9200_models[STAC_9200_MODELS] = { > [STAC_REF] = "ref", > + [STAC_XPS] = "xps", > }; > > static struct snd_pci_quirk stac9200_cfg_tbl[] = { > @@ -480,9 +489,9 @@ static struct snd_pci_quirk stac9200_cfg > SND_PCI_QUIRK(PCI_VENDOR_ID_DELL, 0x01cd, > "Dell Inspiron E1705/9400", STAC_REF), > SND_PCI_QUIRK(PCI_VENDOR_ID_DELL, 0x01ce, > - "Dell XPS M1710", STAC_REF), > + "Dell XPS M1710", STAC_XPS), > SND_PCI_QUIRK(PCI_VENDOR_ID_DELL, 0x01cf, > - "Dell Precision M90", STAC_REF), > + "Dell Precision M90", STAC_XPS), > SND_PCI_QUIRK(PCI_VENDOR_ID_DELL, 0x01d6, > "unknown Dell", STAC_REF), > SND_PCI_QUIRK(PCI_VENDOR_ID_DELL, 0x01d8, > @@ -596,11 +605,18 @@ static unsigned int stac922x_dell_pin_co > 0x50a003f3, 0x405003f4 > }; > > +static unsigned int stac922x_dell2_pin_configs[10] = { > + 0x02211230, 0x01a1902e, 0x01111212, 0x01114210, > + 0x02a11021, 0x01116211, 0x404003f0, 0x40c003f1, > + 0x01813120, 0x400003f2 > +}; > + > static unsigned int *stac922x_brd_tbl[STAC_922X_MODELS] = { > [STAC_D945_REF] = ref922x_pin_configs, > [STAC_D945GTP3] = d945gtp3_pin_configs, > [STAC_D945GTP5] = d945gtp5_pin_configs, > [STAC_922X_DELL] = stac922x_dell_pin_configs, > + [STAC_922X_DELL2] = stac922x_dell2_pin_configs, > [STAC_INTEL_MAC_V1] = intel_mac_v1_pin_configs, > [STAC_INTEL_MAC_V2] = intel_mac_v2_pin_configs, > [STAC_INTEL_MAC_V3] = intel_mac_v3_pin_configs, > @@ -620,6 +636,7 @@ static const char *stac922x_models[STAC_ > [STAC_D945GTP5] = "5stack", > [STAC_D945GTP3] = "3stack", > [STAC_922X_DELL] = "dell", > + [STAC_922X_DELL2] = "dell2", > [STAC_INTEL_MAC_V1] = "intel-mac-v1", > [STAC_INTEL_MAC_V2] = "intel-mac-v2", > [STAC_INTEL_MAC_V3] = "intel-mac-v3", > @@ -697,6 +714,7 @@ static struct snd_pci_quirk stac922x_cfg > "Mac Mini", STAC_INTEL_MAC_V3), > /* Dell */ > SND_PCI_QUIRK(0x1028, 0x01d7, "Dell XPS M1210", STAC_922X_DELL), > + SND_PCI_QUIRK(0x1028, 0x01d2, "Dell ", STAC_922X_DELL2), > > {} /* terminator */ > }; > diff -r d6314f13dcc7 Documentation/ALSA-Configuration.txt > --- a/Documentation/ALSA-Configuration.txt Fri Aug 17 09:17:36 2007 +0200 > +++ b/Documentation/ALSA-Configuration.txt Sun Aug 19 12:51:56 2007 -0700 > @@ -954,7 +954,11 @@ Prior to version 0.9.0rc4 options had a > can be adjusted. Appearing only when compiled with > $CONFIG_SND_DEBUG=y > > - STAC9200/9205/9254 > + STAC9200 > + ref Reference board > + XPS Dell XPS Systems > + > + STAC9205/9254 > ref Reference board > dell-m43 Dell Precision > dell-m44 Dell Inspiron
At Mon, 20 Aug 2007 07:17:28 -0700, Tobin Davis wrote:
I did. It worked yesterday according to Michael Gerdau, but apparently it doesn't today.
Well, that's what I wondered.
Michael, could you check the default pincfg entries in /proc/asound/card0/codec#* whether they really match with the values in patch_sigmatel.c?
Takashi
Michael, could you check the default pincfg entries in /proc/asound/card0/codec#* whether they really match with the values in patch_sigmatel.c?
Yes, sure. However it will be later tonight.
Best, Michael
I did. It worked yesterday according to Michael Gerdau, but apparently it doesn't today.
Well, that's what I wondered.
Michael, could you check the default pincfg entries in /proc/asound/card0/codec#* whether they really match with the values in patch_sigmatel.c?
Ok, took me some more time (didn't want to test while almost asleep...)
/proc/asound/card0/codec#* _do_ reflect the pincfg entries as of patch_sigmatel.c
I also once again have my sound working with both Dell-XPS.patch (the one I originally tested) as well as the later version Dell.patch. I also have a working LFE control with both patches.
I have not yet a clear understanding why I did not have sound and LFE yesterday. However the thing that's different is that yesterday morning I booted into a kernel with the patched alsa and did no other alsa mod unload/load until I reverted the patch (when I got back my sound).
When doing my initial test as well as todays test I booted into the pristine hg20070818 (or hg20070817 back then) driver and loaded the patched modules via '/etc/init.d/alsasound restart'.
I can't reboot the machine until about 16:00 today to check whether booting into the patched driver does make a difference or not but I will do so this afternoon and report my results (of course this could be a red hering).
FYI I've attached /proc/asound/card0/codec#* to this mail in case that is of interest.
Best, Michael
At Tue, 21 Aug 2007 10:51:12 +0200, Michael Gerdau wrote:
I did. It worked yesterday according to Michael Gerdau, but apparently it doesn't today.
Well, that's what I wondered.
Michael, could you check the default pincfg entries in /proc/asound/card0/codec#* whether they really match with the values in patch_sigmatel.c?
Ok, took me some more time (didn't want to test while almost asleep...)
/proc/asound/card0/codec#* _do_ reflect the pincfg entries as of patch_sigmatel.c
I also once again have my sound working with both Dell-XPS.patch (the one I originally tested) as well as the later version Dell.patch. I also have a working LFE control with both patches.
I have not yet a clear understanding why I did not have sound and LFE yesterday. However the thing that's different is that yesterday morning I booted into a kernel with the patched alsa and did no other alsa mod unload/load until I reverted the patch (when I got back my sound).
When doing my initial test as well as todays test I booted into the pristine hg20070818 (or hg20070817 back then) driver and loaded the patched modules via '/etc/init.d/alsasound restart'.
I can't reboot the machine until about 16:00 today to check whether booting into the patched driver does make a difference or not but I will do so this afternoon and report my results (of course this could be a red hering).
FYI I've attached /proc/asound/card0/codec#* to this mail in case that is of interest.
OK, I'll let people playing the individual patch for one or two days. If you or no other people find similar regressions, I'll merge Tobin's latest patch to HG tree after that.
thanks,
Takashi
At Tue, 21 Aug 2007 14:27:47 +0200, I wrote:
At Tue, 21 Aug 2007 10:51:12 +0200, Michael Gerdau wrote:
I did. It worked yesterday according to Michael Gerdau, but apparently it doesn't today.
Well, that's what I wondered.
Michael, could you check the default pincfg entries in /proc/asound/card0/codec#* whether they really match with the values in patch_sigmatel.c?
Ok, took me some more time (didn't want to test while almost asleep...)
/proc/asound/card0/codec#* _do_ reflect the pincfg entries as of patch_sigmatel.c
I also once again have my sound working with both Dell-XPS.patch (the one I originally tested) as well as the later version Dell.patch. I also have a working LFE control with both patches.
I have not yet a clear understanding why I did not have sound and LFE yesterday. However the thing that's different is that yesterday morning I booted into a kernel with the patched alsa and did no other alsa mod unload/load until I reverted the patch (when I got back my sound).
When doing my initial test as well as todays test I booted into the pristine hg20070818 (or hg20070817 back then) driver and loaded the patched modules via '/etc/init.d/alsasound restart'.
I can't reboot the machine until about 16:00 today to check whether booting into the patched driver does make a difference or not but I will do so this afternoon and report my results (of course this could be a red hering).
FYI I've attached /proc/asound/card0/codec#* to this mail in case that is of interest.
OK, I'll let people playing the individual patch for one or two days. If you or no other people find similar regressions, I'll merge Tobin's
^ don't
latest patch to HG tree after that.
thanks,
Takashi _______________________________________________ Alsa-devel mailing list Alsa-devel@alsa-project.org http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
OK, I'll let people playing the individual patch for one or two days. If you or no other people find similar regressions, I'll merge Tobin's latest patch to HG tree after that.
I've now rebooted into the patch posted to the ML and everything works fine. Don't know what had happened yesterday -- can't reproduce the problems.
Best, Michael who happily listens to the enhanced Basssound of his XPS M1710 :-)
The only difference between the Dell-XPS.patch I sent you, and the Dell.patch I sent to the list was that I added more Dell systems to the STAC922x portion of the driver (you're using the STAC9200). Since both patches were for the same driver, it was just easier this way. And the 922x portion was tested on #alsa irc.
Tobin
On Tue, 2007-08-21 at 10:51 +0200, Michael Gerdau wrote:
I did. It worked yesterday according to Michael Gerdau, but apparently it doesn't today.
Well, that's what I wondered.
Michael, could you check the default pincfg entries in /proc/asound/card0/codec#* whether they really match with the values in patch_sigmatel.c?
Ok, took me some more time (didn't want to test while almost asleep...)
/proc/asound/card0/codec#* _do_ reflect the pincfg entries as of patch_sigmatel.c
I also once again have my sound working with both Dell-XPS.patch (the one I originally tested) as well as the later version Dell.patch. I also have a working LFE control with both patches.
I have not yet a clear understanding why I did not have sound and LFE yesterday. However the thing that's different is that yesterday morning I booted into a kernel with the patched alsa and did no other alsa mod unload/load until I reverted the patch (when I got back my sound).
When doing my initial test as well as todays test I booted into the pristine hg20070818 (or hg20070817 back then) driver and loaded the patched modules via '/etc/init.d/alsasound restart'.
I can't reboot the machine until about 16:00 today to check whether booting into the patched driver does make a difference or not but I will do so this afternoon and report my results (of course this could be a red hering).
FYI I've attached /proc/asound/card0/codec#* to this mail in case that is of interest.
Best, Michael
participants (3)
-
Michael Gerdau -
Takashi Iwai -
Tobin Davis