Alsa-Devel Archive on lore.kernel.org
 help / color / mirror / Atom feed
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

  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