Skip to content

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

@RanderWang RanderWang requested a review from marc-hb January 17, 2024 05:43
@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>
@lgirdwood lgirdwood added this to the TBD milestone Apr 17, 2024
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