Diff: btle/btle_gattc.cpp

Revision:

239:693a1f145b5a

Parent:

238:6e85a2484de7

Child:

240:75b69581d1dd

diff -r 6e85a2484de7 -r 693a1f145b5a btle/btle_gattc.cpp
--- a/btle/btle_gattc.cpp	Fri Jun 19 15:55:21 2015 +0100
+++ b/btle/btle_gattc.cpp	Fri Jun 19 15:55:21 2015 +0100
@@ -16,268 +16,22 @@
 
 #include "btle_gattc.h"
 #include "UUID.h"
-
-#define BLE_DB_DISCOVERY_MAX_SRV          4  /**< Maximum number of services supported by this module. This also indicates the maximum number of users allowed to be registered to this module. (one user per service). */
-#define BLE_DB_DISCOVERY_MAX_CHAR_PER_SRV 4  /**< Maximum number of characteristics per service supported by this module. */
-
-#define SRV_DISC_START_HANDLE             0x0001 /**< The start handle value used during service discovery. */
-
-
-/**@brief Structure for holding information about the service and the characteristics found during
- *        the discovery process.
- */
-struct DiscoveredService {
-    void setup(ShortUUIDBytes_t uuidIn, Gap::Handle_t start, Gap::Handle_t end) {
-        uuid        = uuidIn;
-        startHandle = start;
-        endHandle   = end;
-    }
-
-    ShortUUIDBytes_t uuid;        /**< UUID of the service. */
-    Gap::Handle_t    startHandle; /**< Service Handle Range. */
-    Gap::Handle_t    endHandle;   /**< Service Handle Range. */
-};
-
-/**@brief Structure for holding information about the service and the characteristics found during
- *        the discovery process.
- */
-struct DiscoveredCharacteristic {
-    struct Properties_t {
-        static const uint8_t BROADCAST_PROPERTY_MASK         = 0x01;
-        static const uint8_t READ_PROPERTY_MASK              = 0x02;
-        static const uint8_t WRITE_WO_RESPONSE_PROPERTY_MASK = 0x04;
-        static const uint8_t WRITE_PROPERTY_MASK             = 0x08;
-        static const uint8_t NOTIFY_PROPERTY_MASK            = 0x10;
-        static const uint8_t INDICATE_PROPERTY_MASK          = 0x20;
-        static const uint8_t AUTH_SIGNED_PROPERTY_MASK       = 0x40;
-
-        Properties_t() : broadcast(0), read(0), write_wo_resp(0), write(0), notify(0), indicate(0), auth_signed_wr(0) {
-            /* empty */
-        }
-
-        Properties_t(uint8_t props) :
-            broadcast(props & BROADCAST_PROPERTY_MASK),
-            read(props & READ_PROPERTY_MASK),
-            write_wo_resp(props & WRITE_WO_RESPONSE_PROPERTY_MASK),
-            write(props & WRITE_PROPERTY_MASK),
-            notify(props & NOTIFY_PROPERTY_MASK),
-            indicate(props & INDICATE_PROPERTY_MASK),
-            auth_signed_wr(props & AUTH_SIGNED_PROPERTY_MASK) {
-            /* empty*/
-        }
-
-        uint8_t broadcast       :1; /**< Broadcasting of the value permitted. */
-        uint8_t read            :1; /**< Reading the value permitted. */
-        uint8_t write_wo_resp   :1; /**< Writing the value with Write Command permitted. */
-        uint8_t write           :1; /**< Writing the value with Write Request permitted. */
-        uint8_t notify          :1; /**< Notications of the value permitted. */
-        uint8_t indicate        :1; /**< Indications of the value permitted. */
-        uint8_t auth_signed_wr  :1; /**< Writing the value with Signed Write Command permitted. */
-    };
-
-    void setup(ShortUUIDBytes_t uuidIn, Properties_t propsIn, Gap::Handle_t declHandleIn, Gap::Handle_t valueHandleIn) {
-        uuid        = uuidIn;
-        props       = propsIn;
-        declHandle  = declHandleIn;
-        valueHandle = valueHandleIn;
-    }
-
-    ShortUUIDBytes_t uuid;
-    Properties_t     props;
-    Gap::Handle_t    declHandle;
-    Gap::Handle_t    valueHandle;
-};
-
-struct DiscoveryStatus {
-    void terminateServiceDiscovery(void) {
-        serviceDiscoveryInProgress = false;
-        printf("end of service discovery\r\n");
-    }
-
-    void terminateCharacteristicDiscovery(void) {
-        characteristicDiscoveryInProgress = false;
-        serviceDiscoveryInProgress        = true;
-        currSrvInd++; /* Progress service index to keep discovery alive. */
-    }
-
-    void resetDiscoveredServices(void) {
-        srvCount   = 0;
-        currSrvInd = 0;
-        memset(services, 0, sizeof(DiscoveredService) * BLE_DB_DISCOVERY_MAX_SRV);
-    }
-
-    void resetDiscoveredCharacteristics(void) {
-        charCount   = 0;
-        currCharInd = 0;
-        memset(characteristics, 0, sizeof(DiscoveredCharacteristic) * BLE_DB_DISCOVERY_MAX_CHAR_PER_SRV);
-    }
-
-    void serviceDiscoveryStarted(Gap::Handle_t connectionHandle) {
-        connHandle                        = connectionHandle;
-        resetDiscoveredServices();
-        serviceDiscoveryInProgress        = true;
-        characteristicDiscoveryInProgress = false;
-    }
-
-    void characteristicDiscoveryStarted(Gap::Handle_t connectionHandle) {
-        connHandle                        = connectionHandle;
-        resetDiscoveredCharacteristics();
-        characteristicDiscoveryInProgress = true;
-        serviceDiscoveryInProgress        = false;
-    }
-
-    void setupDiscoveredServices(const ble_gattc_evt_prim_srvc_disc_rsp_t *response) {
-        currSrvInd = 0;
-        srvCount   = response->count;
-
-        /* Account for the limitation on the number of discovered services we can handle at a time. */
-        if (srvCount > BLE_DB_DISCOVERY_MAX_SRV) {
-            srvCount = BLE_DB_DISCOVERY_MAX_SRV;
-        }
-
-        for (unsigned serviceIndex = 0; serviceIndex < srvCount; serviceIndex++) {
-            services[serviceIndex].setup(response->services[serviceIndex].uuid.uuid,
-                                         response->services[serviceIndex].handle_range.start_handle,
-                                         response->services[serviceIndex].handle_range.end_handle);
-        }
-    }
-
-    void setupDiscoveredCharacteristics(const ble_gattc_evt_char_disc_rsp_t *response) {
-        currCharInd = 0;
-        charCount   = response->count;
-
-        /* Account for the limitation on the number of discovered characteristics we can handle at a time. */
-        if (charCount > BLE_DB_DISCOVERY_MAX_CHAR_PER_SRV) {
-            charCount = BLE_DB_DISCOVERY_MAX_CHAR_PER_SRV;
-        }
-
-        for (unsigned charIndex = 0; charIndex < charCount; charIndex++) {
-            characteristics[charIndex].setup(response->chars[charIndex].uuid.uuid,
-                                             *(const uint8_t *)(&response->chars[charIndex].char_props),
-                                             response->chars[charIndex].handle_decl,
-                                             response->chars[charIndex].handle_value);
-        }
-    }
-
-    void progressCharacteristicDiscovery() {
-        while (characteristicDiscoveryInProgress && (currCharInd < charCount)) {
-            /* THIS IS WHERE THE CALLBACK WILL GO */
-            printf("%x [%u]\r\n", characteristics[currCharInd].uuid, characteristics[currCharInd].valueHandle);
-
-            currCharInd++;
-        }
-
-        if (characteristicDiscoveryInProgress) {
-            Gap::Handle_t startHandle = characteristics[currCharInd - 1].valueHandle + 1;
-            Gap::Handle_t endHandle   = services[currSrvInd].endHandle;
-            resetDiscoveredCharacteristics();
-
-            if (startHandle < endHandle) {
-                ble_gattc_handle_range_t handleRange = {
-                    .start_handle = startHandle,
-                    .end_handle   = endHandle
-                };
-                printf("char discovery returned %u\r\n", sd_ble_gattc_characteristics_discover(connHandle, &handleRange));
-            } else {
-               terminateCharacteristicDiscovery();
-            }
-        }
-    }
-
-    void progressServiceDiscovery() {
-        while (serviceDiscoveryInProgress && (currSrvInd < srvCount)) {
-            /* THIS IS WHERE THE CALLBACK WILL GO */
-            printf("%x [%u %u]\r\n", services[currSrvInd].uuid, services[currSrvInd].startHandle, services[currSrvInd].endHandle);
-
-            if (true) { /* characteristic discovery is optional. */
-                launchCharacteristicDiscovery(connHandle, services[currSrvInd].startHandle, services[currSrvInd].endHandle);
-            } else {
-                currSrvInd++; /* Progress service index to keep discovery alive. */
-            }
-        }
-        if (serviceDiscoveryInProgress && (srvCount > 0) && (currSrvInd > 0)) {
-            Gap::Handle_t endHandle = services[currSrvInd - 1].endHandle;
-            resetDiscoveredServices();
-
-            printf("services discover returned %u\r\n", sd_ble_gattc_primary_services_discover(connHandle, endHandle, NULL));
-        }
-    }
-
-    DiscoveredService        services[BLE_DB_DISCOVERY_MAX_SRV];  /**< Information related to the current service being discovered.
-                                                                   *  This is intended for internal use during service discovery. */
-    DiscoveredCharacteristic characteristics[BLE_DB_DISCOVERY_MAX_CHAR_PER_SRV];
-
-    uint16_t connHandle;  /**< Connection handle as provided by the SoftDevice. */
-    uint8_t  currSrvInd;  /**< Index of the current service being discovered. This is intended for internal use during service discovery.*/
-    uint8_t  srvCount;    /**< Number of services at the peers GATT database.*/
-    uint8_t  currCharInd; /**< Index of the current characteristic being discovered. This is intended for internal use during service discovery.*/
-    uint8_t  charCount;    /**< Number of characteristics within the service.*/
-
-    bool     serviceDiscoveryInProgress;
-    bool     characteristicDiscoveryInProgress;
-};
-
-static DiscoveryStatus discoveryStatus;
-
-ble_error_t launchServiceDiscovery(Gap::Handle_t connectionHandle)
-{
-    discoveryStatus.serviceDiscoveryStarted(connectionHandle);
-
-    uint32_t rc;
-    if ((rc = sd_ble_gattc_primary_services_discover(connectionHandle, SRV_DISC_START_HANDLE, NULL)) != NRF_SUCCESS) {
-        discoveryStatus.terminateServiceDiscovery();
-        switch (rc) {
-            case NRF_ERROR_INVALID_PARAM:
-            case BLE_ERROR_INVALID_CONN_HANDLE:
-                return BLE_ERROR_INVALID_PARAM;
-            case NRF_ERROR_BUSY:
-                return BLE_STACK_BUSY;
-            default:
-            case NRF_ERROR_INVALID_STATE:
-                return BLE_ERROR_INVALID_STATE;
-        }
-    }
-
-    return BLE_ERROR_NONE;
-}
-
-ble_error_t launchCharacteristicDiscovery(Gap::Handle_t connectionHandle, Gap::Handle_t startHandle, Gap::Handle_t endHandle) {
-    discoveryStatus.characteristicDiscoveryStarted(connectionHandle);
-
-    ble_gattc_handle_range_t handleRange = {
-        .start_handle = startHandle,
-        .end_handle   = endHandle
-    };
-    uint32_t rc;
-    if ((rc = sd_ble_gattc_characteristics_discover(connectionHandle, &handleRange)) != NRF_SUCCESS) {
-        discoveryStatus.terminateCharacteristicDiscovery();
-        switch (rc) {
-            case BLE_ERROR_INVALID_CONN_HANDLE:
-            case NRF_ERROR_INVALID_ADDR:
-                return BLE_ERROR_INVALID_PARAM;
-            case NRF_ERROR_BUSY:
-                return BLE_STACK_BUSY;
-            default:
-            case NRF_ERROR_INVALID_STATE:
-                return BLE_ERROR_INVALID_STATE;
-        }
-    }
-
-    return BLE_ERROR_NONE;
-}
+#include "btle_discovery.h"
 
 void bleGattcEventHandler(const ble_evt_t *p_ble_evt)
 {
+    ServiceDiscovery *singleton = ServiceDiscovery::getSingleton();
+
     switch (p_ble_evt->header.evt_id) {
         case BLE_GATTC_EVT_PRIM_SRVC_DISC_RSP:
             switch (p_ble_evt->evt.gattc_evt.gatt_status) {
                 case BLE_GATT_STATUS_SUCCESS:
-                    discoveryStatus.setupDiscoveredServices(&p_ble_evt->evt.gattc_evt.params.prim_srvc_disc_rsp);
+                    singleton->setupDiscoveredServices(&p_ble_evt->evt.gattc_evt.params.prim_srvc_disc_rsp);
                     break;
 
                 case BLE_GATT_STATUS_ATTERR_ATTRIBUTE_NOT_FOUND:
                 default:
-                    discoveryStatus.terminateServiceDiscovery();
+                    singleton->terminate();
                     break;
             }
             break;
@@ -285,17 +39,17 @@
         case BLE_GATTC_EVT_CHAR_DISC_RSP:
             switch (p_ble_evt->evt.gattc_evt.gatt_status) {
                 case BLE_GATT_STATUS_SUCCESS:
-                    discoveryStatus.setupDiscoveredCharacteristics(&p_ble_evt->evt.gattc_evt.params.char_disc_rsp);
+                    singleton->setupDiscoveredCharacteristics(&p_ble_evt->evt.gattc_evt.params.char_disc_rsp);
                     break;
 
                 case BLE_GATT_STATUS_ATTERR_ATTRIBUTE_NOT_FOUND:
                 default:
-                    discoveryStatus.terminateCharacteristicDiscovery();
+                    singleton->terminateCharacteristicDiscovery();
                     break;
             }
             break;
     }
 
-    discoveryStatus.progressCharacteristicDiscovery();
-    discoveryStatus.progressServiceDiscovery();
+    singleton->progressCharacteristicDiscovery();
+    singleton->progressServiceDiscovery();
 }
\ No newline at end of file