Skip to content

[SDW][BUG]Pop noise occurred when terminate aplay0,2(RT1308) or aplay0,0(RT711) via "Ctrl+C" #1865

Closed

Description

@YvonneYang2

Describe the bug
Pop noise occurred when terminate aplay0,2(RT1308) or aplay0,0(RT711) via Ctrl-C.

To Reproduce

  1. Boot up.
  2. Aplay 0,2 via "aplay -Dhw:0,2 -r48000 -c2 -fs16_le wav/48California_Dreaming.wav -vv -i"
  3. Terminate it by "Ctrl+C". If issue not reproduced, please try step2/3 more times.

frequency
Not each terminate will occur pop noise. Do step2/3 10 times, could reproduce about 5 times.

Expected result
Terminate it via "Ctrl+C" has no pop noise.

Actual result
Terminate it via "Ctrl+C" has pop noise.

Environment
Kernel:release/sdw_cml_hotfix, commit:38d0dcd
FW: commits/cml-010-hot-fix, commit: 0ed0dba
tplg: Same with FW branch, sof-cml-rt711-rt1308-mono-rt715.tplg
platform: CML-U Laptop with ALC711+ ALC1308+ ALC715 in SDW mode
ucm: https://github.com/thesofproject/alsa-ucm-conf.git+PR#24 or PR#28

ctrlc_demsg.log
ctrlc_soflogger.log

Activity

  1. added
    bugSomething isn't working
    CMLApplies to Comet Lake platform
    SDWApplies to SoundWire bus for codec connection
    on Mar 6, 2020
  2. changed the title [-]v[SDW][BUG]Pop noise occurred when terminate aplay0,2(RT1308) or aplay0,0(RT711) via Ctrl-C[/-] [+][SDW][BUG]Pop noise occurred when terminate aplay0,2(RT1308) or aplay0,0(RT711) via "Ctrl+C"[/+] on Mar 6, 2020
  3. YvonneYang2 commented on Mar 6, 2020

    @YvonneYang2
    Author

    Tested with nocodec tplg, please see the picture and attached wave.

    ctl-cToStop

  4. sinahuang commented on Mar 6, 2020

    @sinahuang

    Issue also can be reproduced on CML-H Laptop with codec ALC711, 2 x RT1308, RT715 in SDW mode, but can't be reproduced on TGL-U RVP with codec ALC711 in SDW mode and RT1308 in I2S mode.

  5. plbossart commented on Mar 6, 2020

    @plbossart
    Member

    @YvonneYang2 does this happen with the master branches for fw and kernel?

    I don't know if we have a good solution, when playback is interrupted the link remains on for some time but the data transfers will stop abruptly. Even if we push zeroes on the link we'd still have discontinuities so maybe this is an amplifier issue.

    @shumingfan @bardliao what do you think?

  6. bardliao commented on Mar 9, 2020

    @bardliao
    Collaborator

    abruptly

    @plbossart I can see this issue with latest kernel and fw code. And it happens not only when terminate but also start. The pop doesn't sound like discontinuities. The pop is much louder than the music if I set the volume to a very low level. However, there is no pop if I play with /dev/zero.

  7. YvonneYang2 commented on Mar 9, 2020

    @YvonneYang2
    Author

    Error still seen when playing same level audio file 

  8. shumingfan commented on Mar 9, 2020

    @shumingfan

    @plbossart I want to do a silent playback test and @bardliao had completed it. If there is no pop with this test, I think the pop noise doesn't come from the codec.

  9. YvonneYang2 commented on Mar 9, 2020

    @YvonneYang2
    Author

    Checked on latest sof-dev + master, still can produce it.
    Then tried headset loopback noise, got the snapshot and wav.

    Sine wav:
    Ctrl_C
    sinewave.zip

    Same level wav:
    Ctrl_C
    samelevel.zip

  10. ClarexZhou commented on Mar 16, 2020

    @ClarexZhou

    Error is much improved with PR#1897. But still has very light pop via scenario "pause aplay first then Ctl+c to release aplay". Can observe this pop sound when increase speaker volume to Max via PGA3.0 3
    Tested with kernel sof-dev, commit: 0f90705, contains PR #1887 + PR #1897 and FW: https://github.com/thesofproject/sof/tree/cml-010-hot-fix, commit thesofproject/sof@0ed0dba + commit thesofproject/sof@f73332e
    And with same kernel sof-dev, commit: 0f90705, contains PR #1887 + PR #1897 and FW: Master, commit: thesofproject/sof@afd9770,

  11. YvonneYang2 commented on Apr 22, 2020

    @YvonneYang2
    Author

    Since #1897 has been merged, retest it om cml-u laptop, issue cannot be reproduced with latest sof-dev(8dc1789) and master(0befebb). Close it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    CMLApplies to Comet Lake platformPop NoiseSDWApplies to SoundWire bus for codec connectionbugSomething isn't working

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions