diff options
| author | Selene ToyKeeper | 2024-03-29 08:21:07 -0600 |
|---|---|---|
| committer | Selene ToyKeeper | 2024-03-29 08:21:07 -0600 |
| commit | 3cafb86c00a95a70e58144230f641c2c94208e7b (patch) | |
| tree | f0da65c64f98b253c2a611f7c1d672ff60765568 /hw/hank/emisar-d3aa/hwdef.c | |
| parent | weak battery detection: use different thresholds for AA and Li-Ion (diff) | |
| download | anduril-3cafb86c00a95a70e58144230f641c2c94208e7b.tar.gz anduril-3cafb86c00a95a70e58144230f641c2c94208e7b.tar.bz2 anduril-3cafb86c00a95a70e58144230f641c2c94208e7b.zip | |
d3aa weak battery test: blink 3x instead of 2x, and omit number readout
Diffstat (limited to '')
| -rw-r--r-- | hw/hank/emisar-d3aa/hwdef.c | 16 |
1 files changed, 12 insertions, 4 deletions
diff --git a/hw/hank/emisar-d3aa/hwdef.c b/hw/hank/emisar-d3aa/hwdef.c index 8aaa893..36c6fed 100644 --- a/hw/hank/emisar-d3aa/hwdef.c +++ b/hw/hank/emisar-d3aa/hwdef.c @@ -123,9 +123,10 @@ uint8_t quick_volt_measurement() { void detect_weak_battery() { // guess at the cell strength with a load test... - // - measure voltage while LEDs off + // - measure voltage with LEDs off // - measure again with LEDs on // - determine how much to limit power + // (ramp up until battery becomes unstable) // - blink to indicate weak battery mode, if active ramp_level_hard_limit = 0; @@ -162,17 +163,23 @@ void detect_weak_battery() { } set_level(0); - // TODO? limit NiMH to ~half power, even if it passed the sag test? - // (and if so, blink 2X for NiMH, 3X for alkaline) + // Blink again if not in full-power mode: + // - 1 blink total: Strong Li-ion cell, full power enabled + // - 2 blinks: Strong AA cell, max AA power enabled + // (not used on this driver, strong AA uses mode 1) + // - 3 blinks: Weak battery, power severely limited uint8_t extra_blinks = 0; - if (ramp_level_hard_limit) extra_blinks ++; + if (ramp_level_hard_limit) extra_blinks += 2; for (uint8_t i=0; i<extra_blinks; i++) { delay_4ms(300/4); blink_once(); } + #ifdef USE_WEAK_BATTERY_PROTECTION_READOUT + // this numeric display isn't really needed by default, + // but the code remains in case anyone wants to use it if (ramp_level_hard_limit) { delay_4ms(255); // not booted far enough for this to work yet @@ -190,6 +197,7 @@ void detect_weak_battery() { blink_once(); } } + #endif // ifdef USE_WEAK_BATTERY_PROTECTION_READOUT } #endif // ifdef USE_WEAK_BATTERY_PROTECTION |