Skip to content

## Pull Request: Disable INA226 Current Sensing by Default (Hardware Compatibility) #4

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
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

## Pull Request: Disable INA226 Current Sensing by Default (Hardware Compatibility) #4

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
e81d940
Stop tracking .vscode files
tazomatalax Apr 15, 2025
4889cc5
implment AS5600 sensor via i2c
tazomatalax May 15, 2025
a03a12f
docs: clarify INA226 current sensing is disabled by default
tazomatalax May 20, 2025
00e9771
update nodered flow file
tazomatalax May 27, 2025
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
367 changes: 0 additions & 367 deletions .vscode/c_cpp_properties.json
View file
Open in desktop

This file was deleted.

10 changes: 0 additions & 10 deletions .vscode/extensions.json
View file
Open in desktop

This file was deleted.

47 changes: 0 additions & 47 deletions .vscode/launch.json
View file
Open in desktop

This file was deleted.

5 changes: 0 additions & 5 deletions .vscode/settings.json
View file
Open in desktop

This file was deleted.

18 changes: 16 additions & 2 deletions README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,13 +12,14 @@ A Raspberry Pi Pico-based sensor interface project that provides real-time monit
- Dual-core utilization for efficient task handling
- Motor power control functionality
- Thread-safe sensor data handling using FreeRTOS mutexes
- **Current sensing code is present but disabled by default for hardware compatibility**

## Hardware Requirements

- Raspberry Pi Pico
- 4 - 20mA Sensors (100R shunt resistor)
- Active low RPM sensor (1 pulse per revolution)
- Current monitoring over I2C
- Current monitoring over I2C (INA226, optional/disabled by default)
- USB Serial Modbus RTU communication

![Basic circuit diagram](/schematic/circuit.png)
Expand Down Expand Up @@ -49,6 +50,19 @@ A Raspberry Pi Pico-based sensor interface project that provides real-time monit
- `/include` - Header files
- `/test` - Test files

## Important Note on Current Sensing (INA226)

The codebase includes support for current sensing using the INA226 over I2C. However, due to a hardware issue, all INA226-related code (initialization, runtime, and configuration) is commented out by default in `main.cpp` and `sys_init.h`.

- **To enable current sensing:**
- Uncomment the INA226-related lines in `main.cpp` and `sys_init.h`.
- Ensure your hardware is working and the INA226 is connected properly.
- **If left commented:**
- The rest of the system (ADC, RPM, Modbus, etc.) will function normally.
- No current measurements will be reported in the Modbus registers.

This approach allows easy re-enabling of current sensing in the future by simply uncommenting the relevant code blocks.

## Configuration

The project can be configured through the `platformio.ini` file. Current settings:
Expand Down Expand Up @@ -79,7 +93,7 @@ The following table describes the Modbus holding registers used in this device:
| 40003 (0x0002) | CH1_MA | FLOAT32 | Channel 1 current measurement in mA |
| 40005 (0x0004) | CH2_MA | FLOAT32 | Channel 2 current measurement in mA |
| 40007 (0x0006) | PULSE_RATE | FLOAT32 | Measured pulse rate in Hz |
| 40009 (0x0008) | MOTOR_MA | FLOAT32 | Motor current measurement in mA |
| 40009 (0x0008) | MOTOR_MA | FLOAT32 | Motor current measurement in mA (only if INA226 enabled) |

Notes:
- All FLOAT32 values use two consecutive registers (32 bits)
Expand Down
Loading