All of lore.kernel.org
 help / color / mirror / Atom feed
From: David Henningsson <david.henningsson@canonical.com>
To: "Yang, Libin" <libin.yang@intel.com>,
	'Takashi Iwai' <tiwai@suse.de>,
	"Lin, Mengdong" <mengdong.lin@intel.com>,
	"tanuk@iki.fi" <tanuk@iki.fi>,
	"Girdwood, Liam R" <liam.r.girdwood@intel.com>
Cc: "airlied@linux.ie" <airlied@linux.ie>,
	"alsa-devel@alsa-project.org" <alsa-devel@alsa-project.org>
Subject: Re: DP1.2 MST audio support discussion
Date: Tue, 13 Oct 2015 08:47:45 +0200	[thread overview]
Message-ID: <561CA911.7070405@canonical.com> (raw)
In-Reply-To: <96A12704CE18D347B625EE2D4A099D1974C250@SHSMSX103.ccr.corp.intel.com>

Hi Libin,

On 2015-10-13 08:25, Yang, Libin wrote:
> Hi Takashi and all,
>
> We are planning to enable DP1.2 MST (Multi-Stream Transport)
> audio.

This was also discussed during the audio mini-summit in Dublin last week.

>
> Based on the previous discussion, we will extend the
> struct hdmi_spec_per_pin to support MST audio device entry.
> So the struct hdmi_spec_per_pin can be a real pin or a device
> entry in the pin.  The idea is to add a member dev_idx in the
> struct hdmi_spec_per_pin. Dev_idx, together with pin_nid,
> can represent a device entry.
>
> 1. Dynamic PCM assignment
> We will use dynamic PCM assignment for MST audio. This
> means we will create a fixed number of PCMs (the number
> is the same convertor number). All the created PCMs will not
> be assigned to any pin (device entry). When there is a monitor
> connected, an available PCM will be assigned to the pin. And
> it will be de-assigned when the monitor is disconnected.
> Userspace can fetch the HW param when monitor connection
> status is changed.

For the dynamic PCM assignment, the suggestion is to try to use static 
as much as possible. That is, first try the existing static mapping 
(port B -> 3, port C -> 7, port D -> 8), and only if that PCM is already 
being used, try another PCM.

One could then allocate two extra PCMs from the start (9 and 10) to try 
in case the other PCM is busy (my preference), or one could steal one of 
the existing non-busy ones (Takashi's preference).

> I'm not sure how to notify the userspace, such as notifying
> pulseaudio the PCM is assigned or de-assigned. Any ideas?

Userspace will receive change events on Jack and ELD kctls which can be 
used for this. However, there's not yet code in PulseAudio that will 
re-probe a PCM when a Jack event is received.

> 2. Compatibility.
> We will patch patch_hdmi.c to support the MST audio.
> Will we use mst audio driver to support the old mode
> or we use a flag, when HW doesn't support MST audio,
> we will use the old code? Suppose MST audio driver should
> be able support both MST audio and non-MST audio.

Apparently, the suggestion seems to be three hdmi codec drivers for the 
same hardware? One with MST audio, one without MST audio, and one ported 
to the ASoC framework :-/


-- 
David Henningsson, Canonical Ltd.
https://launchpad.net/~diwic

  reply	other threads:[~2015-10-13  6:47 UTC|newest]

Thread overview: 57+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-10-13  6:25 DP1.2 MST audio support discussion Yang, Libin
2015-10-13  6:47 ` David Henningsson [this message]
2015-10-13  7:34   ` Yang, Libin
2015-10-13  9:20     ` Takashi Iwai
2015-10-13 12:31       ` Yang, Libin
2015-10-13 14:03         ` Takashi Iwai
2015-10-13 14:12           ` Yang, Libin
2015-10-13 16:20             ` Takashi Iwai
2015-10-14  2:03               ` Yang, Libin
2015-10-13 23:52       ` Raymond Yau
2015-10-14  2:15         ` Yang, Libin
2015-10-14  6:44         ` Takashi Iwai
2015-10-16  0:32           ` Raymond Yau
2015-10-16  6:38             ` Takashi Iwai
2015-10-16  8:51               ` Yang, Libin
2015-10-16  9:00                 ` Takashi Iwai
2015-10-16 11:55                   ` Yang, Libin
2015-10-19  5:16               ` Yang, Libin
2015-10-22  1:31             ` Yang, Libin
2015-10-22  6:51               ` David Henningsson
2015-10-22  6:56                 ` Yang, Libin
2015-10-22  7:27                   ` David Henningsson
2015-10-22  7:40                     ` Yang, Libin
2015-10-22  8:52                       ` David Henningsson
2015-10-22 11:21                         ` Yang, Libin
2015-10-22 17:42                         ` Takashi Iwai
2015-10-23  5:30                         ` Lin, Mengdong
2015-10-23  5:53                           ` Takashi Iwai
2015-10-23  8:35                             ` Lin, Mengdong
2015-10-23  8:44                               ` Takashi Iwai
2015-10-23 10:15                                 ` Lin, Mengdong
2015-11-13  7:27                             ` Raymond Yau
2015-10-23 10:55                           ` David Henningsson
2015-10-23 12:35                             ` Lin, Mengdong
2015-10-27  8:45                               ` Yang, Libin
2015-10-30 11:27                                 ` Takashi Iwai
2015-11-01  8:53                                   ` Raymond Yau
2015-11-02  7:54                                     ` Yang, Libin
2015-11-02  7:30                                   ` Yang, Libin
2015-11-02  7:46                                     ` Takashi Iwai
2015-11-02  7:55                                     ` David Henningsson
2015-11-04 14:17                                       ` Yang, Libin
2015-11-04 15:04                                         ` Jani Nikula
2015-11-05  9:07                                         ` David Henningsson
2015-11-05  9:39                                           ` Raymond Yau
2015-11-10  6:46                                           ` Yang, Libin
2015-11-10  7:45                                             ` David Henningsson
2015-11-11  2:04                                               ` Yang, Libin
2015-11-11  7:58                                                 ` Yang, Libin
2015-11-11  8:05                                                   ` Takashi Iwai
2015-11-11  8:11                                                     ` Yang, Libin
2015-11-11  8:26                                                       ` Takashi Iwai
2015-11-11  8:33                                                         ` Yang, Libin
2015-11-11  8:52                                                           ` Takashi Iwai
2015-10-23  9:40                 ` Raymond Yau
2015-10-23 13:08                   ` Lin, Mengdong
2015-10-16  1:11 ` Yang, Libin

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=561CA911.7070405@canonical.com \
    --to=david.henningsson@canonical.com \
    --cc=airlied@linux.ie \
    --cc=alsa-devel@alsa-project.org \
    --cc=liam.r.girdwood@intel.com \
    --cc=libin.yang@intel.com \
    --cc=mengdong.lin@intel.com \
    --cc=tanuk@iki.fi \
    --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.