Thanks to visit codestin.com
Credit goes to github.com

Skip to content

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

Merged
merged 3 commits into from
Apr 25, 2022
Merged

Switch to adafruit_bus_device dependency #21

merged 3 commits into from
Apr 25, 2022

Conversation

tekktrik
Copy link
Member

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.

Copy link
Contributor

@FoamyGuy FoamyGuy left a 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.

@tannewt tannewt requested a review from a team March 14, 2022 17:33
Copy link

@jepler jepler left a 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!

@tekktrik tekktrik marked this pull request as ready for review March 18, 2022 15:53
@tekktrik
Copy link
Member Author

Should I wait for a test or go ahead and merge with the approvals?

@tekktrik tekktrik added the enhancement New feature or request label Mar 24, 2022
# Conflicts:
#	adafruit_sharpmemorydisplay.py
Copy link
Contributor

@FoamyGuy FoamyGuy left a 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.

@FoamyGuy FoamyGuy merged commit e1f8e38 into adafruit:main Apr 25, 2022
adafruit-adabot added a commit to adafruit/Adafruit_CircuitPython_Bundle that referenced this pull request Apr 26, 2022
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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Switch to BusDevice dependency
3 participants