From: Takashi Iwai <tiwai@suse.de>
Subject: Re: [PATCH] ALSA: hda_intel: add
	AZX_DCAPS_I915_POWERWELL for skl
Date: Fri, 27 Mar 2015 09:19:28 +0100
Message-ID: <s5hegoa97jj.wl-tiwai@suse.de>
References: <1427440204-49331-1-git-send-email-libin.yang@intel.com>
 <s5hfv8q98lm.wl-tiwai@suse.de>
 <96A12704CE18D347B625EE2D4A099D19622F1C@SHSMSX103.ccr.corp.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: <alsa-devel-bounces@alsa-project.org>
Received: from mx2.suse.de (cantor2.suse.de [195.135.220.15])
 by alsa0.perex.cz (Postfix) with ESMTP id 0C24B260410
 for <alsa-devel@alsa-project.org>; Fri, 27 Mar 2015 09:19:29 +0100 (CET)
In-Reply-To: <96A12704CE18D347B625EE2D4A099D19622F1C@SHSMSX103.ccr.corp.intel.com>
List-Unsubscribe: <http://mailman.alsa-project.org/mailman/options/alsa-devel>,
 <mailto:alsa-devel-request@alsa-project.org?subject=unsubscribe>
List-Archive: <http://mailman.alsa-project.org/pipermail/alsa-devel/>
List-Post: <mailto:alsa-devel@alsa-project.org>
List-Help: <mailto:alsa-devel-request@alsa-project.org?subject=help>
List-Subscribe: <http://mailman.alsa-project.org/mailman/listinfo/alsa-devel>,
 <mailto:alsa-devel-request@alsa-project.org?subject=subscribe>
Errors-To: alsa-devel-bounces@alsa-project.org
Sender: alsa-devel-bounces@alsa-project.org
To: "Yang, Libin" <libin.yang@intel.com>
Cc: "alsa-devel@alsa-project.org" <alsa-devel@alsa-project.org>
List-Id: alsa-devel@alsa-project.org

At Fri, 27 Mar 2015 08:02:54 +0000,
Yang, Libin wrote:
> 
> Hi Takashi,
> 
> 
> 
> > -----Original Message-----
> > From: Takashi Iwai [mailto:tiwai@suse.de]
> > Sent: Friday, March 27, 2015 3:57 PM
> > To: Yang, Libin
> > Cc: alsa-devel@alsa-project.org
> > Subject: Re: [PATCH] ALSA: hda_intel: add
> > AZX_DCAPS_I915_POWERWELL for skl
> > 
> > At Fri, 27 Mar 2015 15:10:04 +0800,
> > libin.yang@intel.com wrote:
> > >
> > > From: Libin Yang <libin.yang@intel.com>
> > >
> > > HDMI/DP codec on SKL is in the power well.
> > > The power well must be turned on before probing the
> > > HDMI/DP codec.
> > >
> > > Signed-off-by: Libin Yang <libin.yang@intel.com>
> > 
> > So, was the previous question clarified?
> 
> Yes, I have confirmed with our silicon team.
> 
> > 
> > This certainly sucks.  It means that the powerwell is on even you
> > don't use the HDMI/DP at all.  If this is intended as a temporarily
> > workaround, it should be mentioned so.  Please give more comments
> > and
> > backgrounds.
> 
> Yes, as this is added in the skl audio controller, even there is no HDMI/DP
> codec, we should also add this flag. Otherwise the HDMI/DP codec
> may not be detected correctly.

But it's possible to do it only at probing, not permanently.  If so,
we'll have another patch in future.

Please write more information in the changelog and resubmit.


Takashi