Mercurial > public > ostc4
changeset 228:f9ba924d188e div-fixes-4-1
Bugfix: set battery percentage correctly after RTE update
When the RTE is rebooted, the battery percentage on the display is zeroed.
However, when selecting SYS2-reboot-maintenance, the previously known
battery percentage can be manually restored. Interestingly, when
the restore reported a percentage A, choosing that resulted in a
percentage B to be displayed again. With B much smaller than A.
So, rebooting the RTE multiple times resulted in an seemingly
empty battery, while it definitely is not.
The reason for this is a relatively simple bug in the RTE code. This
commit fixes the conversion between the internal LTC2941 registers
and the percentage value to be displayed. Obviously, from and to
the internal registers need to be symmetrical.
Signed-off-by: Jan Mulder <jlmulder@xs4all.nl>
author | Jan Mulder <jlmulder@xs4all.nl> |
---|---|
date | Tue, 02 Apr 2019 14:53:15 +0200 |
parents | d5891007fc4c |
children | 2c0b502b0a72 |
files | Small_CPU/Src/batteryGasGauge.c |
diffstat | 1 files changed, 15 insertions(+), 8 deletions(-) [+] |
line wrap: on
line diff
--- a/Small_CPU/Src/batteryGasGauge.c Tue Apr 02 18:18:22 2019 +0200 +++ b/Small_CPU/Src/batteryGasGauge.c Tue Apr 02 14:53:15 2019 +0200 @@ -27,11 +27,11 @@ #include "stm32f4xx_hal.h" #include "i2c.h" -float battery_f_voltage = 0; -float battery_f_charge_percent = 0; +static float battery_f_voltage = 0; +static float battery_f_charge_percent = 0; #define BGG_BATTERY_OFFSET (26123) //; 65536-(3,35Ah/0,085mAh) -#define BGG_BATTERY_DEVIDER (394) //; 3,35Ah/0,085mAh/100 [%] +#define BGG_BATTERY_DIVIDER (394) //; 3,35Ah/0,085mAh/100 [%] float get_voltage(void) { @@ -61,8 +61,14 @@ uint8_t buffer[2]; buffer[0] = 0x01; - buffer[1] = 0xF8;// true: F8 = 11111000, wrong/old comment: 11101000 - I2C_Master_Transmit( DEVICE_BATTERYGAUGE, buffer, 2); + + // F8 = 11111000: + // Vbat 3.0V (11) + // Prescale M = 128 (111) + // AL/CC pin disable (0) + // Shutdown (0) + buffer[1] = 0xF8; + I2C_Master_Transmit(DEVICE_BATTERYGAUGE, buffer, 2); } @@ -86,7 +92,7 @@ battery_f_charge_percent_local = (float)(bufferReceive[2] * 256); battery_f_charge_percent_local += (float)(bufferReceive[3]); battery_f_charge_percent_local -= BGG_BATTERY_OFFSET; - battery_f_charge_percent_local /= BGG_BATTERY_DEVIDER; + battery_f_charge_percent_local /= BGG_BATTERY_DIVIDER; if(battery_f_charge_percent_local < 0) battery_f_charge_percent_local = 0; @@ -120,8 +126,9 @@ if(percentage >= 100) mAhSend = 0xFFFF; - else - mAhSend = (uint16_t)(percentage * 655.35f); + else { + mAhSend = (percentage * BGG_BATTERY_DIVIDER) + BGG_BATTERY_OFFSET; + } uint8_t bufferSend[3]; bufferSend[0] = 0x02;