Skip to content

Document alt INA3221 addr, avoid INA219 conflict #1441

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.

Sign up for GitHub

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

Open
tahnok wants to merge 1 commit into
base: dev
Choose a base branch
from
Open

Document alt INA3221 addr, avoid INA219 conflict #1441

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
11 changes: 10 additions & 1 deletion src/helpers/sensors/EnvironmentSensorManager.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 +61,9 @@ static SensirionI2cSht4x SHT4X;
#endif

#if ENV_INCLUDE_INA3221
#define TELEM_INA3221_ADDRESS 0x42 // INA3221 3 channel current sensor I2C address
#ifndef TELEM_INA3221_ADDRESS
#define TELEM_INA3221_ADDRESS 0x42 // INA3221 3 channel current sensor I2C address (0x40-0x43 based on A0 pin)
#endif
#define TELEM_INA3221_SHUNT_VALUE 0.100 // most variants will have a 0.1 ohm shunts
#define TELEM_INA3221_NUM_CHANNELS 3
#include <Adafruit_INA3221.h>
Expand Down Expand Up @@ -268,6 +270,13 @@ bool EnvironmentSensorManager::begin() {
#endif

#if ENV_INCLUDE_INA219
// Skip INA219 if INA3221 is at the same address (they conflict)
#if ENV_INCLUDE_INA3221
if (INA3221_initialized && TELEM_INA219_ADDRESS == TELEM_INA3221_ADDRESS) {
INA219_initialized = false;
MESH_DEBUG_PRINTLN("INA219 skipped - INA3221 at same address %02X", TELEM_INA219_ADDRESS);
} else
#endif
if (INA219.begin(TELEM_WIRE)) {
MESH_DEBUG_PRINTLN("Found INA219 at address: %02X", TELEM_INA219_ADDRESS);
INA219_initialized = true;
Expand Down