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

Search all 16 addresses for INA #52

Merged
merged 1 commit into from Mar 24, 2020
Merged

Conversation

nathancheek
Copy link
Contributor

Description

The INA220 supports 16 addresses: 0x40 through 0x4F. This bugfix allows the library to search for all 16 devices instead of just 15.

Fixes #51

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

  1. Confirm that INA220 at address 0x4F is not detected
  2. Make code change
  3. Confirm that INA220 at address 0x4F is detected

Test Configuration:

  • Arduino version: 1.8.12
  • Arduino Hardware: ATmega128
  • SDK: Arduino IDE

@nathancheek nathancheek changed the title Search for 16 devices Search all 16 addresses for INA Mar 24, 2020
@SV-Zanshin SV-Zanshin merged commit 6832f4c into Zanduino:master Mar 24, 2020
@SV-Zanshin SV-Zanshin added the bug Something isn't working label Mar 24, 2020
@SV-Zanshin
Copy link
Collaborator

Merged with master

SV-Zanshin pushed a commit that referenced this pull request Mar 24, 2020
Changed if-condition from "<0x50" to "<=0x4F" to make immediately clear what the I2C range is
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

begin() only looks for 15 devices
2 participants