Semaphore Initialization - Linux and FreeRTOS #89

Merged
gaisser merged 16 commits from KSat/fsfw:mueller_binSemaphoreInit into master 2020-08-28 13:25:07 +02:00
Owner

Includes all new semaphore features.

Implemented for Linux and FreeRTOS.

fixes #53

Includes all new semaphore features. Implemented for Linux and FreeRTOS. fixes #53
muellerr added the
feature
label 2020-05-29 13:50:31 +02:00
muellerr changed title from Semaphore Initialization to Semaphore Initialization - Linux and FreeRTOS 2020-05-29 14:25:31 +02:00
Author
Owner

I tested this code with an internal unit tester (for linux and freeRTOS, and this test can also be used for rtems when the implementation is finished). I will add the code to the fsfw test repository.

I tested this code with an internal unit tester (for linux and freeRTOS, and this test can also be used for rtems when the implementation is finished). I will add the code to the fsfw test repository.
gaisser self-assigned this 2020-08-25 12:35:41 +02:00
Author
Owner

UPDATE: interface will be adapted to be similar to new MutexIF interface.
Also, include replacements still missing

UPDATE: interface will be adapted to be similar to new MutexIF interface. Also, include replacements still missing
muellerr changed title from Semaphore Initialization - Linux and FreeRTOS to WIP: Semaphore Initialization - Linux and FreeRTOS 2020-08-25 18:34:59 +02:00
muellerr changed title from WIP: Semaphore Initialization - Linux and FreeRTOS to Semaphore Initialization - Linux and FreeRTOS 2020-08-27 17:15:30 +02:00
gaisser reviewed 2020-08-28 11:46:23 +02:00
@ -0,0 +31,4 @@
virtual~ SemaphoreIF() {};
static const uint8_t INTERFACE_ID = CLASS_ID::SEMAPHORE_IF;
Owner

I don't see the addition of the class id in FwClassIds. Could you add this change as well?

I don't see the addition of the class id in FwClassIds. Could you add this change as well?
Author
Owner

It's actually in there, I did a compile check

It's actually in there, I did a compile check
Owner

Interesting, I don't find it in the commit view.

Interesting, I don't find it in the commit view.
Owner

Oh it's part of the master.

Oh it's part of the master.
gaisser marked this conversation as resolved
gaisser merged commit ce4d010b5c into master 2020-08-28 13:25:06 +02:00
gaisser deleted branch mueller_binSemaphoreInit 2020-08-28 13:25:11 +02:00
Sign in to join this conversation.
No description provided.