- Alsa-devel - mailman.alsa-project.org

Re: [alsa-devel] [PATCH] ASoC TLV320AIC3X codec driver
by Vladimir A. Barinov 14 Nov '07

14 Nov '07

Hello Takashi, Thank you for the review. Takashi Iwai wrote: > At Tue, 13 Nov 2007 22:41:25 +0300, > Vladimir Barinov wrote: > >> --- /dev/null >> +++ linux-2.6.24.rc2.alsa/sound/soc/codecs/tlv320aic3x.c >> > (snip) > >> + >> +struct snd_soc_codec_device soc_codec_dev_aic3x; >> > > Do you need this here? It's defined in the later position. > You are right. Thanks for pointing to this. That is an artefact of the driver that I based on. > >> +/* >> + * All input lines are connected when !0xf and disconnected with 0xf bit field, >> > > Please keep the line within 80 chars. Try to run checkpatch.pl in > linuxkernel/scripts for checking such minor coding-style issues. > But this line is less then 80 chars :) I've applied checkpatch.pl before sending the patch. The driver really has 3 lines with 80 chars warning, but they are less then 82 char. It's in the codec interconnection map for DAPM. If that is a strong demand with 80 chars _warning_ then I'll try to rename the mixers/path/endpoint names to match, if not then the code will be more understandable. That lines are: + {"Right HPCOM Mux", "differential of HPROUT", "Right Line2 Bypass Mixer"}, + {"Right HPCOM Mux", "differential of HPLCOM", "Right Line2 Bypass Mixer"}, If I'll split it with 2 lines each then will it be good to watch in accordance with DAPM usage strategy? Destination Widget <=== Path Name <=== Source Widget > >> +static const char *aic3x_left_dac_mux[] = { "DAC_L1", "DAC_L3", "DAC_L2" }; >> +static const char *aic3x_right_dac_mux[] = { "DAC_R1", "DAC_R3", "DAC_R2" }; >> +static const char *aic3x_left_hpcom_mux[] = >> + { "differential of HPLOUT", "constant VCM", "single-ended" }; >> +static const char *aic3x_right_hpcom_mux[] = >> + { "differential of HPROUT", "constant VCM", "single-ended", >> + "differential of HPLCOM", "external feedback" }; >> +static const char *aic3x_linein_mode_mux[] = { "single-ended", "differential" }; >> > > Ditto. > Ditto :) These lines are less then 80 chars. > >> +#define AIC3X_RATES SNDRV_PCM_RATE_8000_96000 >> +#define AIC3X_FORMATS (SNDRV_PCM_FMTBIT_S16_LE | SNDRV_PCM_FMTBIT_S20_3LE | \ >> + SNDRV_PCM_FMTBIT_S24_LE | SNDRV_PCM_FMTBIT_S32_LE) >> > > Are you sure that it's FMTBIT_S24_LE? It's not packed in 3 bytes but > uses lower 3 bytes of 4 bytes frame. > > Aic3x supports 16/20/24/32 bits data word length and in accordance with aic33 documentation the number of clocks per half-frame for each channel are equal to exact number of bits of the word. Unfortunately, now I have no h/w to test 24bits mode. > >> Index: linux-2.6.24.rc2.alsa/sound/soc/codecs/Kconfig >> =================================================================== >> --- linux-2.6.24.rc2.alsa.orig/sound/soc/codecs/Kconfig >> +++ linux-2.6.24.rc2.alsa/sound/soc/codecs/Kconfig >> @@ -37,3 +37,6 @@ config SND_SOC_CS4270_VD33_ERRATA >> bool >> depends on SND_SOC_CS4270 >> >> +config SND_SOC_TLV320AIC3X >> + tristate >> + depends on SND_SOC >> > > Currently, there is a known problem with dependency on I2C over soc > codecs drivers. If CONFIG_I2C=m and CONFIG_SND_SOC_*=y, you'll get > unresolved symbols. > > I don't know how to resolve this issue right now. But, the drivers > that support *only* I2C right now can add "depends on I2C" to its > Kconfig. That doesn't work for CS4270 which may work without I2C as > standalone mode, though. > Ok, will add I2C dependency. Regards, Vladimir

3 4

[alsa-devel] Naming multiple USB devices (solved)
by Giovanni Maruzzelli 14 Nov '07

14 Nov '07

Hi list, I've added the following script to http://alsa.opensrc.org/index.php/Udev (it's not the most elegant solution, but it works for me): BEWARE of the linebreaks, they are mangled by the mail client ================================= #!/usr/bin/perl # fixed persistent naming for multiple usb soundcards # gmaruzz (at) celliax.org # This is to be executed by udev with the following rules: # KERNEL=="controlC[0-9]*", DRIVERS=="usb",PROGRAM="/usr/bin/alsa_name.pl %k", NAME="snd/%c{1}" # KERNEL=="hwC[D0-9]*", DRIVERS=="usb", PROGRAM="/usr/bin/alsa_name.pl %k", NAME="snd/%c{1}" # KERNEL=="midiC[D0-9]*", DRIVERS=="usb",PROGRAM="/usr/bin/alsa_name.pl %k", NAME="snd/%c{1}" # KERNEL=="pcmC[D0-9cp]*", DRIVERS=="usb",PROGRAM="/usr/bin/alsa_name.pl %k", NAME="snd/%c{1}" use strict; use warnings; my $alsaname = $ARGV[0]; #udev called us with this argument (%k) my $physdevpath = $ENV{PHYSDEVPATH}; #udev put this in our environment my $alsanum = "cucu"; #you can find the physdevpath of a device with "udevinfo -a -p $(udevinfo -q path -n /dev/snd/pcmC0D0c)" $physdevpath =~ s/.*\/([^\/]*)/$1/; #eliminate until last slash $physdevpath =~ s/([^:]*):.*/$1/; #eliminate from twopoint (:) to end_of_line if($physdevpath eq "1-5.2") # you can find this value with "dmesg" { $alsanum="11"; #start from "10" (easier for debugging), "0" is for motherboard soundcard, max is "31" } if($physdevpath eq "1-5.3") # you can find this value with "dmesg" { $alsanum="12"; #start from "10" (easier for debugging), "0" is for motherboard soundcard, max is "31" } if($physdevpath eq "1-5.4") # you can find this value with "dmesg" { $alsanum="13"; #start from "10" (easier for debugging), "0" is for motherboard soundcard, max is "31" } if($physdevpath eq "3-2") # you can find this value with "dmesg" { $alsanum="14"; #start from "10" (easier for debugging), "0" is for motherboard soundcard, max is "31" } # other bus positions.... if($alsanum ne "cucu") { $alsaname=~ s/(.*)C([0-9]+)(.*)/$1C$alsanum$3/; } print $alsaname; exit 0; ======================= Ciao, Giovanni On Feb 10, 2007 8:31 PM, Jean-Michel Pouré <jmpoure(a)free.fr> wrote: > Le samedi 10 février 2007 à 18:53 +0100, Ingo Müller a écrit : > > I'm going to play with udev now, too, so if I find something, I'll > > tell > > you. In return, it would be awesome if you could write down - even it > > is > > only short - a howto about how to manage two identical devices, once > > we > > got it working. > > > > Fantastic. > > Thank you very much for your help. > I will add a page on the wiki about udev rules. > > Kind regards, > Jean-Michel > > > > ------------------------------------------------------------------------- > Using Tomcat but need to do more? Need to support web services, security? > Get stuff done quickly with pre-integrated technology to make your job easier. > Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo > http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642 > _______________________________________________ > Alsa-devel mailing list > Alsa-devel(a)lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/alsa-devel >

1 0

[alsa-devel] Query regarding snd_pcm_lib_preallocate_pages_for_all()
by Pharaoh . 14 Nov '07

14 Nov '07

Hi, I am trying to use snd_pcm_lib_preallocate_pages_for_all() on an arm based platform for preallocating the DMA buffer. I am using it as snd_pcm_lib_preallocate_pages_for_all(pcm, SNDRV_DMA_TYPE_DEV, NULL, 64 * 1024, 64 * 1024); since I dont know what should be passed as the 3'rd param. I tried passing my device pointer i.e. card->dev after correctly populating it but I get an error as 'DMA coherent mask in not set' and buffer is not allocated. If I pass NULL as shown above, which I think is appropriate, the allocation is done well and I can play some samples too using aplay, but when I do a rmmod all hell breaks free. I am thinking this is corrupting memory some where, since when I preallocate using SNDRV_DMA_TYPE_CONTINUOUS, everything works well but I since I need to mmap the buffer I dont use it. I get following: c001f7e0 c001f7e0 c001f814 c03b3040 0000000b c34c1c98 c34c1c88 1c80: c003642c c0071978 c34c0000 c34c1cb8 c34c1c9c c003b780 c00363fc c34c0000 1ca0: c34c0000 c03b3040 00000001 c34c1cd4 c34c1cbc c003bd94 c003b690 c34c1ce8 1cc0: 60000093 00000001 c34c1ce8 c34c1cd8 c002908c c003bbe8 bf000000 c34c1d08 1ce0: c34c1cec c002b2ac c0028d84 ffffffec c03b3040 c34733c8 c001f7e0 c34c1d40 1d00: c34c1d0c c002b588 c002b250 c03a7060 c34c1d94 00000017 ffffffff c026dd40 1d20: 00000017 c34c1dec 00000004 00000093 40130000 c34c1de8 c34c1d44 c002b688 1d40: c002b3a0 0000007e 00000000 00000000 00000000 00000000 00000000 00000000 1d60: 00000000 c03a7060 c34c1e14 c03b3040 c022eef4 c34c1f14 00000000 00000a00 1d80: c34c1e80 c34c1d90 c019ee84 c01bb390 00000000 7a035438 00000005 00000000 1da0: 00000a00 c03a7060 c0271cdc c34c1d50 00000000 c34c1e08 c34c1dc4 c0063a94 1dc0: c0063984 00000044 ffffffff c34c1e20 00000000 c02eabe0 c347317c c34c1e44 1de0: c34c1dec c00249a0 c002b65c c02eabe0 00000000 00000013 00000093 c34c0000 1e00: c0306200 00000000 c02eabe0 c347317c c001f7e0 40130000 c34c1e44 c02eabe0 1e20: c34c1e34 c02c0bfc c0067478 00000093 ffffffff 10d5f0ff c34c1e7c c34c1e48 1e40: c006d658 c0067444 c3541cc0 00000000 00000002 11b100df 000004c0 c347317c 1e60: c34c0000 00000001 c001f7e0 40130000 c34c1ecc c34c1e80 c006e61c c006d1b8 1e80: c3545000 c001f820 11b100df 0000007e 00000000 00000010 c3545000 00000800 1ea0: c34c0000 ffffffeb c03b3040 c347317c c001f7e0 c001f814 c34c1fb0 40130958 1ec0: c34c1f04 c34c1ed0 c002b470 c006dedc c004822c c03b307c 0000081f ffffffff 1ee0: c026ddc0 0000081f c34c1fb0 40130958 00000055 4012f000 c34c1fac c34c1f08 1f00: c002b688 c002b3a0 c34c1f14 c0222148 c0221a50 c34c1e94 0000000c c34c1e84 1f20: 00000001 00000000 00000000 c1b01040 00000000 00000009 0001d2b4 00000066 1f40: c0024fe4 00000001 c34c1f6c c34c1f58 c01a02c0 c01a01c0 00000000 00000000 1f60: c34c1fa4 c34c1f70 c01a09d0 c01a02b0 c34c1e70 c34c1e74 00000004 0001d2b4 1f80: 00000a00 00000000 0000008e ffffffff 00000115 0001d2dd 00000000 00000000 1fa0: 00000000 c34c1fb0 c0024de8 c002b65c 00000000 0000b05c 54434100 00000001 1fc0: 40130958 00000115 0001d2dd 00000000 00000000 00000055 4012f000 00000007 1fe0: 0001d150 be8ba5ec 0000b05c 40082fbc 60000010 ffffffff 00000000 00000000 Backtrace: [<c0073880>] (page_remove_rmap+0x0/0x118) from [<c006c628>] (unmap_vmas+0x3c4/0x60c) r4 = C02EABE0 [<c006c264>] (unmap_vmas+0x0/0x60c) from [<c00719e8>] (exit_mmap+0x80/0x150) [<c0071968>] (exit_mmap+0x0/0x150) from [<c003642c>] (mmput+0x40/0xe4) r8 = 0000000B r7 = C03B3040 r6 = C001F814 r5 = C001F7E0 r4 = C001F7E0 [<c00363ec>] (mmput+0x0/0xe4) from [<c003b780>] (exit_mm+0x100/0x10c) r4 = C34C0000 [<c003b680>] (exit_mm+0x0/0x10c) from [<c003bd94>] (do_exit+0x1bc/0x9c4) r7 = 00000001 r6 = C03B3040 r5 = C34C0000 r4 = C34C0000 [<c003bbd8>] (do_exit+0x0/0x9c4) from [<c002908c>] (die+0x318/0x36c) [<c0028d74>] (die+0x0/0x36c) from [<c002b2ac>] (__do_kernel_fault+0x6c/0x7c) [<c002b240>] (__do_kernel_fault+0x0/0x7c) from [<c002b588>] (do_page_fault+0x1f8/0x210) r7 = C001F7E0 r6 = C34733C8 r5 = C03B3040 r4 = FFFFFFEC [<c002b390>] (do_page_fault+0x0/0x210) from [<c002b688>] (do_DataAbort+0x3c/0xa0) [<c002b64c>] (do_DataAbort+0x0/0xa0) from [<c00249a0>] (__dabt_svc+0x40/0x60) r8 = C347317C r7 = C02EABE0 r6 = 00000000 r5 = C34C1E20 r4 = FFFFFFFF [<c0067434>] (lru_cache_add_active+0x0/0xa0) from [<c006d658>] (do_wp_page+0x4b0/0x574) r4 = 10D5F0FF [<c006d1a8>] (do_wp_page+0x0/0x574) from [<c006e61c>] (__handle_mm_fault+0x750/0x824) [<c006decc>] (__handle_mm_fault+0x0/0x824) from [<c002b470>] (do_page_fault+0xe0/0x210) [<c002b390>] (do_page_fault+0x0/0x210) from [<c002b688>] (do_DataAbort+0x3c/0xa0) [<c002b64c>] (do_DataAbort+0x0/0xa0) from [<c0024de8>] (ret_from_exception+0x0/0x10) r8 = 00000000 r7 = 00000000 r6 = 0001D2DD r5 = 00000115 r4 = FFFFFFFF Code: e3130901 e1a03004 1594300c e59f008c (e5931004) <1>Fixing recursive fault but reboot is needed! BUG: scheduling while atomic: udevd/0x00000004/245 Bad page state in process 'exe' page:c02eaa60 flags:0x00004000 mapping:00000000 mapcount:0 count:0 Trying to fix it up, but a reboot is needed Backtrace: Bad page state in process 'exe' page:c02eaa60 flags:0x00004000 mapping:00000000 mapcount:0 count:0 Trying to fix it up, but a reboot is needed Backtrace: Bad page state in process 'exe' page:c02eaa80 flags:0x00004000 mapping:00000000 mapcount:0 count:0 Trying to fix it up, but a reboot is needed Backtrace: Bad page state in process 'exe' page:c02eaa80 flags:0x00004000 mapping:00000000 mapcount:0 count:0 Trying to fix it up, but a reboot is needed Backtrace: Bad page state in process 'exe' page:c02eaaa0 flags:0x00004000 mapping:00000000 mapcount:0 count:0 Trying to fix it up, but a reboot is needed Backtrace: Bad page state in process 'exe' page:c02eaaa0 flags:0x00004000 mapping:00000000 mapcount:0 count:0 Trying to fix it up, but a reboot is needed Backtrace: Bad page state in process 'exe' page:c02eaac0 flags:0x00004000 mapping:00000000 mapcount:0 count:0 Trying to fix it up, but a reboot is needed Backtrace: Bad page state in process 'exe' page:c02eaac0 flags:0x00004000 mapping:00000000 mapcount:0 count:0 Trying to fix it up, but a reboot is needed Backtrace: Bad page state in process 'exe' page:c02eaae0 flags:0x00004000 mapping:00000000 mapcount:0 count:0 Trying to fix it up, but a reboot is needed Backtrace: Bad page state in process 'exe' page:c02eaae0 flags:0x00004000 mapping:00000000 mapcount:0 count:0 Trying to fix it up, but a reboot is needed Backtrace: Bad page state in process 'exe' page:c02eab00 flags:0x00004000 mapping:00000000 mapcount:0 count:0 Trying to fix it up, but a reboot is needed Backtrace: Bad page state in process 'exe' page:c02eab00 flags:0x00004000 mapping:00000000 mapcount:0 count:0 Trying to fix it up, but a reboot is needed Backtrace: Bad page state in process 'exe' page:c02eab20 flags:0x00004000 mapping:00000000 mapcount:0 count:0 Trying to fix it up, but a reboot is needed Backtrace: Bad page state in process 'exe' page:c02eab20 flags:0x00004000 mapping:00000000 mapcount:0 count:0 Trying to fix it up, but a reboot is needed Backtrace: Bad page state in process 'exe' page:c02eab40 flags:0x00004000 mapping:00000000 mapcount:0 count:0 Trying to fix it up, but a reboot is needed Backtrace: Bad page state in process 'exe' page:c02eab40 flags:0x00004000 mapping:00000000 mapcount:0 count:0 Trying to fix it up, but a reboot is needed Backtrace: Bad page state in process 'exe' page:c02eab60 flags:0x00004000 mapping:00000000 mapcount:0 count:0 Trying to fix it up, but a reboot is needed Backtrace: Bad page state in process 'exe' page:c02eab60 flags:0x00004000 mapping:00000000 mapcount:0 count:0 Trying to fix it up, but a reboot is needed Backtrace: Bad page state in process 'exe' page:c02eab80 flags:0x00004000 mapping:00000000 mapcount:0 count:0 Trying to fix it up, but a reboot is needed Backtrace: Bad page state in process 'exe' page:c02eab80 flags:0x00004000 mapping:00000000 mapcount:0 count:0 Trying to fix it up, but a reboot is needed Backtrace: Bad page state in process 'exe' page:c02eaba0 flags:0x00004000 mapping:00000000 mapcount:0 count:0 Trying to fix it up, but a reboot is needed Backtrace: Bad page state in process 'exe' page:c02eaba0 flags:0x00004000 mapping:00000000 mapcount:0 count:0 Trying to fix it up, but a reboot is needed Backtrace: Bad page state in process 'exe' page:c02eabc0 flags:0x00004000 mapping:00000000 mapcount:0 count:0 Trying to fix it up, but a reboot is needed Backtrace: Bad page state in process 'exe' page:c02eabc0 flags:0x00004000 mapping:00000000 mapcount:0 count:0 Trying to fix it up, but a reboot is needed Backtrace: so on

3 4

Re: [alsa-devel] [Pkg-alsa-devel] Bug#450805: alsa-tools-gui: hdspmixer initializes only first Hammerfall DSP card
by Elimar Riesebieter 14 Nov '07

14 Nov '07

forwarded 450805 alsa-devel(a)alsa-project.org tags 450805 pending patch Thanks Hi ALSA developers, does it make sense to apply the patch to HEAD? I don't have a Hammerfall handy and can not test it. Thanks for cooperation. Elimar On Sun, 11 Nov 2007 the mental interface of Frank Heckenbach told: Package: alsa-tools-gui Version: 1.0.13-1 Severity: normal Tags: patch We are using Hammerfall DSP cards. After booting, their audio output remains silent until hdspmixer is started. No interaction in the GUI of hdspmixer is necessary to unmute the first HDSP card; however, further cards are only unmuted when activating the respective GUI page ("2", "3"). Apparently, hdspmixer does some automatic initialization of the card when activating the page. Since we'd like to have a fully automatic startup, the following patch activates the page for each existing card on startup, thereby initializing them. There are surely more elegant solutions, but this patch is tested and solves the problem for us. --- alsa-tools-1.0.13/hdspmixer/src/HDSPMixerCardSelector.h.orig 2007-04-16 19:49:22.000000000 +0200 +++ alsa-tools-1.0.13/hdspmixer/src/HDSPMixerCardSelector.h 2007-04-16 19:50:51.000000000 +0200 @@ -37,6 +37,7 @@ int card; HDSPMixerWindow *basew; HDSPMixerCardSelector(int x, int y, int w, int h, int card); + void ActivateCard (int i); void draw(); int handle(int e); }; --- alsa-tools-1.0.13/hdspmixer/src/HDSPMixerCardSelector.cxx.orig 2007-04-16 19:49:23.000000000 +0200 +++ alsa-tools-1.0.13/hdspmixer/src/HDSPMixerCardSelector.cxx 2007-04-16 19:53:45.000000000 +0200 @@ -45,27 +45,25 @@ } } +void HDSPMixerCardSelector::ActivateCard (int i) +{ + card = i + 1; + basew->current_card = i; + basew->cards[i]->setMode (basew->cards[i]->getSpeed ()); + redraw (); +} + int HDSPMixerCardSelector::handle(int e) { int xpos = Fl::event_x()-x(); switch (e) { case FL_PUSH: - if (xpos < 13 && card != 1) { - card = 1; - basew->current_card = 0; - basew->cards[0]->setMode(basew->cards[0]->getSpeed()); - redraw(); - } else if (xpos >= 24 && xpos < 37 && card != 2 && basew->cards[1] != NULL) { - card = 2; - basew->current_card = 1; - basew->cards[1]->setMode(basew->cards[1]->getSpeed()); - redraw(); - } else if (xpos >= 48 && card != 3 && basew->cards[2] != NULL) { - card = 3; - basew->current_card = 2; - basew->cards[2]->setMode(basew->cards[2]->getSpeed()); - redraw(); - } + if (xpos < 13 && card != 1) + ActivateCard (0); + else if (xpos >= 24 && xpos < 37 && card != 2 && basew->cards[1] != NULL) + ActivateCard (1); + else if (xpos >= 48 && card != 3 && basew->cards[2] != NULL) + ActivateCard (2); return 1; default: return Fl_Widget::handle(e); --- alsa-tools-1.0.13/hdspmixer/src/HDSPMixerWindow.cxx.orig 2007-04-16 19:41:21.000000000 +0200 +++ alsa-tools-1.0.13/hdspmixer/src/HDSPMixerWindow.cxx 2007-04-16 19:55:13.000000000 +0200 @@ -700,6 +701,9 @@ Fl::atclose = atclose_cb; Fl::add_handler(handler_cb); Fl::add_timeout(0.030, readregisters_cb, this); + i = 0; + while (cards[i] != NULL) + inputs->buttons->cardselector->ActivateCard (i++); } int HDSPMixerWindow::handle(int e) -- Experience is something you don't get until just after you need it!

2 1

[alsa-devel] [ALSA] fix private data pointer calculation in CS4270 driver
by Timur Tabi 14 Nov '07

14 Nov '07

Fix the calculation of the private_data pointer in the CS4270 driver. Signed-off-by: Timur Tabi <timur(a)freescale.com> --- sound/soc/codecs/cs4270.c | 3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/sound/soc/codecs/cs4270.c b/sound/soc/codecs/cs4270.c index 5d601ad..abac628 100644 --- a/sound/soc/codecs/cs4270.c +++ b/sound/soc/codecs/cs4270.c @@ -725,7 +725,8 @@ static int cs4270_probe(struct platform_device *pdev) codec->owner = THIS_MODULE; codec->dai = &cs4270_dai; codec->num_dai = 1; - codec->private_data = codec + ALIGN(sizeof(struct snd_soc_codec), 4); + codec->private_data = (void *) codec + + ALIGN(sizeof(struct snd_soc_codec), 4); socdev->codec = codec; -- 1.5.2.4

2 1

[alsa-devel] mixer API documentation
by Richard Kettlewell 13 Nov '07

13 Nov '07

Hello, Where is the ALSA mixer API documentation? I looked at http://www.alsa-project.org/alsa-doc/alsa-lib/mixer.html, but that only says "Mixer interface is designed to access the abstracted mixer controls. This is an abstraction layer over the hcontrol layer". ttfn/rjk

2 2

[alsa-devel] [PATCH] ASoC TLV320AIC3X codec driver
by Vladimir Barinov 13 Nov '07

13 Nov '07

Hi Liam, This patch adds ALSA SoC support for TI TLV320AIC3X audio codecs. The features that are supported: o Capture/Playback/Bypass. o 16/20/24/32 bit audio. o 8k - 96k sample rates. o codec master only mode o DAPM. Vladimir

1 0

[alsa-devel] Intel HD Audio
by Chris Pemberton 13 Nov '07

13 Nov '07

I've been trying to fix audio on my Lenovo 3000 N100 laptop for quite some time; on this list and via bug #0003468. I finally got tired of the xorg graphics woes and hacking for sound, so I am now "temporarily" running Vista. I'd like to get this audio fixed permanently, so if you need any details from the working Vista drivers, let me know before I get sick of Windows and switch back to Linux. Chris

2 3

[alsa-devel] New developer question
by Pete Fawcett 13 Nov '07

13 Nov '07

Dear Alsa Developers I am trying to find out how to submit some fixes to ALSA drivers. All I could find on the web site was to subscribe to this mailing list so I hope this is the correct place to raise this. I have made some changes to the driver for AD1988 device. This is an HDA device and handled by the snd_hda_intel module. The specific hardware is handled by patch_analog.c I have made some changes so that 5.1 surround sound is now output on my motherboard. Previously stereo sound was output to all all three pairs of speakers. That's all fine for me, but in the spirit of open source I thought I should submit these changes for wider use. Please advise me how I proceed from here. Thanks Pete

2 1

[alsa-devel] issues with pulse plugin
by Mike Gorse 13 Nov '07

13 Nov '07

I have run into what is basically another instance of bug 3470 (pulse_hw_params asserting because the stream is already initialized). The application that I am working with is calling snd_pcm_hw_params each time it plays a sample. The following patch makes things work for me (actually the app still doesn't work because it uses poll() in a way that is incompatible with the pulse plugin, but that is a separate issue, and I don't know if it should be considered a bug in the plugin.) --- pulse/pcm_pulse.c.orig 2007-11-08 15:13:12.000000000 -0500 +++ pulse/pcm_pulse.c 2007-11-08 15:13:12.000000000 -0500 @@ -497,7 +497,7 @@ assert(pcm->p); //Resolving bugtrack ID 0003470 - if(!(base && snd_pcm_state(base) == SND_PCM_STATE_PREPARED)) + if(!(base && (snd_pcm_state(base) == SND_PCM_STATE_PREPARED || snd_pcm_state(base) == SND_PCM_STATE_SETUP))) assert(!pcm->stream); pa_threaded_mainloop_lock(pcm->p->mainloop); -- I couldn't add this to the bug because it was marked as resolved. Any comments? Or is the app behaving in a way that is counter to the specification? Also, I'm still encountering bug 2601 (a separate assert failure that happens if period_size == 0) even with the patch to pcm_ioplug.c. I think that PERIOD_TIME and PERIODS need to be constrained; I added a couple of comments to the bug. Thanks, -- Mike Gorse / AIM:linvortex / http://mgorse.freeshell.org --

2 3