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