mpu_prototypes.h
In v10, should there be MPU_x versions of:
BaseTypet xQueuePeek( QueueHandlet xQueue, void * const pvBuffer, TickTypet xTicksToWait );
BaseTypet xQueuePeekFromISR( QueueHandlet pxQueue, void * const pvBuffer );
BaseTypet xQueueSemaphoreTake( QueueHandlet xQueue, TickTypet xTicksToWait );
?
mpu_prototypes.h
BaseTypet xQueuePeek( QueueHandlet xQueue, void * const pvBuffer, TickType_t xTicksToWait );That is in there already.
BaseTypet xQueuePeekFromISR( QueueHandlet pxQueue, void * const pvBuffer );I don’t think ISR safe functions are necessary as they are privileged already and don’t need two switch to privileged mode.
BaseTypet xQueueSemaphoreTake( QueueHandlet xQueue, TickType_t xTicksToWait );That is also already in there.
mpu_prototypes.h
Hi Richard,
Thanks for the quick reply, and nice to meet you.
Yes, I agree that the ISR routine shouldn’t be necessary. Let me double check my download, but what I’m saying is that in mpu_prototypes.h that the function should be defined as:
MPUxQueuePeek() and MPUxQueueSemaphoreTake(),
mpu_prototypes.h
Double checked. I’m not sure why it would be necessary, but there is a function in mpuwrappers.c called
BaseTypet MPU_xQueuePeekFromISR(), for whatever reason, it raises the privilege, although I would have thought an ISR is already at the raised privilege.
I also double checked the MPUx prototypes in mpuprototypes.h, and the ones cited do seem to be missing the MPU_ tag.
Paul
mpu_prototypes.h
Ah – I see what you mean now. I was looking in mcuwrappers.h not mcuprototypes.h. Will catch that too.
mpu_prototypes.h
Thanks Richard, all good!