using newer bin semaph create call.
architecture dependant function call is external now and shall be implemented by developer
This commit is contained in:
parent
44d4678089
commit
1415cd2339
@ -8,16 +8,13 @@
|
||||
|
||||
#include <framework/serviceinterface/ServiceInterfaceStream.h>
|
||||
|
||||
extern "C" {
|
||||
#include "portmacro.h"
|
||||
#include "task.h"
|
||||
}
|
||||
|
||||
BinarySemaphore::BinarySemaphore() {
|
||||
xSemaphoreCreateBinary(handle);
|
||||
handle = xSemaphoreCreateBinary();
|
||||
if(handle == nullptr) {
|
||||
|
||||
error << "Binary semaphore creation failure" << std::endl;
|
||||
}
|
||||
xSemaphoreGive(handle);
|
||||
}
|
||||
|
||||
BinarySemaphore::~BinarySemaphore() {
|
||||
@ -27,36 +24,40 @@ BinarySemaphore::~BinarySemaphore() {
|
||||
// This copy ctor is important as it prevents the assignment to a ressource
|
||||
// (other.handle) variable which is later deleted!
|
||||
BinarySemaphore::BinarySemaphore(const BinarySemaphore& other) {
|
||||
xSemaphoreCreateBinary(handle);
|
||||
handle = xSemaphoreCreateBinary();
|
||||
if(handle == nullptr) {
|
||||
error << "Binary semaphore creation failure" << std::endl;
|
||||
}
|
||||
xSemaphoreGive(handle);
|
||||
}
|
||||
|
||||
BinarySemaphore& BinarySemaphore::operator =(const BinarySemaphore& s) {
|
||||
if(this != &s) {
|
||||
xSemaphoreCreateBinary(handle);
|
||||
handle = xSemaphoreCreateBinary();
|
||||
if(handle == nullptr) {
|
||||
error << "Binary semaphore creation failure" << std::endl;
|
||||
}
|
||||
xSemaphoreGive(handle);
|
||||
}
|
||||
return *this;
|
||||
}
|
||||
|
||||
BinarySemaphore::BinarySemaphore(BinarySemaphore&& s) {
|
||||
xSemaphoreCreateBinary(handle);
|
||||
handle = xSemaphoreCreateBinary();
|
||||
if(handle == nullptr) {
|
||||
error << "Binary semaphore creation failure" << std::endl;
|
||||
}
|
||||
xSemaphoreGive(handle);
|
||||
}
|
||||
|
||||
BinarySemaphore& BinarySemaphore::operator =(
|
||||
BinarySemaphore&& s) {
|
||||
if(&s != this) {
|
||||
xSemaphoreCreateBinary(handle);
|
||||
handle = xSemaphoreCreateBinary();
|
||||
if(handle == nullptr) {
|
||||
error << "Binary semaphore creation failure" << std::endl;
|
||||
}
|
||||
xSemaphoreGive(handle);
|
||||
}
|
||||
return *this;
|
||||
}
|
||||
@ -127,7 +128,8 @@ ReturnValue_t BinarySemaphore::giveBinarySemaphore(SemaphoreHandle_t semaphore)
|
||||
void BinarySemaphore::resetSemaphore() {
|
||||
if(handle != nullptr) {
|
||||
vSemaphoreDelete(handle);
|
||||
xSemaphoreCreateBinary(handle);
|
||||
handle = xSemaphoreCreateBinary();
|
||||
xSemaphoreGive(handle);
|
||||
}
|
||||
}
|
||||
|
||||
|
@ -18,6 +18,10 @@ extern "C" {
|
||||
* @brief OS Tool to achieve synchronization of between tasks or between task and ISR
|
||||
* @details
|
||||
* Documentation: https://www.freertos.org/Embedded-RTOS-Binary-Semaphores.html
|
||||
*
|
||||
* SHOULDDO: check freeRTOS version and use new task notifications,
|
||||
* if non-ancient freeRTOS version is used.
|
||||
*
|
||||
* @ingroup osal
|
||||
*/
|
||||
class BinarySemaphore: public HasReturnvaluesIF {
|
||||
@ -68,8 +72,8 @@ public:
|
||||
|
||||
/**
|
||||
* Take the binary semaphore.
|
||||
* If the semaphore has already been taken, the task will be blocked for a maximum
|
||||
* of #timeoutMs or until the semaphore is given back,
|
||||
* If the semaphore has already been taken, the task will be blocked
|
||||
* for a maximum of #timeoutMs or until the semaphore is given back,
|
||||
* for example by an ISR or another task.
|
||||
* @param timeoutMs
|
||||
* @return -@c RETURN_OK on success
|
||||
|
@ -5,16 +5,12 @@
|
||||
*
|
||||
*/
|
||||
#include <framework/osal/FreeRTOS/TaskManagement.h>
|
||||
#include <FreeRTOS.h>
|
||||
#include "portmacro.h"
|
||||
#include "task.h"
|
||||
|
||||
/**
|
||||
* TODO: This stuff is hardware and architecture and mission dependant...
|
||||
* Some FreeRTOS implementations might be able to determine their own task context for example.
|
||||
* If not ISRs are used, or task preemption is enabled, some of this stuff might
|
||||
* not be necessary anyway. Maybe there is a better solution?
|
||||
*/
|
||||
extern "C" {
|
||||
#include "FreeRTOS.h"
|
||||
#include "task.h"
|
||||
}
|
||||
|
||||
void TaskManagement::requestContextSwitchFromTask() {
|
||||
vTaskDelay(0);
|
||||
}
|
||||
@ -22,7 +18,7 @@ void TaskManagement::requestContextSwitchFromTask() {
|
||||
void TaskManagement::requestContextSwitch(CallContext callContext = CallContext::task) {
|
||||
if(callContext == CallContext::isr) {
|
||||
// This function depends on the partmacro.h definition for the specific device
|
||||
portYIELD_FROM_ISR();
|
||||
requestContextSwitchFromISR();
|
||||
} else {
|
||||
requestContextSwitchFromTask();
|
||||
}
|
||||
|
@ -7,6 +7,12 @@
|
||||
#ifndef FRAMEWORK_OSAL_FREERTOS_TASKMANAGEMENT_H_
|
||||
#define FRAMEWORK_OSAL_FREERTOS_TASKMANAGEMENT_H_
|
||||
|
||||
/**
|
||||
* Architecture dependant portmacro.h function call.
|
||||
* Should be implemented in bsp.
|
||||
*/
|
||||
extern "C" void requestContextSwitchFromISR();
|
||||
|
||||
/*!
|
||||
* Used by functions to tell if they are being called from
|
||||
* within an ISR or from a regular task. This is required because FreeRTOS
|
||||
@ -18,6 +24,7 @@ enum CallContext {
|
||||
isr = 0xFF //!< isr_context
|
||||
};
|
||||
|
||||
|
||||
class TaskManagement {
|
||||
public:
|
||||
/**
|
||||
|
Loading…
Reference in New Issue
Block a user