Log the power state of adapter with INFO logging level |
|||||||||
Issue descriptionThe power state change of adapter is currently logged at DEBUG level which doesn't show up in the system log from feedback reports. And this info is important for WiFi debugging in terms of co-ex issue.
,
May 17 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromiumos/third_party/bluez/+/2893fffbde42917b5994bca569f23c71dcd3dda4 commit 2893fffbde42917b5994bca569f23c71dcd3dda4 Author: Miao-chen Chou <mcchou@chromium.org> Date: Wed May 17 19:17:53 2017 CHROMIUM: Log the power state change of adapter This changes the log level of messages in adapter_start() and adapter_stop() in order to have better visibility on the power state of adapter in /var/log/messages. BUG= chromium:723781 TEST=build, deploy and check /var/log/messages Change-Id: If2ec798e5285fc1f26fb63eaca7be148c8d868d1 Reviewed-on: https://chromium-review.googlesource.com/508049 Reviewed-by: Miao-chen Chou <mcchou@chromium.org> Commit-Queue: Miao-chen Chou <mcchou@chromium.org> Tested-by: Miao-chen Chou <mcchou@chromium.org> Trybot-Ready: Miao-chen Chou <mcchou@chromium.org> [modify] https://crrev.com/2893fffbde42917b5994bca569f23c71dcd3dda4/src/adapter.c
,
May 18 2017
Your change meets the bar and is auto-approved for M59. Please go ahead and merge the CL to branch 3071 manually. Please contact milestone owner if you have questions. Owners: amineer@(Android), cmasso@(iOS), gkihumba@(ChromeOS), Abdul Syed@(Desktop) For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
May 20 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromiumos/third_party/bluez/+/898f66756430236a234b92142488fc6bd55f921c commit 898f66756430236a234b92142488fc6bd55f921c Author: Miao-chen Chou <mcchou@chromium.org> Date: Sat May 20 10:55:16 2017 CHROMIUM: Log the power state change of adapter This changes the log level of messages in adapter_start() and adapter_stop() in order to have better visibility on the power state of adapter in /var/log/messages. BUG= chromium:723781 TEST=build, deploy and check /var/log/messages Change-Id: If2ec798e5285fc1f26fb63eaca7be148c8d868d1 Reviewed-on: https://chromium-review.googlesource.com/508168 Commit-Ready: Miao-chen Chou <mcchou@chromium.org> Tested-by: Miao-chen Chou <mcchou@chromium.org> Reviewed-by: Sameer Nanda <snanda@chromium.org> Reviewed-by: Miao-chen Chou <mcchou@chromium.org> [modify] https://crrev.com/898f66756430236a234b92142488fc6bd55f921c/src/adapter.c
,
May 22 2017
This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible! If all merges have been completed, please remove any remaining Merge-Approved labels from this issue. Thanks for your time! To disable nags, add the Disable-Nags label. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
May 26 2017
This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible! If all merges have been completed, please remove any remaining Merge-Approved labels from this issue. Thanks for your time! To disable nags, add the Disable-Nags label. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Jul 10 2017
This issue hasn't been updated in the last 6 weeks, so removing its merge approval label. Please re-request a merge if needed. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Oct 31 2017
,
Jan 22 2018
,
Jan 23 2018
,
Jul 12
|
|||||||||
►
Sign in to add a comment |
|||||||||
Comment 1 by mcchou@chromium.org
, May 17 2017