riot-ci/RIOT
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

sys/event: Made trivial functions static inline

Browse Source 
When the expected ROM overhead of a function is bigger than the actual function,
it is better to provide the function as static inline function in the header.
This commit is contained in:
Marian Buschsieweke 2020-08-04 12:58:39 +02:00
parent 649486f5cd
commit 2c03dfca13
No known key found for this signature in database
GPG Key ID: 61F64C6599B1539F
2 changed files with 27 additions and 33 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

28
sys/event/event.c
View File

@ -33,25 +33,6 @@
#include "xtimer.h" #include "xtimer.h"
#endif #endif
void event_queue_init_detached(event_queue_t *queue)
{
assert(queue);
memset(queue, '\0', sizeof(*queue));
}
void event_queue_init(event_queue_t *queue)
{
assert(queue);
memset(queue, '\0', sizeof(*queue));
queue->waiter = (thread_t *)sched_active_thread;
}
void event_queue_claim(event_queue_t *queue)
{
assert(queue && (queue->waiter == NULL));
queue->waiter = (thread_t *)sched_active_thread;
}
void event_post(event_queue_t *queue, event_t *event) void event_post(event_queue_t *queue, event_t *event)
{ {
assert(queue && event); assert(queue && event);
@ -152,12 +133,3 @@ event_t *event_wait_timeout64(event_queue_t *queue, uint64_t timeout)
return _wait_timeout(queue, &timer); return _wait_timeout(queue, &timer);
} }
#endif #endif
void event_loop_multi(event_queue_t *queues, size_t n_queues)
{
event_t *event;
while ((event = event_wait_multi(queues, n_queues))) {
event->handler(event);
}
}

32
sys/include/event.h
View File

@ -97,10 +97,12 @@
#define EVENT_H #define EVENT_H
#include <stdint.h> #include <stdint.h>
#include <string.h>
#include "assert.h"
#include "clist.h"
#include "irq.h" #include "irq.h"
#include "thread_flags.h" #include "thread_flags.h"
#include "clist.h"
#ifdef __cplusplus #ifdef __cplusplus
extern "C" { extern "C" {
@ -156,14 +158,23 @@ typedef struct {
* *
* @param[out] queue event queue object to initialize * @param[out] queue event queue object to initialize
*/ */
void event_queue_init(event_queue_t *queue); static inline void event_queue_init(event_queue_t *queue)
{
assert(queue);
memset(queue, '\0', sizeof(*queue));
queue->waiter = (thread_t *)sched_active_thread;
}
/** /**
* @brief Initialize an event queue not binding it to a thread * @brief Initialize an event queue not binding it to a thread
* *
* @param[out] queue event queue object to initialize * @param[out] queue event queue object to initialize
*/ */
void event_queue_init_detached(event_queue_t *queue); static inline void event_queue_init_detached(event_queue_t *queue)
{
assert(queue);
memset(queue, '\0', sizeof(*queue));
}
/** /**
* @brief Bind an event queue to the calling thread * @brief Bind an event queue to the calling thread
@ -175,7 +186,11 @@ void event_queue_init_detached(event_queue_t *queue);
* *
* @param[out] queue event queue object to bind to a thread * @param[out] queue event queue object to bind to a thread
*/ */
void event_queue_claim(event_queue_t *queue); static inline void event_queue_claim(event_queue_t *queue)
{
assert(queue && (queue->waiter == NULL));
queue->waiter = (thread_t *)sched_active_thread;
}
/** /**
* @brief Queue an event * @brief Queue an event
@ -308,7 +323,14 @@ event_t *event_wait_timeout64(event_queue_t *queue, uint64_t timeout);
* @param[in] queues Event queues to process * @param[in] queues Event queues to process
* @param[in] n_queues Number of queues passed with @p queues * @param[in] n_queues Number of queues passed with @p queues
*/ */
void event_loop_multi(event_queue_t *queues, size_t n_queues); static inline void event_loop_multi(event_queue_t *queues, size_t n_queues)
{
event_t *event;
while ((event = event_wait_multi(queues, n_queues))) {
event->handler(event);
}
}
/** /**
* @brief Simple event loop * @brief Simple event loop