Runnig blecent app on Linux does not work about mynewt-nimble

[pkarashchenko]

The unknown event types seems to be reported by Linux HCI socket interface:

I did a quick experiment and see that only lower 4 bits seems to be valid and changing code:

diff --git a/nimble/host/src/ble_hs_hci_evt.c b/nimble/host/src/ble_hs_hci_evt.c
index e7af2243..3096b4fc 100644
--- a/nimble/host/src/ble_hs_hci_evt.c
+++ b/nimble/host/src/ble_hs_hci_evt.c
@@ -481,7 +481,7 @@ ble_hs_hci_evt_le_adv_rpt(uint8_t subevent, const void *data, unsigned int len)
 
         data += sizeof(rpt) + rpt->data_len + 1;
 
-        desc.event_type = rpt->type;
+        desc.event_type = rpt->type & 0xF;
         desc.addr.type = rpt->addr_type;
         memcpy(desc.addr.val, rpt->addr, BLE_DEV_ADDR_LEN);
         desc.length_data = rpt->data_len;
@@ -510,7 +510,7 @@ ble_hs_hci_evt_le_dir_adv_rpt(uint8_t subevent, const void *data, unsigned int l
     desc.length_data = 0;
 
     for (i = 0; i < ev->num_reports; i++) {
-        desc.event_type = ev->reports[i].type;
+        desc.event_type = ev->reports[i].type & 0xF;
         desc.addr.type = ev->reports[i].addr_type;
         memcpy(desc.addr.val, ev->reports[i].addr, BLE_DEV_ADDR_LEN);
         desc.direct_addr.type = ev->reports[i].dir_addr_type;

seems to solve the issue. I'm not sure if that is a correct solution

[sjanc]

Hi,

this is odd, 0xa1 doesn't even match report type from extended advertising report... what controller is this?

As for the patch, it looks OK, please send PR (we may discuss if this should be under some ifdef ie, for non-mynewt controllers only, but this can happen in PR itself)

[pkarashchenko]

I will check the controller model. Actually in this case I'm running Ubuntu 20.04 on MacBook Pro :)
I will get the controller details and also will check if this is a driver issue. Also I will check if the same happens if I run exactly the same OS (installed on a portable SSD) on a Dell laptop.

[pkarashchenko]

Seems to be Bluetooth driver instability when running Ubuntu on MacBook.