Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix #1244, Add RTEMS timebase callback wrapper #1276

Closed
wants to merge 2 commits into from
Closed

Fix #1244, Add RTEMS timebase callback wrapper #1276

Changes from 1 commit
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
942a1f6
Fix #1244, Add RTEMS timebase callback wrapper
pepepr08 Aug 1, 2022
97441c8
Fix #1244, removed cast that hid the bug at compile time
pepepr08 Aug 4, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
24 changes: 21 additions & 3 deletions src/os/rtems/src/os-impl-timebase.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -286,6 +286,24 @@ void OS_UsecsToTicks(uint32 usecs, rtems_interval *ticks)
/* The user may specify whether to use priority inheritance on mutexes via osconfig.h */
#define OSAL_TIMEBASE_MUTEX_ATTRIBS RTEMS_PRIORITY | RTEMS_BINARY_SEMAPHORE | RTEMS_INHERIT_PRIORITY

/*----------------------------------------------------------------
*
* Function: OS_TimeBase_CallbackThreadEntry
*
* Purpose: Local helper routine, not part of OSAL API.
* Wrapper function used by OS_TimeBaseCreate_Impl to
* convert the rtems_task_argument on newly created
* timebase task into an osal_id_t used by the
* OS_TimeBase_CallbackThread.
*
*-----------------------------------------------------------------*/
static void OS_TimeBase_CallbackThreadEntry(rtems_task_argument arg)
{
osal_id_t id;
id = OS_ObjectIdFromInteger(arg);
OS_TimeBase_CallbackThread(id);
}

/*----------------------------------------------------------------
*
* Function: OS_TimeBaseCreate_Impl
Expand Down Expand Up @@ -390,9 +408,9 @@ int32 OS_TimeBaseCreate_Impl(const OS_object_token_t *token)
else
{
/* will place the task in 'ready for scheduling' state */
rtems_sc = rtems_task_start(local->handler_task, /*rtems task id*/
(rtems_task_entry)OS_TimeBase_CallbackThread, /* task entry point */
(rtems_task_argument)r_name); /* passed argument */
rtems_sc = rtems_task_start(local->handler_task, /*rtems task id*/
(rtems_task_entry)OS_TimeBase_CallbackThreadEntry, /* task entry point */
pepepr08 marked this conversation as resolved.
Show resolved Hide resolved
(rtems_task_argument)r_name); /* passed argument */

if (rtems_sc != RTEMS_SUCCESSFUL)
{
Expand Down