Skip to content
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 topology for testing SOF plugin #8815

Merged
merged 1 commit into from
Feb 5, 2024

Conversation

ranj063
Copy link
Collaborator

@ranj063 ranj063 commented Jan 29, 2024

The topology uses passthrough pipelines for playback/capture and supports only 48k, 2ch, 16bit format for now.

The topology uses passthrough pipelines for playback/capture and
supports only 48k, 2ch, 16bit format for now.

Signed-off-by: Ranjani Sridharan <[email protected]>
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.

I have zero background on the plugin....

@@ -66,9 +66,9 @@ The above example needs to be 48k as example pipe has no SRC/ASRC.
Likewise

```
arecord -Dsof:tgl-nocodec:1:default:default:48k2c16b -f dat file.wav
arecord -Dsof:plugin:1:default:default:48k2c16b -f dat file.wav
Copy link
Member

Choose a reason for hiding this comment

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

it:would:not:hurt:to:document:all:these:options:never:seen:this:before

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@plbossart it is documented in the plugin usage help message and in the same README file below


Object.Widget.dai-copier."1" {
dai_index 1
dai_type "SSP"
Copy link
Member

Choose a reason for hiding this comment

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

does the concept of SSP make any sense for a plugin at the ALSA-lib level? Why is this needed?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

it is not really needed for the plugin but these are mandatory attributes for a dai-copier and I cant create one without it. I will have to update the host/dai copier to remove these but for now, we have no choice

Copy link
Member

@lgirdwood lgirdwood left a comment

Choose a reason for hiding this comment

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

Plugin is still at the "staging" level, so we need to continue and capture our TODOs e.g. needing SSP for copier is a new TODO

Copy link
Collaborator

@singalsu singalsu left a comment

Choose a reason for hiding this comment

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

Thanks, really useful to have a working example to start other topologies plugin development.

@kv2019i kv2019i merged commit a5bdcce into thesofproject:main Feb 5, 2024
39 of 42 checks passed
@ranj063 ranj063 deleted the plugin-topology branch February 5, 2024 17:10
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.

5 participants