From: Lars-Peter Clausen <lars@metafoo.de>
To: Ricard Wanderlof <ricard.wanderlof@axis.com>,
Felipe Ferreri Tonello <eu@felipetonello.com>
Cc: alsa-devel <alsa-devel@alsa-project.org>
Subject: Re: Unregister a card from userspace?
Date: Wed, 11 Nov 2015 14:09:30 +0100 [thread overview]
Message-ID: <56433E0A.20804@metafoo.de> (raw)
In-Reply-To: <alpine.DEB.2.02.1511111348380.11990@lnxricardw1.se.axis.com>
On 11/11/2015 01:59 PM, Ricard Wanderlof wrote:
>
> On Wed, 11 Nov 2015, Felipe Ferreri Tonello wrote:
>
>>> But that is precisely the problem. The framework that manages the
>>> insertion and deletion of modules notes that my module has a usage count
>>> that is > 0 (in my case it is 2, which I'm convinced is because the card
>>> has been registered with ALSA), and refuses to free the module. So there
>>> is now way any unregister_card call can be made when my module is freed,
>>> as due to the usage count it would never even be attempted.
>>
>> Do you mean that unbind is never called? Well, the framework should call
>> bind/unbind for each user of your module.
>
> Ok, I didn't know that. It's the first time I've been writing an ALSA
> driver as a loadable module.
>
>> How is your architecture? If you use one card for each probe of module
>> then each bind should create one device, right?
>
> In this case, it is a device which is located on the SoC. There is only
> one instance of it. When the module is loaded, it loads a codec driver
> with its associated DAI driver, a PCM driver, snd-soc-dummy-dai, and then
> a machine driver to tie it all together, finally registering the card with
> ALSA.
>
> Looking in /sys/bus/platform/devices, there seems to be one device created
> for the codec itself, and one for the machine driver. I don't know if
> that's right or wrong, the ALSA driver hierarchy works in the sense that I
> can capture audio from it anyway.
Which one is the module you can't unload? The module that contains the
machine driver should only have its reference count increment when a
playback or capture stream is active so you can't remove it when it is
actively in use.
- Lars
next prev parent reply other threads:[~2015-11-11 13:09 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-11-04 9:36 Unregister a card from userspace? Ricard Wanderlof
2015-11-04 11:24 ` Clemens Ladisch
2015-11-12 13:07 ` Ricard Wanderlof
2015-11-12 13:26 ` Lars-Peter Clausen
2015-11-12 15:38 ` Ricard Wanderlof
2015-11-09 11:24 ` Felipe Ferreri Tonello
2015-11-10 7:55 ` Ricard Wanderlof
2015-11-11 10:34 ` Felipe Ferreri Tonello
2015-11-11 12:59 ` Ricard Wanderlof
2015-11-11 13:09 ` Lars-Peter Clausen [this message]
2015-11-11 15:24 ` Ricard Wanderlof
2015-11-11 14:50 ` Felipe Ferreri Tonello
2015-11-11 15:33 ` Ricard Wanderlof
2015-11-12 10:31 ` Felipe Ferreri Tonello
2015-11-12 14:02 ` Ricard Wanderlof
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=56433E0A.20804@metafoo.de \
--to=lars@metafoo.de \
--cc=alsa-devel@alsa-project.org \
--cc=eu@felipetonello.com \
--cc=ricard.wanderlof@axis.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox