From: "Dagg Stompler" <daggs@gmx.com>
To: Takashi Iwai <tiwai@suse.de>, Clemens Ladisch <clemens@ladisch.de>
Cc: "Deucher, Alexander" <Alexander.Deucher@amd.com>,
alsa-user@lists.sourceforge.net, alsa-devel@alsa-project.org
Subject: Re: [Alsa-user] hdmi sound rerouted?
Date: Tue, 05 Nov 2013 10:49:56 +0100 [thread overview]
Message-ID: <20131105094956.135600@gmx.com> (raw)
> ----- 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: <http://thread.gmane.org/gmane.linux.alsa.user/37732>.
> >
> > 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 <clemens@ladisch.de>
> >
> > --- 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.
next reply other threads:[~2013-11-05 9:49 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-11-05 9:49 Dagg Stompler [this message]
-- strict thread matches above, loose matches on Subject: below --
2013-11-04 18:53 [Alsa-user] hdmi sound rerouted? Dagg Stompler
2013-11-05 7:14 ` Takashi Iwai
2013-11-05 8:27 ` Clemens Ladisch
2013-11-05 8:49 ` Takashi Iwai
2013-11-04 12:21 daggs
2013-11-04 13:04 ` Takashi Iwai
2013-11-04 13:47 ` Deucher, Alexander
2013-11-04 14:06 ` Takashi Iwai
2013-11-03 7:50 Dagg Stompler
2013-11-04 8:43 ` Takashi Iwai
2013-10-19 9:49 [alsa-devel] " Dagg Stompler
2013-10-19 12:59 ` [Alsa-user] " Deucher, Alexander
[not found] <20130927185045.47660@gmx.com>
2013-09-28 22:32 ` [alsa-devel] " Clemens Ladisch
2013-09-30 2:23 ` [Alsa-user] " Deucher, Alexander
2013-09-30 7:05 ` Clemens Ladisch
2013-09-30 9:18 ` Takashi Iwai
2013-10-07 13:36 ` Deucher, Alexander
2013-10-08 7:49 ` Takashi Iwai
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20131105094956.135600@gmx.com \
--to=daggs@gmx.com \
--cc=Alexander.Deucher@amd.com \
--cc=alsa-devel@alsa-project.org \
--cc=alsa-user@lists.sourceforge.net \
--cc=clemens@ladisch.de \
--cc=tiwai@suse.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.