Skip to content
This repository has been archived by the owner on Apr 18, 2024. It is now read-only.

Add status logging #215

Merged
merged 3 commits into from
Mar 5, 2019
Merged

Add status logging #215

merged 3 commits into from
Mar 5, 2019

Conversation

kenr
Copy link
Contributor

@kenr kenr commented Mar 5, 2019

We get spurious 0x8029 (NRF_ERROR_SD_RPC_H5_TRANSPORT_STATE) errors using Nordic Semiconductor USB CDC. Adding more logging to capture the status when this happens.

Copy link
Contributor

@chunfantasy chunfantasy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@kenr kenr merged commit b04c4a9 into master Mar 5, 2019
chunfantasy added a commit that referenced this pull request Mar 5, 2019
…e-driver

* 'master' of https://github.com/NordicSemiconductor/pc-ble-driver: (32 commits)
  Add status logging (#215)
  Feature/remove 115k2 baud rate support (#214)
  Fix/install dirs for tests examples (#213)
  Force armgcc toolchain version (#210)
  Adjust line in doc (#211)
  Remove test/example subdirectories, use what is provided by cmake (#212)
  Update documentation (#209)
  Remove baud wiggle (#204)
  Make creation of run test targets optional (#208)
  Check if status callback returns errors (#207)
  Update CMakeLists.txt (#203)
  Remove newline in log and status messages (#199)
  Fix/spurious errors advertising test (#197)
  Make terminate event happen earlier and stop adv/scan before closing (#196)
  Enable use of VCPKG_ROOT env variable (#195)
  Add hex as cmake module resource (#194)
  Fix/spurious error and compilation warning (#193)
  Add check of armgcc toolchain version (#192)
  Fix/advertise buffers (#191)
  Fix/support install prefix take 2 (#190)
  ...
@kenr kenr deleted the feature/add-more-logging-in-tests branch March 20, 2019 13:56
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants