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

Play with trinkey #669

Draft
wants to merge 47 commits into
base: master
Choose a base branch
from
Draft

Play with trinkey #669

wants to merge 47 commits into from

Conversation

paulz
Copy link

@paulz paulz commented Jan 29, 2023

Summary

[describe your changes here]

Checklist

  • CHANGELOG.md for the BSP or HAL updated
  • All new or modified code is well documented, especially public items
  • No new warnings or clippy suggestions have been introduced (see CI or check locally)

If Adding a new Board

  • Board CI added to crates.json
  • Board is properly following "Tier 2" conventions, unless otherwise decided to be "Tier 1"

If Adding a new cargo feature to the HAL

  • Feature is added to the test matrix for applicable boards / PACs in crates.json

paulz and others added 30 commits January 22, 2023 12:22
I am definitely struggling a bit with what's coming back from the
USB calls, and I don't know yet how really to initialize the neopixel

There's a bunch of commented out code that is in the direction of what
I mean to do.
WIP: trying to have the morse code generation at least emit .__. to
the serial port, but reinitializing the interface doesn't make
sense. Need to refactor so that the global abstraction for the hardware
we've configured is more globally accessible (or easier to pass around.)
we do handle an empty queue.

Next, make the queue into a circular buffer, and push stuff into
it.

It would sure be nice to have some tests.
drain the last state.)

Board now blinks S-N-S, followed after an interval with a 10 unit
long ON pulse.
imf and others added 17 commits February 5, 2023 18:48
independently.

Note that the queue doesn't (currently) worry about overflow. It
CAN and WILL overflow if it's filled faster than it drains.
That said, all that results is some states not being rendered to
the pins.
It still would benefit from a letter queue, but this works as long
as you don't type too fast.

Needs some cleanup though.
I left the stubs of the character queue in place, so I can finish
that out, but this is way shorter now.

I wish I had a way to generate the initial queue state, but iterators
are part of std, which we don't have access to.
Next we should use it.
Tried to make the USB loop provide queue length stats, but not having
std in embedded rust meant I wasn't sure how to convert my usize into
a string. (I made it a u32 at least.)
QA'd all letters and numbers but haven't verified punctuation.
@jbeaurivage jbeaurivage added the board support Related to support for a particular board label May 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
board support Related to support for a particular board
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants