Seppo Ingalsuo

> The testbench build failures are compile warnings that are probably easy to fix: Yes, there's an unused variable. ``` /home/runner/work/sof/sof/src/audio/google/google_ctc_audio_processing.c: In function ‘ctc_init’: /home/runner/work/sof/sof/src/audio/google/google_ctc_audio_processing.c:265:29: error: unused variable ‘md’ [-Werror=unused-variable]...

> one question: will testbench support IPC3 and IPC4 both or after change, testbench only support IPC4? I think there need to be separate executable build for IPC versions. The...

@mszleszy It's more efficient for MCPS to treat DMIC FIFO as S32_LE data, no need to mask and shift every PCM sample. The LSB bits b(7:2) are zeros except b(1:0)....

> @singalsu could you please verify config issues? Good that the mtrace.txt was attached. It can be seen that an IIR response (0) is assigned correctly to every channel 1...

There was a mistake in previous PR version with sof-mtl-max98360a-rt5682 PCM IDs. DMIC0 should be 99 (it was 100) and I decided to set DMIC1 PCM ID to 100. The...

Note: This PR needs https://github.com/alsa-project/alsa-utils/pull/250

> for SoundWire, best to follow the list I came up with a couple of years ago... > > [PCMDeviceList.txt](https://github.com/thesofproject/sof/files/14021187/PCMDeviceList.txt) > > Device 11 is "just" for 16khz, for KPB/WOV...

> @singalsu sorry if I am perceived as the painful reviewer, but I have to ask a background technical question: > > In a recent thread (#8834), @perexg mentioned that...

> @lgirdwood @singalsu Unfortunately, Intel Internal CI shows an error in the tools step: make tools cmake Is the alsa-utils up to date there? It needs to have recent changes,...

> Just to clarify, is the topology going to support both 48+96kHz, or just 96kHz? The latter case is problematic, I have no real issues with the former. The changes...