At Tue, 18 Jun 2013 21:42:14 +0800,
Wang Xingchao wrote:
>
> For Intel Haswell HDMI codecs, the pins choose converter 0 by default.
> This would cause conflict when playing audio on unused pins,the pin with
> physical device connected would get audio data too.
> i.e. Pin 0/1/2 default choose converter 0, pin 1 has HDMI monitor connected.
> when play audio on Pin 0 or pin 2, pin 1 could get audio data too.
>
> This patch configure unused pins to choose different converter.
>
> Signed-off-by: Wang Xingchao <xingchao.w...@linux.intel.com>
> ---
> sound/pci/hda/patch_hdmi.c | 91
> ++++++++++++++++++++++++++++++++++++--------
> 1 file changed, 76 insertions(+), 15 deletions(-)
>
> diff --git a/sound/pci/hda/patch_hdmi.c b/sound/pci/hda/patch_hdmi.c
> index 8983747..43f8b76 100644
> --- a/sound/pci/hda/patch_hdmi.c
> +++ b/sound/pci/hda/patch_hdmi.c
> @@ -1110,26 +1110,15 @@ static int hdmi_setup_stream(struct hda_codec *codec,
> hda_nid_t cvt_nid,
> return 0;
> }
>
> -/*
> - * HDA PCM callbacks
> - */
> -static int hdmi_pcm_open(struct hda_pcm_stream *hinfo,
> - struct hda_codec *codec,
> - struct snd_pcm_substream *substream)
> +static int hdmi_choose_cvt(struct hda_codec *codec,
> + int pin_idx, int *cvt_id, int *mux_id)
> {
> struct hdmi_spec *spec = codec->spec;
> - struct snd_pcm_runtime *runtime = substream->runtime;
> - int pin_idx, cvt_idx, mux_idx = 0;
> struct hdmi_spec_per_pin *per_pin;
> - struct hdmi_eld *eld;
> struct hdmi_spec_per_cvt *per_cvt = NULL;
> + int cvt_idx, mux_idx = 0;
>
> - /* Validate hinfo */
> - pin_idx = hinfo_to_pin_index(spec, hinfo);
> - if (snd_BUG_ON(pin_idx < 0))
> - return -EINVAL;
> per_pin = get_pin(spec, pin_idx);
> - eld = &per_pin->sink_eld;
>
> /* Dynamically assign converter to stream */
> for (cvt_idx = 0; cvt_idx < spec->num_cvts; cvt_idx++) {
> @@ -1147,17 +1136,89 @@ static int hdmi_pcm_open(struct hda_pcm_stream *hinfo,
> continue;
> break;
> }
> +
> /* No free converters */
> if (cvt_idx == spec->num_cvts)
> return -ENODEV;
>
> + if (cvt_id)
> + *cvt_id = cvt_idx;
> + if (mux_id)
> + *mux_id = mux_idx;
> +
> + return 0;
> +}
> +
> +static void haswell_config_cvts(struct hda_codec *codec,
> + int pin_id, int mux_id)
> +{
> + struct hdmi_spec *spec = codec->spec;
> + struct hdmi_spec_per_pin *per_pin;
> + int pin_idx, mux_idx;
> + int curr;
> + int err;
> +
> + for (pin_idx = 0; pin_idx < spec->num_pins; pin_idx++) {
> + per_pin = get_pin(spec, pin_idx);
> +
> + if (pin_idx == pin_id)
> + continue;
> +
> + curr = snd_hda_codec_read(codec, per_pin->pin_nid, 0,
> + AC_VERB_GET_CONNECT_SEL, 0);
> +
> + /* Choose another unused converter */
> + if (curr == mux_id) {
> + err = hdmi_choose_cvt(codec, pin_idx, NULL, &mux_idx);
> + if (err < 0)
> + return;
> + snd_printdd("HDMI: choose converter %d for pin %d\n",
> mux_idx, pin_idx);
> + snd_hda_codec_write_cache(codec, per_pin->pin_nid, 0,
> + AC_VERB_SET_CONNECT_SEL,
> + mux_idx);
> + }
> + }
> +}
> +
> +/*
> + * HDA PCM callbacks
> + */
> +static int hdmi_pcm_open(struct hda_pcm_stream *hinfo,
> + struct hda_codec *codec,
> + struct snd_pcm_substream *substream)
> +{
> + struct hdmi_spec *spec = codec->spec;
> + struct snd_pcm_runtime *runtime = substream->runtime;
> + int pin_idx, cvt_idx, mux_idx = 0;
> + struct hdmi_spec_per_pin *per_pin;
> + struct hdmi_eld *eld;
> + struct hdmi_spec_per_cvt *per_cvt = NULL;
> + int err;
> +
> + /* Validate hinfo */
> + pin_idx = hinfo_to_pin_index(spec, hinfo);
> + if (snd_BUG_ON(pin_idx < 0))
> + return -EINVAL;
> + per_pin = get_pin(spec, pin_idx);
> + eld = &per_pin->sink_eld;
> +
> + err = hdmi_choose_cvt(codec, pin_idx, &cvt_idx, &mux_idx);
> + if (err < 0)
> + return err;
> +
> + per_cvt = get_cvt(spec, cvt_idx);
> /* Claim converter */
> per_cvt->assigned = 1;
> hinfo->nid = per_cvt->cvt_nid;
>
> - snd_hda_codec_write(codec, per_pin->pin_nid, 0,
> + snd_hda_codec_write_cache(codec, per_pin->pin_nid, 0,
> AC_VERB_SET_CONNECT_SEL,
> mux_idx);
You shouldn't mix this fix into your patch.
This change should be applied as a separate patch and go to stable
kernel.
I'll cook up your patch to remove that chunk and apply the rest as
is.
thanks,
Takashi
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
