dmammolo commented on issue #11594: URL: https://github.com/apache/nuttx/issues/11594#issuecomment-1908246967
@davids5 regarding the diff to the PX4 version (mutex instead of semaphore) are you talking about this part of the code: https://github.com/PX4/PX4-Autopilot/blob/da28d9a7f2c9d664631c3ee536fa6f1c116d65de/src/lib/drivers/device/nuttx/SPI.cpp#L130-L163 ? My code should be based on v1.13.3 and not touch this part of the logic. For me the `_locking_mode` is `LOCK_NONE`. To have a mutex I assume `LOCK_PREEMPTION` would be desired? If I understand the Dcache functionalities, invalidating triggers a sync of the the SRAM to Dcache memory. So IMO it does not make sense to do this if the SPI transfer/read and DMA completion is not done yet. @alandeassis this hardware change is not so easy to achieve with a pix32v6 and as mentioned this issue pops up only with specific code changes (which I can't share unfortunately). -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
