From: Julien Bramary <julien.bramary@blueyonder.co.uk>
To: Takashi Iwai <tiwai@suse.de>
Cc: Lee Revell <rlrevell@joe-job.com>, alsa-devel@lists.sourceforge.net
Subject: Re: Trigger Callback
Date: Wed, 02 Aug 2006 13:16:10 +0100 [thread overview]
Message-ID: <44D0978A.10401@blueyonder.co.uk> (raw)
In-Reply-To: <s5hslkf9xzj.wl%tiwai@suse.de>
Takashi Iwai wrote:
> At Tue, 01 Aug 2006 21:14:24 +0100,
> Julien Bramary wrote:
>
>> Lee Revell wrote:
>>
>>
>>>> > "PCM engine" refers to the hardware's PCM engine, not ALSA's. It's 100%
>>>> > device dependent - you need to consult the docs for your sound device to
>>>> > determine how to start the PCM.
>>>> >
>>>> > Lee
>>>>
>> Thanks, I see now why it starts, I'm doing some inits in my dac in the
>> open callback... I should do those here in the trigger.
>>
>
> Remember that the trigger callback is designed really as a "trigger".
> That is, it should be executed as shortly as possible. The callback
> is even in the atomic context within spinlock and irq disablement, so
> any function that my sleep shall cause lockup.
>
> If you need to allocate something, it should be done in either
> open, hw_params or prepare callback. Only start and stop are done in
> trigger callback.
>
>
> Takashi
>
>
Thanks for the reminder. I think this should be ok. The only function I
now have in the trigger is the following 'write' (atomic enough):
/* enable SSC data Tx */
at91_ssc_write(AT91_SSC_CR, AT91_SSC_TXEN);
Cheers,
Julien
-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys -- and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
prev parent reply other threads:[~2006-08-02 12:16 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-08-01 19:48 Trigger Callback Julien Bramary
2006-08-01 19:55 ` Lee Revell
2006-08-01 20:14 ` Julien Bramary
2006-08-02 11:29 ` Takashi Iwai
2006-08-02 12:16 ` Julien Bramary [this message]
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=44D0978A.10401@blueyonder.co.uk \
--to=julien.bramary@blueyonder.co.uk \
--cc=alsa-devel@lists.sourceforge.net \
--cc=rlrevell@joe-job.com \
--cc=tiwai@suse.de \
/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