-
Notifications
You must be signed in to change notification settings - Fork 13.7k
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
[WIP] Added TI ina226 I2C power monitor #11755
Conversation
What are the desired use cases?
|
@dagar Long term analog will go away. This will replace that function. 1 per brick on separate buses. |
Address changed to 0x41 to match latest HW. Re based on master |
I brought one, glad to test this driver later, my module have a R100(0.1ohm) regsister on it, looks like it's too big for drone, I need to change it. Happy to see there is a shunt parameter for this sensor. |
I used a wrong formula earlier. Right way to calculate the power of this resistor is |
@dagar were you going to merge this? |
@bys1123 What flight controller and what connector did you plug it into? |
@davids5 It's Pixhawk4, on I2C A port. |
The board I am testing is at 0x41, we should add a scan and a range. |
You can change this line for now |
OK, no problem. I'm try it later. |
Do you think scan this large range would cause other issue? Maybe a parameter for this address is better? Just a suggestion, let's find out. |
@dagar - prelim for discussion on how to factor in this part and finally refactor the batery_status system_power.
There is extra raw values as debug code in the msg for now.