Revision 818:61c62a287194, committed 2015-09-29

Comitter:

rgrover1

Date:

Tue Sep 29 09:49:23 2015 +0100

Parent:

817:7a23f308cfe1

Child:

819:3d50ed3cda4b

Commit message:

Synchronized with git rev ad8e92ac
Author: Rohit Grover
adding an initial prototype for a yotta-config based initialization for BLE transports

Changed in this revision

--- a/ble/BLE.h	Tue Sep 29 09:49:23 2015 +0100
+++ b/ble/BLE.h	Tue Sep 29 09:49:23 2015 +0100
@@ -23,6 +23,8 @@
 #include "GattClient.h"
 #include "BLEInstanceBase.h"
 
+#include "mbed_error.h"
+
 /**
  * The base class used to abstract away BLE capable radio transceivers or SOCs,
  * to enable this BLE API to work with any radio transparently.
@@ -50,6 +52,9 @@
      */
     ble_error_t shutdown(void) {
         clearAdvertisingPayload();
+        if (!transport) {
+            error("bad handle to underlying transport");
+        }
         return transport->shutdown();
     }
 
@@ -60,6 +65,9 @@
      *          Note: The string is owned by the BLE_API.
      */
     const char *getVersion(void) {
+        if (!transport) {
+            error("bad handle to underlying transport");
+        }
         return transport->getVersion();
     }
 
@@ -68,9 +76,15 @@
      * going through this accessor.
      */
     const Gap &gap() const {
+        if (!transport) {
+            error("bad handle to underlying transport");
+        }
         return transport->getGap();
     }
     Gap &gap() {
+        if (!transport) {
+            error("bad handle to underlying transport");
+        }
         return transport->getGap();
     }
 
@@ -79,9 +93,15 @@
      * functionality requires going through this accessor.
      */
     const GattServer& gattServer() const {
+        if (!transport) {
+            error("bad handle to underlying transport");
+        }
         return transport->getGattServer();
     }
     GattServer& gattServer() {
+        if (!transport) {
+            error("bad handle to underlying transport");
+        }
         return transport->getGattServer();
     }
 
@@ -90,9 +110,15 @@
      * functionality requires going through this accessor.
      */
     const GattClient& gattClient() const {
+        if (!transport) {
+            error("bad handle to underlying transport");
+        }
         return transport->getGattClient();
     }
     GattClient& gattClient() {
+        if (!transport) {
+            error("bad handle to underlying transport");
+        }
         return transport->getGattClient();
     }
 
@@ -102,9 +128,15 @@
      * accessor.
      */
     const SecurityManager& securityManager() const {
+        if (!transport) {
+            error("bad handle to underlying transport");
+        }
         return transport->getSecurityManager();
     }
     SecurityManager& securityManager() {
+        if (!transport) {
+            error("bad handle to underlying transport");
+        }
         return transport->getSecurityManager();
     }
 
@@ -116,9 +148,49 @@
      * WFE().
      */
     void waitForEvent(void) {
+        if (!transport) {
+            error("bad handle to underlying transport");
+        }
         transport->waitForEvent();
     }
 
+public:
+    typedef unsigned InstanceID_t;
+    static const InstanceID_t DEFAULT_INSTANCE = 0;
+#ifndef YOTTA_CFG_BLE_INSTANCES_COUNT
+    static const InstanceID_t NUM_INSTANCES = 1;
+#else
+    static const InstanceID_t NUM_INSTANCES = YOTTA_CFG_BLE_INSTANCES_COUNT;
+#endif
+
+    /**
+     * Get a reference to the BLE singleton corresponding to a given interface.
+     * There is a static array of BLE singletons.
+     *
+     * @Note: Calling Instance() is preferred over constructing a BLE object
+     * directly, as it returns references to singletons.
+     *
+     * @param[in] id
+     *              Instance-ID. This should be less than NUM_INSTANCES in order
+     *              for the returned BLE singleton to be useful.
+     *
+     * @return a reference to a single object
+     */
+    static BLE &Instance(InstanceID_t id = DEFAULT_INSTANCE);
+
+    /**
+     * Constructor for a handle to a BLE instance (i.e. BLE stack). BLE handles
+     * are thin wrappers around a transport object (i.e. ptr. to
+     * BLEInstanceBase).
+     *
+     * BLE objects are are better created as singletons accessed through the
+     * Instance() method. If multiple BLE handles are constructed for the same
+     * interface (using this constructor), they will share the same underlying
+     * transport object.
+     */
+    BLE(InstanceID_t instanceID = DEFAULT_INSTANCE);
+
+
     /*
      * Deprecation alert!
      * All of the following are deprecated and may be dropped in a future
@@ -1327,17 +1399,12 @@
         return securityManager().onPasskeyDisplay(callback);
     }
 
-public:
-    BLE() : transport(createBLEInstance()) {
-        /* empty */
-    }
-
 private:
     BLE(const BLE&);
     BLE &operator=(const BLE &);
 
 private:
-    BLEInstanceBase *const transport; /* the device specific backend */
+    BLEInstanceBase *transport; /* the device specific backend */
 };
 
 typedef BLE BLEDevice; /* DEPRECATED. This type alias is retained for the sake of compatibility with older

--- a/source/BLE.cpp	Tue Sep 29 09:49:23 2015 +0100
+++ b/source/BLE.cpp	Tue Sep 29 09:49:23 2015 +0100
@@ -79,7 +79,7 @@
 #endif
 };
 
-BLE &
+BLE::BLE &
 BLE::Instance(InstanceID_t id)
 {
     static BLE *singletons[NUM_INSTANCES];