Repository navigation
[BUG][TGL][DSM]Sound from speaker0,0 is distortion when playback and capture on speaker at the same time #3073
Description
Describe the bug
Sound from speaker0,0 is distortion when playback and capture on speaker at the same time
To Reproduce
1.Run "aplay -Dhw:0,0 -f s32_le -c 2 -r 48000 sine_48k_32bit_10s_stereo_l5_r2.wav -d 5 & arecord -Dhw:0,0 -f s32_le -c 4 -r 48000 record.wav -d 5"
2.Check the sound output from speaker.
Reproduction Rate
5/5
Expected behavior
Sound from speaker0,0 should be normally when playback and capture on speaker at the same time
Impact
Sound from speaker0,0 is distortion when playback and capture on speaker at the same time
Environment
Kernel:https://github.com/thesofproject/linux/commits/topic/sof-dev (commit: 1b2f260)
- PR#2208 for DP fix
- PR#2184 for wov S0i2
- 0001-hack-hack-98373-for-feedback-data-validation.patch.txt
For volteer:
- revert regression on USB-C 6a7c533 for volteer
- revert d4b41f8bb
- revert regression on PINCTRL cd0a323 for volteer
https://github.com/thesofproject/sof/tree/releases/tgl/v7.0-rc2 commit: b36dabe - revert e35e91d
- PR 3069
FW:
https://github.com/thesofproject/sof/tree/releases/tgl/v7.0-rc2 commit: b36dabe - revert e35e91d
- PR 3069
TPLG:sof-tgl-max98373-rt5682.tplg
Platform:Chromebook with codec 98373 in I2S mode.
Activity
sinahuang commentedon Jun 22, 2020
Error not seen with tgl-006-drop-stable released FW + tplg on https://github.com/thesofproject/sof/tree/releases/tgl/v7.0-rc2 +pr3069 branch and
kernel https://github.com/thesofproject/linux/commits/topic/sof-dev +PR#2208
PR#2184 +0001-hack-hack-98373-for-feedback-data-validation.patch.txt .
ClarexZhou commentedon Jun 22, 2020
Error only occurred if build fw with chrome_override changes in config
make tigerlake_chrome_overridedefconfig
Error not seen if build fw with only defaultconfig
jajanusz commentedon Jun 22, 2020
Problem is caused by not fitting into 120MIPS (there is 400mhz clock with defconfig and 120mhz in chrome config)
mengdonglin commentedon Jun 22, 2020
@jajanusz So this is a performance restriction caused by lower clock frequency?
jajanusz commentedon Jun 22, 2020
Yes, we are testing some optimization fix, but if it's not enough we will need to use dynamic clock switching.
ClarexZhou commentedon Jun 22, 2020
issue only occurred after enable LPRO
Error not seen if build with LPS enabled, and disable LPRO.
lgirdwood commentedon Jun 22, 2020
@sinahuang @ClarexZhou are we using any demux in the topology ? We have an updated DEMUX pending that reduces from 70MCPS to 8.5 MCPS
sathya-nujella commentedon Jun 22, 2020
this is a blocker for us: we can not enable WoV + DSM feature at this moment..
RDharageswari commentedon Jun 22, 2020
We see I/O error when we open the (feedback)capture path while speaker playback
localhost ~ # arecord -Dhw:0,0 try.wav -c4 -fs32_le -d10 -r48000
Recording WAVE 'try.wav' : Signed 32 bit Little Endian, Rate 48000 Hz, Channels 4
arecord: pcm_read:2151: read error: Input/output error
sathya-nujella commentedon Jun 22, 2020
Is it like using LPRO for only S0ix+Wov and for all other cases use HPRO?
1 remaining item
keyonjie commentedon Jun 23, 2020
Yes, we use demux there. @sinahuang @ClarexZhou can you help to try if Bartosz's PR help for this?
#3000
ClarexZhou commentedon Jun 23, 2020
Error still can be reproduced with https://github.com/thesofproject/sof/commits/tgl-007-drop-stable, commit f16bf1e. Error can be reproduced even after disable LPRO in config with this commit.
Error not seen with PR#3000, even with LPRO enabled config
slawblauciak commentedon Jun 23, 2020
The mentioned PR will not fix the problem if the DSM processing is enabled.
lgirdwood commentedon Jun 23, 2020
Btw, there was another DEMUX corruption fix applied yesterday (sorry, cant remeber PR), we should be applying that and #3000 as dependencies in order to fix this issue (since they fix other related issues that can impact this issue). There may be other PRs still needed though for final fix.
ClarexZhou commentedon Jun 24, 2020
No distortion can be heard with tgl-007-drop-stable, tag: releases/tgl/v7.0-rc5, commi: 1dd07a8 , but the record wav files is not as smooth as previously TGL 006 release. This isn't caused by c27ed8e , create new issue to track #3092
ClarexZhou commentedon Jun 29, 2020
Keep open, we are waiting for PR on Master.
slawblauciak commentedon Jul 28, 2020
PR to be merged: #3205
slawblauciak commentedon Aug 18, 2020
@ClarexZhou PR merged, good to close?
keqiaozhang commentedon Sep 3, 2020
Close for not reproducible.