[alsa-devel] [RESEND][PATCH] ASoC: hdac_hdmi: Program pin-port-mux during jack report event

[Takashi Iwai]

On Thu, 21 Jun 2018 12:45:34 +0200,
Sriram Periyasamy wrote:
> 
> On Thu, Jun 21, 2018 at 10:28:33AM +0200, Takashi Iwai wrote:
> > On Thu, 21 Jun 2018 09:54:20 +0200,
> > Sriram Periyasamy wrote:
> > > 
> > > On Wed, Jun 20, 2018 at 03:01:13PM +0200, Takashi Iwai wrote:
> > > > On Wed, 20 Jun 2018 13:08:21 +0200,
> > > > Sriram Periyasamy wrote:
> > > > > 
> > > > > During d3/d0 cycle, the connection selection index of all pins points
> > > > > to the default value. This needs to be restored to ensure audio is
> > > > > restored after d3/d0 cycle.
> > > > > 
> > > > > So store the connection selection index and program it during jack
> > > > > report event which gets invoked in cases like d3/d0 cycle, hot plug
> > > > > detection when multiple displays are connected.
> > > > 
> > > > Hm, I thought *_CONNECT_SEL being cached and restored at resume
> > > > already by regmap?
> > > > 
> > > 
> > > Yes, legacy HDA HDMI driver has this method whereas ASoC based HDAC HDMI
> > > driver doesn't use regmap. Hence we needed this solution.
> > 
> > If so, why not just use the standard snd_hdac_codec_write()?
> > 
> 
> Because snd_hdac_codec_write doesn't power up the device to send the verb
> across the link when the device is runtime suspended.

Then why hdac_hdmi_port_select_set() would work?  It's using
snd_hdac_codec_write(), too.


Takashi