Skip to content

Dsm reference capture channelmap - #3421

Merged
lgirdwood merged 2 commits into
masterfrom
DSM_Reference_capture_channelmap
Sep 16, 2020
Merged

lgirdwood merged 2 commits into
masterfrom
DSM_Reference_capture_channelmap

Conversation

@RDharageswari

Copy link
Copy Markdown

This patch set creates macros for the channel map of Feedback and reference streams. Also, modifies the feedback channels and channel map of the Demux configuration to send voltage and current data to the smart amplifier component and only voltage data to the host as Echo Reference stream. Also modifies the number of reference feedback channels from the
Demux component to the host capture stream in sof-tgl-max98373-rt5682.

Dharageswari R and others added 2 commits September 15, 2020 19:25
This patch creates macros for the channel map of Feedback
and reference streams. Also, modifies the feedback channels and channel map
of the Demux configuration to send voltage and current data to the
smart amplifier component and only voltage data to the host as Feedback
stream.

Signed-off-by: Dharageswari R <dharageswari.r@intel.com>
This patch modifies the number of reference feedback channels from the
Demux component to the host capture stream in sof-tgl-max98373-rt5682.

Signed-off-by: Dharageswari R <dharageswari.r@intel.com>

@keyonjie keyonjie 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.

Looks good to me, thanks for the fix @RDharageswari

)

ifelse(SMART_REF_CHANNELS, `8',
`define(`REF_CHMAP',`0x01,0x02,0x04,0x08,0x00,0x00,0x00,0x00')',

@keyonjie keyonjie Sep 16, 2020 •

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.

it's fine to keep this for smart_amp_test at the moment, we might need to pick the 4 'V's for 4 speakers in the future.

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