Skip to content

Topology: Use pipe-volume-switch-capture in nocodec topologies - #3783

Merged
lgirdwood merged 1 commit into
thesofproject:masterfrom
singalsu:add_volume_switch_to_nocodec_capture_newtry
Feb 18, 2021
Merged

lgirdwood merged 1 commit into
thesofproject:masterfrom
singalsu:add_volume_switch_to_nocodec_capture_newtry

Conversation

@singalsu

Copy link
Copy Markdown
Collaborator

This patch replaces use of pipe-volume-capture macro or
pipe-low-latency-capture to pipe-volume-switch-capture in nocodec
topologies for APL, BDW, CHT, CNL, ICL, and TGL based platforms. It
allows to test of volume component mute switch control in SSP loopback
that is used by default in nocodec topologies. The testing of mute
switch feature is simplest to do with loopback topologies. Nocodec
topologies are not in mainstream usage so this is the safest option to
enable testing.

The mute switch add should be possible to all capture topologies but
it can be done later once it is confirmed it is safe to do (avoid
accidental muted audio or problems with UCM).

Signed-off-by: Seppo Ingalsuo seppo.ingalsuo@linux.intel.com

This patch replaces use of pipe-volume-capture macro or
pipe-low-latency-capture to pipe-volume-switch-capture in nocodec
topologies for APL, BDW, CHT, CNL, ICL, and TGL based platforms. It
allows to test of volume component mute switch control in SSP loopback
that is used by default in nocodec topologies. The testing of mute
switch feature is simplest to do with loopback topologies. Nocodec
topologies are not in mainstream usage so this is the safest option to
enable testing.

The mute switch add should be possible to all capture topologies but
it can be done later once it is confirmed it is safe to do (avoid
accidental muted audio or problems with UCM).

Signed-off-by: Seppo Ingalsuo <seppo.ingalsuo@linux.intel.com>

@lgirdwood lgirdwood left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@plbossart dont we use nocodec topologies on developer boards like UP^2 etc ?

@singalsu

Copy link
Copy Markdown
Collaborator Author

@lgirdwood Yes, I'm using nocodec topologies in UP2 and Minnowboard when I have no audio peripherals hooked. Also it's practical to set nocodec mode via kconfig force nocodec option even if audio HW would be detected. I'm using such configuration for sof-test runs those rely on SSP LBM loopback.

@singalsu
singalsu marked this pull request as ready for review February 15, 2021 17:26
@singalsu

singalsu commented Feb 15, 2021 •

Copy link
Copy Markdown
Collaborator Author

CI results look now OK with this topology change. The patch is the same but apparently the tests are now more robust.

@lgirdwood

Copy link
Copy Markdown
Member

@plbossart any comments ?

@plbossart plbossart left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

The nocodec topologies used to be plain vanilla ones with no processing, it seems they are moving towards a more full-fledged topology. That's fine I guess.

@lgirdwood

Copy link
Copy Markdown
Member

BYT alsa bat signal overflow - unrelated since PR does not change BYT.

@lgirdwood
lgirdwood merged commit b0faccc into thesofproject:master Feb 18, 2021
@singalsu
singalsu deleted the add_volume_switch_to_nocodec_capture_newtry branch June 15, 2021 08:21
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