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

[alsa-devel] [PATCH] configure.in: Fix variable name
by Ozan Çağlayan 26 Nov '10

26 Nov '10

Fix variable name for --with-asound-state-dir as currently we have to pass --with-ASOUND_STATE_DIR= which is wrong and inconsistent with the other switches. Signed-off-by: Ozan Çağlayan <ozan(a)pardus.org.tr> --- configure.in | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/configure.in b/configure.in index 94a2b17..77778da 100644 --- a/configure.in +++ b/configure.in @@ -289,7 +289,7 @@ if test "x$with_systemdsystemunitdir" != xno; then fi AM_CONDITIONAL(HAVE_SYSTEMD, [test -n "$with_systemdsystemunitdir" -a "x$with_systemdsystemunitdir" != xno ]) -AC_ARG_WITH(ASOUND_STATE_DIR, +AC_ARG_WITH([asound-state-dir], AS_HELP_STRING([--with-asound-state-dir=DIR], [Directory to place asound.state file in]), [ASOUND_STATE_DIR="$withval"], [ASOUND_STATE_DIR="/var/lib/alsa"]) -- 1.7.2.1

2 1

Re: [alsa-devel] ALSA Multichannel through HDMI
by John Ettedgui 26 Nov '10

26 Nov '10

On Thu, Nov 25, 2010 at 3:34 PM, John Ettedgui <john.ettedgui(a)gmail.com> wrote: > On Thu, Nov 25, 2010 at 3:15 PM, Anssi Hannula <anssi.hannula(a)iki.fi> wrote: >> On 26.11.2010 01:11, John Ettedgui wrote: >>> On Thu, Nov 25, 2010 at 2:58 PM, Anssi Hannula <anssi.hannula(a)iki.fi> wrote: >>>> On 26.11.2010 00:42, John Ettedgui wrote: >>>>> On Thu, Nov 25, 2010 at 11:19 AM, Anssi Hannula <anssi.hannula(a)iki.fi> wrote: >>>>>> On 25.11.2010 14:21, John Ettedgui wrote: >>>>>>> On Thu, Nov 25, 2010 at 3:40 AM, Anssi Hannula <anssi.hannula(a)iki.fi> wrote: >>>>>>>> On 25.11.2010 13:09, John Ettedgui wrote: >>>>>>>>> On Thu, Nov 25, 2010 at 2:49 AM, Anssi Hannula <anssi.hannula(a)iki.fi> wrote: >>>>>>>>>> On 25.11.2010 06:12, John Ettedgui wrote: >>>>>>>>>>> I'm actually trying to get a 5.1 setup working without passthrough, if >>>>>>>>>>> I use mplayer with passthrough my receiver behaves just fine, but I am >>>>>>>>>>> trying to send the 6 channels from the computer. >>>>>>>>>>> >>>>>>>>>>> I was actually told that the current ALSA driver for radeon's hdmi >>>>>>>>>>> does not support more than 2 channels, and that would explain why I'm >>>>>>>>>>> getting this behavior. Is that true though? >>>>>>>>>> >>>>>>>>>> Yes. >>>>>>>>>> >>>>>>>>>> >>>>>>>>>> However, strangely your log output seems to show the intel's hdmi parser >>>>>>>>>> being used instead: >>>>>>>>>> >>>>>>>>>>> ALSA sound/pci/hda/patch_intelhdmi.c:842: hdmi_setup_stream: NID=0x2, >>>>>>>>>>> stream=0x1, new-format=0x11 >>>>>>>>>> >>>>>>>>>> Were you just trying some patch at the time? >>>>>>>>> >>>>>>>>> yes Wu gave me a patch to try against the kernel, I'm guessing this is >>>>>>>>> the reason you see something unexpected. >>>>>>>> >>>>>>>> Did 6 channels work with that patch? (possibly with a wrong channel order) >>>>>>> Not any better as far as I remember. >>>>>> >>>>>> OK. Did 2 channel audio continue to work with it? >>>>>> >>>>>> If you are not sure, here's the patch doing the same thing against >>>>>> current alsa driver: >>>>>> http://stuff.onse.fi/0001-ALSA-hda-Use-generic-HDMI-code-for-ATI-HDMI-codec… >>>>>> >>>>>> Confirming that would help (it would allow to switch the ATI chip to use >>>>>> the generic parser instead of the ATI-specific one), though not with the >>>>>> multichannel issue (see below). >>>>>> >>>>>> [...] >>>>>> >>>>>>>>> Since I just moved to jack/rca cables I'm in no hurry anymore, but is >>>>>>>>> there any reason why the driver cannot work with more than 2 channels? >>>>>>>> >>>>>>>> I don't really know (it is simply not implemented). >>>>>>> Alright. >>>>>>>> >>>>>>>>> Anything I could do to help with that? >>>>>>>> >>>>>>>> Possibly. But see above, so that we won't try the same things you >>>>>>>> already tried :) >>>>>>>> >>>>>>> Sure :) >>>>>> >>>>>> Looking again at the the alsa info output you provided, I don't think I >>>>>> have any great ideas. It looks like the chip simply reports that it >>>>>> supports a maximum of 2 channels. >>>>>> >>>>>> One option could be hardcoding "chans = 8;" in generic_hdmi_build_pcms() >>>>>> in sound/pci/hda/patch_hdmi.c (assuming you have the above patch and 2 >>>>>> channel audio works with it). However, I find it rather unlikely that it >>>>>> would make any difference. >>>>>> >>>>>> -- >>>>>> Anssi Hannula >>>>>> >>>>> >>>>> Anssi, >>>>> the patch fails on my kernel, I cannot find the hda_codec_preset >>>>> manually in there either. >>>>> Did you want me to try that against a 2.6.37 rc instead of a 2.6.36? >>>> >>>> For 2.6.36 I think you can just use your earlier patch. >>>> >>>> Note that on that kernel generic_hdmi_build_pcms() is >>>> intel_hdmi_build_pcms() in patch_intelhdmi.c. >>>> >>> ok >>>> BTW, I've just been informed by another ATI user that his card was >>>> actually stereo-only, while he thought earlier it was multichannel >>>> capable. So, are you sure your card is a multichannel one? >>>> (if not, it would explain why the card says it is not) >>>> >>>> -- >>>> Anssi Hannula >>>> >>> Yes, it works fine in Window so I believe the hardware part is good. >> >> And you are sure it is multichannel PCM, and not e.g. multichannel DTS >> or multichannel AC-3 with on-the-fly compression, which are possible >> with stereo hdmi as well? >> >> Just making sure :) >> >> -- >> Anssi Hannula >> > > Well, I have not super tested it, but the reason I believe it is is > simple, when I use passthrough my receiver says Dolby digital or > something like that, but in Windows just going around or playing games > it says Multichannel. > > I'll get the patch applied in a few minutes, I was running out of > memory and my system was getting really slow... :) > I am not sure if I did it right but now I don't even get hdmi sound working anymore. It is not listed in aplay-l, and dmesg gives me this: [drm:radeon_dvi_detect] *ERROR* HDMI Type A-1: probed a monitor but no|invalid EDID I've attached the patch I used, since I could not used the previous one I had to hack it, maybe I broke something? (I have not set channels to 8 yet, just changed the parser) Thanks, John

2 5

Re: [alsa-devel] ALSA Multichannel through HDMI
by John Ettedgui 26 Nov '10

26 Nov '10

On Tue, Jan 12, 2010 at 9:02 PM, Wu Fengguang <fengguang.wu(a)intel.com> wrote: > On Wed, Jan 13, 2010 at 12:55:09PM +0800, John Ettedgui wrote: >> On Tue, Jan 12, 2010 at 8:52 PM, Wu Fengguang <fengguang.wu(a)intel.com> wrote: >> > On Wed, Jan 13, 2010 at 12:48:13PM +0800, John Ettedgui wrote: >> >> On Tue, Jan 12, 2010 at 5:45 PM, Wu Fengguang <fengguang.wu(a)intel.com> wrote: >> >> > On Wed, Jan 13, 2010 at 09:32:04AM +0800, John Ettedgui wrote: >> >> >> On Tue, Jan 12, 2010 at 5:09 PM, Wu Fengguang <fengguang.wu(a)intel.com> wrote: >> >> > >> >> >> >> > You can play a 5.1 track and find something like this in dmesg: >> >> >> >> > >> >> >> >> > HDMI: ASP channel %d => slot %d >> >> >> >> > HDMI: ASP channel %d => slot %d >> >> >> >> > HDMI: ASP channel %d => slot %d >> >> >> >> > HDMI: ASP channel %d => slot %d >> >> >> >> > HDMI: ASP channel %d => slot %d >> >> >> >> > HDMI: ASP channel %d => slot %d >> >> >> >> > HDMI: ASP channel %d => slot %d >> >> >> >> > HDMI: ASP channel %d => slot %d >> >> >> >> > >> >> >> >> After trying speaker-test -c6, I could not find anything like that. >> >> >> >> If I try mplayer -channel 6, it just does not output any sound. >> >> >> > >> >> >> > Did you enable CONFIG_SND_DEBUG_VERBOSE? >> >> >> > >> >> >> >> >> >> Yes. >> >> > >> >> > Then we need more debugging.. >> >> > >> >> > Can you send me the file /proc/asound/card*/eld*? >> >> Attached to this email >> >> > And the alsainfo output: >> >> > http://www.alsa-project.org/alsa-info.sh >> >> and this one is at >> >> http://www.alsa-project.org/db/?f=59773f9117f653f3d20d21b90dc2104eeb5d1ecd >> > >> > John, please try this and then redo speaker-test and check dmesg: >> > >> > # echo monitor_present 1 > /proc/asound/card*/eld* >> > >> > Thanks, >> > Fengguang >> > >> Alright, I guess this is the output we are looking for: >> >> >> ALSA sound/pci/hda/patch_intelhdmi.c:842: hdmi_setup_stream: NID=0x2, >> stream=0x1, new-format=0x11 >> ALSA sound/pci/hda/hda_intel.c:1641: azx_pcm_prepare: bufsize=0x10000, >> format=0x11 >> HDMI: ASP channel 0 => slot 0 >> HDMI: ASP channel 0 => slot 0 >> HDMI: ASP channel 0 => slot 0 >> HDMI: ASP channel 0 => slot 0 >> HDMI: ASP channel 0 => slot 0 >> HDMI: ASP channel 0 => slot 0 >> HDMI: ASP channel 0 => slot 0 >> HDMI: ASP channel 0 => slot 0 > > Oh well, I guess your hardware doesn't support channel mapping.. > > You can still do it in user space: > http://article.gmane.org/gmane.linux.alsa.user/33393 > > Thanks, > Fengguang > I am sorry to resurrect this thread after so long, but I still have not been able to get anywhere. Also it just came to my attention that if my receiver always sees the hdmi connection as stereo (apart from when using ac3/dts passthrough) channel remapping is not the solution, but somehow something else is messed up. I think I should try some options in modprobe.conf for snd-hda-intel but I am not sure which ones, and from all the examples I can find on the web, it seems none of them are talking about a radeon hdmi connection... So I'm asking for advice here again :) Thanks!

3 13

Re: [alsa-devel] terrible sound from my ALC850 / 880 [Alsa driver version 1.0.21] debian ( squeeze )
by Colin Kincaid Williams 26 Nov '10

26 Nov '10

On Wed, Nov 24, 2010 at 12:12 AM, Colin Kincaid Williams <discord(a)uw.edu> wrote: > Hello, > > I'm experiencing crap sound preformance from my ALC 850/880 chipset. > I'm first wondering if their is a way to set the model other than the > /etc/modprobe.d/alsa-base.conf file? Can I try different models > without rebooting? So far I've tried the model 6stack, since that > sounds like what I have on my motherboard. Furthermore, does anybody > have any ideas on what else I can do so that my computer can sound at > least as good as my $20 coby mp3 player? > > > cat /proc/asound/version > Advanced Linux Sound Architecture Driver Version 1.0.21. > > > > The output from lspci : > > > 00:10.1 Audio device [0403]: nVidia Corporation MCP51 High Definition > Audio [10de:026c] (rev a2) > Subsystem: Giga-byte Technology Device [1458:a102] > Control: I/O- Mem+ BusMaster+ SpecCycle- MemWINV- VGASnoop- ParErr- > Stepping- SERR- FastB2B- DisINTx- > Status: Cap+ 66MHz+ UDF- FastB2B+ ParErr- DEVSEL=fast >TAbort- > <TAbort- <MAbort- >SERR- <PERR- INTx- > Latency: 0 (500ns min, 1250ns max) > Interrupt: pin B routed to IRQ 22 > Region 0: Memory at f5100000 (32-bit, non-prefetchable) [size=16K] > Capabilities: [44] Power Management version 2 > Flags: PMEClk- DSI- D1- D2- AuxCurrent=0mA PME(D0-,D1-,D2-,D3hot+,D3cold+) > Status: D0 NoSoftRst- PME-Enable- DSel=0 DScale=0 PME- > Capabilities: [50] MSI: Enable- Count=1/1 Maskable+ 64bit+ > Address: 0000000000000000 Data: 0000 > Masking: 00000000 Pending: 00000000 > Capabilities: [6c] HyperTransport: MSI Mapping Enable+ Fixed+ > Kernel driver in use: HDA Intel >

1 0

[alsa-devel] Alsa-user list down
by Colin Kincaid Williams 26 Nov '10

26 Nov '10

I've tried subscribing to the Alsa-user list and get no response, it seems that the list is down.

1 0

[alsa-devel] [PATCH] ASoC: tlv320aic3x - fix variable may be used uninitialized warning
by Axel Lin 25 Nov '10

25 Nov '10

If aic3x_read failed , val is used uninitialized. Fix it by initializing val to 0. This patch fixes below compile warning: sound/soc/codecs/tlv320aic3x.c: In function 'aic3x_get_gpio': sound/soc/codecs/tlv320aic3x.c:1183: warning: 'val' may be used uninitialized in this function sound/soc/codecs/tlv320aic3x.c: In function 'aic3x_headset_detected': sound/soc/codecs/tlv320aic3x.c:1211: warning: 'val' may be used uninitialized in this function sound/soc/codecs/tlv320aic3x.c: In function 'aic3x_button_pressed': sound/soc/codecs/tlv320aic3x.c:1219: warning: 'val' may be used uninitialized in this function Signed-off-by: Axel Lin <axel.lin(a)gmail.com> --- sound/soc/codecs/tlv320aic3x.c | 6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/sound/soc/codecs/tlv320aic3x.c b/sound/soc/codecs/tlv320aic3x.c index bfd036a..9de7c38 100644 --- a/sound/soc/codecs/tlv320aic3x.c +++ b/sound/soc/codecs/tlv320aic3x.c @@ -1180,7 +1180,7 @@ EXPORT_SYMBOL_GPL(aic3x_set_gpio); int aic3x_get_gpio(struct snd_soc_codec *codec, int gpio) { u8 reg = gpio ? AIC3X_GPIO2_REG : AIC3X_GPIO1_REG; - u8 val, bit = gpio ? 2: 1; + u8 val = 0, bit = gpio ? 2 : 1; aic3x_read(codec, reg, &val); return (val >> bit) & 1; @@ -1208,7 +1208,7 @@ EXPORT_SYMBOL_GPL(aic3x_set_headset_detection); int aic3x_headset_detected(struct snd_soc_codec *codec) { - u8 val; + u8 val = 0; aic3x_read(codec, AIC3X_HEADSET_DETECT_CTRL_B, &val); return (val >> 4) & 1; } @@ -1216,7 +1216,7 @@ EXPORT_SYMBOL_GPL(aic3x_headset_detected); int aic3x_button_pressed(struct snd_soc_codec *codec) { - u8 val; + u8 val = 0; aic3x_read(codec, AIC3X_HEADSET_DETECT_CTRL_B, &val); return (val >> 5) & 1; } -- 1.7.2

2 2

[alsa-devel] [PATCH 2/2 RESEND] ASoC: Fix resource reclaim for osk5912
by Axel Lin 25 Nov '10

25 Nov '10

In current implementation, there are resources leak in the error path. This patch properly reclaims the allocated resources in the error path. Also adds a missing clk_put in osk_soc_exit. Signed-off-by: Axel Lin <axel.lin(a)gmail.com> Acked-by: Jarkko Nikula <jhnikula(a)gmail.com> --- sound/soc/omap/osk5912.c | 11 ++++++++--- 1 files changed, 8 insertions(+), 3 deletions(-) diff --git a/sound/soc/omap/osk5912.c b/sound/soc/omap/osk5912.c index 18d053d..7e75e77 100644 --- a/sound/soc/omap/osk5912.c +++ b/sound/soc/omap/osk5912.c @@ -177,7 +177,8 @@ static int __init osk_soc_init(void) tlv320aic23_mclk = clk_get(dev, "mclk"); if (IS_ERR(tlv320aic23_mclk)) { printk(KERN_ERR "Could not get mclk clock\n"); - return -ENODEV; + err = PTR_ERR(tlv320aic23_mclk); + goto err2; } /* @@ -188,7 +189,7 @@ static int __init osk_soc_init(void) if (clk_set_rate(tlv320aic23_mclk, CODEC_CLOCK)) { printk(KERN_ERR "Cannot set MCLK for AIC23 CODEC\n"); err = -ECANCELED; - goto err1; + goto err3; } } @@ -196,9 +197,12 @@ static int __init osk_soc_init(void) (uint) clk_get_rate(tlv320aic23_mclk), CODEC_CLOCK); return 0; -err1: + +err3: clk_put(tlv320aic23_mclk); +err2: platform_device_del(osk_snd_device); +err1: platform_device_put(osk_snd_device); return err; @@ -207,6 +211,7 @@ err1: static void __exit osk_soc_exit(void) { + clk_put(tlv320aic23_mclk); platform_device_unregister(osk_snd_device); } -- 1.7.2

2 1

[PATCH] alsactl: Add a --with-udev-rules-dir configure option.
by Colin Guthrie 25 Nov '10

25 Nov '10

After the previous patch to install udev rules it is impossible to do a non-root, custom-prefix make install. This is generally not a problem when building official packages but it can be a pain for developers and debuggers etc. This is essentially the same patch as was already made in PulseAudio in commit e8a5746f2fcae59bfd18d39b621509b3ef130453. --- alsactl/Makefile.am | 2 -- configure.in | 6 ++++++ 2 files changed, 6 insertions(+), 2 deletions(-) diff --git a/alsactl/Makefile.am b/alsactl/Makefile.am index 721b619..5cfc415 100644 --- a/alsactl/Makefile.am +++ b/alsactl/Makefile.am @@ -11,8 +11,6 @@ alsactl_SOURCES=alsactl.c state.c utils.c init_parse.c alsactl_CFLAGS=$(AM_CFLAGS) -DSYS_ASOUNDRC=\"$(ASOUND_STATE_DIR)/asound.state\" noinst_HEADERS=alsactl.h list.h init_sysdeps.c init_utils_string.c init_utils_run.c init_sysfs.c -udevrulesdir=/lib/udev/rules.d - dist_udevrules_DATA = \ 90-alsa-restore.rules diff --git a/configure.in b/configure.in index 94a2b17..5b3491d 100644 --- a/configure.in +++ b/configure.in @@ -112,6 +112,12 @@ if test x"$alsaconf" = xtrue; then fi AM_CONDITIONAL(USE_XMLTO, test x"$xmlto" = xyes) +AC_ARG_WITH( + [udev-rules-dir], + AS_HELP_STRING([--with-udev-rules-dir],[Directory where to install udev rules to (defaults to /lib/udev/rules.d)]), + [udevrulesdir=$withval], [udevrulesdir="/lib/udev/rules.d"]) +AC_SUBST(udevrulesdir) + dnl Checks for header files. AC_HEADER_STDC if test x$alsamixer = xtrue; then -- 1.7.3.2 --------------050704060108030906000201--

1 0

[alsa-devel] [PATCH 2.6.37-rc1] ASoC: OMAP: fix OMAP1 compilation problem
by Janusz Krzysztofik 25 Nov '10

25 Nov '10

In the new code introduced with commit cf4c87abe238ec17cd0255b4e21abd949d7f811e, "OMAP: McBSP: implement McBSP CLKR and FSR signal muxing via mach-omap2/mcbsp.c", the way omap1 build is supposed to bypass omap2 specific functionality doesn't optimize out all omap2 specific stuff. This breaks linking phase for omap1 machines, giving "undefined reference to `omap2_mcbsp1_mux_clkr_src'" and "undefined reference to `omap2_mcbsp1_mux_fsr_src'" errors. Fix it. Created and tested against linux-2.6.37-rc1. Signed-off-by: Janusz Krzysztofik <jkrzyszt(a)tis.icnet.pl> --- sound/soc/omap/omap-mcbsp.c | 8 ++++++++ 1 file changed, 8 insertions(+) --- linux-2.6.37-rc1/sound/soc/omap/omap-mcbsp.c.orig 2010-11-01 22:43:16.000000000 +0100 +++ linux-2.6.37-rc1/sound/soc/omap/omap-mcbsp.c 2010-11-02 15:41:42.000000000 +0100 @@ -644,15 +644,23 @@ static int omap_mcbsp_dai_set_dai_sysclk case OMAP_MCBSP_CLKR_SRC_CLKR: + if (cpu_class_is_omap1()) + break; omap2_mcbsp1_mux_clkr_src(CLKR_SRC_CLKR); break; case OMAP_MCBSP_CLKR_SRC_CLKX: + if (cpu_class_is_omap1()) + break; omap2_mcbsp1_mux_clkr_src(CLKR_SRC_CLKX); break; case OMAP_MCBSP_FSR_SRC_FSR: + if (cpu_class_is_omap1()) + break; omap2_mcbsp1_mux_fsr_src(FSR_SRC_FSR); break; case OMAP_MCBSP_FSR_SRC_FSX: + if (cpu_class_is_omap1()) + break; omap2_mcbsp1_mux_fsr_src(FSR_SRC_FSX); break; default:

6 14

[alsa-devel] [PATCH] ASoC: neo1973_gta02_wm8753: fix wrong parameter for snd_soc_register_dai and snd_soc_unregister_dai
by Axel Lin 25 Nov '10

25 Nov '10

Signed-off-by: Axel Lin <axel.lin(a)gmail.com> --- sound/soc/samsung/neo1973_gta02_wm8753.c | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/sound/soc/samsung/neo1973_gta02_wm8753.c b/sound/soc/samsung/neo1973_gta02_wm8753.c index 920e8d1..8c65b63 100644 --- a/sound/soc/samsung/neo1973_gta02_wm8753.c +++ b/sound/soc/samsung/neo1973_gta02_wm8753.c @@ -438,7 +438,7 @@ static int __init neo1973_gta02_init(void) return -ENOMEM; /* register bluetooth DAI here */ - ret = snd_soc_register_dai(&neo1973_gta02_snd_device->dev, -1, &bt_dai); + ret = snd_soc_register_dai(&neo1973_gta02_snd_device->dev, &bt_dai); if (ret) { platform_device_put(neo1973_gta02_snd_device); return ret; @@ -491,7 +491,7 @@ module_init(neo1973_gta02_init); static void __exit neo1973_gta02_exit(void) { - snd_soc_unregister_dai(&neo1973_gta02_snd_device->dev, -1); + snd_soc_unregister_dai(&neo1973_gta02_snd_device->dev); platform_device_unregister(neo1973_gta02_snd_device); gpio_free(GTA02_GPIO_HP_IN); gpio_free(GTA02_GPIO_AMP_SHUT); -- 1.7.2

3 2