Implement hasReset() to track Reset Complete response.

sparkfun / SparkFun_BNO080_Arduino_Library

An Arduino Library for the BNO080 IMU combination triple axis accelerometer/gyro/magnetometer packaged with an ARM Cortex M0+ running powerful algorithms.

Other

77 stars 62 forks source link

Implement hasReset() to track Reset Complete response. #92

Closed mattbradford83 closed 3 years ago

mattbradford83 commented 3 years ago

Great library with lots of great features for debugging this troublesome chip. One thing that appears to be missing is the tracking of the "Reset Complete" Executable response packet (BNO08X documentation, P24, fig 1-27). This is needed to determine if the chip resets unexpectedly, so that reports can be re-set. It also pairs really well with the resetReason() function. I've added some code to document this working, along with an example of how to use it to show a software reset.

PaulZC commented 3 years ago

Thank you @mattbradford83 , It may take us a few days before we are able to merge and test this (we have many other things going on at the moment). Please feel free to give us a nudge here if you think this is taking too long. Thanks again - and very best wishes, Paul

mattbradford83 commented 3 years ago

Hey @PaulZC how's this one going?

PaulZC commented 3 years ago

Hi @mattbradford83 ,

Thanks for the reminder! ;-)

Can you please re-target your PR at the release-candidate branch so I can merge and test it? Contributing guidelines are here.

Many thanks, Paul

PaulZC commented 3 years ago

Well, there you go. I learned something new today! ;-) I can change the targeted branch. (Previously I thought only the person submitting the PR could do that...) Merging... Thanks! Paul