-
Notifications
You must be signed in to change notification settings - Fork 7.5k
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
PeriMan: Improve log messages #8812
Conversation
@SuGlider Please check if you want any other log messages for the PeriMan. |
061449c
to
519327e
Compare
Further investigation shows that the CPU hangs waiting for an interrupt. The following sketch with debug level set to verbose produces this misbehavior (tested on ESP32-S3-DevKitC):
The backtrace of when the processor hangs:
@SuGlider Might this be related to the issue you are seeing in #8809 ? |
maybe the fact that you are modifying the default UART pins, without deinitializing the UART first, is the problem :) |
these issues that Lucas has reported are solved with #8820 - including hanging while using logging. |
The issue with verbose was related to dead lock. Solved with #8820 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
* Improve log messages * Add name function --------- Co-authored-by: Jan Procházka <[email protected]>
Description of Change
This PR aims to improve the Peripheral Manager log messages to help debugging processes.