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

HW Bug: R3 on REB Rev C causes Low Battery Detection to never work #35

Closed
ZAdamMac opened this issue Sep 20, 2023 · 1 comment
Closed
Assignees
Labels
bug Something isn't working
Milestone

Comments

@ZAdamMac
Copy link
Owner

ZAdamMac commented Sep 20, 2023

The position of Resistor R3 is wrong, and worse, the resistor itself is superfluous.

Because it stands between the Low Battery (LBO) pin on the TPS module and the matched GPIO pin on the microcontroller, the line is constantly pulled high to the level of the main 3.3v rail; that is, the low battery signal is never sent.

By removing this resistor the issue is resolved in f/power-management-enhance's recent commits; using the internal pullup resistor on the MSP430 more than suffices.

Users of revision C development kits should cut resistor R3 off the board or otherwise remove it. TODO: remove from revision D before committing the schematics.

@ZAdamMac ZAdamMac added this to the 0.3.0 milestone Sep 20, 2023
@ZAdamMac
Copy link
Owner Author

Fix for this will be included on the Revision D schematics.

@ZAdamMac ZAdamMac self-assigned this Sep 20, 2023
@ZAdamMac ZAdamMac added the bug Something isn't working label Sep 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Development

No branches or pull requests

1 participant