Skip to content

test-case: simultaneous-playback-capture.sh to remove 'both' keyword - #81

Merged
stripes416 merged 1 commit into
thesofproject:masterfrom
Bin-QA:simultaneous
Feb 25, 2020
Merged

stripes416 merged 1 commit into
thesofproject:masterfrom
Bin-QA:simultaneous

Conversation

@Bin-QA

@Bin-QA Bin-QA commented Feb 19, 2020 •

Copy link
Copy Markdown
Contributor

Need to split both type pipeline into playback and capture
Change both keyword into check duplicate pcm id

Wait patch sof-test: split both type pipeline into playback and capture can be merge this PR

Because without that change, this pr couldn't worked

Signed-off-by: Wu, BinX binx.wu@intel.com

@Bin-QA
Bin-QA requested a review from aiChaoSONG as a code owner February 19, 2020 10:01
@stripes416

Copy link
Copy Markdown
Contributor

Okay, so we need to wait until #80 (which is also dependent on #10) is merged before merging this PR?

xiulipan
xiulipan previously approved these changes Feb 21, 2020
@xiulipan

Copy link
Copy Markdown
Contributor

@stripes416 @aiChaoSONG Please help to review and approve as all dependency is merged.

@aiChaoSONG

Copy link
Copy Markdown

@Bin-QA still need some fix. hw:0,1, hw:0,2 shouldn't be present

2020-02-25 08:26:23 UTC [COMMAND] aplay -D hw:0,0 -c 2 -r 48000 -f S16_LE /dev/zero -q &
2020-02-25 08:26:23 UTC [COMMAND] arecord -D hw:0,0 -c 2 -r 48000 -f S16_LE /dev/null -q &
2020-02-25 08:26:23 UTC [INFO] Preparing to sleep for 5
2020-02-25 08:26:28 UTC [INFO] check pipeline after 5s
2020-02-25 08:26:28 UTC [COMMAND] killing all pipelines
2020-02-25 08:26:28 UTC [COMMAND] aplay -D hw:0,1 -c 4 -r 48000 -f S32_LE /dev/zero -q &
2020-02-25 08:26:28 UTC [COMMAND] arecord -D hw:0,2 -c 2 -r 48000 -f S16_LE /dev/null -q &
aplay: main:788: audio open error: No such file or directory
2020-02-25 08:26:28 UTC [INFO] Preparing to sleep for 5
arecord: main:788: audio open error: No such file or directory
2020-02-25 08:26:33 UTC [INFO] check pipeline after 5s
...
2020-02-25 08:26:35 UTC [INFO] Test Result: FAIL!

Need to split both type pipeline into playback and capture
Change both keyword into check duplicate pcm id

Signed-off-by: Wu, BinX <binx.wu@intel.com>
@Bin-QA

Bin-QA commented Feb 25, 2020

Copy link
Copy Markdown
Contributor Author

@Bin-QA still need some fix. hw:0,1, hw:0,2 shouldn't be present

The root cause is by typo, because we current use func_pipeline_export
which will help to check tplg file name, here use the error path & tplg, so the filter is not work
already update to fix it

@aiChaoSONG aiChaoSONG left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, on helios works fine.

@stripes416 stripes416 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works on apl-up2-nocodec. LGTM

kledford@apl-up2-nocodec:~/sof-test/test-case$ ./simultaneous-playback-capture.sh 
2020-02-25 18:17:47 UTC [INFO] ./simultaneous-playback-capture.sh using /lib/firmware/intel/sof-tplg/sof-apl-nocodec.tplg as target TPLG to run the test case
2020-02-25 18:17:47 UTC [INFO] Catch block option from TPLG_BLOCK_LST will block 'pcm=HDA Digital,Media Playback,DMIC16k' for /lib/firmware/intel/sof-tplg/sof-apl-nocodec.tplg
2020-02-25 18:17:47 UTC [INFO] Run command: 'sof-tplgreader.py /lib/firmware/intel/sof-tplg/sof-apl-nocodec.tplg -f id:0,1,3,5 -b pcm:'HDA Digital,Media Playback,DMIC16k' -s 0 -e' to get BASH Array
2020-02-25 18:17:47 UTC [INFO] Testing: (Loop: 1/1)
2020-02-25 18:17:48 UTC [COMMAND] aplay -D hw:0,0 -c 2 -r 48000 -f S16_LE /dev/zero -q &
2020-02-25 18:17:48 UTC [COMMAND] arecord -D hw:0,0 -c 2 -r 48000 -f S16_LE /dev/null -q &
2020-02-25 18:17:48 UTC [INFO] Preparing to sleep for 5
2020-02-25 18:17:53 UTC [INFO] check pipeline after 5s
2020-02-25 18:17:53 UTC [COMMAND] killing all pipelines
2020-02-25 18:17:53 UTC [COMMAND] aplay -D hw:0,1 -c 2 -r 48000 -f S16_LE /dev/zero -q &
2020-02-25 18:17:53 UTC [COMMAND] arecord -D hw:0,1 -c 2 -r 48000 -f S16_LE /dev/null -q &
2020-02-25 18:17:53 UTC [INFO] Preparing to sleep for 5
2020-02-25 18:17:58 UTC [INFO] check pipeline after 5s
2020-02-25 18:17:58 UTC [COMMAND] killing all pipelines
2020-02-25 18:17:58 UTC [COMMAND] aplay -D hw:0,3 -c 2 -r 48000 -f S16_LE /dev/zero -q &
2020-02-25 18:17:58 UTC [COMMAND] arecord -D hw:0,3 -c 2 -r 48000 -f S16_LE /dev/null -q &
2020-02-25 18:17:58 UTC [INFO] Preparing to sleep for 5
2020-02-25 18:18:03 UTC [INFO] check pipeline after 5s
2020-02-25 18:18:03 UTC [COMMAND] killing all pipelines
2020-02-25 18:18:03 UTC [COMMAND] aplay -D hw:0,5 -c 2 -r 48000 -f S16_LE /dev/zero -q &
2020-02-25 18:18:03 UTC [COMMAND] arecord -D hw:0,5 -c 2 -r 48000 -f S16_LE /dev/null -q &
2020-02-25 18:18:03 UTC [INFO] Preparing to sleep for 5
2020-02-25 18:18:08 UTC [INFO] check pipeline after 5s
2020-02-25 18:18:08 UTC [COMMAND] killing all pipelines
2020-02-25 18:18:09 UTC [INFO] Test Result: PASS!

@stripes416
stripes416 merged commit 8596965 into thesofproject:master Feb 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants