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

WIP: ws2812: add support for 125MHz for RP2040 #374

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from
Open

Conversation

bgould
Copy link
Member

@bgould bgould commented Feb 14, 2022

This adds support for 125Mhz CPU frequency to WS2812

FYI I accidentally pushed and then reverted this change to dev; was sure of the best way to clean that up, hope I did it correctly.

accidental push: b4eb406
revert: 0ced126

@aykevl
Copy link
Member

aykevl commented Feb 14, 2022

This looks good to me. Is this still a WIP pull request?

@bgould
Copy link
Member Author

bgould commented Feb 18, 2022

@aykevl I'm getting this asm error when trying to compile using RP2040 (using this branch in TinyGo: https://github.com/tinygo-org/tinygo/tree/macropad-rp2040)

bcg@bcg-x12:~/src/drivers$ tinygo version
tinygo version 0.23.0-dev linux/amd64 (using go version go1.17.3 and LLVM version 13.0.1)
bcg@bcg-x12:~/src/drivers$ tinygo build -o test.hex -target=macropad-rp2040 ./examples/ws2812
error: <inline asm>:278:4: out of range pc-relative fixup value
          bne.n 1b                       @ [1/3] send_bit
          ^

(haven't added a smoke test for this to drivers yet, was going to do that after macropad-rp2040 is merged in TinyGo)

@aykevl
Copy link
Member

aykevl commented Mar 1, 2022

Here is a fix: #391

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.

None yet

2 participants