Conversation
XuGuohui
commented
Jul 31, 2014
- Make library compatible with Due except the examples
- Add board definition in boards.h and do radio reset on every RedBearLab boards
There was a problem hiding this comment.
This should not be required for the V2 shield, Blend, and, Blend micro as the nRF8001 Reset pin is not inverted in those boards. This change was required only for the v1.1 Shield and the v2012_07 Shield. Please confirm.
There was a problem hiding this comment.
These code is for the radio reset, not for the hareware reset, so it dose not care if the Reset pin is inverted or not. The radio reset is required for all RedBearLab boards, otherwise, if Reset pin is UNUSED, nRF8001 will not work properly, such as can not set local name if re-upload sketch.
There was a problem hiding this comment.
I think in the V2.0 and later boards, the Arduino RESET line is also being routed to the nRF8001 so even if the radio reset command is not sent, the code will work. The the code above is a work around specific to the v1.1 and v2012_07 shield versions which did not map any Arduino pin to the nRF8001 RESET and only had a POR circuit.
Since the code here is only a work around I am very reluctant to add it for all redbearlab boards that do not have this problem. Please confirm my understanding.
There was a problem hiding this comment.
I totally understand what you mean. There is one case I tried that if uploaded sketch and BLE works well,and then I change the BLE Local Name in the sketch and upload again without preesing the Arduino RESET, then the BLE Local Name showed on App will not change. Please help confirm what I tried and tell me the reason if so. Thanks a lot!
|
guohui seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account. You have signed the CLA already but the status is still pending? Let us recheck it. |