Yes, there also seems to be some missing if I'm not mistaken
(immediately I notice the ones for switching output and input.) If you
lack those, just setting up the DSP is almost pointless.
On Thu, Aug 30, 2018 at 4:08 PM, Takashi Iwai <tiwai@xxxxxxx> wrote:
> On Thu, 30 Aug 2018 20:07:38 +0200,
> Sasha Levin wrote:
>>
>> From: Connor McAdams <conmanx360@xxxxxxxxx>
>>
>> [ Upstream commit 8f8c523c4604afe231196920bf08310141a4f0ba ]
>>
>> This patch adds the PCI subsys ID for the Recon3D that has been tested,
>> and adds the QUIRK_R3D enumeration.
>>
>> Signed-off-by: Connor McAdams <conmanx360@xxxxxxxxx>
>> Signed-off-by: Takashi Iwai <tiwai@xxxxxxx>
>> Signed-off-by: Sasha Levin <alexander.levin@xxxxxxxxxxxxx>
>
> Is the order or patches correct?
> This one has to be applied before patch 77, at least.
>
>
> thanks,
>
> Takashi
>
>> ---
>> sound/pci/hda/patch_ca0132.c | 2 ++
>> 1 file changed, 2 insertions(+)
>>
>> diff --git a/sound/pci/hda/patch_ca0132.c b/sound/pci/hda/patch_ca0132.c
>> index 27ecff0d5c72..3288bbc3687c 100644
>> --- a/sound/pci/hda/patch_ca0132.c
>> +++ b/sound/pci/hda/patch_ca0132.c
>> @@ -994,6 +994,7 @@ enum {
>> QUIRK_ALIENWARE_M17XR4,
>> QUIRK_SBZ,
>> QUIRK_R3DI,
>> + QUIRK_R3D,
>> };
>>
>> static const struct hda_pintbl alienware_pincfgs[] = {
>> @@ -1050,6 +1051,7 @@ static const struct snd_pci_quirk ca0132_quirks[] = {
>> SND_PCI_QUIRK(0x1458, 0xA016, "Recon3Di", QUIRK_R3DI),
>> SND_PCI_QUIRK(0x1458, 0xA026, "Gigabyte G1.Sniper Z97", QUIRK_R3DI),
>> SND_PCI_QUIRK(0x1458, 0xA036, "Gigabyte GA-Z170X-Gaming 7", QUIRK_R3DI),
>> + SND_PCI_QUIRK(0x1102, 0x0013, "Recon3D", QUIRK_R3D),
>> {}
>> };
>>
>> --
>> 2.17.1
>>
