Diff: main.cpp

Revision:

19:7e760f65ed78

Parent:

14:1ac188d5404d

Child:

23:64f3f756acf6

--- a/main.cpp	Fri Jan 04 18:45:54 2019 +0000
+++ b/main.cpp	Tue Feb 19 22:34:59 2019 +0000
@@ -22,10 +22,6 @@
 #include "FATFileSystem.h"
 #include "LittleFileSystem.h"
 
-// An event queue is a very useful structure to debounce information between contexts (e.g. ISR and normal threads)
-// This is great because things such as network operations are illegal in ISR, so updating a resource in a button's fall() function is not allowed
-EventQueue eventQueue;
-
 // Default network interface object. Don't forget to change the WiFi SSID/password in mbed_app.json if you're using WiFi.
 NetworkInterface *net = NetworkInterface::get_default_instance();
 
@@ -51,43 +47,51 @@
 
 #define SENSORS_POLL_INTERVAL 1.0
 
-// Declaring pointers for access to Pelion Client resources outside of main()
-MbedCloudClientResource *res_button;
-MbedCloudClientResource *res_led;
-MbedCloudClientResource *res_temperature;
-MbedCloudClientResource *res_voltage;
+// Declaring pointers for access to Pelion Device Management Client resources outside of main()
+MbedCloudClientResource *button_res;
+MbedCloudClientResource *led_res;
+MbedCloudClientResource *post_res;
 
-// When the device is registered, this variable will be used to access various useful information, like device ID etc.
-static const ConnectorClientEndpointInfo* endpointInfo;
+MbedCloudClientResource *temp_res;
+MbedCloudClientResource *volt_res;
+
+
+// An event queue is a very useful structure to debounce information between contexts (e.g. ISR and normal threads)
+// This is great because things such as network operations are illegal in ISR, so updating a resource in a button's fall() function is not allowed
+EventQueue eventQueue;
 
 /**
- * PUT handler
+ * PUT handler - sets the value of the built-in LED
  * @param resource The resource that triggered the callback
  * @param newValue Updated value for the resource
  */
-void led_put_callback(MbedCloudClientResource *resource, m2m::String newValue) {
+void put_callback(MbedCloudClientResource *resource, m2m::String newValue) {
     printf("PUT received. New value: %s\n", newValue.c_str());
     led = atoi(newValue.c_str());
 }
 
 /**
- * POST handler
+ * POST handler - prints the content of the payload
  * @param resource The resource that triggered the callback
  * @param buffer If a body was passed to the POST function, this contains the data.
  *               Note that the buffer is deallocated after leaving this function, so copy it if you need it longer.
  * @param size Size of the body
  */
-void led_post_callback(MbedCloudClientResource *resource, const uint8_t *buffer, uint16_t size) {
-    printf("POST received. Payload: %s\n", res_led->get_value().c_str());
-    led = atoi(res_led->get_value().c_str());
+void post_callback(MbedCloudClientResource *resource, const uint8_t *buffer, uint16_t size) {
+    printf("POST received (length %u). Payload: ", size);
+    for (size_t ix = 0; ix < size; ix++) {
+        printf("%02x ", buffer[ix]);
+    }
+    printf("\n");
 }
 
 /**
- * Button function triggered by the physical button press.
+ * Button handler
+ * This function will be triggered either by a physical button press or by a ticker every 5 seconds (see below)
  */
 void button_press() {
-    int v = res_button->get_value_int() + 1;
-    res_button->set_value(v);
+    int v = button_res->get_value_int() + 1;
+    button_res->set_value(v);
     printf("Button clicked %d times\n", v);
 }
 
@@ -103,7 +107,10 @@
 /**
  * Registration callback handler
  * @param endpoint Information about the registered endpoint such as the name (so you can find it back in portal)
+ * When the device is registered, this variable will be used to access various useful information, like device ID etc.
  */
+
+static const ConnectorClientEndpointInfo* endpointInfo;
 void registered(const ConnectorClientEndpointInfo *endpoint) {
     printf("Registered to Pelion Device Management. Endpoint Name: %s\n", endpoint->internal_endpoint_name.c_str());
     endpointInfo = endpoint;
@@ -118,8 +125,8 @@
     float vref = adc_vref.read();
     printf("ADC temp:  %6.4f C,  vref: %6.4f %%\r\n", temp, vref);
     if (endpointInfo) {
-        res_temperature->set_value(temp);
-        res_voltage->set_value(vref);
+        temp_res->set_value(temp);
+        volt_res->set_value(vref);
     }
 }
 
@@ -128,14 +135,7 @@
     printf("\nStarting Simple Pelion Device Management Client example\n");
 
     // If the User button is pressed ons start, then format storage.
-    DigitalIn *user_button = new DigitalIn(BUTTON1);
-#if TARGET_DISCO_L475VG_IOT01A
-    // The user button on DISCO_L475VG_IOT01A works the other way around
-    const int PRESSED = 0;
-#else
-    const int PRESSED = 1;
-#endif
-    if (user_button->read() == PRESSED) {
+    if (button.read() == MBED_CONF_APP_BUTTON_PRESSED_STATE) {
         printf("User button is pushed on start. Formatting the storage...\n");
         int storage_status = fs.reformat(bd);
         if (storage_status != 0) {
@@ -146,66 +146,57 @@
                 }
             }
         }
-        if (storage_status != 0) {
-            printf("ERROR: Failed to reformat the storage (%d).\n", storage_status);
-        }
     } else {
         printf("You can hold the user button during boot to format the storage and change the device identity.\n");
     }
 
-    // Connect to the internet (DHCP is expected to be on)
+    // Connect to the Internet (DHCP is expected to be on)
     printf("Connecting to the network using the default network interface...\n");
     net = NetworkInterface::get_default_instance();
 
-    nsapi_error_t net_status = -1;
-    for (int tries = 0; tries < 3; tries++) {
-        net_status = net->connect();
-        if (net_status == NSAPI_ERROR_OK) {
-            break;
-        } else {
-            printf("Unable to connect to network. Retrying...\n");
-        }
-    }
-
-    if (net_status != NSAPI_ERROR_OK) {
-        printf("ERROR: Connecting to the network failed (%d)!\n", net_status);
-        return -1;
+    nsapi_error_t net_status = NSAPI_ERROR_NO_CONNECTION;
+    while ((net_status = net->connect()) != NSAPI_ERROR_OK) {
+        printf("Unable to connect to network (%d). Retrying...\n", net_status);
     }
 
     printf("Connected to the network successfully. IP address: %s\n", net->get_ip_address());
 
     printf("Initializing Pelion Device Management Client...\n");
 
-    // SimpleMbedCloudClient handles registering over LwM2M to Pelion DM
+    // SimpleMbedCloudClient handles registering over LwM2M to Pelion Device Management
     SimpleMbedCloudClient client(net, bd, &fs);
     int client_status = client.init();
     if (client_status != 0) {
-        printf("ERROR: Pelion Client initialization failed (%d)\n", client_status);
+        printf("Pelion Client initialization failed (%d)\n", client_status);
         return -1;
     }
 
     // Creating resources, which can be written or read from the cloud
-    res_button = client.create_resource("3200/0/5501", "button_count");
-    res_button->set_value(0);
-    res_button->methods(M2MMethod::GET);
-    res_button->observable(true);
-    res_button->attach_notification_callback(button_callback);
+    button_res = client.create_resource("3200/0/5501", "button_count");
+    button_res->set_value(0);
+    button_res->methods(M2MMethod::GET);
+    button_res->observable(true);
+    button_res->attach_notification_callback(button_callback);
 
-    res_led = client.create_resource("3201/0/5853", "led_state");
-    res_led->set_value(1);
-    res_led->methods(M2MMethod::GET | M2MMethod::PUT);
-    res_led->attach_put_callback(led_put_callback);
+    led_res = client.create_resource("3201/0/5853", "led_state");
+    led_res->set_value(led.read());
+    led_res->methods(M2MMethod::GET | M2MMethod::PUT);
+    led_res->attach_put_callback(put_callback);
+
+    post_res = client.create_resource("3300/0/5605", "execute_function");
+    post_res->methods(M2MMethod::POST);
+    post_res->attach_post_callback(post_callback);
 
     // Sensor resources
-    res_temperature = client.create_resource("3303/0/5700", "temperature");
-    res_temperature->set_value(0);
-    res_temperature->methods(M2MMethod::GET);
-    res_temperature->observable(true);
+    temp_res = client.create_resource("3303/0/5700", "temperature");
+    temp_res->set_value(0);
+    temp_res->methods(M2MMethod::GET);
+    temp_res->observable(true);
 
-    res_voltage = client.create_resource("3316/0/5700", "voltage");
-    res_voltage->set_value(0);
-    res_voltage->methods(M2MMethod::GET);
-    res_voltage->observable(true);
+    volt_res = client.create_resource("3316/0/5700", "voltage");
+    volt_res->set_value(0);
+    volt_res->methods(M2MMethod::GET);
+    volt_res->observable(true);
 
     printf("Initialized Pelion Device Management Client. Registering...\n");
 
@@ -215,19 +206,15 @@
     // Register with Pelion DM
     client.register_and_connect();
 
-    int i = 600; // wait up 60 seconds before attaching sensors and button events
-    while (i-- > 0 && !client.is_client_registered()) {
-        wait_ms(100);
-    }
-
+    // The button fires on an interrupt context, but debounces it to the eventqueue, so it's safe to do network operations
     button.fall(eventQueue.event(&button_press));
     printf("Press the user button to increment the LwM2M resource value...\n");
 
-    // The timer fires on an interrupt context, but debounces it to the eventqueue, so it's safe to do network operations
     Ticker timer;
     timer.attach(eventQueue.event(&sensors_update), SENSORS_POLL_INTERVAL);
 
     // You can easily run the eventQueue in a separate thread if required
     eventQueue.dispatch_forever();
 }
-#endif
+
+#endif /* MBED_TEST_MODE */