-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Realtek-RTL8195AM-Hide_Debugging_Messages #9202
Realtek-RTL8195AM-Hide_Debugging_Messages #9202
Conversation
This PR addresses the issue of ARMmbed#8903. A new boolean variable 'debug' is added to turn off the display of debugging message (by default). It can be turned on when needed. Description 1. Add new boolean parameter ‘debug’ in function ‘RTWInterface’ in file "RTWInterface.h". 2. Add conditional statement to enable/disable variable ‘GlobalDebugEnable’ in file "RTWInterface.h" to control whether debugging message should be displayed.
@AdamZhang0124, thank you for your changes. |
@MarceloSalazar @screamerbg @ashok-rao Do we have a precedent with partners supplying their own debug symbols? @AdamZhang0124 I'm fine with the fix, but considering that Mbed OS has it's own debug symbols, I'm wondering if those should be used instead. No change on your side is required yet. |
Hi @cmonr , |
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.
Various drivers use bool debug parameter as well, should be good
CI started |
Test run: FAILEDSummary: 1 of 11 test jobs failed Failed test jobs:
|
There is a device problem, we will restart tests once this is resolved (thus the job was aborted) |
@0xc0170 , |
CI restarted |
This PR addresses the issue of #8903.
A new boolean variable 'debug' is added to turn off the display of debugging message (by default). It can be turned on when needed.
Description
Pull request type
Reviewers