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 driver #85

Closed
wants to merge 2 commits into from
Closed

[WIP] WS2812 driver #85

wants to merge 2 commits into from

Conversation

johnnyman727
Copy link
Contributor

No description provided.

@@ -104,6 +104,9 @@ int main(void) {
port_init(&port_b, 2, &PORT_B, GCLK_PORT_B,
TCC_PORT_B, DMA_PORT_B_TX, DMA_PORT_B_RX);

// TEMPORARY
ws2812_init(PORT_B.power);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe I'm misunderstanding, but this means that this driver is now "on by default"? I don't think this is a good idea.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@rwaldron I was just using this to check that my makefile changes worked and I could call the functions I wrote :) It won't be on by default in production.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Got it!

@johnnyman727
Copy link
Contributor Author

@nodebotanist in case you go looking, I attempted to get the Neopixel driver started here. I pretty much got to the point where I had an API laid out and the codebase compiled but I didn't actually write any of the driver. :) I probably won't get to this for a few months if you want to work on it. Some notes I've written on the topic.

@scott1702
Copy link

@johnnyman727 @nodebotanist Any plans for this to be continued? Just got my T2 and hoping to get it working with neopixels :)

@johnnyman727
Copy link
Contributor Author

@scott1702 my hands are pretty full right now but I'd be happy to help advise if you want to take up the mantle.

@scott1702
Copy link

Haha yeah I can imagine, especially with everyone getting their tessels now. I would take up the challenge but unfortunately I know nothing about C 😞

@johnnyman727
Copy link
Contributor Author

@scott1702 it's more about reading datasheets than writing C but I understand if it's outside of your comfort zone.

@rwaldron
Copy link
Contributor

I don't think this should be included with the t2-firmware/node/tessel.js/tessel-export.js code, but instead as a separate module file

@johnnyman727
Copy link
Contributor Author

Closing this issue because I don't plan to work on it further.

@Frijol Frijol deleted the jon-ws2812 branch June 5, 2016 09:14
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.

3 participants