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

topology: nhlt: intel: support more device types and directions #226

Closed
wants to merge 1 commit into from

Conversation

brentlu
Copy link
Contributor

@brentlu brentlu commented Jul 26, 2023

In current NHLT table the device type of all SSP endpoints are set to BT Sideband(0) instead of SSP Analog Codec(4) and the direction only supports Render(0) and Capture(1).

Here we introduce two new quirks from topology to set the device type correctly and support two more directions: Render with loopback(2) and Feedback for render(3) for speakers with echo reference or IV sense feedback.

@brentlu
Copy link
Contributor Author

brentlu commented Jul 26, 2023

PR:
linux: thesofproject/linux#4492
alsa-utils: #226
topology: thesofproject/sof#7984

In current NHLT table the device type of all SSP endpoints are set to
BT Sideband(0) instead of SSP Analog Codec(4) and the direction only
supports Render(0) and Capture(1).

Here we introduce two new quirks from topology to set the device type
correctly and support two more directions: Render with loopback(2)
and Feedback for render(3) for speakers with echo reference or IV
sense feedback.

Signed-off-by: Brent Lu <brent.lu@intel.com>
@perexg perexg closed this in 3a47ef2 Aug 1, 2023
@ranj063
Copy link
Contributor

ranj063 commented Aug 4, 2023

@perexg @brentlu this change breaks the current IPC4 implementation in the SOF driver as there's no support for dealing with the newly added device types and directions yet. Can we please revert this change temporarily until we have finalized the kernel side changes? @plbossart FYI

@brentlu
Copy link
Contributor Author

brentlu commented Aug 8, 2023

Hi @perexg this PR is uploaded for review with Linux and topology side change and can't be merged alone. Sorry I should make the PR descriptor more clear in advance. Please revert it and sorry for the trouble.

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.

2 participants