Hi Takashi,

Attach patch for fix AMD no sound issue. It also could solve Intel ALC700 no sound issue. Thanks.

BR, Kailang

-----Original Message----- From: Takashi Iwai [mailto:tiwai@suse.de] Sent: Tuesday, November 21, 2017 2:40 PM To: Kailang Subject: Re: AMD Raven Platform

On Tue, 21 Nov 2017 04:13:16 +0100, Kailang wrote:

...

Hi Takashi,

AMD guys add below patch. To support their new HDA controller. But it still no sound. This platform codec is ALC701. But ALC701 was worked with Intel platform on Intel site. My test platform was ICH7. ALC701 and ICH7 was also worked

on my site.

...

I check alsa-info dump as attach file. It's very well.

The patch that AMD guys assigned ATI_SB flag. I think no sound issue maybe the HDA controller setting relation.

Could you can find any difference?

First off, if the main target is the onboard analog I/O, I'd pass index=1,0 option to snd-hda-intel module to swap the HDMI/DP and the analog I/O. This will allow testing with the native ALSA API more easily.

Did you pass need some COEF setups as we applied for Intel reference board? There is even another patch that isn't still included to upstream (the difference is COEF 0x4a with bit 0x2000).

In the controller side, one thing that comes to my mind about AMD is the PCI snooping. Try AZX_DCAPS_PRESET_ATI_HDMI_NS instead as a test. The essential difference with this preset is AZX_DCAPS_SNOOP_OFF bits. Other than that, the recent Intel chip requires the different slot assignment, and AZX_DCAPS_SEPARATE_STREAM_TAG is used. But this is set automatically also when the number of streams is greater than 15, too. (You'll see a warning.)

Feel free to put me to Cc in the communication with AMD people directly.

thanks,

Takashi

...

Many Thanks. Kailang --- a/sound/pci/hda/hda_intel.c +++ b/sound/pci/hda/hda_intel.c @@ -2454,6 +2454,8 @@ static const struct pci_device_id

azx_ids[] = {

...

    /* AMD Hudson */
    { PCI_DEVICE(0x1022, 0x780d),
      .driver_data = AZX_DRIVER_GENERIC | 

AZX_DCAPS_PRESET_ATI_SB

...

},

•   { PCI_DEVICE(0x1022, 0x15e3),
  

•     .driver_data = AZX_DRIVER_GENERIC | 
  

AZX_DCAPS_PRESET_ATI_SB

...

• }, /* ATI HDMI */ { PCI_DEVICE(0x1002, 0x0002), .driver_data = AZX_DRIVER_ATIHDMI_NS |

AZX_DCAPS_PRESET_ATI_HDMI_NS },

------Please consider the environment before printing this e-mail.