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 INA219 driver + Current stubs #484

Merged
merged 2 commits into from
Sep 5, 2023
Merged

Add INA219 driver + Current stubs #484

merged 2 commits into from
Sep 5, 2023

Conversation

tyeth
Copy link
Contributor

@tyeth tyeth commented Sep 5, 2023

Adds the methods and polling fields for Current which was previously unavailable to new sensors.

Adds the INA219 driver which uses the new getEventCurrent polling method. Tested+Working on address 40 or 44.
image

@brentru
Copy link
Member

brentru commented Sep 5, 2023

@tyeth there's a merge conflict in library.properties btw

Copy link
Member

@brentru brentru left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM - you can merge when passing

@tyeth tyeth merged commit 144ea5d into adafruit:main Sep 5, 2023
@tyeth tyeth deleted the add-INA219 branch September 5, 2023 16:52
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