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

Add optional parameter for using internal pull-up for int1 #79

Merged
merged 1 commit into from
Aug 13, 2023

Conversation

perja12
Copy link
Contributor

@perja12 perja12 commented Aug 13, 2023

This is useful in cases where there is no support for internal pull-up resistor on the board (for example Binho Nova host adapter). On such devices, one have to use an external pull-up resistor.

@perja12 perja12 force-pushed the perja/internal_pull_up branch 2 times, most recently from 111bc2c to 2f41bbf Compare August 13, 2023 09:19
@ladyada
Copy link
Member

ladyada commented Aug 13, 2023

please replace with a try/except so that if htere is no internal pullup support, it just gets skipped

This is useful in cases where there is no support for internal pull-up resistor on the board (for example Binho Nova host adapter). On such devices, one have to use an external pull-up resistor.
@perja12
Copy link
Contributor Author

perja12 commented Aug 13, 2023

Thanks @ladyada. I have updated the code. Please take a look again.

@ladyada
Copy link
Member

ladyada commented Aug 13, 2023

great work!

@ladyada ladyada merged commit 0ff4058 into adafruit:main Aug 13, 2023
1 check passed
adafruit-adabot added a commit to adafruit/Adafruit_CircuitPython_Bundle that referenced this pull request Aug 29, 2023
Updating https://github.com/adafruit/Adafruit_CircuitPython_LIS3DH to 5.2.0 from 5.1.23:
  > Merge pull request adafruit/Adafruit_CircuitPython_LIS3DH#79 from perja12/perja/internal_pull_up

Updating https://github.com/adafruit/Adafruit_CircuitPython_TCA8418 to 1.0.8 from 1.0.7:
  > Merge pull request adafruit/Adafruit_CircuitPython_TCA8418#12 from adam-huganir/docs-fix-tutorial-link-update

Updating https://github.com/adafruit/Adafruit_CircuitPython_MatrixPortal to 3.1.11 from 3.1.0:
  > Merge pull request adafruit/Adafruit_CircuitPython_MatrixPortal#92 from FoamyGuy/docs_fix
  > Merge pull request adafruit/Adafruit_CircuitPython_MatrixPortal#91 from rhooper/main

Updating https://github.com/adafruit/Adafruit_CircuitPython_Bundle/circuitpython_library_list.md to NA from NA:
  > Updated download stats for the libraries
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.

2 participants