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

Re: [alsa-devel] how to enumerate useful ALSA devices?
by Andrew Eikum 11 Mar '12

11 Mar '12

> The 2008 audio guide recommends > http://0pointer.de/blog/projects/guide-to-sound-apis.html > "Do not use the snd_card_xxx() APIs. For enumerating use > snd_device_name_hint() (and related functions). > snd_card_xxx() is obsolete... Bluetooth audio are not included" > > Is that correct advice? > > Wine still uses snd_card, so it knows only about "default" and > "plughw:x,y". I believe that's wrong but don't know what should be > used instead in order not to come up with a long list of artificial > names mentioned in /usr/share/alsa/alsa.conf such as surround51 when > the poor on-board card knows nothing but 16 bit stereo. > > I'm looking for a list of working devices on this machine, which likely > should contain "default" (usually leading to PA on a recent Ubuntu machine), > "plug:dmix" (PA getting out of the way when trying to use other devices), > "hw:0,0" and presumably not much more. Bump, still hoping for an answer to this. I tried experimenting with snd_device_name_hint(). It gave me what looked like a promising list, consisting of: null front:CARD=SB,DEV=0 surround40:CARD=SB,DEV=0 surround41:CARD=SB,DEV=0 surround50:CARD=SB,DEV=0 surround51:CARD=SB,DEV=0 surround71:CARD=SB,DEV=0 iec958:CARD=SB,DEV=0 This is identical to the list from "aplay -L". But, passing these to aplay fails if I have PulseAudio running ("device in use"). If I kill PA, then they don't fail, but give warnings about rate conversion and suggest the plug plugin. So I stick "plug:" in front of the device name, and they fail again ("unknown parameter front:CARD"). Clearly I'm doing something very wrong. But I don't know what! Then it occurred to me that this list also contains neither "default" nor the "pulse" device in my asound.conf[1]. Both of these work when passed to aplay. Looking to other projects for help, I find that VLC uses the snd_card_ functions, just like Wine currently does. Of course, that fails to find the "pulse" device from asound.conf. And mplayer's libao2 uses a couple of hard-coded device names, which is even worse (right?). So I'm really stuck here. Where am I supposed to get a list of devices that I can present to the user, including their Bluetooth devices and virtual devices from asound.conf? [1] asound.conf from Arch Linux pulseaudio-alsa package: http://projects.archlinux.org/svntogit/packages.git/tree/trunk/asound.conf?… Andrew

4 10

Re: [alsa-devel] Port C Out selector , Port E Out selector of ad1988 and ad1988_3stack_dac_nids
by Raymond Yau 10 Mar '12

10 Mar '12

2011/9/16 Takashi Iwai <tiwai(a)suse.de>: > At Fri, 16 Sep 2011 11:59:54 +0800, > Raymond Yau wrote: >> >> 2011/9/12 Takashi Iwai <tiwai(a)suse.de>: >> > At Wed, 31 Aug 2011 11:21:00 +0800, >> > Raymond Yau wrote: >> >> >> >> HDA - Add Headphone Playback Volume control for ad1988/ad1989 >> >> >> >> - use DAC0 instead of DAC1 for Port-A Headphone >> >> - assign 0x03 to spec->multiout.hp_nid except model="6stack-dig-fp" >> > >> > Thanks, applied now. >> > >> > >> > Takashi >> > >> >> while testing the headphone volume control , >> >> I discover that the "Port C Out selector" and "Port E Out Selector" >> (i.e. node 0x31 and 0x32 ) by commit in patch_anlog.c >> >> fd66e0d0591dd12eb0bea1e9f3aa194bb93cebbd >> >> >> >> * AD1988 >> * >> * Output pins and routes >> * >> * Pin Mix Sel DAC >> * port-A 0x11 (mute/hp) <- 0x22 <- 0x37 <- 03/04/06 >> * port-B 0x14 (mute/hp) <- 0x2b <- 0x30 <- 03/04/06 >> * port-C 0x15 (mute) <- 0x2c <- 0x31 <- 05/0a >> * port-D 0x12 (mute/hp) <- 0x29 <- 04 >> * port-E 0x17 (mute/hp) <- 0x26 <- 0x32 <- 05/0a >> >> >> >> The connection list of node 0x31 and 0x32 are different from datasheet >> >> >> Table 10. Audio Widget >> >> 31 Audio selector Selects DAC_2 and DAC_4 for PORT-C >> PORT-E Out Selector >> 32 Audio selector Selects DAC_2 and DAC_4 for PORT-E >> PORT-C In Selector >> >> >> >> Codec: Analog Devices AD1988 >> Address: 0 >> AFG Function Id: 0x1 (unsol 0) >> Vendor Id: 0x11d41988 >> Subsystem Id: 0x104381e1 >> Revision Id: 0x100400 >> >> >> Node 0x31 [Audio Selector] wcaps 0x300101: Stereo >> Connection: 2 >> 0x04* 0x0a >> Node 0x32 [Audio Selector] wcaps 0x300101: Stereo >> Connection: 2 >> 0x05* 0x04 >> >> >> >> is the following patch based on real hardware ? >> >> >> commit 8c7c7b8dd2828b42c1230c6b0235e7d1dcf57e5 >> >> hda-codec - Fix surrounds on 3stack mode of AD1988 > > I don't see this commit id in my tree... > > > Takashi > http://git.alsa-project.org/?p=alsa-kernel.git;a=commit;h=fd66e0d0591dd12eb… Thu, 17 Nov 2005 14:31:34 +0000 (15:31 +0100) Add AD1988 codec support to hda-codec driver. http://git.alsa-project.org/?p=alsa-kernel.git;a=commitdiff;h=f8c7c7b8dd282… Thu, 24 Nov 2005 15:17:20 +0000 (16:17 +0100)] Fixed the swapped surround/CLFE on 3stack mode of AD1988. hda-emu codecs/ad1988a -m "3stack" route -a 0x15 Out[0a] -x Sel[31] -- Mix[2c] -> Pin[15] Out[04] -> Sel[31] -- Mix[2c] -> Pin[15] route -a 0x17 Out[04] -> Sel[32] -- Mix[26] -> Pin[17] Out[05] -x Sel[32] -- Mix[26] -> Pin[17]

2 4

[alsa-devel] [PATCH][RFC] intel-hda: proper support for Asus N55SF/N75SF laptops with external subwoofer
by Andrew 06 Mar '12

06 Mar '12

Hello all, The following patch adds full support for Asus Laptops N55SF/N75SF (Possibly more models) with external subwoofer. It adds a new model (asus-nx5) and proper PCIID detection. Example asoundrc for upmixing and dmix is attached as well. (Thanks to K900 from AgiliaLinux dev. conference for it as well as testing on N55SF) Currently everything is supported except Subwoofer pin autosensing. It is somehow implemented in windows drivers, but I couldn't get it neither via codec's gpio or regular 0x16 unsol event. In 2ch mode the subwoofer is off, so should save a little bit of battery. I couldn't test automic feature and spdif, because I don't have the hardware around. Since alc662_quirks.c looks a little bit overbloated, I moved my quirks to a separate file. If that's not OK, I'll resubmit the patch. (This is my first patch to alsa-devel, so I might be missing something) Regards, Andrew

2 1

[alsa-devel] ALC892, Clevo P150HM: Input Source gets reset to "Mic" every resume
by Julian Sikorski 02 Mar '12

02 Mar '12

Hi, whenever I resume the machine, the Input Source would get changed back from "Internal Mic" to "Mic". This is a bit annoying, because pulseaudio is unaware of that change. As a result, everything looks fine in pavucontrol but no sound gets recorded. A quick look at alsamixer -c0 revealed the problem. My alsa info: http://www.alsa-project.org/db/?f=c5776b615a5f3b9ae624e4250fb0c2b69118ce26 This is unrelated to the other problems I have been reporting, since this happens on a mostly-stock Fedora 3.1.2 kernel (the patch I added deals with xhci, not alsa). Regards, Julian

3 8

[alsa-devel] [PATCH] ALSA : Au88x0 - Support 4 channels playback when AC97 codec has SDAC bit , Reduce the number of playback devices of au8830 from 32 to 16
by Raymond Yau 28 Feb '12

28 Feb '12

[PATCH - ALSA 1/2] Au88x0 - Support 4 channels playback when AC97 codecs has SDAC bit - Check SDAC bit of AC97 codec for supporting 4 channels playback. [PATCH - ALSA 2/2] Au88x0 - Reduce the number of playback subdevices of au8830 from 32 to 16 - The maximum number of playback streams depend on the number of sample rate conveters (16) and the number of DMA channels (32).

2 3

Re: [alsa-devel] [RFC PATCH] ARM ASoC: add sound driver for imx27pdk using mc13783 codec
by Fabio Estevam 09 Feb '12

09 Feb '12

Hi Philippe, On Wed, Jan 25, 2012 at 1:45 PM, Philippe Rétornaz <philippe.retornaz(a)epfl.ch> wrote: > I have a working sound support for the mx31moboard boards available here: > http://github.com/retornaz/linux/commits/moboard-sound That's cool :-) > It's basically Sascha mc13783 codec driver (from his git tree) rebased on 3.2 > and some mx31moboad specific code. > > I can playback & record sound on our board (2 channels@16Khz) with the sdma. > > Sascha, do you have any plan to upstream the mc13783 codec driver ? In case Sascha doesn't have plans to upstream mc13783, is this something you could do? Thanks, Fabio Estevam

3 6

[alsa-devel] [RESEND PATCH] davinci: CQ93VC: remove machine specific header file inclusion from codec driver
by Manjunath Hadli 08 Feb '12

08 Feb '12

remove unnecessary inclusion of machine specific header file mach/dm365.h from cq93vc.c voice codec driver which comes in the way of platform code consolidation. Signed-off-by: Manjunath Hadli <manjunath.hadli(a)ti.com> Cc: Liam Girdwood <lrg(a)ti.com> Cc: Mark Brown <broonie(a)opensource.wolfsonmicro.com> Cc: Jaroslav Kysela <perex(a)perex.cz> CC: Takashi Iwai <tiwai(a)suse.de> Cc: <alsa-devel(a)alsa-project.org> Cc: <linux-kernel(a)vger.kernel.org> --- sound/soc/codecs/cq93vc.c | 2 -- 1 files changed, 0 insertions(+), 2 deletions(-) diff --git a/sound/soc/codecs/cq93vc.c b/sound/soc/codecs/cq93vc.c index 46dbfd0..ff4eb7a 100644 --- a/sound/soc/codecs/cq93vc.c +++ b/sound/soc/codecs/cq93vc.c @@ -38,8 +38,6 @@ #include <sound/soc.h> #include <sound/initval.h> -#include <mach/dm365.h> - static inline unsigned int cq93vc_read(struct snd_soc_codec *codec, unsigned int reg) { -- 1.6.2.4

3 3

[alsa-devel] [PATCH 1/15] sound/soc/mxs/mxs-saif.c: add missing iounmap
by Julia Lawall 08 Feb '12

08 Feb '12

From: Julia Lawall <Julia.Lawall(a)lip6.fr> Add missing iounmap in error handling code, in a case where the function already preforms iounmap on some other execution path. A simplified version of the semantic match that finds this problem is as follows: (http://coccinelle.lip6.fr/) // <smpl> @@ expression e; statement S,S1; int ret; @@ e = \(ioremap\|ioremap_nocache\)(...) ... when != iounmap(e) if (<+...e...+>) S ... when any when != iounmap(e) *if (...) { ... when != iounmap(e) return ...; } ... when any iounmap(e); // </smpl> Signed-off-by: Julia Lawall <Julia.Lawall(a)lip6.fr> --- sound/soc/mxs/mxs-saif.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/sound/soc/mxs/mxs-saif.c b/sound/soc/mxs/mxs-saif.c index 049e543..5ee0adb 100644 --- a/sound/soc/mxs/mxs-saif.c +++ b/sound/soc/mxs/mxs-saif.c @@ -680,7 +680,7 @@ static int mxs_saif_probe(struct platform_device *pdev) ret = -ENODEV; dev_err(&pdev->dev, "failed to get dma resource: %d\n", ret); - goto failed_ioremap; + goto failed_get_resource; } saif->dma_param.chan_num = dmares->start; @@ -739,6 +739,7 @@ failed_register: failed_get_irq2: free_irq(saif->irq, saif); failed_get_irq1: +failed_get_resource: iounmap(saif->base); failed_ioremap: release_mem_region(iores->start, resource_size(iores));

6 29

[alsa-devel] [PATCH 1/2] ASoC: Convert kirkwood-t5325 to use snd_soc_register_card()
by Axel Lin 07 Feb '12

07 Feb '12

Use snd_soc_register_card() instead of creating a "soc-audio" platform device. Signed-off-by: Axel Lin <axel.lin(a)gmail.com> --- I don't have this hardware, I'd appreciate if someone can test this patch. Axel arch/arm/mach-kirkwood/t5325-setup.c | 6 ++++ sound/soc/kirkwood/kirkwood-t5325.c | 47 +++++++++++++++++---------------- 2 files changed, 30 insertions(+), 23 deletions(-) diff --git a/arch/arm/mach-kirkwood/t5325-setup.c b/arch/arm/mach-kirkwood/t5325-setup.c index 966b2b3..f9d2a11 100644 --- a/arch/arm/mach-kirkwood/t5325-setup.c +++ b/arch/arm/mach-kirkwood/t5325-setup.c @@ -106,6 +106,11 @@ static struct platform_device hp_t5325_button_device = { } }; +static struct platform_device hp_t5325_audio_device = { + .name = "t5325-audio", + .id = -1, +}; + static unsigned int hp_t5325_mpp_config[] __initdata = { MPP0_NF_IO2, MPP1_SPI_MOSI, @@ -179,6 +184,7 @@ static void __init hp_t5325_init(void) kirkwood_sata_init(&hp_t5325_sata_data); kirkwood_ehci_init(); platform_device_register(&hp_t5325_button_device); + platform_device_register(&hp_t5325_audio_device); i2c_register_board_info(0, i2c_board_info, ARRAY_SIZE(i2c_board_info)); kirkwood_audio_init(); diff --git a/sound/soc/kirkwood/kirkwood-t5325.c b/sound/soc/kirkwood/kirkwood-t5325.c index 16b83e5..ce4f408 100644 --- a/sound/soc/kirkwood/kirkwood-t5325.c +++ b/sound/soc/kirkwood/kirkwood-t5325.c @@ -80,7 +80,6 @@ static struct snd_soc_dai_link t5325_dai[] = { }, }; - static struct snd_soc_card t5325 = { .name = "t5325", .owner = THIS_MODULE, @@ -93,38 +92,40 @@ static struct snd_soc_card t5325 = { .num_dapm_routes = ARRAY_SIZE(t5325_route), }; -static struct platform_device *t5325_snd_device; - -static int __init t5325_init(void) +static int __devinit t5325_probe(struct platform_device *pdev) { + struct snd_soc_card *card = &t5325; int ret; - if (!machine_is_t5325()) - return 0; - - t5325_snd_device = platform_device_alloc("soc-audio", -1); - if (!t5325_snd_device) - return -ENOMEM; - - platform_set_drvdata(t5325_snd_device, - &t5325); - - ret = platform_device_add(t5325_snd_device); - if (ret) { - printk(KERN_ERR "%s: platform_device_add failed\n", __func__); - platform_device_put(t5325_snd_device); - } + card->dev = &pdev->dev; + ret = snd_soc_register_card(card); + if (ret) + dev_err(&pdev->dev, "snd_soc_register_card() failed: %d\n", + ret); return ret; } -module_init(t5325_init); -static void __exit t5325_exit(void) +static int __devexit t5325_remove(struct platform_device *pdev) { - platform_device_unregister(t5325_snd_device); + struct snd_soc_card *card = platform_get_drvdata(pdev); + + snd_soc_unregister_card(card); + return 0; } -module_exit(t5325_exit); + +static struct platform_driver t5325_driver = { + .driver = { + .name = "t5325-audio", + .owner = THIS_MODULE, + }, + .probe = t5325_probe, + .remove = __devexit_p(t5325_remove), +}; + +module_platform_driver(t5325_driver); MODULE_AUTHOR("Arnaud Patard <arnaud.patard(a)rtp-net.org>"); MODULE_DESCRIPTION("ALSA SoC t5325 audio client"); MODULE_LICENSE("GPL"); +MODULE_ALIAS("platform:t5325-audio"); -- 1.7.5.4

2 2

Re: [alsa-devel] [PATCH 1/7 v2] dmaengine: add a simple dma library
by Guennadi Liakhovetski 06 Feb '12

06 Feb '12

Hi Shimoda-san On Fri, 27 Jan 2012, Shimoda, Yoshihiro wrote: > Hello Guennadi-san, > > 2012/01/26 23:56, Guennadi Liakhovetski wrote: > > This patch adds a library of functions, helping to implement dmaengine > > drivers for hardware, unable to handle scatter-gather lists natively. > > The first version of this driver only supports memcpy and slave DMA > > operation. > > > > Signed-off-by: Guennadi Liakhovetski <g.liakhovetski(a)gmx.de> > > --- > > > > v2: > > > > 1. switch from using a tasklet to threaded IRQ, which allowed to > > I tested this driver on 2 environment, but it could not work correctly. > - renesas_usbhs driver with shdma driver on the mackerel board > - renesas_usbhs driver with experimental SUDMAC driver > > In this case, should we modify the renesas_usbhs driver? Yes, you do. Please, see the respective version of the shdma patch. It doesn't request channel IRQs any more directly, instead, it uses a dma-simple wrapper function. Also operations change a bit. Thanks Guennadi > For reference, if I changed the threaded IRQ to tasklet, > the 2 environment worked correctly: > ============== > diff --git a/drivers/dma/dma-simple.c b/drivers/dma/dma-simple.c > index 49d8f7d..65a5170 100644 > --- a/drivers/dma/dma-simple.c > +++ b/drivers/dma/dma-simple.c > @@ -715,16 +715,18 @@ static irqreturn_t chan_irq(int irq, void *dev) > > spin_lock(&schan->chan_lock); > > - ret = ops->chan_irq(schan, irq) ? IRQ_WAKE_THREAD : IRQ_NONE; > + ret = ops->chan_irq(schan, irq) ? IRQ_HANDLED : IRQ_NONE; > + if (ret == IRQ_HANDLED) > + tasklet_schedule(&schan->tasklet); > > spin_unlock(&schan->chan_lock); > > return ret; > } > > -static irqreturn_t chan_irqt(int irq, void *dev) > +static void chan_irqt(unsigned long dev) > { > - struct dma_simple_chan *schan = dev; > + struct dma_simple_chan *schan = (struct dma_simple_chan *)dev; > const struct dma_simple_ops *ops = > to_simple_dev(schan->dma_chan.device)->ops; > struct dma_simple_desc *sdesc; > @@ -744,15 +746,13 @@ static irqreturn_t chan_irqt(int irq, void *dev) > spin_unlock_irq(&schan->chan_lock); > > simple_chan_ld_cleanup(schan, false); > - > - return IRQ_HANDLED; > } > > int dma_simple_request_irq(struct dma_simple_chan *schan, int irq, > unsigned long flags, const char *name) > { > - int ret = request_threaded_irq(irq, chan_irq, chan_irqt, > - flags, name, schan); > + int ret = request_irq(irq, chan_irq, flags, name, schan); > + tasklet_init(&schan->tasklet, chan_irqt, (unsigned long)schan); > > schan->irq = ret < 0 ? ret : irq; > > diff --git a/include/linux/dma-simple.h b/include/linux/dma-simple.h > index 5336674..beb1489 100644 > --- a/include/linux/dma-simple.h > +++ b/include/linux/dma-simple.h > @@ -68,6 +68,7 @@ struct dma_simple_chan { > int id; /* Raw id of this channel */ > int irq; /* Channel IRQ */ > enum dma_simple_pm_state pm_state; > + struct tasklet_struct tasklet; > }; > > /** > ============== > > Best regards, > Yoshihiro Shimoda > --- Guennadi Liakhovetski, Ph.D. Freelance Open-Source Software Developer http://www.open-technology.de/

3 11