Skip to content
This repository has been archived by the owner on Jun 19, 2024. It is now read-only.

feat: ADI Addressible LEDs #91

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from
Draft

Conversation

Tropix126
Copy link
Member

@Tropix126 Tropix126 commented Feb 14, 2024

Describe the changes this PR makes. Why should it be merged?

Support for controlling WS2812B smart LEDs over ADI.

Additional Context

Blocked by (several) bugs in PROS at the moment unfortunately.

  • ext_adi_led_set_pixel deadlocks the entire program due to an off-by-one-error in their port mutex handling for this function. We currently get around this by manually manipulating the buffer ourselves.
  • All LED setters appear to be broken to some extent. They don't update the actual strip the first time they're called, and there needs to be a delay between update calls for them to properly update. This is possibly an SDK bug (:c). There are potential ways we could get around this, but it would likely involve either making all LED setters blocking or running a background processing daemon just for LEDs. Neither of these solutions are great.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant