-
Notifications
You must be signed in to change notification settings - Fork 322
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
[DNM][DNR] TEST #8902
Closed
LaurentiuM1234
wants to merge
11
commits into
thesofproject:main
from
LaurentiuM1234:test_perp_patches
Closed
[DNM][DNR] TEST #8902
LaurentiuM1234
wants to merge
11
commits into
thesofproject:main
from
LaurentiuM1234:test_perp_patches
Conversation
This file contains 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
Since the DMAC is configured in a somewhat different way in the native case vs the non-native case, add a native version of the ipc_get_page_descriptors() function and remove the Zephyr-specific functions from the non-native version of said function. Signed-off-by: Laurentiu Mihalcea <[email protected]>
With every copy() operation, the source and destination addresses keep getting modified. As such, for each re-configuration performed during host_copy_one_shot(), the HOST DMAC needs to be made aware of these changes. This commit changes host_copy_one_shot() such that on each dma_config() call the DMAC driver receives the updated addresses and size. Signed-off-by: Laurentiu Mihalcea <[email protected]>
Source and destination addresses cannot be NULL. As such, set them to the values found in the first element of hd->config's elem_array. This is fine to do because the host component uses only 1 block. Signed-off-by: Laurentiu Mihalcea <[email protected]>
Since the Zephyr SAI driver is now enabled, add an entry in the zephyr_dev array, which will resolve to multiple "struct device *", one for each SAI node specified in the DTS. Signed-off-by: Laurentiu Mihalcea <[email protected]>
Since the EDMA and HOST DMA nodes have been introduced to the i.MX93 overlay, add entries in the dma array which will create a struct dma for each of these DMACs. Signed-off-by: Laurentiu Mihalcea <[email protected]>
Since now there's a Zephyr driver for NXP's SAI, the dai_set_config() should be modified to also allow the configuration of the SAI. As such, this commit introduces a new case for NXP's SAI that does exactly that. Signed-off-by: Laurentiu Mihalcea <[email protected]>
Some DMACs (e.g: NXP's EDMA) can automatically adjust the source and destination addresses upon transfer completion. As such, we need to indicate how the adjustment should be performed. In the case of playback, the source address should be decremented, while in the case of capture, the destination address should be decremented. Signed-off-by: Laurentiu Mihalcea <[email protected]>
Currently, the DAI component's state is not updated on dai_trigger() operation, which leads to pipeline_comp_copy() skipping the dai_copy() operation (since the DAI component never transitions to the ACTIVE state). To fix this, add a state transition in dai_comp_trigger_internal(). Also, make sure not to trigger the DAI component if already in the requested state. Signed-off-by: Laurentiu Mihalcea <[email protected]>
In the case of some DAIs, the DMA slot may be encoded differently in the DAI configuration handshake. As such, we can't count on the fact that the handshake itself can be used as the DMA slot. To fix this, add a function which parses the handshake and allows each DAI to use its own encoding of the DMA slot inside the handshake. Signed-off-by: Laurentiu Mihalcea <[email protected]>
The native SAI and ESAI drivers use a different handshake encoding. As such, when using native Zephyr drivers use a different function for decoding the channel from the handshake. Signed-off-by: Laurentiu Mihalcea <[email protected]>
LaurentiuM1234
force-pushed
the
test_perp_patches
branch
from
March 5, 2024 12:03
a9dd166
to
ccad0ba
Compare
Signed-off-by: Laurentiu Mihalcea <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
No description provided.