Skip to content

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

Closed
IhorNehrutsa wants to merge 4 commits into
micropython:masterfrom
IhorNehrutsa:hour-WDT
Closed

docs/esp32/quickref.rst: Reinitialize watchdog timer with longer timeout.#7770
IhorNehrutsa wants to merge 4 commits into
micropython:masterfrom
IhorNehrutsa:hour-WDT

Conversation

@IhorNehrutsa
Copy link
Copy Markdown
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
Comment thread docs/esp32/quickref.rst Outdated
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
Copy Markdown
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
Copy Markdown

github-actions Bot commented Mar 2, 2023

Code size report:


@IhorNehrutsa
Copy link
Copy Markdown
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