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.

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

RGB LED blink #111

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

frank26080115
Copy link

added RGB LED, including a blink pattern engine

it uses the 10 kHz routine, but the blink pattern is executed at 100ms time frames

I have defined some patterns. The three primary patterns are

  • unarmed - just red
  • armed - green slow blink
  • running - green fast blink

primary patterns are run in a continuous loop

then there are event indicators, called secondary patterns, they execute only once and then goes back to the previous primary pattern

the events are defined for current limit, stall, desync, and low batt, but I only actually put current limit and low batt into this particular pull request. I don't have enough knowledge about the code or a test setup to test stall and desync effectively

I've added the Aria 70A target as the test target, giving a template on how to define the RGB pins for a target. I've also attempted to make sure that other targets retain their previous behavior. My code does work together with USE_LED_STRIP

Please review

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

Successfully merging this pull request may close these issues.

2 participants