[alsa-devel] [PATCH] ALSA: hda_intel: add AZX_DCAPS_I915_POWERWELL for skl

Takashi Iwai tiwai at suse.de
Fri Mar 27 08:56:37 CET 2015


At Fri, 27 Mar 2015 15:10:04 +0800,
libin.yang at intel.com wrote:
> 
> From: Libin Yang <libin.yang at 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 at intel.com>

So, was the previous question clarified?

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.


thanks,

Takashi

> ---
>  sound/pci/hda/hda_intel.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/sound/pci/hda/hda_intel.c b/sound/pci/hda/hda_intel.c
> index 15a8299..4744a96 100644
> --- a/sound/pci/hda/hda_intel.c
> +++ b/sound/pci/hda/hda_intel.c
> @@ -298,7 +298,8 @@ enum {
>  	 AZX_DCAPS_SNOOP_TYPE(SCH))
>  
>  #define AZX_DCAPS_INTEL_SKYLAKE \
> -	(AZX_DCAPS_INTEL_PCH | AZX_DCAPS_SEPARATE_STREAM_TAG)
> +	(AZX_DCAPS_INTEL_PCH | AZX_DCAPS_SEPARATE_STREAM_TAG |\
> +	 AZX_DCAPS_I915_POWERWELL)
>  
>  /* quirks for ATI SB / AMD Hudson */
>  #define AZX_DCAPS_PRESET_ATI_SB \
> -- 
> 1.9.1
> 


More information about the Alsa-devel mailing list