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

idf_monitor.py: allow monitor without resetting target (IDFGH-5963) #7651

Closed
bugadani opened this issue Oct 6, 2021 · 3 comments · Fixed by #8788
Closed

idf_monitor.py: allow monitor without resetting target (IDFGH-5963) #7651

bugadani opened this issue Oct 6, 2021 · 3 comments · Fixed by #8788
Labels
Resolution: Done Issue is done internally Status: Done Issue is done internally Type: Feature Request Feature request for IDF

Comments

@bugadani
Copy link
Contributor

bugadani commented Oct 6, 2021

Is your feature request related to a problem? Please describe.

We have some bugs that occur after a long uptime. It would help if monitor would not reset our system when connected.

Describe the solution you'd like

A new command line switch (e.g. idf.py monitor --running) that would monitor the serial output of a running target without requiring establishing a connection.

Describe alternatives you've considered

  • The idf_monitor.py shipped with esp-idf 4.2 worked well. We started monitor without connecting the debug wires, then connected our device. Unfortunately, with esp-idf v4.3, the tool requires a connection, therefore it can't be used this way.
  • we can use putty or other software, but it lacks the address processing features of idf_monitor.py
@bugadani bugadani added the Type: Feature Request Feature request for IDF label Oct 6, 2021
@espressif-bot espressif-bot added the Status: Opened Issue is new label Oct 6, 2021
@github-actions github-actions bot changed the title esptool: allow monitor without resetting target esptool: allow monitor without resetting target (IDFGH-5963) Oct 6, 2021
@dobairoland
Copy link
Collaborator

Hi @bugadani. Your request about connecting idf_monitor.py to an ESP chip without reset is clear to me.

However, please explain how this relates to esptool used for flashing. After flashing the chip, you need to restart it so the uploaded firmware could start executing. It is possible to keep the chip in bootloader mode and not reset it but that is unusable for you because the firmware won't be executed at all. The update of esptool from v3.0 to v3.1 is not changing the behaviour of idf_monitor.py.

@bugadani
Copy link
Contributor Author

bugadani commented Oct 6, 2021

@dobairoland I didn't mention flashing at all, since in my use case, there is no flashing involved. The target device is up and running, and has been running for some time when I would like to connect the monitor.

I might be confusing the exact tools involved, though I hope that's not an issue here. I think I was confusing this together with an unrelated issue that did involve esptool.py. Anyway, I've updated the issue accordingly, hopefully now with the correct script.

@bugadani bugadani changed the title esptool: allow monitor without resetting target (IDFGH-5963) idf_monitor.py: allow monitor without resetting target (IDFGH-5963) Oct 6, 2021
@dobairoland
Copy link
Collaborator

Thanks for the clarification!

@dobairoland dobairoland linked a pull request Apr 19, 2022 that will close this issue
@espressif-bot espressif-bot added Resolution: NA Issue resolution is unavailable Status: Done Issue is done internally Resolution: Done Issue is done internally and removed Status: Opened Issue is new Resolution: NA Issue resolution is unavailable labels May 10, 2022
espressif-bot pushed a commit that referenced this issue May 15, 2022
…ing the chip target upon connection

Closes #8889

Closes IDFGH-7189, IDFGH-7301, IDFGH-5963

Closes #7651

Merges #8788
espressif-bot pushed a commit to espressif/esp-idf-monitor that referenced this issue Jan 16, 2023
…ing the chip target upon connection

Closes espressif/esp-idf#8889

Closes IDFGH-7189, IDFGH-7301, IDFGH-5963

Closes espressif/esp-idf#7651

Merges espressif/esp-idf#8788
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

Successfully merging a pull request may close this issue.

3 participants