Skip to content

Conversation

@jsarha
Copy link
Contributor

@jsarha jsarha commented May 17, 2023

Rename mixin and mixout Analog Playback volumes. Rewrite the names of the mixers to better reflect their position in the topology.

As a result of this commit mixers are renamed in sof-hda-generic.tplg.

'gain.1.1 1 2nd Playback Volume' becomes
'gain.1.1 Pre Mixer Analog Playback Volume'

and

'gain.2.1 2 Main Playback Volume' becomes
'gain.2.1 Post Mixer Analog Playback Volume'

Rename mixin and mixout Analog Playback volumes. Rewrite the names of
the mixers to better reflect their position in the topology.

As a result of this commit mixers are renamed in sof-hda-generic.tplg.

'gain.1.1 1 2nd Playback Volume' becomes
'gain.1.1 Pre Mixer Analog Playback Volume'

and

'gain.2.1 2 Main Playback Volume' becomes
'gain.2.1 Post Mixer Analog Playback Volume'

Signed-off-by: Jyri Sarha <jyri.sarha@linux.intel.com>
@jsarha jsarha requested a review from ranj063 as a code owner May 17, 2023 21:53
@ranj063 ranj063 requested a review from plbossart May 19, 2023 15:15
Copy link
Member

@plbossart plbossart left a comment

Choose a reason for hiding this comment

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

Sounds ok @jsarha @ranj063

The volumes are not used by UCM so we can select whatever we want.

@ranj063
Copy link
Collaborator

ranj063 commented May 19, 2023

@jsarha can we apply the same pre-mixer/post-mixer convention to all the mixer names in the other topologies as well? Thanks!

@ranj063 ranj063 merged commit af9bedb into thesofproject:main May 19, 2023
@jsarha
Copy link
Contributor Author

jsarha commented May 22, 2023

@jsarha can we apply the same pre-mixer/post-mixer convention to all the mixer names in the other topologies as well? Thanks!

Working on it. Just wanted to send this first to know that I am on the right track. On some topologies, nocodec for instance, its not as easy to refer to the related PCM name.

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