Remove complex timer based toggling and switch to RTOS task with semaphore based delay #18

2026-06-15T14:00:25+02:00

eeeck commented

2026-06-15 14:00:25 +02:00

No description provided.

eeeck added this to the v0.1.0 milestone 2026-06-15 14:00:25 +02:00

eeeck added the enhancement label 2026-06-15 14:00:25 +02:00

eeeck self-assigned this 2026-06-15 14:00:25 +02:00

eeeck added this to the Thesis SCRUM project 2026-06-15 14:00:25 +02:00

eeeck force-pushed dev-rtos-semaphore from dca7707d28 to 657d6914ac

2026-06-15 14:02:03 +02:00

Compare

eeeck changed title from ~~WIP: Remove complex timer based toggling to task+semaphore based delay~~ to WIP: Remove complex timer based toggling and switch to RTOS task with semaphore based delay

2026-06-15 14:02:51 +02:00

eeeck force-pushed dev-rtos-semaphore from 657d6914ac to b3c4f6b131

2026-06-15 14:06:31 +02:00

Compare

eeeck added 1 commit 2026-06-15 14:07:00 +02:00

Transition from timer based to task based LED signal ab8b68313b

- Remove RTOS timers
- Create LED task
- Avoid polling, instead wait for semaphore

eeeck force-pushed dev-rtos-semaphore from b3c4f6b131 to ab8b68313b

2026-06-15 14:07:00 +02:00

Compare

eeeck added 2 commits 2026-06-15 14:11:17 +02:00

Release CANrx semaphore in ISR 8c8fe93d5b

Move before starting scheduler to avoid race condition ce85e473b3

- If a CAN message arrives during boot time ISR will fire and try to put
  data into xCAN1RxQueue. At that time queue is NULL: calling RTOS APIs
  on NULL handles will cause a HardFault

eeeck added 1 commit 2026-06-15 14:12:57 +02:00

Move to RTOS_THREADS c1d2e65e20

- ops...

eeeck added 3 commits 2026-06-15 14:30:43 +02:00

Fix double semaphore release b1744ce941

Indentation fixes 975acb8c3e

Better way to avoid race condition, only notification lives in CAN task 2c14d095f7

- Runs only once
- Only critical notification is inside task
- CAN message during boot will not be lost, instead put inside FIFO0

eeeck marked the pull request as ready for review 2026-06-15 14:33:12 +02:00

eeeck merged commit b737c27b49 into main

2026-06-15 14:33:39 +02:00

eeeck deleted branch dev-rtos-semaphore

2026-06-15 14:33:39 +02:00

eeeck referenced this pull request

2026-06-15 15:31:18 +02:00

LED will not be triggered again after timer set #15

eeeck moved this to Done in Thesis SCRUM on 2026-06-15 15:33:57 +02:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: eeeck/CANdigger#18