Skip to content

[BUG]DMIC/ Headset will capture empty sound after turn down volume to zero then turn it up. #2438

Closed

Description

@ClarexZhou

Describe the bug
System boot up or wake up from s3, then do arecord via DMIC / Headset. Turn down the capture volume to zero then turn it up, DMIC/Headset will capture empty sound.

To Reproduce

  1. Boot up system
  2. Terminal 1: 'arecord -Dhw:0,4 -f dat 1.wav -vv -i'
  3. Terminal 2: 'alsamixer', then turn down the PGA5.0 5 volume till zero then turn it up.

Reproduced rate
4/5

Expected result
DMIC capture normally.

Actual result
DMIC capture empty sound after turn up input volume from zero.

Environment
kernel: https://github.com/thesofproject/linux/tree/release/sdw_cml_hotfix commit: 38d0dcd5
FW: https://github.com/thesofproject/sof/commits/cml-010-drop-stable commit: 5e5f69e
TPLG: Same with FW branch+PR #2431 , sof-cml-rt711-rt1308-mono-rt715.tplg rename to sof-cml-rt711-rt1308-rt715.tplg
Platform: CML Laptop with ALC711+ ALC1308+ ALC715 in SDW mode

No error message in dmesg.
error sof logger

 CORE  LEVEL      COMP_ID                TIMESTAMP            DELTA                FILE_NAME	CONTENT
    0      1           SA          17794084.322917  17794084.000000      src/lib/agent.c:55  	validate(), ll drift detected, delta = 25509
    0      1       VOLUME 5.19     30862011.354167  13067927.000000 udio/volume/volume.c:285 	volume_set_chan: Limited request 0 to VOL_MIN.
    0      1       VOLUME 5.19     30862868.020833       856.666687 udio/volume/volume.c:285 	volume_set_chan: Limited request 0 to VOL_MIN.
    0      1       VOLUME 5.19     30862878.489583        10.468750 udio/volume/volume.c:285 	volume_set_chan: Limited request 0 to VOL_MIN.
    0      1       VOLUME 5.19     32648222.760417   1785344.250000 udio/volume/volume.c:285 	volume_set_chan: Limited request 0 to VOL_MIN.

dmesg.txt
errorSofLogger.txt
fullSofLogger.txt

Activity

  1. added
    bugSomething isn't working as expected
    CMLApplies to Comet Lake platform
    SDWSoundWire
    on Feb 26, 2020
  2. ClarexZhou commented on Feb 26, 2020

    @ClarexZhou
    Author

    Error not seen when using tplg without PR #2431, and same cml-010-drop-stable commit: 5e5f69e FW.
    Error not seen when using tplg with PR #2431 , but using Master branch FW, commit a6726cf.

    PR #2431 is to enable MIC mute LED in topology for CML-U SDW laptop.
    @libinyang Could you comment on this PR?

  3. libinyang commented on Feb 26, 2020

    @libinyang
    Contributor

    The PR add 2 "switch" controls in the tplg, one is for headset MIC, the other is for DMIC. User can use the "switch" controls to mute the capture pipeline (the volume pipeline with 'switch' support). The driver need the switch control because the driver need the 'switch' controls to support MIC LED.

  4. changed the title [-][BUG][BUG][UCM]DMIC will capture empty sound after turn down volume to zero then turn it up.[/-] [+][BUG]DMIC will capture empty sound after turn down volume to zero then turn it up.[/+] on Feb 26, 2020
  5. juimonen commented on Feb 26, 2020

    @juimonen

    so we have couple of patches in master that will probably affect this:

    related PR's
    #2196
    62dbfcf

    #2109
    e525c17
    901a963
    dcf722f
    9d4ca10

    so we should try out these on top of the 010-drop-stable

  6. juimonen commented on Feb 26, 2020

    @juimonen

    #2441
    @ClarexZhou can you try if above PR helps?

  7. lgirdwood commented on Feb 26, 2020

    @lgirdwood
    Member

    @ClarexZhou does this bug occur in master ?

  8. lgirdwood commented on Feb 26, 2020

    @lgirdwood
    Member

    @singalsu any comments ?

  9. ClarexZhou commented on Feb 27, 2020

    @ClarexZhou
    Author

    @ClarexZhou does this bug occur in master ?

    Error not occurred in master

  10. ClarexZhou commented on Feb 27, 2020

    @ClarexZhou
    Author

    #2441
    @ClarexZhou can you try if above PR helps?
    Can't apply this patch on cml-010-drop-stable directly for merge failed.

  11. ClarexZhou commented on Feb 27, 2020

    @ClarexZhou
    Author

    Error also can be reproduced via rt711 headset capture.
    Following below steps:

    1. Boot up system
    2. Terminal 1: 'arecord -Dhw:0,1 -f dat 1.wav -vv -i'
      3 Terminal 2: 'alsamixer', then turn down the PGA2.0 2 volume till zero then turn it up.
  12. changed the title [-][BUG]DMIC will capture empty sound after turn down volume to zero then turn it up.[/-] [+][BUG]DMIC/ Headset will capture empty sound after turn down volume to zero then turn it up.[/+] on Feb 27, 2020
  13. juimonen commented on Feb 27, 2020

    @juimonen

    @ClarexZhou can you just try the PR 2441 itself? it is on top of 010-drop-stable already...

  14. libinyang commented on Feb 27, 2020

    @libinyang
    Contributor

    agree with Jaska, especially for 62dbfcf. The description is very similar with our issue.

  15. ClarexZhou commented on Feb 27, 2020

    @ClarexZhou
    Author

    Error not seen with PR #2441 on cml-010-drop-stable

  16. ClarexZhou commented on Feb 28, 2020

    @ClarexZhou
    Author

    Error is fixed on cml-010-hot-fix branch. Close

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 platformNOT ON MASTERIssue is not seen on masterSDWSoundWirebugSomething isn't working as expected

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions