Write total_operating_time and cycles_numbers to EEPROM every 5 min #7

Breath4Life / controller

5 stars 1 forks source link

Write total_operating_time and cycles_numbers to EEPROM every 5 min #7 #23

Closed jonadem closed 4 years ago

jonadem commented 4 years ago

To test it, it is easier to change WRITE_EEPROM_PERIOD_MS from "3000 1000L" to "5 1000L".

incrementCyclesNumber() need to be called from another task (and the current call need to be removed [there is a FIXME]).

anpar commented 4 years ago

Looks good! Will be tested today.

anpar commented 4 years ago

Seems to work for reading/writing EEPROM but breaks the LCD at some point for a yet unknown reason.

20200409_112023

cassiersg commented 4 years ago

Seems to work for reading/writing EEPROM but breaks the LCD at some point for a yet unknown reason.

Probably due to disabling interrupts for too long. (Maybe try without the taskEnterCritical / taskExitCritical ?)

anpar commented 4 years ago

Seems to work for reading/writing EEPROM but breaks the LCD at some point for a yet unknown reason.

Probably due to disabling interrupts for too long. (Maybe try without the taskEnterCritical / taskExitCritical ?)

I will. But for now I'm struggling to reproduce the bug while I haven't changed anything. Weird.