bug: sleep code commented out, WDTE config word disabled — no deep sleep #40

New issue

Closed

opened 2026-08-08 03:19:07 +00:00 by time · 1 comment

time commented

2026-08-08 03:19:07 +00:00

Contributor

In main.c, the sleep sequence is commented out (//stop_fvr_tmp_adc(), //wdt_clear(), //go_to_sleep()) and replaced with __delay_ms(4000). Additionally, the WDTE config word is commented out (//#pragma config WDTE = ON), meaning the WDT is disabled. The device loops indefinitely without entering deep sleep, which defeats the battery-life design goal. Fix: uncomment the sleep code and WDTE, and ensure the WDT is properly configured to wake the device from sleep.

In main.c, the sleep sequence is commented out (`//stop_fvr_tmp_adc()`, `//wdt_clear()`, `//go_to_sleep()`) and replaced with `__delay_ms(4000)`. Additionally, the WDTE config word is commented out (`//#pragma config WDTE = ON`), meaning the WDT is disabled. The device loops indefinitely without entering deep sleep, which defeats the battery-life design goal. Fix: uncomment the sleep code and WDTE, and ensure the WDT is properly configured to wake the device from sleep.

time referenced this issue from a pull request that will close it,

2026-08-08 07:19:59 +00:00

fix: implement WDT-driven sleep mode with debug toggle #41

time commented

2026-08-08 07:20:05 +00:00

Author

Contributor

Fixed in PR #41: WDT (256s, WDTE=ON) is now the wakeup source for deep sleep. Added DEBUG_MODE define to toggle between sleep and static delay. Verified the WDT-sleep wake logic against the PIC12LF1840T39A datasheet — WDT timeout during sleep wakes without Reset.