-
Notifications
You must be signed in to change notification settings - Fork 349
topology2: add second capture stream for DMIC0 for MTL topology #7085
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 second capture stream for DMIC0 for MTL topology #7085
Conversation
|
@jairaj-arava is there array of four DMIC for rt5682 ? or two ? If only two, we don't need to support 4 channel stream |
@RanderWang for Chromebooks we always assume there are four mics, and then CRAS removes the channels that don't exist using the mapping found in the platform-specific UCM extensions. At the fimware/topology level we always expose 4 channels to reuse the same thing and not worry about hardware changes. |
2b25f46 to
bd6c69f
Compare
bd6c69f to
d50a295
Compare
|
@jairaj-arava can you please have a look at the change in #7113? We are switching to using arrays and I'd like all new changes to follow that |
|
@jairaj-arava array-support merged in #7113 so rebase will be needed |
|
d50a295 to
d853fdc
Compare
|
d853fdc to
15e6e3d
Compare
This patch adds second capture stream by means of copier module Signed-off-by: jairaj-arava <jairaj.arava@intel.com>
|
Can one of the admins verify this patch? |
|
One DUT not run in https://sof-ci.01.org/sofpr/PR7085/build4015/devicetest/index.html test, but not relevant for this PR, so proceeding with merge. |
These changes adds a second stream for DMIC to the MTL Chromebook topology. The second stream can be used as a reference stream by the user. All the needed supporting changes were already merged via #6783.

Verified the functionality by simulataneously capturing data via both the streams and didn't see any issue in the recorded data as of now.