Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

check-pause-release-suspend-resume: timing fixes + switch to sof-tplgreader #885

Closed
wants to merge 7 commits into from
Closed

check-pause-release-suspend-resume: timing fixes + switch to sof-tplgreader #885

wants to merge 7 commits into from

Commits on Apr 26, 2022

  1. check-pause-release-suspend-resume: fix the shellcheck 

    warning and code cleanup

fix one shellcheck warning and some code cleanup.

Signed-off-by: Keqiao Zhang <[email protected]>
    @keqiaozhang
    keqiaozhang committed Apr 26, 2022
    Configuration menu
    Copy the full SHA
    24de38f View commit details
    Browse the repository at this point in the history

  2. check-pause-release-suspend-resume: use sof-tplgreader to 

    parse pipelines instead of specifying PCM to test

Consistent with other test scripts to use sof-tplgreader
to parse audio pipelines designed in tplg and use -m option
to select part of them to test.

Signed-off-by: Keqiao Zhang <[email protected]>
    @keqiaozhang
    keqiaozhang committed Apr 26, 2022
    Configuration menu
    Copy the full SHA
    0b0a3a8 View commit details
    Browse the repository at this point in the history

  3. check-pause-release-suspend-resume: extend the interval before 

    checking the aplay/arecord status after pause/release

This interval is too short which may casue timing issue
on some platforms. Extend the interval to 500ms to avoid
such issues.

Signed-off-by: Keqiao Zhang <[email protected]>
    @keqiaozhang
    keqiaozhang committed Apr 26, 2022
    Configuration menu
    Copy the full SHA
    bc784a4 View commit details
    Browse the repository at this point in the history

  4. check-pause-release-suspend-resume: add a -T option to specify 

    the sleep type for suspend/resuse

There are 2 sleep types designed in /sys/power/mem_sleep
add a -T option to specify which one you want to test.

Signed-off-by: Keqiao Zhang <[email protected]>
    @keqiaozhang
    keqiaozhang committed Apr 26, 2022
    Configuration menu
    Copy the full SHA
    8fb0258 View commit details
    Browse the repository at this point in the history

  5. check-pause-release-suspend-resume: add a delay time before 

    pausing the audio stream

There's a latency when starting audio stream, if pause
the audio stream directly after it starts, it may lead
to some unexpected errors. Add a delay to ensure that
audio stream is in a normal state before pusing.

Signed-off-by: Keqiao Zhang <[email protected]>
    @keqiaozhang
    keqiaozhang committed Apr 26, 2022
    Configuration menu
    Copy the full SHA
    89fbe4a View commit details
    Browse the repository at this point in the history

  6. check-pause-release-suspend-resume: move the code to a function 

    Move the code to a function to make the code easy
to understand and maintain.

Signed-off-by: Keqiao Zhang <[email protected]>
    @keqiaozhang
    keqiaozhang committed Apr 26, 2022
    Configuration menu
    Copy the full SHA
    909b76e View commit details
    Browse the repository at this point in the history

  7. run-all-tests: add pause-release-suspend-resume related cases 

    Add 2 new cases to run all test:
1. pause-release-suspend-resume-with-playback
2. pause-release-suspend-resume-with-capture

Signed-off-by: Keqiao Zhang <[email protected]>
    @keqiaozhang
    keqiaozhang committed Apr 26, 2022
    Configuration menu
    Copy the full SHA
    61a1972 View commit details
    Browse the repository at this point in the history