Hello Mark, Liam On Tue, Aug 31, 2010 at 1:01 PM, Mark Brown <broonie@xxxxxxxxxxxxxxxxxxxxxxxxxxx> wrote: > On Thu, Aug 26, 2010 at 02:53:51PM +0200, Manuel Lauss wrote: >> This patch fixes up the au1x audio platform after the multi-component >> merge: >> - compile fixes and updates to get DB1200 platform audio working again, >> - removal of global variables in AC97/I2S/DMA(PCM) modules. >> >> The AC97 part is limited to one instance only for now due to issues >> with getting at driver data in the soc_ac97_ops. >> >> Signed-off-by: Manuel Lauss <manuel.lauss@xxxxxxxxxxxxxx> > > Applied, thanks - we can work on fixing up the remaining issues with > AC'97 incrementally. Here's a snippet of the kernel log with linked-list debugging enabled. Looks like the list issue Liam fixed ~1.5 years ago: ( see http://mailman.alsa-project.org/pipermail/alsa-devel/2009-February/014757.html ) AC97 SoC Audio Codec 0.6 asoc: ac97-hifi <-> au1xpsc_ac97.1 mapping ok kobject (8f798da0): tried to init an initialized object, something is seriously wrong. Call Trace: [<801082d8>] dump_stack+0x8/0x34 [<802d2bf0>] kobject_init+0xc4/0xd4 [<80310758>] device_initialize+0x2c/0x70 [<803110ac>] device_register+0x14/0x28 [<803d1ab4>] snd_soc_instantiate_cards+0xa60/0xbc0 [<803d1d50>] soc_probe+0x13c/0x154 [<80313970>] driver_probe_device+0xc4/0x204 [<80312b90>] bus_for_each_drv+0x68/0xd0 [<80313cd4>] device_attach+0xc0/0xfc [<80312928>] bus_probe_device+0x38/0x54 [<80310fd8>] device_add+0x4b8/0x578 [<8031560c>] platform_device_add+0x12c/0x24c [<8060da40>] db1200_audio_load+0x78/0xa4 [<8010aa40>] do_one_initcall+0x40/0x1e8 [<805fb338>] kernel_init+0xd4/0x174 [<8010e734>] kernel_thread_helper+0x10/0x18 ------------[ cut here ]------------ WARNING: at /home/mano/db1200/kernel/linux-2.6.git/lib/list_debug.c:30 __list_add+0xa8/0xb0() list_add corruption. prev->next should be next (8fe44fa8), but was 8f7eefc4. (prev=8f7eefc4). Modules linked in: Call Trace: [<801082d8>] dump_stack+0x8/0x34 [<8012a2dc>] warn_slowpath_common+0x78/0xa4 [<8012a390>] warn_slowpath_fmt+0x2c/0x38 [<802e046c>] __list_add+0xa8/0xb0 [<80312b18>] bus_add_device+0x1d4/0x1e4 [<80310f88>] device_add+0x468/0x578 [<803d1ab4>] snd_soc_instantiate_cards+0xa60/0xbc0 [<803d1d50>] soc_probe+0x13c/0x154 [<80313970>] driver_probe_device+0xc4/0x204 [<80312b90>] bus_for_each_drv+0x68/0xd0 [<80313cd4>] device_attach+0xc0/0xfc [<80312928>] bus_probe_device+0x38/0x54 [<80310fd8>] device_add+0x4b8/0x578 [<8031560c>] platform_device_add+0x12c/0x24c [<8060da40>] db1200_audio_load+0x78/0xa4 [<8010aa40>] do_one_initcall+0x40/0x1e8 [<805fb338>] kernel_init+0xd4/0x174 [<8010e734>] kernel_thread_helper+0x10/0x18 ---[ end trace 8e74e4819d0d06c3 ]--- ------------[ cut here ]------------ WARNING: at /home/mano/db1200/kernel/linux-2.6.git/lib/list_debug.c:30 __list_add+0xa8/0xb0() list_add corruption. prev->next should be next (8f788f90), but was 8f7eefa4. (prev=8f7eefa4). Modules linked in: Call Trace: [<801082d8>] dump_stack+0x8/0x34 [<8012a2dc>] warn_slowpath_common+0x78/0xa4 [<8012a390>] warn_slowpath_fmt+0x2c/0x38 [<802e046c>] __list_add+0xa8/0xb0 [<80310ff0>] device_add+0x4d0/0x578 [<803d1ab4>] snd_soc_instantiate_cards+0xa60/0xbc0 [<803d1d50>] soc_probe+0x13c/0x154 [<80313970>] driver_probe_device+0xc4/0x204 [<80312b90>] bus_for_each_drv+0x68/0xd0 [<80313cd4>] device_attach+0xc0/0xfc [<80312928>] bus_probe_device+0x38/0x54 [<80310fd8>] device_add+0x4b8/0x578 [<8031560c>] platform_device_add+0x12c/0x24c [<8060da40>] db1200_audio_load+0x78/0xa4 [<8010aa40>] do_one_initcall+0x40/0x1e8 [<805fb338>] kernel_init+0xd4/0x174 [<8010e734>] kernel_thread_helper+0x10/0x18 ---[ end trace 8e74e4819d0d06c4 ]--- ALSA device list: #0: DB1200_AC97 Thanks! Manuel Lauss _______________________________________________ Alsa-devel mailing list Alsa-devel@xxxxxxxxxxxxxxxx http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
