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

[alsa-devel] [PATCH 2/2] ARM: S5PV210: Add the GPIO lib
by Chanwoo Choi 29 Jul '10

29 Jul '10

This patch add the definition of GPIO to support WM8994 audio codec on I2C bus. Signed-off-by : Chanwoo Choi <cw00.choi(a)samsung.com> Signed-off-by : Kyungmin Park <kyungmin.park(a)samsung.com> --- arch/arm/mach-s5pv210/include/mach/gpio.h | 10 ++++++++-- 1 files changed, 8 insertions(+), 2 deletions(-) diff --git a/arch/arm/mach-s5pv210/include/mach/gpio.h b/arch/arm/mach-s5pv210/include/mach/gpio.h index d6461ba..33a9867 100644 --- a/arch/arm/mach-s5pv210/include/mach/gpio.h +++ b/arch/arm/mach-s5pv210/include/mach/gpio.h @@ -52,6 +52,8 @@ #define S5PV210_GPIO_MP01_NR (8) #define S5PV210_GPIO_MP02_NR (4) #define S5PV210_GPIO_MP03_NR (8) +#define S5PV210_GPIO_MP04_NR (8) +#define S5PV210_GPIO_MP05_NR (8) /* GPIO bank numbers */ @@ -94,6 +96,8 @@ enum s5p_gpio_number { S5PV210_GPIO_MP01_START = S5PV210_GPIO_NEXT(S5PV210_GPIO_J4), S5PV210_GPIO_MP02_START = S5PV210_GPIO_NEXT(S5PV210_GPIO_MP01), S5PV210_GPIO_MP03_START = S5PV210_GPIO_NEXT(S5PV210_GPIO_MP02), + S5PV210_GPIO_MP04_START = S5PV210_GPIO_NEXT(S5PV210_GPIO_MP03), + S5PV210_GPIO_MP05_START = S5PV210_GPIO_NEXT(S5PV210_GPIO_MP04), }; /* S5PV210 GPIO number definitions */ @@ -127,13 +131,15 @@ enum s5p_gpio_number { #define S5PV210_MP01(_nr) (S5PV210_GPIO_MP01_START + (_nr)) #define S5PV210_MP02(_nr) (S5PV210_GPIO_MP02_START + (_nr)) #define S5PV210_MP03(_nr) (S5PV210_GPIO_MP03_START + (_nr)) +#define S5PV210_MP04(_nr) (S5PV210_GPIO_MP04_START + (_nr)) +#define S5PV210_MP05(_nr) (S5PV210_GPIO_MP05_START + (_nr)) /* the end of the S5PV210 specific gpios */ -#define S5PV210_GPIO_END (S5PV210_MP03(S5PV210_GPIO_MP03_NR) + 1) +#define S5PV210_GPIO_END (S5PV210_MP05(S5PV210_GPIO_MP05_NR) + 1) #define S3C_GPIO_END S5PV210_GPIO_END /* define the number of gpios we need to the one after the MP03() range */ -#define ARCH_NR_GPIOS (S5PV210_MP03(S5PV210_GPIO_MP03_NR) + \ +#define ARCH_NR_GPIOS (S5PV210_MP05(S5PV210_GPIO_MP05_NR) + \ CONFIG_SAMSUNG_GPIO_EXTRA + 1) #include <asm-generic/gpio.h>

2 1

[alsa-devel] [PATCH 0/3] ASoC: fsi: SPDIF support
by Kuninori Morimoto 29 Jul '10

29 Jul '10

Dear Mark, Liam These are FSI driver patches Kuninori Morimoto (3): ASoC: fsi: remove unnecessary clock processing ASoC: fsi: remove device id check ASoC: fsi: Add new funtion for SPDIF These patches are based on Mark's "for-2.6.36" tree. These are needed for FSI2 device and HDMI support. Best regards -- Kuninori Morimoto

3 6

[alsa-devel] acer aspire 8943g with alc670
by Miroslav Šulc 29 Jul '10

29 Jul '10

hello, i'd like to help with making the above soundcard to fully work, but have no clue what to do exactly. currently the soundcard works pretty weird, though at least sound goes out :-) i tried hda analyzer but did not find any documentation for it so have no idea what to do exactly. also, when clicking AUD_IN nodes, i get this error: ** (hda_analyzer.py:10063): WARNING **: column number is a boolean, but will be interpreted as an integer; this is likely not what you intended Traceback (most recent call last): File "/dev/shm/hda-analyzer/hda_analyzer.py", line 308, in selection_changed_cb self.__refresh() File "/dev/shm/hda-analyzer/hda_analyzer.py", line 276, in __refresh self.load() File "/dev/shm/hda-analyzer/hda_analyzer.py", line 336, in load self.node_window.add(NodeGui(node=n)) File "/dev/shm/hda-analyzer/hda_guilib.py", line 86, in __init__ self.__build_node(node, doframe) File "/dev/shm/hda-analyzer/hda_guilib.py", line 682, in __build_node vbox.pack_start(self.__build_amps(node), False, False) File "/dev/shm/hda-analyzer/hda_guilib.py", line 365, in __build_amps node.in_amp and node.amp_vals_in or None) File "/dev/shm/hda-analyzer/hda_guilib.py", line 328, in build_caps if ctl.hdactl.amp_index_match(idx): File "/dev/shm/hda-analyzer/hda_proc.py", line 170, in amp_index_match count = (self.amp_chs & 1) + ((self.amp_chs >> 1) & 1) TypeError: unsupported operand type(s) for &: 'NoneType' and 'int' can anybody pls direct me to some documentation that could help me to master hda analyzer and find the way to make the card work correctly? miroslav

3 7

[alsa-devel] mmotm 2010-07-27 - hda-intel audio fails to initialize
by Valdis.Kletnieks＠vt.edu 29 Jul '10

29 Jul '10

On Tue, 27 Jul 2010 14:56:50 PDT, akpm(a)linux-foundation.org said: > The mm-of-the-moment snapshot 2010-07-27-14-56 has been uploaded to > > http://userweb.kernel.org/~akpm/mmotm/ Dell Latitude E6500. Soundcard has gone astray, it appears. _From 2.6.35-rc3-mmotm0719 dmesg: [ 3.372928] usbcore: registered new interface driver usbhid [ 3.372931] usbhid: USB HID core driver [ 3.395281] input: Dell WMI hotkeys as /devices/virtual/input/input5 [ 3.396477] HDA Intel 0000:00:1b.0: PCI INT A -> GSI 21 (level, low) -> IRQ 21 [ 3.433604] input: HDA Intel Mic at Sep Left Jack as /devices/pci0000:00/0000:00:1b.0/sound/card0/input6 [ 3.433922] input: HDA Intel Mic at Ext Right Jack as /devices/pci0000:00/0000:00:1b.0/sound/card0/input7 [ 3.434402] input: HDA Intel Line Out at Sep Left Jack as /devices/pci0000:00/0000:00:1b.0/sound/card0/input8 [ 3.434668] input: HDA Intel HP Out at Ext Right Jack as /devices/pci0000:00/0000:00:1b.0/sound/card0/input9 [ 3.435552] ALSA device list: [ 3.435554] #0: HDA Intel at 0xf6fdc000 irq 48 [ 3.435588] Netfilter messages via NETLINK v0.30. [ 3.435608] nf_conntrack version 0.5.0 (16384 buckets, 65536 max) _From 2.6.35-rc6-mmotm0727: [ 3.065797] usbcore: registered new interface driver usbhid [ 3.065800] usbhid: USB HID core driver [ 3.085379] input: Dell WMI hotkeys as /devices/virtual/input/input5 [ 3.086706] HDA Intel 0000:00:1b.0: PCI INT A -> GSI 21 (level, low) -> IRQ 21 [ 3.086788] HDA Intel 0000:00:1b.0: irq 48 for MSI/MSI-X [ 3.086841] HDA Intel 0000:00:1b.0: setting latency timer to 64 [ 3.095165] ALSA sound/pci/hda/hda_intel.c:1450: no codecs initialized [ 3.095492] HDA Intel 0000:00:1b.0: PCI INT A disabled [ 3.095640] ALSA device list: [ 3.095642] No soundcards found. [ 3.095676] Netfilter messages via NETLINK v0.30. Am I hitting a known issue? If not, what other info would help?

2 2

[alsa-devel] [PATCH] Two patches for Alsa-plugins (pulse)
by David Henningsson 29 Jul '10

29 Jul '10

These two patches are being used in Ubuntu Lucid (released this April) and are working well enough to have people asking us to backport them to Karmic. Today I got an email, asking for the upstream status of one of these patches, since he wanted them in Fedora. If posting them here isn't the correct way to upstream them, please tell me so. The first one (Fix invalid buffer pointer return value) fixes broken logic: This patch improves recovering from underruns, and prevents hangs inside snd_pcm_write* and snd_pcm_read* due to snd_pcm_avail* returning too low values. It especially helps low latency situations. The second one (Do not report underruns to the ALSA layer) is more a change of behavior, which could be questioned. The arguments for removing that code are these: * If pulseaudio gets an underrun, the normal way to end that underrun is to feed it with more buffers. This is different from the ALSA way of dealing with underruns, which requires hardware buffer pointers to be reset. * In addition, underrun signals are delivered asynchronously from pulseaudio. This means that there might be more buffers on the way to pulseaudio when the underrun is reported, making the underrun obsolete. Unfortunately, there is currently no known way to determine whether this is the case or not. // David

5 20

[alsa-devel] [PATCH] Rename iMic to Int Mic on Lenovo NB0763
by David Henningsson 29 Jul '10

29 Jul '10

The non-standard name "iMic" makes PulseAudio ignore the microphone. BugLink: https://launchpad.net/bugs/605101 -- David Henningsson, Canonical Ltd. http://launchpad.net/~diwic

2 3

Re: [alsa-devel] Regression 2.6.35-rc6: ALSA Intel HDA/Realtek: missing Beep
by Takashi Iwai 29 Jul '10

29 Jul '10

At Thu, 29 Jul 2010 10:09:30 +0200, Mario 'BitKoenig' Holbe wrote: > > On Thu, Jul 29, 2010 at 07:44:38AM +0200, Takashi Iwai wrote: > > Mario 'BitKoenig' Holbe wrote: > > > But the BIOS itself beeps through the sound-card at boot :/ > > But BIOS tells that the HD-audio codec shouldn't use, so the driver > > follows it. > > Even grub's beep (play 480 440 1) goes through the sound-card. So it > seems like the BIOS leaves everything set up working as well. Well, my point is that BIOS gives the wrong SSID value. It's bad that the value is compliant with Realtek codecs specification, but it clears the PC-beep bit explicitly. In most cases, SSID value isn't compliant (so BIOS gives a wrong but it's less wrong :), thus the driver takes a fallback to PC-beep enabled. > Please don't get me wrong. I'm not saying the driver does something > wrong, I'm sure it doesn't. I'm just sure that if I could convince it to > behave as if it would have detected the Beep pin everything would work > fine again because it did before... FYI, you can change SSID by writing /sys/class/sound/hwC*D*/subsystem_id file. Then reconfigure the codec via triggering /sys/class/sound/hwC*D*/reconfig. > > Please give alsa-info.sh output instead of codec proc file. It's more > > comprehensive. > > Attached. > > This is from a kernel with both patches applied you sent me. > > > With the patch below, you'll likely have back the system beep sound. > > Nope, no sound. But I guess this wasn't the intention of the patch. Now, > no beep input is registered anymore - which was the intention, I guess. It's the intention. If SSID set by BIOS doesn't indicate the presence of PC-speaker bit *explicitly*, we shouldn't touch beep stuff in codec. > > But it doesn't go through codec, thus no volume control. > > If you mean it should go through the 5V PC Speaker (i.e. pcspkr) - I > don't have such a thing connected. I always appreciated having volume- > and mute control over the beep at night when you can't get away from > work but don't like to wake up anybody just because command completion > beeps, because you pasted something in the wrong window, or whatever. For the driver, the outside connection doesn't matter. The information is given as the spec bit, and just follows it :) So, the fix is likely to override SSID value, or create a special quirk rule to enable PC-beep for known white-list, supposing BIOS won't be fixed in any future... Takashi

2 6

[PATCH 4/4] Rename iMic to Int Mic on Lenovo NB0763
by David Henningsson 29 Jul '10

29 Jul '10

The non-standard name "iMic" makes PulseAudio ignore the microphone. BugLink: https://launchpad.net/bugs/605101 Signed-off-by: David Henningsson <david.henningsson(a)canonical.com> --- sound/pci/hda/patch_realtek.c | 6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/sound/pci/hda/patch_realtek.c b/sound/pci/hda/patch_realtek.c index 1744d4d..b4b7f58 100644 --- a/sound/pci/hda/patch_realtek.c +++ b/sound/pci/hda/patch_realtek.c @@ -7087,7 +7087,7 @@ static struct hda_input_mux alc883_lenovo_nb0763_capture_source = { .num_items = 4, .items = { { "Mic", 0x0 }, - { "iMic", 0x1 }, + { "Int Mic", 0x1 }, { "Line", 0x2 }, { "CD", 0x4 }, }, @@ -8657,8 +8657,8 @@ static struct snd_kcontrol_new alc883_lenovo_nb0763_mixer[] = { HDA_CODEC_MUTE("CD Playback Switch", 0x0b, 0x04, HDA_INPUT), HDA_CODEC_VOLUME("Mic Playback Volume", 0x0b, 0x0, HDA_INPUT), HDA_CODEC_MUTE("Mic Playback Switch", 0x0b, 0x0, HDA_INPUT), - HDA_CODEC_VOLUME("iMic Playback Volume", 0x0b, 0x1, HDA_INPUT), - HDA_CODEC_MUTE("iMic Playback Switch", 0x0b, 0x1, HDA_INPUT), + HDA_CODEC_VOLUME("Int Mic Playback Volume", 0x0b, 0x1, HDA_INPUT), + HDA_CODEC_MUTE("Int Mic Playback Switch", 0x0b, 0x1, HDA_INPUT), { } /* end */ }; -- 1.7.0.4 --------------000003000303050509090808--

1 0

[PATCH 4/4] Rename iMic to Int Mic on Lenovo NB0763
by David Henningsson 29 Jul '10

29 Jul '10

The non-standard name "iMic" makes PulseAudio ignore the microphone. BugLink: https://launchpad.net/bugs/605101 --- sound/pci/hda/patch_realtek.c | 6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/sound/pci/hda/patch_realtek.c b/sound/pci/hda/patch_realtek.c index 1744d4d..b4b7f58 100644 --- a/sound/pci/hda/patch_realtek.c +++ b/sound/pci/hda/patch_realtek.c @@ -7087,7 +7087,7 @@ static struct hda_input_mux alc883_lenovo_nb0763_capture_source = { .num_items = 4, .items = { { "Mic", 0x0 }, - { "iMic", 0x1 }, + { "Int Mic", 0x1 }, { "Line", 0x2 }, { "CD", 0x4 }, }, @@ -8657,8 +8657,8 @@ static struct snd_kcontrol_new alc883_lenovo_nb0763_mixer[] = { HDA_CODEC_MUTE("CD Playback Switch", 0x0b, 0x04, HDA_INPUT), HDA_CODEC_VOLUME("Mic Playback Volume", 0x0b, 0x0, HDA_INPUT), HDA_CODEC_MUTE("Mic Playback Switch", 0x0b, 0x0, HDA_INPUT), - HDA_CODEC_VOLUME("iMic Playback Volume", 0x0b, 0x1, HDA_INPUT), - HDA_CODEC_MUTE("iMic Playback Switch", 0x0b, 0x1, HDA_INPUT), + HDA_CODEC_VOLUME("Int Mic Playback Volume", 0x0b, 0x1, HDA_INPUT), + HDA_CODEC_MUTE("Int Mic Playback Switch", 0x0b, 0x1, HDA_INPUT), { } /* end */ }; -- 1.7.0.4 --------------030908020603010700040801--

1 0

Re: [alsa-devel] [PATCH 06/10 v2] ALSA: msnd: check request_region() return value
by Takashi Iwai 29 Jul '10

29 Jul '10

At Thu, 29 Jul 2010 14:45:24 +0400, Kulikov Vasiliy wrote: > > request_region() may fail, if so return -EBUSY. > > Signed-off-by: Kulikov Vasiliy <segooon(a)gmail.com> Thanks, applied both patches now. Takashi > --- > sound/isa/msnd/msnd_pinnacle.c | 5 ++++- > 1 files changed, 4 insertions(+), 1 deletions(-) > > diff --git a/sound/isa/msnd/msnd_pinnacle.c b/sound/isa/msnd/msnd_pinnacle.c > index 60b6abd..ed42f8e 100644 > --- a/sound/isa/msnd/msnd_pinnacle.c > +++ b/sound/isa/msnd/msnd_pinnacle.c > @@ -549,7 +549,10 @@ static int __devinit snd_msnd_attach(struct snd_card *card) > printk(KERN_ERR LOGNAME ": Couldn't grab IRQ %d\n", chip->irq); > return err; > } > - request_region(chip->io, DSP_NUMIO, card->shortname); > + if (request_region(chip->io, DSP_NUMIO, card->shortname) == NULL) { > + free_irq(chip->irq, chip); > + return -EBUSY; > + } > > if (!request_mem_region(chip->base, BUFFSIZE, card->shortname)) { > printk(KERN_ERR LOGNAME > -- > 1.7.0.4 >

1 0