IPC4: TGL: Add multicore support#3711
Closed
fredoh9 wants to merge 5 commits intothesofproject:topic/sof-devfrom
Closed
IPC4: TGL: Add multicore support#3711fredoh9 wants to merge 5 commits intothesofproject:topic/sof-devfrom
fredoh9 wants to merge 5 commits intothesofproject:topic/sof-devfrom
Conversation
Dsp converts pcm rate to the one defined by dai. When SRC is used, the pcm runtime rate is different with dai rate and we need to fix it up for BE components. Signed-off-by: Rander Wang <rander.wang@intel.com>
Src module only needs two parameters : base module config and sink rate. This PR add prepare and setup for src widgets. Signed-off-by: Rander Wang <rander.wang@intel.com>
Currently the domain bit in ipc msg for module initialization is set to lp mode for pipeline. This is not correct since it is for module domain type: ll domain or dp domain which are for scheduler in fw. The domain bit depends on dp domain setting. Signed-off-by: Rander Wang <rander.wang@intel.com>
Add support for parsing core token. Signed-off-by: Ranjani Sridharan <ranjani.sridharan@linux.intel.com>
Add core_get/core_put to support core power up/down for IPC4. Signed-off-by: Fred Oh <fred.oh@linux.intel.com>
plbossart
requested changes
Jun 22, 2022
| msg.data_ptr = &dx_info; | ||
| msg.data_size = sizeof(dx_info); | ||
|
|
||
| /* now send the iPC */ |
| msg.data_ptr = &dx_info; | ||
| msg.data_size = sizeof(dx_info); | ||
|
|
||
| /* now send the iPC */ |
sound/soc/sof/ipc4-topology.h
Outdated
| struct sof_ipc4_available_audio_format available_fmt; | ||
| struct sof_ipc4_msg msg; | ||
| }; | ||
|
|
| */ | ||
| struct sof_ipc4_dx_info { | ||
| u32 core_mask; | ||
| u32 dx_mask; |
Member
There was a problem hiding this comment.
this is very confusing, please describe more.
is core_mask the active cores, the managed cores? Why do we need a new definition?
Likewise Dx means 1 for D0 and 0 for D3? What about D0ix for core0?
Collaborator
Author
|
Will merge this to #3679 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Currently this is based on Rander's SRC PR, #3692. Last two are additional changes for IPC4 TGL multicore support.
The topology I tested is also based on Rander's PR, thesofproject/sof#5913. And I set core_id 1 on SRC module. Topology graph is like this.