Re: [PATCH] ASoC: Only deregister AC97 dev if it's name was not"AC97"

From: Atsushi Nemoto
Date: Mon Mar 16 2009 - 21:18:54 EST


On Mon, 16 Mar 2009 16:01:37 +0000, Mark Brown <broonie@xxxxxxxxxxxxxxxxxxxxxxxxxxx> wrote:
> > Hm, codec->ac97->dev.bus should be NULL unless it's registered,
> > so calling soc_ac97_dev_unregister() should be harmless...
>
> Atsushi, are you using ac97.c as your codec driver? Could you please
> post the actual backtrace you're seeing?

Yes. I'm using ac97.c and here is the backtrace. The first argument
of sysfs_get_dirent (dir_sd) was NULL. The bottom
platform_device_unregister call is for "soc-audio" device.

[<801fac24>] sysfs_get_dirent+0x24/0xd0
[<801fcb60>] sysfs_remove_group+0x34/0x144
[<802cd3b0>] device_del+0x48/0x178
[<802cd4f4>] device_unregister+0x14/0x28
[<c01afa88>] snd_ac97_dev_disconnect+0x24/0x34 [snd_ac97_codec]
[<c00d1b4c>] snd_device_disconnect+0x48/0x9c [snd]
[<c00d1be0>] snd_device_disconnect_all+0x40/0x80 [snd]
[<c00ccc30>] snd_card_disconnect+0x25c/0x2e0 [snd]
[<c00ccce4>] snd_card_free+0x30/0xf0 [snd]
[<c014c68c>] snd_soc_free_pcms+0xcc/0x110 [snd_soc_core]
[<c01ce0ec>] ac97_soc_remove+0x2c/0x60 [snd_soc_ac97]
[<c014b8b8>] soc_remove+0x64/0x128 [snd_soc_core]
[<802cfad8>] __device_release_driver+0x80/0xb8
[<802cfc40>] device_release_driver+0x28/0x40
[<802cef34>] bus_remove_device+0xc0/0x100
[<802cd490>] device_del+0x128/0x178
[<802d13d4>] platform_device_del+0x2c/0x9c
[<802d1944>] platform_device_unregister+0x14/0x28

---
Atsushi Nemoto
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/