[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <20090317.101838.104710820.nemoto@toshiba-tops.co.jp>
Date: Tue, 17 Mar 2009 10:18:38 +0900 (JST)
From: Atsushi Nemoto <anemo@....ocn.ne.jp>
To: broonie@...nsource.wolfsonmicro.com
Cc: tiwai@...e.de, mano@...rinelk.homelinux.net,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH] ASoC: Only deregister AC97 dev if it's name was not
"AC97"
On Mon, 16 Mar 2009 16:01:37 +0000, Mark Brown <broonie@...nsource.wolfsonmicro.com> 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@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists