Skip to content

docs/esp32/quickref.rst: Reinitialize watchdog timer with longer timeout. #7770

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

Closed
wants to merge 4 commits into from

Conversation

IhorNehrutsa
Copy link
Contributor

@IhorNehrutsa IhorNehrutsa commented Sep 10, 2021

You need enough time to upload files to the device after exiting to REPL(WebREPL).

Discussion in
ESP32/machine_wdt.c: Add WDT.deinit() method. #6631

The best explanation at #6631 (comment)

@dpgeorge dpgeorge added the docs label Sep 13, 2021
@IhorNehrutsa IhorNehrutsa changed the title docs/machine.WDT.rst: Reinitialise the watchdog with a longer timeout docs/esp32/quickref.rst: Reinitialize watchdog timer with longer timeout. Dec 15, 2021
you don't have enough time to copy the updates to the device.
ESP32 alows reinitialise the watchdog with a longer timeout – like an hour.

More comprehansive example usage::
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A long example like this doesn't belong in the quickref. The paragraph above is enough (to mention that the timeout can be reinitialised).

@github-actions
Copy link

github-actions bot commented Mar 2, 2023

Code size report:


@IhorNehrutsa
Copy link
Contributor Author

Crash when git rebase.
Continue at:
docs/esp32: Reinitialize watchdog timer with longer timeout. #10901

@IhorNehrutsa IhorNehrutsa deleted the hour-WDT branch March 2, 2023 00:46
tannewt pushed a commit to tannewt/circuitpython that referenced this pull request Mar 24, 2023
This reverts commit 7e6e824.

Fixes micropython#7770

The change in micropython#7623 needs to be revered; the raise-site added in micropython#7632
is the correct one and the one in socketpool needs to be reverted.

This is not affecting 8.0.x because micropython#7623 was not back-ported to there
before we realized it was not a full fix.

Both micropython#7770 and micropython#7606 should be re-tested. I didn't test.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants