ASoC: Intel: soc-acpi-intel-ptl-match: Add support for MSI Prestige 14/16 Flip AI+ (D3MTG/C3MTG)

[EricLin0509]

Summary

The dmesg log shows this hardware layout

[   13.220268] sof-audio-pci-intel-ptl 0000:00:1f.3: No SoundWire machine driver found for the ACPI-reported configuration:
[   13.220270] sof-audio-pci-intel-ptl 0000:00:1f.3: link 2 mfg_id 0x025d part_id 0x1320 version 0x3
[   13.220271] sof-audio-pci-intel-ptl 0000:00:1f.3: link 2 mfg_id 0x025d part_id 0x1320 version 0x3
[   13.220272] sof-audio-pci-intel-ptl 0000:00:1f.3: link 3 mfg_id 0x025d part_id 0x0713 version 0x3

This hardware has one rt713 for microphones and headphone jacks and two rt1320 for two woofers and two tweeters (4 Speakers in total)

Code chanages

• sound/soc/intel/common/soc-acpi-intel-ptl-match.c
  • adds rt1320_2_group2_lr_adr for the two RT1320 which on the same link 2
  • adds link_mask = BIT(2) | BIT(3) entry referencing sof-ptl-rt713-l3-rt1320-l2.tplg

Dependencies

The topology sof-ptl-rt713-l3-rt1320-l2.tplg is required, I just created a new pull request in sof.

See: thesofproject/sof#10948

[sofci]

Can one of the admins verify this patch?

reply test this please to run this test once

[bardliao]

In theory, we don't need to create a new match table for new audio configs on PTL. Let's see what happened if you don't apply the change.
Please charry pick https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git/commit/?id=c84179a1d36bebe99d9694502737ae9f3a90d2bc to your kernel.
Also, please rename sof-dyndbg.conf.txt to sof-dyndbg.conf and put it in the /etc/modprobe.d/ folder and get the dmesg log after rebooting.

[EricLin0509]

In theory, we don't need to create a new match table for new audio configs on PTL. Let's see what happened if you don't apply the change. Please charry pick https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git/commit/?id=c84179a1d36bebe99d9694502737ae9f3a90d2bc to your kernel. Also, please rename sof-dyndbg.conf.txt to sof-dyndbg.conf and put it in the /etc/modprobe.d/ folder and get the dmesg log after rebooting.

After Reverting my changes, cherry-pick this patch and adding sof-dyndbg.conf, it will fallback to the default driver

[   14.807437] sof-audio-pci-intel-ptl 0000:00:1f.3: SoundWire enabled on CannonLake+ platform, using SOF driver
[   14.807445] sof-audio-pci-intel-ptl 0000:00:1f.3: enabling device (0000 -> 0002)
[   14.807640] sof-audio-pci-intel-ptl 0000:00:1f.3: DSP detected with PCI class/subclass/prog-if 0x040100
[   14.807691] sof-audio-pci-intel-ptl 0000:00:1f.3: bound 0000:00:02.0 (ops intel_audio_component_bind_ops [xe])
[   14.814323] sof-audio-pci-intel-ptl 0000:00:1f.3: use msi interrupt mode
[   14.824388] acpi device:37: SDCA function SmartAmp (type 1) at 0x4
[   14.824585] acpi device:39: SDCA function SmartAmp (type 1) at 0x4
[   14.824870] acpi device:34: SDCA function UAJ (type 6) at 0x1
[   14.824877] acpi device:35: SDCA function SmartMic (type 3) at 0x2
[   14.851455] sof-audio-pci-intel-ptl 0000:00:1f.3: No SoundWire machine driver found for the ACPI-reported configuration:
[   14.851459] sof-audio-pci-intel-ptl 0000:00:1f.3: link 2 mfg_id 0x025d part_id 0x1320 version 0x3
[   14.851462] sof-audio-pci-intel-ptl 0000:00:1f.3: link 2 mfg_id 0x025d part_id 0x1320 version 0x3
[   14.851464] sof-audio-pci-intel-ptl 0000:00:1f.3: link 3 mfg_id 0x025d part_id 0x0713 version 0x3
[   14.851474] sof-audio-pci-intel-ptl 0000:00:1f.3: Use SoundWire default machine driver with function topologies

Here is the full log: log.txt

[bardliao]

In theory, we don't need to create a new match table for new audio configs on PTL. Let's see what happened if you don't apply the change. Please charry pick https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git/commit/?id=c84179a1d36bebe99d9694502737ae9f3a90d2bc to your kernel. Also, please rename sof-dyndbg.conf.txt to sof-dyndbg.conf and put it in the /etc/modprobe.d/ folder and get the dmesg log after rebooting.

After Reverting my changes, cherry-pick this patch and adding sof-dyndbg.conf, it will fallback to the default driver

Yes, that is expected. Do you see any audio issue?

[EricLin0509]

Yes, that is expected. Do you see any audio issue?

Well, there is no audio issue present.

[bardliao]

Yes, that is expected. Do you see any audio issue?

Well, there is no audio issue present.

Can we close the PR then?

[EricLin0509]

Can we close the PR then?

Okay, Close it now.