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

LMV324LIPT not behaving like true rail-to-rail op-amp #24

Open
poornack opened this issue May 17, 2018 · 2 comments
Open

LMV324LIPT not behaving like true rail-to-rail op-amp #24

poornack opened this issue May 17, 2018 · 2 comments
Labels
Hardware All issue related to modifying or designing hardware wontfix

Comments

@poornack
Copy link
Contributor

The LMV324LIPT op-amp's output reaches a maximum of 2.7V and a minimum of 45mV when supplied 3.3V and Ground. This op-amp returns values from the two battery current sensors, external pressure sensor and the water sensor. This shouldn't cause an issue when operating under normal condition so I am marking this as WONTFIX for now. We can revisit swapping this op-amp for another in the future.

@poornack poornack added wontfix Hardware All issue related to modifying or designing hardware labels May 17, 2018
@lymacasm
Copy link
Collaborator

Should be fine. If we're worried, we can also change the value of the resistor at the output of the current monitor IC, as that resistor is used to adjust the gain. With the current resistor (time current, not electron current), the only time we'll have a 3.3V output is when we have ~120A output from the batteries, which is both the maximum battery output current, and a scenario that will likely never occur, especially since the board is fused. But, if we are worried about measuring the current in the upper threshold, the easiest thing to do would be to swap the resistor to one that will produce 2.7V at 120A.

@lymacasm
Copy link
Collaborator

lymacasm commented May 17, 2018

At 2.7V in the ADC, the battery current should be at approximately 98A (120A * (2.7/3.3)) which is too large anyways, so we can assume there's a problem if we see that. With the chosen resistor, the currents will actually be a little higher than that, as 3.3V actually represents a current a little larger than 120A, but it's not significantly different.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Hardware All issue related to modifying or designing hardware wontfix
Projects
None yet
Development

No branches or pull requests

2 participants