Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Feature Request] sdkconfig: "Print registers and reboot after delay" would be useful (IDFGH-7947) #9462

Closed
chipweinberger opened this issue Jul 29, 2022 · 6 comments
Assignees
Labels
Resolution: Done Issue is done internally Status: Done Issue is done internally Type: Feature Request Feature request for IDF

Comments

@chipweinberger
Copy link
Contributor

chipweinberger commented Jul 29, 2022

A CONFIG_ESP_SYSTEM_PANIC_PRINT_DELAY_REBOOT option would be usable for both development & production, and would be most ideal for Beta testing phase.

  1. "print & reboot" is annoying for development. A Boot Loop will log to serial too much, forever.
  2. "print registers and halt" is not good for users.

We need a middle approach, "print & delay & reboot, with a configurable delay. e.g. 6 seconds.

Thanks for your consideration.

@chipweinberger chipweinberger added the Type: Feature Request Feature request for IDF label Jul 29, 2022
@espressif-bot espressif-bot added the Status: Opened Issue is new label Jul 29, 2022
@github-actions github-actions bot changed the title [Feature Request] sdkconfig: "Print registers and reboot after delay" would be useful [Feature Request] sdkconfig: "Print registers and reboot after delay" would be useful (IDFGH-7947) Jul 29, 2022
@chipweinberger
Copy link
Contributor Author

Bump.

A CONFIG_ESP_SYSTEM_PANIC_PRINT_DELAY_REBOOT option would let me keep it the same between development and production, and it just generally more useful IMO. 20htz boot loops are annoying =)

@chipweinberger
Copy link
Contributor Author

@ESP-Marius, @igrr, @Alvin1Zhang

Just want to make sure 1 person sees this.

@igrr
Copy link
Member

igrr commented Aug 11, 2022

Hi @chipweinberger,
All issues posted on Github are synced to our internal issue tracker, categorized and land one of the teams' backlogs. We've seen this feature request but currently consider it relatively low priority. Most efforts right now are focused on v5.0 release and various bug fixes.

Have you checked this example demonstrating how to set up builds for multiple configurations (development/production)? https://github.com/espressif/esp-idf/blob/master/examples/build_system/cmake/multi_config/README.md
I hope it can satisfy your requirement for now.

@chipweinberger
Copy link
Contributor Author

chipweinberger commented Aug 11, 2022

Thanks for your input Ivan, as always.

That is a useful page, for now =)

My Use case: For Beta testing, reboot after delay is most ideal. A beta device may need to be debugged "live", but must still reboot automatically.

@chipweinberger
Copy link
Contributor Author

chipweinberger commented Oct 12, 2022

Pull Request: #9962

@igrr, hope you consider it! No worries either way, but I did test it without issue. Super nice to have!

@chipweinberger
Copy link
Contributor Author

merged into master.

@espressif-bot espressif-bot added Status: Done Issue is done internally Resolution: Done Issue is done internally and removed Status: Opened Issue is new labels May 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Resolution: Done Issue is done internally Status: Done Issue is done internally Type: Feature Request Feature request for IDF
Projects
None yet
Development

No branches or pull requests

4 participants