From mboxrd@z Thu Jan 1 00:00:00 1970 From: Takashi Iwai Subject: Re: [PATCH RFC 4/4] ALSA: hda - Use component ops for i915 HDMI/DP audio jack handling Date: Fri, 13 Nov 2015 06:56:17 +0100 Message-ID: References: <1447345214-7212-1-git-send-email-tiwai@suse.de> <1447345214-7212-5-git-send-email-tiwai@suse.de> <5645430D.8060903@linux.intel.com> Mime-Version: 1.0 (generated by SEMI 1.14.6 - "Maruoka") Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mx2.suse.de (mx2.suse.de [195.135.220.15]) by alsa0.perex.cz (Postfix) with ESMTP id F02652612D0 for ; Fri, 13 Nov 2015 11:15:18 +0100 (CET) In-Reply-To: <5645430D.8060903@linux.intel.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: alsa-devel-bounces@alsa-project.org Sender: alsa-devel-bounces@alsa-project.org To: Libin Yang Cc: alsa-devel@alsa-project.org, mengdong.lin@linux.intel.com, David Henningsson List-Id: alsa-devel@alsa-project.org On Fri, 13 Nov 2015 02:55:25 +0100, Libin Yang wrote: > > > On 11/13/2015 12:20 AM, Takashi Iwai wrote: > > Since we have a new audio component ops to fetch the current ELD and > > state now, we can reduce the usage of unsol event of HDMI/DP pins. > > The unsol event isn't only unreliable, but it also needs the power > > up/down of the codec and link at each time, which is a significant > > power and time loss. > > > > In this patch, the jack creation and unsol/jack event handling are > > modified to use the audio component for the dedicated Intel chips. > > > > The jack handling got slightly more codes than a simple usage of > > hda_jack layer since we need to deal directly with snd_jack object; > > the hda_jack layer is basically designed for the pin sense read and > > unsol events, both of which aren't used any longer in our case. > > > > Signed-off-by: Takashi Iwai > > --- > > sound/pci/hda/patch_hdmi.c | 83 ++++++++++++++++++++++++++++++++++++++++++++-- > > 1 file changed, 81 insertions(+), 2 deletions(-) > > > > diff --git a/sound/pci/hda/patch_hdmi.c b/sound/pci/hda/patch_hdmi.c > > index 60cd9e700909..ca1d2d4a295e 100644 > > --- a/sound/pci/hda/patch_hdmi.c > > +++ b/sound/pci/hda/patch_hdmi.c > > @@ -83,6 +83,7 @@ struct hdmi_spec_per_pin { > > struct mutex lock; > > struct delayed_work work; > > struct snd_kcontrol *eld_ctl; > > + struct snd_jack *acomp_jack; /* jack via audio component */ > > int repoll_count; > > bool setup; /* the stream has been set up by prepare callback */ > > int channels; /* current number of channels */ > > @@ -141,6 +142,7 @@ struct hdmi_spec { > > struct hdmi_ops ops; > > > > bool dyn_pin_out; > > + bool use_acomp; /* use audio component for ELD notify/update */ > > > > /* > > * Non-generic VIA/NVIDIA specific > > @@ -1530,6 +1532,9 @@ static int hdmi_read_pin_conn(struct hda_codec *codec, int pin_idx) > > return 0; > > } > > > > +static void sync_eld_via_acomp(struct hda_codec *codec, > > + struct hdmi_spec_per_pin *per_pin); > > + > > static bool hdmi_present_sense(struct hdmi_spec_per_pin *per_pin, int repoll) > > { > > struct hda_jack_tbl *jack; > > @@ -1551,6 +1556,11 @@ static bool hdmi_present_sense(struct hdmi_spec_per_pin *per_pin, int repoll) > > bool eld_changed = false; > > bool ret; > > > > + if (spec->use_acomp) { > > + sync_eld_via_acomp(codec, per_pin); > > + return false; /* don't call snd_hda_jack_report_sync() */ > > I think we still need call intel_verify_pin_cvt_connect(), > intel_not_share_assigned_cvt() and hdmi_setup_audio_infoframe() as > before if eld_valid is true. Right, also ELD kctl notification is missing, too. It implies that we'd better refactor the ELD update code, split the code from hdmi_present_sense() to do both commonly. Takashi