Analog-Devices-MSDK / MAX-BLE-HCI

Python3 BLE HCI Directed to MAX Vendor Specific Commands
Apache License 2.0
2 stars 1 forks source link

Connectable ADV from CLI #29

Closed kevin-gillespie closed 3 months ago

kevin-gillespie commented 4 months ago

Connectable advertising is not being set because adv_params is not None. Not sure how to properly set the adv_type in max_ble_hci_cli.py.

diff --git a/src/max_ble_hci/ble_hci.py b/src/max_ble_hci/ble_hci.py
index b88ff8f..0661af1 100644
--- a/src/max_ble_hci/ble_hci.py
+++ b/src/max_ble_hci/ble_hci.py
@@ -238,6 +238,8 @@ class BleHci(BleStandardCmds, VendorSpecificCmds):
         if adv_params is None:
             adv_type = 0 if connect else 3
             adv_params = AdvParams(adv_type=adv_type)
+
+        print(adv_params)
         self.set_adv_params(adv_params)

         status = self.enable_adv(True)
diff --git a/src/max_ble_hci_cli.py b/src/max_ble_hci_cli.py
index 318c949..9829162 100644
--- a/src/max_ble_hci_cli.py
+++ b/src/max_ble_hci_cli.py
@@ -316,6 +316,7 @@ def main():
                 adv_params=AdvParams(
                     interval_min=args.adv_interval,
                     interval_max=args.adv_interval,
+                    adv_type=0
                 ),
             ),
         ),
EricB-ADI commented 3 months ago

fixed in #32