-
Notifications
You must be signed in to change notification settings - Fork 11
Switch to adafruit_bus_device dependency #21
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code change makes sense and looks good to me.
I don't have the Sharp display to test it though. Will leave open for now, hopefully we can get someone with the display to try it out.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems fine, also didn't test. Thank you!
Should I wait for a test or go ahead and merge with the approvals? |
# Conflicts: # adafruit_sharpmemorydisplay.py
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I merged main to resolve conflicts and tested this successfully on Feather RP2040 with https://www.adafruit.com/product/3502 using the simpletest example in this repo. Everything is working as intended as far as I can tell.
Updating https://github.com/adafruit/Adafruit_CircuitPython_SD to 3.3.9 from 3.3.8: > Merge pull request adafruit/Adafruit_CircuitPython_SD#48 from tekktrik/doc/add-typing > change discord badge > Patch: Replaced discord badge image > Updated gitignore > Update Black to latest. Updating https://github.com/adafruit/Adafruit_CircuitPython_SharpMemoryDisplay to 1.4.0 from 1.2.9: > Merge pull request adafruit/Adafruit_CircuitPython_SharpMemoryDisplay#21 from tekktrik/dev/switch-bus-device > Merge pull request adafruit/Adafruit_CircuitPython_SharpMemoryDisplay#19 from ilikecake/image-function-numpy > change discord badge > Patch: Replaced discord badge image > Updated gitignore > Update Black to latest. > Fixed readthedocs build > Consolidate Documentation sections of README Updating https://github.com/adafruit/Adafruit_CircuitPython_PortalBase to 1.13.0 from 1.12.0: > Merge pull request adafruit/Adafruit_CircuitPython_PortalBase#70 from FoamyGuy/dont_require_secrets_if_no_network > change discord badge > Patch: Replaced discord badge image > Updated gitignore
Since
adafruit_bus_device.spi_device.SPIDevice
can now handle inverted CS pins, this PR switches it over to using it! Also should add everything needed in the infrastructure and documentation to support this change.Resolves #20
Still needs to be tested.