Fix recover from overvoltage/undervoltage state in bms_bq769x0.c

LibreSolar / bms-firmware

Firmware for LibreSolar BMS boards based on bq769x0, bq769x2 or ISL94202

https://libre.solar/bms-firmware/

Apache License 2.0

152 stars 69 forks source link

Fix recover from overvoltage/undervoltage state in bms_bq769x0.c #18

Closed xsider closed 3 years ago

xsider commented 3 years ago

BMS with BQ76930 did not recover from state "overvoltage" or "undervoltage". BMS needs a reset to enable CHG/DSG MOSFETs again.

MOSFET "switch on" depends on ov_reset and uv_reset levels (improved bouncing behaviour)
if first clear of error flag in SYS_STATS fails, it is retried again until flag clears (not dependent on ALERT-line interrupt)

martinjaeger commented 3 years ago

There is a small merge conflict now because of the one line that was already fixed in the develop branch. Could you please rebase to develop to fix the conflict and afterwards force-push so that I can merge? Thanks!

martinjaeger commented 3 years ago

Ah, and can you please add a summary of your above comment to the commit message aswell so that the change will be more clear for people not looking at the PR but only the commit message?