drivers : caam : Move descriptors to noncacheable section

nxp-mcuxpresso / mcux-sdk

MCUXpresso SDK

BSD 3-Clause "New" or "Revised" License

301 stars 136 forks source link

drivers : caam : Move descriptors to noncacheable section #159

Closed dleach02 closed 5 months ago

dleach02 commented 5 months ago

Move used job descriptors in the CAAM driver from the stack to noncacheable section.

Prerequisites

[] I have checked latest main branch and the issue still exists.

Describe the pull request Fixes an issue the HAL_NXP patches with coherency of descriptors. (https://github.com/zephyrproject-rtos/hal_nxp/pull/192)

Type of change

[ ] Bug fix (non-breaking change which fixes an issue)

Tests

Part of the Zephyr enablement

VitSolcikNXP commented 5 months ago

Hello David Leach, such modification can have severe impact on driver usage with non-blocking APIs and without RTOS/Mutex in place.

For me it makes more sense to have whole stack in non-cached memory like TCM. Is that feasible in Zephyr project?

If not, maybe we need to find a different way how to handle allocation of descriptor. Maybe create additional APIs that would accept user buffer to be used for descriptors.

dleach02 commented 5 months ago

@VitSolcikNXP, understood. I'm going to close this PR