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

ucm2: soundwire: rt722: add ConflictingDevice in each device #441

Closed

Conversation

shumingfan
Copy link
Contributor

This patch allows only one input and one output to be opened simultaneously.

This patch allows only one input and one output to be opened simultaneously.

Signed-off-by: Shuming Fan <[email protected]>
@perexg
Copy link
Member

perexg commented Sep 12, 2024

What problem are you trying to resolve? If hardware is capable to use all inputs/outputs simultatenously, we should not block this in UCM.

@plbossart
Copy link
Contributor

SDCA functions are supposed to operate independently, I am not sure why there would be a conflict @shumingfan @bardliao? There may be bandwidth restrictions but that's for the Manager to solve with the multi-lane or higher clock rates capabilities.

@bardliao
Copy link

That is about the bandwidth issue. Currently Intel platforms can support 3 devices work concurrently. We are working on the multi-lane and higher clock rates support. This PR can buy us some time.

@plbossart
Copy link
Contributor

the last thing you want is to play the clock with UCM changes, it's nearly impossible to synchronize UCM changes with kernel additions, so the best course of action is to do the right thing at the kernel level.

@shumingfan
Copy link
Contributor Author

@plbossart @bardliao OK, we focus on the kernel level first. closing

@shumingfan shumingfan closed this Sep 20, 2024
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