From: Mark Brown <broonie@kernel.org>
To: Takashi Iwai <tiwai@suse.de>
Cc: alsa-devel@alsa-project.org,
Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
Subject: Re: [PATCH v2 0/2] Add snd_card_disconnect_sync() helper
Date: Fri, 13 Oct 2017 18:11:50 +0100 [thread overview]
Message-ID: <20171013171150.fcpwiaj5issdgmym@sirena.co.uk> (raw)
In-Reply-To: <s5hh8v3pelc.wl-tiwai@suse.de>
[-- Attachment #1.1: Type: text/plain, Size: 1030 bytes --]
On Fri, Oct 13, 2017 at 11:43:59AM +0200, Takashi Iwai wrote:
> Kuninori Morimoto wrote:
> > My environment now is I'm using DPCM.
> > FE : rsnd
> > FE : rsnd
> > BE : ak4613
> > 1st issue is that kernel need below patch.
> > I guess BE is using dummy driver, and it doesn't have ops(?).
> > If this is needed, I can post it.
> No, this can't be right. Every PCM implementation mandates the
> presence of a trigger callback. It's a must. If a fix needed, it has
> to be fixed in the driver side.
I suspect this is a DPCM framework thing - when the paths are joined up
at runtime there's going to be at least one trigger in there but with
DPCM it's most likely going to be in the front end, the back end DMA is
often going to be transparent to the system and so not have a trigger.
However if you just iterate over all the PCMs you'll see the decomposed
front and back end links with their dummies in there. Not sure what the
fix is here, probably we need to be hiding the back end links more from
the ALSA core.
Liam?
[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
[-- Attachment #2: Type: text/plain, Size: 0 bytes --]
next prev parent reply other threads:[~2017-10-13 17:11 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-10-11 10:16 [PATCH v2 0/2] Add snd_card_disconnect_sync() helper Takashi Iwai
2017-10-11 10:16 ` [PATCH v2 1/2] ALSA: add snd_card_disconnect_sync() Takashi Iwai
2017-10-11 11:22 ` Takashi Sakamoto
2017-10-11 12:19 ` Takashi Iwai
2017-10-11 13:13 ` Takashi Iwai
2017-10-11 10:16 ` [PATCH v2 2/2] ALSA: pcm: Forcibly stop at disconnect callback Takashi Iwai
2017-10-13 7:39 ` [PATCH v2 0/2] Add snd_card_disconnect_sync() helper Kuninori Morimoto
2017-10-13 7:44 ` Takashi Iwai
2017-10-13 9:11 ` Kuninori Morimoto
2017-10-13 9:43 ` Takashi Iwai
2017-10-13 17:11 ` Mark Brown [this message]
2017-10-16 2:26 ` Kuninori Morimoto
2017-10-16 15:37 ` Takashi Iwai
2017-10-17 0:59 ` Kuninori Morimoto
2017-10-17 10:37 ` Takashi Iwai
2017-10-17 20:05 ` Mark Brown
2017-10-18 2:08 ` Kuninori Morimoto
2017-10-18 6:10 ` Takashi Iwai
2017-10-18 7:13 ` Kuninori Morimoto
2017-10-17 14:25 ` Takashi Sakamoto
2017-10-17 14:34 ` Takashi Iwai
2017-10-17 15:16 ` Takashi Sakamoto
2017-10-17 16:09 ` Takashi Iwai
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=20171013171150.fcpwiaj5issdgmym@sirena.co.uk \
--to=broonie@kernel.org \
--cc=alsa-devel@alsa-project.org \
--cc=kuninori.morimoto.gx@renesas.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.