Skip to content

use existing spi baudrate instead of hardcoded #23

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

Merged
merged 3 commits into from
Dec 17, 2020

Conversation

FoamyGuy
Copy link
Contributor

resolves #22. I don't have hardware to test it, but wouldn't anticipate problems since it's mostly the same as: #14 from the TLC59711 library.

@evaherrada
Copy link
Collaborator

@FoamyGuy Do you think you could update this to match what @tannewt said in #22? I think that would probably need some testing though

@FoamyGuy
Copy link
Contributor Author

@dherrada thanks for the reminder. I definitely missed that message on the issue.

Latest commit removes the configure. I don't have hardware to test with and it looks like they are showing out of stock at the moment.

I can go back and attempt to make a similar change on the TLC59711 and test it with that device though.

@evaherrada
Copy link
Collaborator

@FoamyGuy Were you ever able to test this?

@FoamyGuy
Copy link
Contributor Author

@dherrada I did test it on a modified branch with the TLC59711 and didn't see any trouble. Though the modified code for it seems to have gotten lost in the shuffle between my old and new laptops.

@evaherrada
Copy link
Collaborator

Awesome. Thanks!

@evaherrada evaherrada merged commit 00be326 into adafruit:master Dec 17, 2020
adafruit-adabot added a commit to adafruit/Adafruit_CircuitPython_Bundle that referenced this pull request Dec 19, 2020
Updating https://github.com/adafruit/Adafruit_CircuitPython_DHT to 3.5.3 from 3.5.2:
  > Merge pull request adafruit/Adafruit_CircuitPython_DHT#55 from yeyeto2788/fix_issue_54

Updating https://github.com/adafruit/Adafruit_CircuitPython_TLC5947 to 1.3.3 from 1.3.2:
  > Merge pull request adafruit/Adafruit_CircuitPython_TLC5947#23 from FoamyGuy/fix_hardcoded_baudrate

Updating https://github.com/adafruit/Adafruit_CircuitPython_BitbangIO to 1.2.0 from 1.1.2:
  > Merge pull request adafruit/Adafruit_CircuitPython_BitbangIO#13 from makermelissa/master
  > Merge pull request adafruit/Adafruit_CircuitPython_BitbangIO#12 from alustig3/master

Updating https://github.com/adafruit/Adafruit_CircuitPython_Slideshow to 1.5.1 from 1.5.0:
  > Merge pull request adafruit/Adafruit_CircuitPython_Slideshow#29 from FoamyGuy/fallback_without_json
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.

SPI baudrate hardcoded, should use existing frequency from SPI object
2 participants