From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Dagg Stompler" Subject: Re: [Alsa-user] hdmi sound rerouted? Date: Tue, 05 Nov 2013 10:49:56 +0100 Message-ID: <20131105094956.135600@gmx.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mout.gmx.net (mout.gmx.net [212.227.15.19]) by alsa0.perex.cz (Postfix) with ESMTP id 2AB2226505A for ; Tue, 5 Nov 2013 10:49:59 +0100 (CET) Received: from mailout-eu.gmx.com ([10.1.101.210]) by mrigmx.server.lan (mrigmx001) with ESMTP (Nemesis) id 0M8npK-1VTgaY3sYm-00CCQ9 for ; Tue, 05 Nov 2013 10:49:58 +0100 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: Takashi Iwai , Clemens Ladisch Cc: "Deucher, Alexander" , alsa-user@lists.sourceforge.net, alsa-devel@alsa-project.org List-Id: alsa-devel@alsa-project.org > ----- Original Message ----- > From: Takashi Iwai > Sent: 11/05/13 10:49 AM > To: Clemens Ladisch > Subject: Re: [alsa-devel] [Alsa-user] hdmi sound rerouted? > > At Tue, 05 Nov 2013 09:27:10 +0100, > Clemens Ladisch wrote: > > > > Takashi Iwai wrote: > > > You wanted to play back via this HDMI and it didn't work? > > > > The ATI card did not show up as "ATI", but the other ATI card without > > HDMI did: . > > > > Anyway, let's close this thread: > > > > --8<---------------------------------------------------------------->8-- > > ALSA: hda: add device IDs for AMD Evergreen/Northern Islands HDMI > > > > The device IDs of the AMD Cypress/Juniper/Redwood/Cedar/Cayman/Antilles/ > > Barts/Turks/Caicos HDMI HDA controllers were missing for some reason. > > Add them so that the name shows up properly as "ATI HDMI" instead of > > "Generic", and so that the driver does not try to detect nonexistent > > codecs. > > Well, there is no actual behavior difference between > AZX_DRIVER_ATIHDMI and AZX_DRIVER_GENERIC about AMD HDMI chips. The > codec-probing is pretty same. The only difference in the current code > is the driver name string and the fallback of number of streams, but > the latter is valid only when GCAP register doesn't give proper > values, and it shouldn't happen with a proper chip. > > That being said, the entries weren't added intentionally just because > they aren't needed to be added explicitly. The generic entry suffices > from the functionality POV. > > But it seems that people misunderstand, so I think it's fine to apply > your patch to give a more clear name. > > > Takashi > > > According to Takashi's tests and the lack of complaints, these devices > > work fine without disabling snooping. > > > > Signed-off-by: Clemens Ladisch > > > > --- a/sound/pci/hda/hda_intel.c > > +++ b/sound/pci/hda/hda_intel.c > > @@ -4079,6 +4079,22 @@ > > .driver_data = AZX_DRIVER_ATIHDMI | AZX_DCAPS_PRESET_ATI_HDMI }, > > { PCI_DEVICE(0x1002, 0xaa48), > > .driver_data = AZX_DRIVER_ATIHDMI | AZX_DCAPS_PRESET_ATI_HDMI }, > > + { PCI_DEVICE(0x1002, 0xaa50), > > + .driver_data = AZX_DRIVER_ATIHDMI | AZX_DCAPS_PRESET_ATI_HDMI }, > > + { PCI_DEVICE(0x1002, 0xaa58), > > + .driver_data = AZX_DRIVER_ATIHDMI | AZX_DCAPS_PRESET_ATI_HDMI }, > > + { PCI_DEVICE(0x1002, 0xaa60), > > + .driver_data = AZX_DRIVER_ATIHDMI | AZX_DCAPS_PRESET_ATI_HDMI }, > > + { PCI_DEVICE(0x1002, 0xaa68), > > + .driver_data = AZX_DRIVER_ATIHDMI | AZX_DCAPS_PRESET_ATI_HDMI }, > > + { PCI_DEVICE(0x1002, 0xaa80), > > + .driver_data = AZX_DRIVER_ATIHDMI | AZX_DCAPS_PRESET_ATI_HDMI }, > > + { PCI_DEVICE(0x1002, 0xaa88), > > + .driver_data = AZX_DRIVER_ATIHDMI | AZX_DCAPS_PRESET_ATI_HDMI }, > > + { PCI_DEVICE(0x1002, 0xaa90), > > + .driver_data = AZX_DRIVER_ATIHDMI | AZX_DCAPS_PRESET_ATI_HDMI }, > > + { PCI_DEVICE(0x1002, 0xaa98), > > + .driver_data = AZX_DRIVER_ATIHDMI | AZX_DCAPS_PRESET_ATI_HDMI }, > > { PCI_DEVICE(0x1002, 0x9902), > > .driver_data = AZX_DRIVER_ATIHDMI_NS | AZX_DCAPS_PRESET_ATI_HDMI }, > > { PCI_DEVICE(0x1002, 0xaaa0), > > I'll apply the patch against 3.12 and will report back if it works.