From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756031AbcBILo3 (ORCPT ); Tue, 9 Feb 2016 06:44:29 -0500 Received: from mx2.suse.de ([195.135.220.15]:51894 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755261AbcBILo1 (ORCPT ); Tue, 9 Feb 2016 06:44:27 -0500 Date: Tue, 09 Feb 2016 12:44:24 +0100 Message-ID: From: Takashi Iwai To: "Martin Kepplinger" Cc: , , , , , , Subject: Re: [BUG] [REGRESSION] [BISECTED] -rc1 breaks audio over HDMI for i915 In-Reply-To: <56B98888.5000702@posteo.de> References: <56B98888.5000702@posteo.de> User-Agent: Wanderlust/2.15.9 (Almost Unreal) SEMI/1.14.6 (Maruoka) FLIM/1.14.9 (=?UTF-8?B?R29qxY0=?=) APEL/10.8 Emacs/24.5 (x86_64-suse-linux-gnu) MULE/6.0 (HANACHIRUSATO) MIME-Version: 1.0 (generated by SEMI 1.14.6 - "Maruoka") Content-Type: text/plain; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, 09 Feb 2016 07:34:48 +0100, Martin Kepplinger wrote: > > The following change: > > 788d441 ALSA: hda - Use component ops for i915 HDMI/DP audio jack handling > > breaks audio over HDMI on my snd_hda_intel laptop. It is the first bad > commit. > > This was merged for -rc1 and isn't fixed until now, so I got nervous. > There are > no errors in the log that stand out. I'm no sound or alsa developer, and > just reverting it isn't easy since there is stuff depending on this. > > I'm happy to test possible fixes and hope v4.5 not to ship with such a big > regression ;) Could you give a bit more detail about your hardware? At best, give alsa-info.sh output (run with --no-upload option). Also, give the output of dmesg, too. In addition, what shows /proc/asound/card*/eld#*.* files? Does any of it show the proper connection state and ELD? A simple revert of the commit is a patch like below. Does it make the HDMI audio working again? thanks, Takashi --- --- a/sound/pci/hda/patch_hdmi.c +++ b/sound/pci/hda/patch_hdmi.c @@ -156,7 +156,7 @@ struct hdmi_spec { bool i915_bound; /* was i915 bound in this driver? */ }; -#ifdef CONFIG_SND_HDA_I915 +#if 0 #define codec_has_acomp(codec) \ ((codec)->bus->core.audio_component != NULL) #else