mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Vinod <vkoul@kernel.org>
To: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Cc: Srinivas Kandagatla <srinivas.kandagatla@linaro.org>,
	broonie@kernel.org, lgirdwood@gmail.com,
	alsa-devel@alsa-project.org, bgoswami@codeaurora.org,
	linux-kernel@vger.kernel.org, tiwai@suse.com,
	rohitkr@codeaurora.org
Subject: Re: [alsa-devel] [PATCH] ASoC: core: add support to card re-bind/unbind using component framework
Date: Thu, 12 Jul 2018 22:13:50 +0530	[thread overview]
Message-ID: <20180712164350.GW3219@vkoul-mobl> (raw)
In-Reply-To: <68715fbc-9447-dfaf-3fe5-4200684c2cbc@linux.intel.com>

On 12-07-18, 10:26, Pierre-Louis Bossart wrote:
> On 7/11/18 3:43 AM, Srinivas Kandagatla wrote:
> > This patch aims at add achieving dynamic behaviour of audio card when
> > the dependent components disappear and reappear.
> > 
> > With this patch the card is removed if any of the dependent component
> > is removed and card is added back if the dependent component comes back.
> > All this is done using component framework and matching based on
> > component name.
> 
> Humm, no real comment on this patch proper but rather on how userspace would
> deal with this dynamic behavior?

As Takashi pointed out card disappearing and appearing should not be an
issue. On this machine, I typically run a BT headset which disappears
and keep appearing, haven't seen issue :)

> We had similar opens when we worked on the BYT/CHT HDMI stuff and ended-up
> with quite a few issues in userspace (PulseAudio mainly) related to dynamic
> behavior at the kernel level as a result of plug/unplug.

IIRC those had something to do with gfx callback, register, interrupt
racing on BYT HDMI. Ramesh/Jerome might have more details if they
remember now :)

> Also not sure how
> an Android HAL would deal with a card disappearing temporarily if DSP
> resources become unavailable or unresponsive. Any thoughts or guidance you
> might think of?

I have not see userspace HAL code but my guess is that it deals with
card not being there.

Btw this was handled/buried in the driver atm and asking realized that
this is better suited in core, so Srini pushed it up. Current QC stack
expects DSP to go off and card disappear as a result. Once DSP is back
it would appear again (DSP is handled thru remoteproc, I think you
should have a look at it and see if it would suit your DSP needs too)

-- 
~Vinod

      parent reply	other threads:[~2018-07-12 16:43 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-07-11  8:43 Srinivas Kandagatla
2018-07-12 10:59 ` Vinod
2018-07-12 12:02   ` Srinivas Kandagatla
2018-07-12 12:39     ` Vinod
     [not found] ` <68715fbc-9447-dfaf-3fe5-4200684c2cbc@linux.intel.com>
2018-07-12 15:42   ` [alsa-devel] " Takashi Iwai
2018-07-12 15:54     ` Pierre-Louis Bossart
2018-07-12 16:17       ` Mark Brown
2018-07-12 15:58     ` Mark Brown
2018-07-12 16:43   ` Vinod [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=20180712164350.GW3219@vkoul-mobl \
    --to=vkoul@kernel.org \
    --cc=alsa-devel@alsa-project.org \
    --cc=bgoswami@codeaurora.org \
    --cc=broonie@kernel.org \
    --cc=lgirdwood@gmail.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=pierre-louis.bossart@linux.intel.com \
    --cc=rohitkr@codeaurora.org \
    --cc=srinivas.kandagatla@linaro.org \
    --cc=tiwai@suse.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

all inboxes | Powered by JetHome®