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

Re: [alsa-devel] sysfs: WARNING: at fs/sysfs/dir.c:424 sysfs_add_one() - with ALSA
by Kamalesh Babulal 30 Oct '07

30 Oct '07

On Fri, Oct 26, 2007 at 12:44:38PM +0530, Amit Shah wrote: > On 2.6.24-rc1, I get this for all my eth interfaces: > > [ 150.389821] sysfs: duplicate filename 'eth1_rename' can not be created > [ 150.389824] WARNING: at fs/sysfs/dir.c:424 sysfs_add_one() > [ 150.389826] > [ 150.389826] Call Trace: > [ 150.389840] [<ffffffff802ee9cf>] sysfs_add_one+0xaf/0xf0 > [ 150.389844] [<ffffffff802ef9fe>] sysfs_create_link+0x9e/0x150 > [ 150.389849] [<ffffffff8038a418>] device_rename+0x1d8/0x230 > [ 150.389854] [<ffffffff803c2626>] dev_change_name+0xb6/0x230 > [ 150.389857] [<ffffffff803c2d5b>] dev_ioctl+0x2bb/0x5a0 > [ 150.389862] [<ffffffff80437436>] _spin_unlock_irqrestore+0x16/0x40 > [ 150.389864] [<ffffffff8043947a>] do_page_fault+0x24a/0x930 > [ 150.389869] [<ffffffff803b341d>] sock_ioctl+0x7d/0x250 > [ 150.389872] [<ffffffff802aafef>] do_ioctl+0x2f/0xa0 > [ 150.389875] [<ffffffff802ab0d4>] vfs_ioctl+0x74/0x2d0 > [ 150.389878] [<ffffffff802ab3c1>] sys_ioctl+0x91/0xb0 > [ 150.389881] [<ffffffff8020c32e>] system_call+0x7e/0x83 > [ 150.389884] > [ 150.389886] net eth1_rename: device_rename: sysfs_create_symlink failed (-17) > [ 151.169923] sysfs: duplicate filename 'eth2' can not be created > [ 151.169927] WARNING: at fs/sysfs/dir.c:424 sysfs_add_one() > [ 151.169929] > [ 151.169930] Call Trace: > [ 151.169946] [<ffffffff802ee9cf>] sysfs_add_one+0xaf/0xf0 > [ 151.169949] [<ffffffff802ef9fe>] sysfs_create_link+0x9e/0x150 > [ 151.169955] [<ffffffff8038a418>] device_rename+0x1d8/0x230 > [ 151.169960] [<ffffffff803c2626>] dev_change_name+0xb6/0x230 > [ 151.169963] [<ffffffff803c2d5b>] dev_ioctl+0x2bb/0x5a0 > [ 151.169967] [<ffffffff80437436>] _spin_unlock_irqrestore+0x16/0x40 > [ 151.169970] [<ffffffff8043947a>] do_page_fault+0x24a/0x930 > [ 151.169974] [<ffffffff803b341d>] sock_ioctl+0x7d/0x250 > [ 151.169978] [<ffffffff802aafef>] do_ioctl+0x2f/0xa0 > [ 151.169980] [<ffffffff802ab0d4>] vfs_ioctl+0x74/0x2d0 > [ 151.169983] [<ffffffff802ab3c1>] sys_ioctl+0x91/0xb0 > [ 151.169987] [<ffffffff8020c32e>] system_call+0x7e/0x83 > [ 151.169990] > [ 151.169992] net eth2: device_rename: sysfs_create_symlink failed (-17) > [ 151.176762] e1000: eth2: e1000_watchdog: NIC Link is Up 100 Mbps > Full Duplex, Flow Control: RX/TX > [ 151.209396] sysfs: duplicate filename 'eth0' can not be created > Hi, I get similar warning, while loading the alsa driver [ 191.933548] Advanced Linux Sound Architecture Driver Version 1.0.15 (Tue Oct 23 06:09:18 2007 UTC). [ 191.996323] ALSA sound/core/seq/oss/seq_oss.c:232: can't register device seq [ 192.046004] sysfs: duplicate filename 'audio' can not be created [ 192.086048] WARNING: at fs/sysfs/dir.c:424 sysfs_add_one() [ 192.122882] [ 192.122883] Call Trace: [ 192.154111] [<ffffffff802fbe74>] sysfs_add_one+0x5c/0xc9 [ 192.190144] [<ffffffff802fc498>] create_dir+0x58/0x93 [ 192.224571] [<ffffffff802fc50b>] sysfs_create_dir+0x38/0x4f [ 192.262068] [<ffffffff813a622b>] _spin_unlock+0x26/0x2a [ 192.297448] [<ffffffff8048eb92>] kobject_add+0x13e/0x1f9 [ 192.333292] [<ffffffff8065c746>] device_add+0xac/0x610 [ 192.367959] [<ffffffff8065ccc3>] device_register+0x19/0x1d [ 192.404621] [<ffffffff8065cda6>] device_create+0xdf/0x110 [ 192.440760] [<ffffffff8049301c>] sprintf+0x68/0x6a [ 192.473288] [<ffffffff80ff9b4a>] sound_insert_unit+0xcf/0x172 [ 192.511443] [<ffffffff80ff9ba4>] sound_insert_unit+0x129/0x172 [ 192.549996] [<ffffffff80ff9deb>] register_sound_special_device+0x14a/0x159 [ 192.594734] [<ffffffff8101ed06>] snd_register_oss_device+0x11d/0x18f [ 192.636281] [<ffffffff81034470>] register_oss_dsp+0x54/0x8a [ 192.673031] [<ffffffff802604e6>] mark_held_locks+0x4a/0x6a [ 192.709275] [<ffffffff813a50f1>] mutex_lock_nested+0x2a8/0x2c3 [ 192.747533] [<ffffffff802604e6>] mark_held_locks+0x4a/0x6a [ 192.783587] [<ffffffff813a4e32>] __mutex_unlock_slowpath+0x12e/0x13a [ 192.824764] [<ffffffff802606d0>] trace_hardirqs_on+0x11c/0x147 [ 192.862739] [<ffffffff810378d4>] snd_pcm_oss_register_minor+0x35/0x16b [ 192.904816] [<ffffffff813a4e47>] mutex_unlock+0x9/0xb [ 192.938016] [<ffffffff810208de>] snd_timer_dev_register+0x125/0x12c [ 192.978469] [<ffffffff8101e27c>] snd_device_register+0x73/0xe0 [ 193.016307] [<ffffffff8102e03d>] snd_pcm_timer_init+0xba/0xf5 [ 193.053660] [<ffffffff810245bd>] snd_pcm_dev_register+0x1d7/0x211 [ 193.093096] [<ffffffff8101e1e1>] snd_device_register_all+0x5d/0x85 [ 193.133041] [<ffffffff810197ac>] snd_card_register+0x4e/0x2ea [ 193.170427] [<ffffffff8104f66d>] snd_dummy_probe+0x201/0x21a [ 193.207240] [<ffffffff806601b5>] platform_drv_probe+0x12/0x14 [ 193.244615] [<ffffffff8065ea16>] driver_probe_device+0x132/0x1d2 [ 193.283580] [<ffffffff8065eab6>] __device_attach+0x0/0xb [ 193.318382] [<ffffffff8065eabf>] __device_attach+0x9/0xb [ 193.353173] [<ffffffff8065df69>] bus_for_each_drv+0x47/0x7d [ 193.389534] [<ffffffff8065eb5a>] device_attach+0x75/0x8d [ 193.424319] [<ffffffff8065dd78>] bus_attach_device+0x2d/0x79 [ 193.461202] [<ffffffff8065ca2d>] device_add+0x393/0x610 [ 193.495910] [<ffffffff806605a5>] platform_device_add+0x133/0x16a [ 193.534870] [<ffffffff806607a8>] platform_device_register_simple+0x46/0x64 [ 193.579013] [<ffffffff8208d029>] alsa_card_dummy_init+0x38/0x7e [ 193.617437] [<ffffffff82028678>] kernel_init+0x176/0x2ed [ 193.652173] [<ffffffff813a5d27>] trace_hardirqs_on_thunk+0x35/0x3a [ 193.692147] [<ffffffff802606d0>] trace_hardirqs_on+0x11c/0x147 [ 193.730030] [<ffffffff8020cef8>] child_rip+0xa/0x12 [ 193.762145] [<ffffffff8020c60f>] restore_args+0x0/0x30 [ 193.795821] [<ffffffff8023fc7f>] release_console_sem+0x4d/0x1f5 [ 193.834171] [<ffffffff82028502>] kernel_init+0x0/0x2ed [ 193.867870] [<ffffffff8020ceee>] child_rip+0x0/0x12 [ 193.899987] [ 193.911270] kobject_add failed for audio with -EEXIST, don't try to register things with the same name in the same directory. [ 193.983989] [ 193.983990] Call Trace: [ 194.012668] [<ffffffff8048e440>] kobject_put+0x19/0x1b [ 194.046571] [<ffffffff8048ec17>] kobject_add+0x1c3/0x1f9 [ 194.081445] [<ffffffff8065c746>] device_add+0xac/0x610 [ 194.115311] [<ffffffff8065ccc3>] device_register+0x19/0x1d [ 194.151267] [<ffffffff8065cda6>] device_create+0xdf/0x110 [ 194.186663] [<ffffffff8049301c>] sprintf+0x68/0x6a [ 194.218463] [<ffffffff80ff9b4a>] sound_insert_unit+0xcf/0x172 [ 194.255953] [<ffffffff80ff9ba4>] sound_insert_unit+0x129/0x172 [ 194.293932] [<ffffffff80ff9deb>] register_sound_special_device+0x14a/0x159 [ 194.338208] [<ffffffff8101ed06>] snd_register_oss_device+0x11d/0x18f [ 194.379363] [<ffffffff81034470>] register_oss_dsp+0x54/0x8a [ 194.415853] [<ffffffff802604e6>] mark_held_locks+0x4a/0x6a [ 194.451820] [<ffffffff813a50f1>] mutex_lock_nested+0x2a8/0x2c3 [ 194.489889] [<ffffffff802604e6>] mark_held_locks+0x4a/0x6a [ 194.525880] [<ffffffff813a4e32>] __mutex_unlock_slowpath+0x12e/0x13a [ 194.567098] [<ffffffff802606d0>] trace_hardirqs_on+0x11c/0x147 [ 194.605551] [<ffffffff810378d4>] snd_pcm_oss_register_minor+0x35/0x16b [ 194.647848] [<ffffffff813a4e47>] mutex_unlock+0x9/0xb [ 194.681271] [<ffffffff810208de>] snd_timer_dev_register+0x125/0x12c [ 194.721968] [<ffffffff8101e27c>] snd_device_register+0x73/0xe0 [ 194.760041] [<ffffffff8102e03d>] snd_pcm_timer_init+0xba/0xf5 [ 194.797490] [<ffffffff810245bd>] snd_pcm_dev_register+0x1d7/0x211 [ 194.836921] [<ffffffff8101e1e1>] snd_device_register_all+0x5d/0x85 [ 194.876879] [<ffffffff810197ac>] snd_card_register+0x4e/0x2ea [ 194.914262] [<ffffffff8104f66d>] snd_dummy_probe+0x201/0x21a [ 194.951096] [<ffffffff806601b5>] platform_drv_probe+0x12/0x14 [ 194.988480] [<ffffffff8065ea16>] driver_probe_device+0x132/0x1d2 [ 195.027449] [<ffffffff8065eab6>] __device_attach+0x0/0xb [ 195.062267] [<ffffffff8065eabf>] __device_attach+0x9/0xb [ 195.097053] [<ffffffff8065df69>] bus_for_each_drv+0x47/0x7d [ 195.133408] [<ffffffff8065eb5a>] device_attach+0x75/0x8d [ 195.168203] [<ffffffff8065dd78>] bus_attach_device+0x2d/0x79 [ 195.205133] [<ffffffff8065ca2d>] device_add+0x393/0x610 [ 195.239455] [<ffffffff806605a5>] platform_device_add+0x133/0x16a [ 195.278432] [<ffffffff806607a8>] platform_device_register_simple+0x46/0x64 [ 195.322586] [<ffffffff8208d029>] alsa_card_dummy_init+0x38/0x7e [ 195.361032] [<ffffffff82028678>] kernel_init+0x176/0x2ed [ 195.395798] [<ffffffff813a5d27>] trace_hardirqs_on_thunk+0x35/0x3a [ 195.435764] [<ffffffff802606d0>] trace_hardirqs_on+0x11c/0x147 [ 195.473657] [<ffffffff8020cef8>] child_rip+0xa/0x12 [ 195.505771] [<ffffffff8020c60f>] restore_args+0x0/0x30 [ 195.539459] [<ffffffff8023fc7f>] release_console_sem+0x4d/0x1f5 [ 195.577825] [<ffffffff82028502>] kernel_init+0x0/0x2ed [ 195.611533] [<ffffffff8020ceee>] child_rip+0x0/0x12 [ 195.643690] [ 195.659326] ALSA sound/core/rawmidi.c:1542: unable to register OSS rawmidi device 1:0 [ 195.712879] no UART detected at 0x1 [ 195.739880] Motu MidiTimePiece on parallel port irq: 7 ioport: 0x378 [ 195.804370] usbcore: registered new interface driver snd-usb-audio [ 195.844370] usbcore: registered new interface driver snd-usb-usx2y [ 195.884286] usbcore: registered new interface driver snd-usb-caiaq [ 195.924293] ASoC version 0.13.1 [ 195.946107] ALSA device list: [ 195.966550] #0: Dummy 1 [ 195.984913] #1: Virtual MIDI Card 1 [ 196.009489] #2: MTPAV on parallel port at 0x378 -- Thanks & Regards, Kamalesh Babulal, Linux Technology Center, IBM, ISTL.

5 9

[alsa-devel] [PATCH] SB Live24-External better handling
by Timofei V. Bondarenko 30 Oct '07

30 Oct '07

Hi. This patch improves support for 'SB Live 24-bit Extarnal' USB card. 1) This card can go into muted state when a headphones connected or disconnected. So notify mixer about changes in headphone jack. 2) Add LED controls and procfs support just as in similar Audigy 2 NX card. 3) Rename 'PCM Capture' conrol to 'Mic Capture' to reflect reality: the card may adjust microphone input level only. Regards. Tim.

2 3

[alsa-devel] Removal of sequencer instrument layer
by Takashi Iwai 30 Oct '07

30 Oct '07

Hi, JFYI, together with OPL3 hwdep loader, $SUBJECT is done as I mentioned earlier. This resulted in removal of 5600 LOC in alsa-kernel tree. Yeah, a good number. Takashi

1 0

[alsa-devel] cmipci CM_FLINKOFF disables multinchannel playback
by Timofei V. Bondarenko 30 Oct '07

30 Oct '07

Hi. I found the latest cmipci driver cannot playback in multichannel mode anymore. My hardware is 'CMI8738 model 55' attempting to play 2 or 4 channels through the 2nd DAC i get no sound at all. 6 ch produces sound only at center/bass while front and rear still mute. I found the root of problem in redefinition of some bits of the CM_REG_MISC_CTRL (0x18) register. Was: CM_FLINKON=0x40, CM_FLINKOFF=0x20 Now it is: CM_FLINKON=0x80 CM_FLINKOFF=0x40 Note the CM_FLINKOFF does collide with the CM_MUTECH1=0x40 bit. This bit disables playback on 2nd DAC. The CM_FLINKOFF/CM_FLINKON bits used in the 'Modem' mixer control only. During startup the mixer turns the modem off somewhere and this disables 2nd DAC as well. Complete removal of this 'Modem' control fixes the problem. Is it an acceptable solution? I guess the snd_cmipci_control_switches could be removed too. Regards. Tim. --- alsa-driver-hg20071027/alsa-kernel/pci/cmipci.c 2007-09-18 04:00:10.000000000 +0400 +++ alsa-driver/alsa-kernel/pci/cmipci.c 2007-10-29 17:09:31.000000000 +0300 @@ -2389,7 +2389,6 @@ DEFINE_BIT_SWITCH_ARG(fourch, CM_REG_MIS // DEFINE_BIT_SWITCH_ARG(line_rear, CM_REG_MIXER1, CM_REAR2LIN, 1, 0); // DEFINE_BIT_SWITCH_ARG(line_bass, CM_REG_LEGACY_CTRL, CM_CENTR2LIN|CM_BASE2LIN, 0, 0); // DEFINE_BIT_SWITCH_ARG(joystick, CM_REG_FUNCTRL1, CM_JYSTK_EN, 0, 0); /* now module option */ -DEFINE_SWITCH_ARG(modem, CM_REG_MISC_CTRL, CM_FLINKON|CM_FLINKOFF, CM_FLINKON, 0, 0); #define DEFINE_SWITCH(sname, stype, sarg) \ { .name = sname, \ @@ -2597,7 +2596,7 @@ static struct snd_kcontrol_new snd_cmipc /* card control switches */ static struct snd_kcontrol_new snd_cmipci_control_switches[] __devinitdata = { // DEFINE_CARD_SWITCH("Joystick", joystick), /* now module option */ - DEFINE_CARD_SWITCH("Modem", modem), +// DEFINE_CARD_SWITCH("Modem", modem), };

2 3

[alsa-devel] PATCH: ln2440sbc ac97 support
by Ivan Kuten 29 Oct '07

29 Oct '07

Hello, This patch adds ac97 support for ln2440sbc board from LittleChips. This board is based on s3c2440 SoC + AC97 Realtek ALC650 codec. Existing s3c2443 implementation is slightly modified because s3c2440 and s3c2443 have different AC97 interrupts. Regards, Ivan Signed-off-by: Ivan Kuten <ivan.kuten(a)promwad.com> diff -urN linux-2.6.23/sound/soc/s3c24xx/Kconfig linux-2.6.23-2440/sound/soc/s3c24xx/Kconfig --- linux-2.6.23/sound/soc/s3c24xx/Kconfig 2007-10-09 22:31:38.000000000 +0200 +++ linux-2.6.23-2440/sound/soc/s3c24xx/Kconfig 2007-10-28 01:18:23.000000000 +0200 @@ -34,4 +34,12 @@ Say Y if you want to add support for SoC audio on smdk2443 with the WM9710. +config SND_S3C24XX_SOC_LN2440SBC_ALC650 + tristate "SoC AC97 Audio support for LN2440SBC - ALC650" + depends on SND_S3C24XX_SOC + select SND_S3C2443_SOC_AC97 + select SND_SOC_AC97_CODEC + help + Say Y if you want to add support for SoC audio on ln2440sbc + with the ALC650. diff -urN linux-2.6.23/sound/soc/s3c24xx/Makefile linux-2.6.23-2440/sound/soc/s3c24xx/Makefile --- linux-2.6.23/sound/soc/s3c24xx/Makefile 2007-10-09 22:31:38.000000000 +0200 +++ linux-2.6.23-2440/sound/soc/s3c24xx/Makefile 2007-10-25 01:28:18.000000000 +0200 @@ -10,6 +10,8 @@ # S3C24XX Machine Support snd-soc-neo1973-wm8753-objs := neo1973_wm8753.o snd-soc-smdk2443-wm9710-objs := smdk2443_wm9710.o +snd-soc-ln2440sbc-alc650-objs := ln2440sbc_alc650.o obj-$(CONFIG_SND_S3C24XX_SOC_NEO1973_WM8753) += snd-soc-neo1973-wm8753.o obj-$(CONFIG_SND_S3C24XX_SOC_SMDK2443_WM9710) += snd-soc-smdk2443-wm9710.o +obj-$(CONFIG_SND_S3C24XX_SOC_LN2440SBC_ALC650) += snd-soc-ln2440sbc-alc650.o diff -urN linux-2.6.23/sound/soc/s3c24xx/ln2440sbc_alc650.c linux-2.6.23-2440/sound/soc/s3c24xx/ln2440sbc_alc650.c --- linux-2.6.23/sound/soc/s3c24xx/ln2440sbc_alc650.c 1970-01-01 01:00:00.000000000 +0100 +++ linux-2.6.23-2440/sound/soc/s3c24xx/ln2440sbc_alc650.c 2007-10-28 00:27:41.000000000 +0200 @@ -0,0 +1,86 @@ +/* + * SoC audio for ln2440sbc + * + * Copyright 2007 KonekTel, a.s. + * Author: Ivan Kuten + * ivan.kuten(a)promwad.com + * + * Heavily based on smdk2443_wm9710.c + * Copyright 2007 Wolfson Microelectronics PLC. + * Author: Graeme Gregory + * graeme.gregory(a)wolfsonmicro.com or linux(a)wolfsonmicro.com + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License version 2 as + * published by the Free Software Foundation. + * + */ + +#include <linux/module.h> +#include <linux/device.h> +#include <sound/driver.h> +#include <sound/core.h> +#include <sound/pcm.h> +#include <sound/soc.h> +#include <sound/soc-dapm.h> + +#include "../codecs/ac97.h" +#include "s3c24xx-pcm.h" +#include "s3c24xx-ac97.h" + +static struct snd_soc_machine ln2440sbc; + +static struct snd_soc_dai_link ln2440sbc_dai[] = { +{ + .name = "AC97", + .stream_name = "AC97 HiFi", + .cpu_dai = &s3c2443_ac97_dai[0], + .codec_dai = &ac97_dai, +}, +}; + +static struct snd_soc_machine ln2440sbc = { + .name = "LN2440SBC", + .dai_link = ln2440sbc_dai, + .num_links = ARRAY_SIZE(ln2440sbc_dai), +}; + +static struct snd_soc_device ln2440sbc_snd_ac97_devdata = { + .machine = &ln2440sbc, + .platform = &s3c24xx_soc_platform, + .codec_dev = &soc_codec_dev_ac97, +}; + +static struct platform_device *ln2440sbc_snd_ac97_device; + +static int __init ln2440sbc_init(void) +{ + int ret; + + ln2440sbc_snd_ac97_device = platform_device_alloc("soc-audio", -1); + if (!ln2440sbc_snd_ac97_device) + return -ENOMEM; + + platform_set_drvdata(ln2440sbc_snd_ac97_device, + &ln2440sbc_snd_ac97_devdata); + ln2440sbc_snd_ac97_devdata.dev = &ln2440sbc_snd_ac97_device->dev; + ret = platform_device_add(ln2440sbc_snd_ac97_device); + + if (ret) + platform_device_put(ln2440sbc_snd_ac97_device); + + return ret; +} + +static void __exit ln2440sbc_exit(void) +{ + platform_device_unregister(ln2440sbc_snd_ac97_device); +} + +module_init(ln2440sbc_init); +module_exit(ln2440sbc_exit); + +/* Module information */ +MODULE_AUTHOR("Ivan Kuten"); +MODULE_DESCRIPTION("ALSA SoC ALC650 LN2440SBC"); +MODULE_LICENSE("GPL"); diff -urN linux-2.6.23/sound/soc/s3c24xx/s3c2443-ac97.c linux-2.6.23-2440/sound/soc/s3c24xx/s3c2443-ac97.c --- linux-2.6.23/sound/soc/s3c24xx/s3c2443-ac97.c 2007-10-09 22:31:38.000000000 +0200 +++ linux-2.6.23-2440/sound/soc/s3c24xx/s3c2443-ac97.c 2007-10-28 00:55:57.000000000 +0200 @@ -32,7 +32,7 @@ #include <asm/hardware.h> #include <asm/io.h> -#include <asm/arch/regs-ac97.h> +#include <asm/plat-s3c/regs-ac97.h> #include <asm/arch/regs-gpio.h> #include <asm/arch/regs-clock.h> #include <asm/arch/audio.h> @@ -253,7 +253,7 @@ ac_glbctrl |= S3C_AC97_GLBCTRL_TRANSFERDATAENABLE; writel(ac_glbctrl, s3c24xx_ac97.regs + S3C_AC97_GLBCTRL); - ret = request_irq(IRQ_S3C2443_AC97, s3c2443_ac97_irq, + ret = request_irq(IRQ_S3C244x_AC97, s3c2443_ac97_irq, IRQF_DISABLED, "AC97", NULL); if (ret < 0) { printk(KERN_ERR "s3c24xx-ac97: interrupt request failed.\n"); @@ -266,7 +266,7 @@ static void s3c2443_ac97_remove(struct platform_device *pdev) { - free_irq(IRQ_S3C2443_AC97, NULL); + free_irq(IRQ_S3C244x_AC97, NULL); clk_disable(s3c24xx_ac97.ac97_clk); clk_put(s3c24xx_ac97.ac97_clk); iounmap(s3c24xx_ac97.regs); diff -urN linux-2.6.23/sound/soc/s3c24xx/s3c24xx-ac97.h linux-2.6.23-2440/sound/soc/s3c24xx/s3c24xx-ac97.h --- linux-2.6.23/sound/soc/s3c24xx/s3c24xx-ac97.h 2007-10-09 22:31:38.000000000 +0200 +++ linux-2.6.23-2440/sound/soc/s3c24xx/s3c24xx-ac97.h 2007-10-28 00:32:04.000000000 +0200 @@ -20,6 +20,12 @@ #define AC_CMD_ADDR(x) (x << 16) #define AC_CMD_DATA(x) (x & 0xffff) +#ifdef CONFIG_CPU_S3C2440 +#define IRQ_S3C244x_AC97 IRQ_S3C2440_AC97 +#else +#define IRQ_S3C244x_AC97 IRQ_S3C2443_AC97 +#endif + extern struct snd_soc_cpu_dai s3c2443_ac97_dai[]; #endif /*S3C24XXAC97_H_*/

3 3

[alsa-devel] [PATCH] ac97_patch: compilation warning fix
by Krzysztof Helt 29 Oct '07

29 Oct '07

From: Krzysztof Helt <krzysztof.h1(a)wp.pl> This patch kills these two compilation warnings if power management is disabled: sound/pci/ac97/ac97_patch.h:86: warning: 'snd_ac97_restore_status' declared 'static' but never defined sound/pci/ac97/ac97_patch.h:87: warning: 'snd_ac97_restore_iec958' declared 'static' but never defined Signed-off-by: Krzysztof Helt <krzysztof.h1(a)wp.pl> --- The patch was done against kernel 2.6.24-rc1-git4. diff -urp linux-ref/sound/pci/ac97/ac97_patch.h linux-new/sound/pci/ac97/ac97_patch.h --- linux-ref/sound/pci/ac97/ac97_patch.h 2007-10-28 11:11:20.000000000 +0100 +++ linux-new/sound/pci/ac97/ac97_patch.h 2007-10-28 12:46:34.000000000 +0100 @@ -83,8 +83,10 @@ static int snd_ac97_swap_ctl(struct snd_ const char *s2, const char *suffix); static void snd_ac97_rename_vol_ctl(struct snd_ac97 *ac97, const char *src, const char *dst); +#ifdef CONFIG_PM static void snd_ac97_restore_status(struct snd_ac97 *ac97); static void snd_ac97_restore_iec958(struct snd_ac97 *ac97); +#endif static int snd_ac97_info_enum_double(struct snd_kcontrol *kcontrol, struct snd_ctl_elem_info *uinfo); static int snd_ac97_get_enum_double(struct snd_kcontrol *kcontrol,

2 1

[alsa-devel] [PATCH] Dreamcast AICA sound - Get rid of annoying compiler warning
by Adrian McMenamin 29 Oct '07

29 Oct '07

This patch supresses an annoying compiler warning that the variable err may be used uninitialised. Signed-off by: Adrian McMenamin <adrian(a)mcmen.demon.co.uk> diff --git a/sound/sh/aica.c b/sound/sh/aica.c index 88dc840..8861d2f 100644 --- a/sound/sh/aica.c +++ b/sound/sh/aica.c @@ -237,6 +237,7 @@ static int aica_dma_transfer(int channels, int buffer_size, struct snd_card_aica *dreamcastcard; struct snd_pcm_runtime *runtime; unsigned long flags; + err = 0; dreamcastcard = substream->pcm->private_data; period_offset = dreamcastcard->clicks; period_offset %= (AICA_PERIOD_NUMBER / channels);

2 1

[alsa-devel] [PATCH] Fix for assertion in pulse plugin when calling hw_params on a prepared stream
by Sean McNamara 29 Oct '07

29 Oct '07

Hello, I have opened a bug and recently submitted a patch for it on bugtrack: https://bugtrack.alsa-project.org/alsa-bug/view.php?id=3470 All the details are there. But here is a 5-minute summary for you: --The Problem-- (1) All plugins implementing the PCM interface, that I've tested so far, allow calling snd_*_hw_params on a stream when snd_*_state returns SND_PCM_STATE_PREPARED. (2) Applications have used this fact to their advantage, and sometimes employ lazy initialization of the hw params (initialize it once, get to another path in code, oh no wait! we need to initialize them differently...) (3) Per the spec, calling snd_*_hw_params also makes the stream PREPARED, so calling snd_*_hw_params on a stream that has _either_ been explicitly prepared, _or_ had its hw_params previously set, will trigger this problem. (4) The ALSA<->PulseAudio plugin is an exception to (1) above; i.e., it triggers an assertion when exercising this functionality, as if to say, "I can't do that!" (5) Indeed, the PulseAudio client API forbids re-initialization of the parameters in question on a PulseAudio stream, after instantiation. --The Solution-- Only perform the assertion if snd_pcm_state(io->pcm) != SND_PCM_STATE_PREPARED. The assertion is still useful for cases where the stream is mysteriously non-NULL and the stream has not already been prepared; I can't think of a valid case where: (1) The state is not SND_PCM_STATE_PREPARED, and (2) The underlying PulseAudio stream (pcm->stream) is non-NULL, and (3) snd_pcm_hw_params would be expected to do anything meaningful or correct. For this reason, I kept the assertion in, rather than just removing it (which by itself "fixes" the problem with alsaplayer and my test case, which is submitted with the patch.) --Why It Works-- (1) This assertion might be a throwback from before ioplug was so smart; but apparently, ioplug will now call pulse_prepare, which reinitializes the underlying pulseaudio stream, after pulse_hw_params. (2) There is nothing in pulse_hw_params that actually requires any particular state out of pcm->stream. The underlying pulse stream is simply not used in this call. (3) ioplug catches the fact that the hw_params have been changed, and calls back to pulse_prepare to handle the underlying stream teardown and reinitialization (which pulse_prepare does correctly). --Impact-- (1) At least one open source application, alsaplayer, can now play through the ALSA<->PulseAudio plugin, where it could never do so before. (2) The ALSA<->PulseAudio plugin is now more tightly compatible with the other alsa-lib and alsa-plugin backends (dmix, hw, asym, oss, jack, ...) (3) Older versions of sox might have also suffered from this bug, according to information here: http://www.pulseaudio.org/ticket/120 (although the Ubuntu 7.10 version of sox does not exhibit this behavior). --Notes-- (1) I submitted a first-attempt patch that fixed the problem, but it was inefficient: it needlessly called pulse_prepare _prior_ to the main pulse_hw_params function body. This was a result of my lack of understanding of how ioplug was actually saving me from disaster in the background. Be sure to grab 2_pcm_pulse.c.patch rather than pcm_pulse.c.patch. (2) I tested both positive and negative cases; the ALSA<->PulseAudio plugin works with applications that hit the modified code, as well as applications that don't. I've tested both branches of the conditional I added. Thanks, Sean

2 1

[alsa-devel] snd-hdsp in 2.6.22 crashes badly
by ilya .d 29 Oct '07

29 Oct '07

Hello. i'm using RME Hammerfall HDSP9632.. it worked fine with 2.6.17 and 2.6.20 (-gentoo), i don't think gentoo team modifies alsa tree .. as soon as i upgraded to 2.6.22[-gentoo-r8], it booted fine, but when i started gnome-settigs-daemon, kernell generated OOPS, and i couldn't inload the module with modprobe. i disabled gnome-settings-daemon, and reboot. i managed to use hdspconf , hdspmixer + a couple of applications (puredata and snd editor), it was all good. as soon as i ran alsamixer (well , normaly you don't need it for snd-hdsp, but i wanted to configure my second soundcard, which is SB_live, but forgot to give it '-c 1') - so here the same crash heppends. i tried 2.6.20 back - and all absolutely fine with it. looking at hdsp.c in both kernels i spoted quite a few changes , however i don't know much about it ;[ so i attached the details below. -- cheers, ilya .d

2 2

[alsa-devel] ALSA dma ring buffer issue on 2.6.14
by Filipe Braz 27 Oct '07

27 Oct '07

Hi, I am trying to use ALSA with the OMAP2430 embedded platform, using kernel 2.6.14 for low latency voip applications. The problem seems to be that the driver ring buffer is not filled by the alsa kernel/core on time to be sent to the hw codec. So, If I use the following large buffer size / period-size it works fine: aplay --period-size=1024 --buffer-size=8192 -f S16_LE -c 2 -r 8000 pattern.pcm But if I use smaller buffer / period sizes such as the following it does not work: aplay --period-size=32 --buffer-size=128 -f S16_LE -c 2 -r 8000 pattern.pcm aplay --period-size=32 --buffer-size=256 -f S16_LE -c 2 -r 8000 pattern.pcm aplay --period-size=64 --buffer-size=512 -f S16_LE -c 2 -r 8000 pattern.pcm aplay --period-size=128 --buffer-size=512 -f S16_LE -c 2 -r 8000 pattern.pcm aplay --period-size=256 --buffer-size=1024 -f S16_LE -c 2 -r 8000 pattern.pcm aplay --period-size=512 --buffer-size=2048 -f S16_LE -c 2 -r 8000 pattern.pcm I am playing a special pattern sound file that I verify before each period is transfered to the codec via dma (on the driver). On smaller buffer sizes. the 1st period AFTER the ringbuffer rotates to the beginning, appears to not contain the pattern data (instead is filled with zeros). This does not happens with larger buffer / periods. On all cases the pattern file is large enough so that the dma ring buffer rotates several times until transfer is complete. I have tested this on the pc and it works just fine. Also if I use the OSS driver for the OMAP2430 instead of the ALSA it works fine. ALSA with OSS emulation works significantly better but there is still some distortion. Any ideas why this happens? Is this a known issue or is there a solution for this problem? Thanks in advance, Filipe

1 0