prasad-alatkar commented on a change in pull request #777: nimble/host: Clear 
master and slave states on host reset
URL: https://github.com/apache/mynewt-nimble/pull/777#discussion_r398555603
 
 

 ##########
 File path: nimble/host/src/ble_gap.c
 ##########
 @@ -1034,6 +1034,12 @@ ble_gap_master_failed(int status)
         ble_gap_master_connect_failure(status);
         break;
 
+    case BLE_GAP_OP_M_DISC:
+        STATS_INC(ble_gap_stats, initiate_fail);
+        ble_gap_disc_complete();
+        ble_gap_master_reset_state();
 
 Review comment:
   Hi @rymanluk I am getting CI failure
   
   ```
   echo 'Building target=nordic_pca10056_bttester'
   
   Building target=nordic_pca10056_bttester
   
   +newt build -q -l info nordic_pca10056_bttester
   
   2020/03/26 11:46:51.604 [INFO] Ignoring 
/home/travis/gopath/src/github.com/apache/mynewt-nimble/repos/apache-mynewt-core/hw/mcu/nordic/src/ext/nrfx/drivers/src/nrfx_power.c
 because package dictates it.
   
   +rc=0
   
   +[[ 0 -ne 0 ]]
   
   +[[ 0 -eq 127 ]]
   
   +for target in '${TARGETS}'
   
   +echo 'Building target=nordic_pca10056_ext_advertiser'
   
   Building target=nordic_pca10056_ext_advertiser
   
   +newt build -q -l info nordic_pca10056_ext_advertiser
   
   2020/03/26 11:47:43.511 [INFO] Ignoring 
/home/travis/gopath/src/github.com/apache/mynewt-nimble/repos/apache-mynewt-core/hw/mcu/nordic/src/ext/nrfx/drivers/src/nrfx_power.c
 because package dictates it.
   
   Error: nimble/host/src/ble_gap.c:1028:1: error: 'ble_gap_master_failed' 
defined but not used [-Werror=unused-function]
   
    ble_gap_master_failed(int status)
   
    ^~~~~~~~~~~~~~~~~~~~~
   
   cc1: all warnings being treated as errors
   
   ```

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to