search

PaulStoffregen / WS2812Serial

Non-Blocking WS2812 LED Library
139 stars 29 forks source link

T4 Support #8

Closed KurtE closed 4 years ago

KurtE commented 4 years ago

I have not tested all pins yet, but I have done simple test of T4 through prop shield to 8 neopixel bar and it worked on all of those valid Serial TX pins...

Updated examples with comments likewise with readme.

Warning: Requires fix in core library as the file teensy4\imxrt.h has the DMAMUX_SOURCE values wrong (TX and RX pins were reversed)

PaulStoffregen commented 4 years ago

Thanks!

KurtE commented 4 years ago

Yep probably wrong pin in comment…

From: Remo Jongeneelen notifications@github.com Sent: Friday, December 20, 2019 3:12 PM To: PaulStoffregen/WS2812Serial WS2812Serial@noreply.github.com Cc: KurtE kurte@rockisland.com; Author author@noreply.github.com Subject: Re: [PaulStoffregen/WS2812Serial] T4 Support (#8)

@rjongeneelen commented on this pull request.

Wrong pin?

In examples/BasicTest/BasicTest.ino https://github.com/PaulStoffregen/WS2812Serial/pull/8#discussion_r360604221 :

@@ -14,6 +14,7 @@ const int pin = 1; // Teensy 3.2: 1, 5, 8, 10, 31 (overclock to 120 MHz for pin 8) // Teensy 3.5: 1, 5, 8, 10, 26, 32, 33, 48 // Teensy 3.6: 1, 5, 8, 10, 26, 32, 33 +// Teensy 4.0: 1, 8, 14, 16, 20, 24, 29, 39

Shouldn't 16 be 17 (TX4)?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/PaulStoffregen/WS2812Serial/pull/8?email_source=notifications&email_token=AAL4MQFADHQMVN6ESBRUT2TQZVGKVA5CNFSM4JQEL6V2YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCQAEH3I#pullrequestreview-335561709 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AAL4MQDOSO3H2UGYVTA6QH3QZVGKVANCNFSM4JQEL6VQ .