From: Takashi Iwai <tiwai@suse.de>
To: Vinod Koul <vinod.koul@intel.com>
Cc: patches.audio@intel.com, alsa-devel@alsa-project.org,
broonie@kernel.org,
"Subhransu S. Prusty" <subhransu.s.prusty@intel.com>,
lgirdwood@gmail.com
Subject: Re: [PATCH v3 14/15] ASoC: hdac_hdmi: Add jack reporting for user space
Date: Tue, 08 Dec 2015 11:28:16 +0100 [thread overview]
Message-ID: <s5hio49kzen.wl-tiwai@suse.de> (raw)
In-Reply-To: <20151208102059.GQ1854@localhost>
On Tue, 08 Dec 2015 11:20:59 +0100,
Vinod Koul wrote:
>
> On Tue, Dec 08, 2015 at 09:42:09AM +0100, Takashi Iwai wrote:
> > > Okay got it now, that is how my 3 limit for MST is coming from as we
> > > have 3 CVTs so we can actually stream to only 3 devices per port
> > > although someone may theoretically connect 64!
> > >
> > > > > > Though, the max number of converters is limited, thus what you can
> > > > > > actually use is defined by this constraint. Due to this, we'll likely
> > > > > > manage MST for the legacy HDA by assigning MST devs dynamically to
> > > > > > pins in a certain procedure to make things compatible.
> > > > >
> > > > > but how many MST devs are you going to create?
> > > >
> > > > This is exactly the point: for the legacy HDA, instead of creating the
> > > > entry for each MST devices, they are assigned dynamically to PCM at
> > > > activation. So the number of devices exposed to user-space is
> > > > limited -- or better to say, user-space won't notice the difference.
> > >
> > > And how many PCMs are you proposing for MST?
> >
> > 5 for Intel, i.e. Nconv * 2 - 1. This could be even Nconv, but we
> > provide the reserved slots just for the compatible behavior that
> > assumes the static pin/slot assignment.
> >
> > > > > > > Back on present :), what do you recommend for us for jack name.
> > > > > >
> > > > > > Depends on your purpose. If you want to keep it compatible, then use
> > > > > > the compatible string.
> > > > >
> > > > > Since compatible string would mean specfying PCM, which we don;t know and is
> > > > > per machine driver dailink defination, we would go ahead with logical
> > > > > "HDMI/DP, Pin=%d Jack" names.
> > > >
> > > > But then it makes little sense. Multiple MST devices can be on the
> > > > same pin.
> > >
> > > Well am not supporting MST at the moment. How is legacy handling this,
> > > how do you report jack for 64 X 3 devices :)
> > >
> > > Or, are you reporting jack for PCMs based on connection to pin/device, I
> > > think latter.
> >
> > Yes, the jack reporting is for PCM in this case, which is I mentioned
> > in the first reply: it's not about pin but PCM.
> >
> > > But then you would hit limit of PCMs, I assume you maybe doing 9 PCMs for
> > > this, so if someone connects 10th device we won't report it's jack?
> >
> > The jack reporting is done actually at the time the audio is
> > enabled/disabled. And at this moment, the PCM assignment is assured
> > as well. So, to user-space, only the actually usable devices are
> > exposed no matter how many devices are plugged.
>
> Yes that was my thinking based on discussion and patches posted by Libin.
> But one more question who does PCM assignment, looking at patches it seems
> driver and no way to change that..
>
> For example I have 10 devices connected and we have 5 PCMs.
> I would like to route to 7th device, how would that be done?
Manage xrandr to enable the target monitor. Then one PCM stream will
be switched to the corresponding audio, and it'll be notified to
user-space.
Takashi
next prev parent reply other threads:[~2015-12-08 10:28 UTC|newest]
Thread overview: 40+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-12-07 21:17 [PATCH v3 00/15] ASoC: hdac_hdmi: Add DP & notification support Subhransu S. Prusty
2015-12-07 20:12 ` Mark Brown
2015-12-08 11:24 ` Subhransu S. Prusty
2015-12-07 21:24 ` [PATCH v3 01/15] ASoC: hdac_hdmi: Fix to check num nodes correctly Subhransu S. Prusty
2015-12-07 21:24 ` [PATCH v3 02/15] ASoC: hdac_hdmi: Fix to warn instead of err for no connected nids Subhransu S. Prusty
2015-12-07 21:24 ` [PATCH v3 03/15] ASoC: hdac_hdmi: Use list to add pins and converters Subhransu S. Prusty
2016-01-08 13:44 ` Applied "ASoC: hdac_hdmi: Use list to add pins and converters" to the asoc tree Mark Brown
2015-12-07 21:24 ` [PATCH v3 04/15] ASoC: hdac_hdmi: Add hotplug notification and read eld Subhransu S. Prusty
2015-12-07 21:24 ` [PATCH v3 05/15] ASoC: hdac_hdmi: Apply constraints based on ELD Subhransu S. Prusty
2015-12-07 21:24 ` [PATCH v3 06/15] ASoC: hdac_hdmi: Enable DP1.2 and all converters/pins Subhransu S. Prusty
2015-12-07 21:24 ` [PATCH v3 07/15] ASoC: hdac_hdmi: create dais based on number of streams Subhransu S. Prusty
2015-12-07 16:11 ` Takashi Iwai
2015-12-07 21:24 ` [PATCH v3 08/15] ASoC: hdac_hdmi: Create widget/route based on nodes enumerated Subhransu S. Prusty
2015-12-07 16:14 ` Takashi Iwai
2015-12-08 11:28 ` Subhransu S. Prusty
2015-12-07 21:24 ` [PATCH v3 09/15] ASoC: hdac_hdmi: Assign pin for stream based on dapm connection Subhransu S. Prusty
2015-12-07 21:24 ` [PATCH v3 10/15] drm/edid: Add API to help find connection type Subhransu S. Prusty
2015-12-08 14:01 ` Jani Nikula
2015-12-07 21:24 ` [PATCH v3 11/15] ASoC: hdac_hdmi: Add infoframe support for dp audio Subhransu S. Prusty
2015-12-07 21:24 ` [PATCH v3 12/15] ASoC: hdac_hdmi: Add codec suspend/resume handler Subhransu S. Prusty
2015-12-07 21:24 ` [PATCH v3 13/15] ASoC: hdac_hdmi: Fix to keep display active while enumerating codec Subhransu S. Prusty
2015-12-07 21:24 ` [PATCH v3 14/15] ASoC: hdac_hdmi: Add jack reporting for user space Subhransu S. Prusty
2015-12-07 16:18 ` Takashi Iwai
2015-12-08 6:31 ` Vinod Koul
2015-12-08 6:38 ` Takashi Iwai
2015-12-08 7:42 ` Vinod Koul
2015-12-08 7:52 ` Takashi Iwai
2015-12-08 8:38 ` Vinod Koul
2015-12-08 8:42 ` Takashi Iwai
2015-12-08 10:20 ` Vinod Koul
2015-12-08 10:28 ` Takashi Iwai [this message]
2015-12-08 10:42 ` Vinod Koul
2015-12-08 10:51 ` Takashi Iwai
2015-12-08 10:59 ` Takashi Iwai
2015-12-09 5:44 ` Vinod Koul
2015-12-09 8:03 ` Subhransu S. Prusty
2015-12-09 8:13 ` Takashi Iwai
2015-12-09 11:14 ` Subhransu S. Prusty
2015-12-09 11:37 ` Takashi Iwai
2015-12-07 21:24 ` [PATCH v3 15/15] ASoC: hdac_hdmi: Fix to enable device configuration in hw_params Subhransu S. Prusty
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=s5hio49kzen.wl-tiwai@suse.de \
--to=tiwai@suse.de \
--cc=alsa-devel@alsa-project.org \
--cc=broonie@kernel.org \
--cc=lgirdwood@gmail.com \
--cc=patches.audio@intel.com \
--cc=subhransu.s.prusty@intel.com \
--cc=vinod.koul@intel.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 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.