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.

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

topology2: add nocodec support for real case validation #8748

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

RanderWang
Copy link
Collaborator

Sof-mtl-max98357a-rt5682 is used by chrome project. This patch uses nocodec backend for validation so that our CI test can check real production case.

@RanderWang
Copy link
Collaborator Author

RanderWang commented Jan 17, 2024

sof-mtl-max98357a-rt5682-nocodec

@mengdonglin
Copy link
Collaborator

Thank you @RanderWang ! This help us simulate Chrome use scenario in our testing. Maybe we can name it "sof-mtl-mock-chrome-nocodec"? But I feel current name is also good.

@plbossart
Copy link
Member

Not following @RanderWang, why not use a 'nocodec' topology we already have? Why do we need multiple flavors of nocodec?

Also what does this bring in terms of validation?

You haven't quite described the problem statement and alternatives....

@mengdonglin
Copy link
Collaborator

@plbossart The background is that we want to evaluate the impact of multicore timeout issue #8642 for Chrome by running a similar topology as Chrome on MTL RVP. That issue is reproduced on MTL RVP with nocodec topology (3 cores, dummy smart amp, 11 concurrent streams). This PR made a topology for 2 cores, no smart amp, AEC on core 2, and less streams.

@lgirdwood
Copy link
Member

Thank you @RanderWang ! This help us simulate Chrome use scenario in our testing. Maybe we can name it "sof-mtl-mock-chrome-nocodec"? But I feel current name is also good.

Ack - this will differentiate and allow stress and developer debug/test.

Sof-mtl-max98357a-rt5682 is used by chrome project. This patch uses
nocodec backend for validation so that our CI test can check real
production case. Currently we use cavs-nocodec.tplg for validation,
but it is far from real usage and makes no sense.

Signed-off-by: Rander Wang <rander.wang@intel.com>
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