From 1820273696f69eb0edc6cc0bc2b93f474fbfbfc6 Mon Sep 17 00:00:00 2001 From: Rahix <rahix@rahix.de> Date: Sun, 6 Oct 2019 18:07:49 +0200 Subject: [PATCH] feat(hw-locks): hwlock_release() should return void With the switch to the new mutex API, hwlock_release() cannot fail under any circumstances. To emphasize this, make it return void instead of int (Previously it just always returned 0). Signed-off-by: Rahix <rahix@rahix.de> --- epicardium/modules/hw-lock.c | 3 +-- epicardium/modules/modules.h | 2 +- 2 files changed, 2 insertions(+), 3 deletions(-) diff --git a/epicardium/modules/hw-lock.c b/epicardium/modules/hw-lock.c index a6f6e227..664403a4 100644 --- a/epicardium/modules/hw-lock.c +++ b/epicardium/modules/hw-lock.c @@ -55,9 +55,8 @@ int hwlock_acquire(enum hwlock_periph p, TickType_t wait) return 0; } -int hwlock_release(enum hwlock_periph p) +void hwlock_release(enum hwlock_periph p) { assert(p < _HWLOCK_MAX); mutex_unlock(&hwlock_mutex[p]); - return 0; } diff --git a/epicardium/modules/modules.h b/epicardium/modules/modules.h index 745ecec2..d83a7d24 100644 --- a/epicardium/modules/modules.h +++ b/epicardium/modules/modules.h @@ -98,7 +98,7 @@ enum hwlock_periph { }; int hwlock_acquire(enum hwlock_periph p, TickType_t wait); -int hwlock_release(enum hwlock_periph p); +void hwlock_release(enum hwlock_periph p); /* ---------- Display ------------------------------------------------------ */ /* Forces an unlock of the display. Only to be used in Epicardium */ -- GitLab