[GitHub] [incubator-nuttx] cwespressif commented on a change in pull request #2351: xtensa/esp32: Fix issues of force-sleep

[GitBox <gi...@apache.org>]

cwespressif commented on a change in pull request #2351:
URL: https://github.com/apache/incubator-nuttx/pull/2351#discussion_r528458218



##########
File path: arch/xtensa/src/esp32/esp32_pm.c
##########
@@ -577,11 +577,11 @@ static int IRAM_ATTR esp32_sleep_start(uint32_t pd_flags)
       esp32_timer_wakeup_prepare();
     }
 
-  esp32_rtc_sleep_start(s_config.wakeup_triggers, 0);
+  result = esp32_rtc_sleep_start(s_config.wakeup_triggers, 0);
 
   /* Restore CPU frequency */
 
-  result = esp32_configure_cpu_freq(cur_freq);
+  esp32_configure_cpu_freq(cur_freq);

Review comment:
       Hi @acassis thanks for your reviewing, I have added “warning” to indicate that there is a problem with the configured cpu frequency.




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org