Merge pull request #762 from arduino/edge_gsm_fixes

Edge control build fixes and CI

Author: facchinm

.github/workflows/compile-examples.yml

@@ -41,6 +41,7 @@ jobs:
           - fqbn: arduino:mbed:opta
           - fqbn: arduino:mbed:giga
           - fqbn: arduino:mbed:nicla_voice
+          - fqbn: arduino:mbed:edge_control
 
         # compile only the examples compatible with each board
         include:
@@ -170,6 +171,15 @@ jobs:
                 version: v0.9.6
               - name: arduino-libg722
                 source-url: https://github.com/pschatzmann/arduino-libg722.git
+          - board:
+              fqbn: arduino:mbed:edge_control
+            additional-libraries: |
+              - name: MicroNMEA
+            additional-sketch-paths: |
+              - libraries/GSM
+              - libraries/USBHID
+              - libraries/USBMSD/examples/Nano33BLE_FlashMassStorage
+              - libraries/ThreadDebug
               
     steps:
       - name: Checkout repository

cores/arduino/mbed/connectivity/cellular/include/cellular/framework/AT/AT_CellularContext.h

@@ -109,7 +109,7 @@ class AT_CellularContext : public CellularContext {
      *  @return     NIDD context text, e.g. Non-IP or NONIP
      */
     virtual const char *get_nonip_context_type_str();
-    virtual void enable_access_technology();
+    virtual nsapi_error_t enable_access_technology();
     virtual void set_cid(int cid);
 
 private:
@@ -135,15 +135,15 @@ class AT_CellularContext : public CellularContext {
 
     PinName _dcd_pin;
     bool _active_high;
-    RadioAccessTechnologyType _rat;
-    FrequencyBand _band;
 
 protected:
     char _found_apn[MAX_APN_LENGTH];
     // flag indicating if CP was requested to be setup
     bool _cp_req;
     bool _is_connected;
     ATHandler &_at;
+    RadioAccessTechnologyType _rat;
+    FrequencyBand _band;
 };
 
 } // namespace mbed

cores/arduino/mbed/connectivity/drivers/cellular/GEMALTO/CINTERION/GEMALTO_CINTERION_CellularContext.h

@@ -34,6 +34,7 @@ class GEMALTO_CINTERION_CellularContext: public AT_CellularContext {
     virtual NetworkStack *get_stack();
 #endif // NSAPI_PPP_AVAILABLE
     virtual nsapi_error_t do_user_authentication();
+    virtual nsapi_error_t enable_access_technology();
 };
 
 } /* namespace mbed */

cores/arduino/mbed/platform/include/platform/Callback.h

@@ -26,14 +26,6 @@
 #include <mstd_type_traits>
 #include <mstd_functional>
 
-#pragma GCC push_options
-// This prevents the GCC compiler from applying optimizations that assume the code follows strict aliasing rules.
-// In order to prevent bugs arising from undefined behavior that is tricky to find in the Callback implementation,
-// or simply from compiler bugs in GCC.
-#pragma GCC optimize("-fno-strict-aliasing")
-// This prevents the GCC compiler from generating incorrect inline code for the Callback constructor.
-#pragma GCC optimize("-fno-inline")
-
 // Controlling switches from config:
 // MBED_CONF_PLATFORM_CALLBACK_NONTRIVIAL - support storing non-trivial function objects
 // MBED_CONF_PLATFORM_CALLBACK_COMPARABLE - support memcmp comparing stored objects (requires zero padding)
@@ -44,6 +36,7 @@
 #define MBED_CONF_PLATFORM_CALLBACK_NONTRIVIAL 1
 #endif
 
+
 namespace mbed {
 /** \addtogroup platform-public-api */
 /** @{*/
@@ -842,6 +835,4 @@ Callback(R(*func)(const volatile T *, ArgTs...), const volatile U *arg) -> Callb
 
 } // namespace mbed
 
-#pragma GCC pop_options
-
 #endif

edge.variables

@@ -1,5 +1,5 @@
 export FLAVOUR="edge"
 export VARIANTS=("EDGE_CONTROL")
 export FQBNS=("edge_control")
-export LIBRARIES=("PDM SPI Wire MRI USBHID USBMSD ThreadDebug Scheduler")
+export LIBRARIES=("SocketWrapper GSM GPS SPI Wire MRI USBHID USBMSD ThreadDebug Scheduler")
 export BOOTLOADERS=("EDGE_CONTROL")

libraries/GPS/library.properties

@@ -6,4 +6,4 @@ sentence=GPS Support Library
 paragraph=
 category=Other
 url=http://www.arduino.cc/en/Reference/GSM
-architectures=mbed,ArduinoCore-mbed,mbed_portenta
+architectures=mbed,ArduinoCore-mbed,mbed_portenta,mbed_edge

libraries/GSM/examples/GNSSClient/GNSSClient.ino

@@ -8,9 +8,20 @@ char username[] = SECRET_USERNAME;
 char pass[]     = SECRET_PASSWORD;
 
 void setup() {
+
+#if defined(ARDUINO_EDGE_CONTROL)
+  // Power ON MKR2
+  pinMode(ON_MKR2, OUTPUT);
+  digitalWrite(ON_MKR2, HIGH);
+#endif
+
   Serial.begin(115200);
   while (!Serial) {}
 
+  // To enable AT Trace debug uncomment the following lines
+  //GSM.trace(Serial);
+  //GSM.setTraceLevel(4);
+
   Serial.println("Starting Carrier Network registration");
   if(!GSM.begin(pin, apn, username, pass, CATNB)){
     Serial.println("The board was not able to register to the network...");

libraries/GSM/examples/GSMClient/GSMClient.ino

@@ -1,7 +1,5 @@
 #include <GSM.h>
 
-REDIRECT_STDOUT_TO(Serial);
-
 #include "arduino_secrets.h" 
 char pin[]      = SECRET_PIN;
 char apn[]      = SECRET_APN;
@@ -14,8 +12,20 @@ int port = 80;
 GSMClient client;
 
 void setup() {
+
+#if defined(ARDUINO_EDGE_CONTROL)
+  // Power ON MKR2
+  pinMode(ON_MKR2, OUTPUT);
+  digitalWrite(ON_MKR2, HIGH);
+#endif
+
   Serial.begin(115200);
   while(!Serial) {}
+
+  // To enable AT Trace debug uncomment the following lines
+  //GSM.trace(Serial);
+  //GSM.setTraceLevel(4);
+
   Serial.println("Starting Carrier Network registration");
   if(!GSM.begin(pin, apn, username, pass, CATNB, BAND_20 | BAND_19)){
     Serial.println("The board was not able to register to the network...");

libraries/GSM/examples/GSMSSLClient/GSMSSLClient.ino

@@ -1,27 +1,46 @@
 /*
   GSMSSLlient
 
-  This sketch connects to a website (https://ifconfig.me)
+  This sketch connects to a website (https://example.com)
   using the Portenta CAT.M1/NB IoT GNSS Shield and TLS.
  
  */
 
 #include <GSM.h>
-
 #include "arduino_secrets.h" 
+
+#if defined(ARDUINO_EDGE_CONTROL)
+  #include "root_ca.h"
+#endif
+
 char pin[]      = SECRET_PIN;
 char apn[]      = SECRET_APN;
 char username[] = SECRET_USERNAME;
 char pass[]     = SECRET_PASSWORD;
 
-const char  server[] = "ifconfig.me";
+const char  server[] = "example.com";
 const char* ip_address;
 int port = 443;
 GSMSSLClient client;
 
 void setup() {
+
+#if defined(ARDUINO_EDGE_CONTROL)
+  // Power ON MKR2
+  pinMode(ON_MKR2, OUTPUT);
+  digitalWrite(ON_MKR2, HIGH);
+
+  // Configure root certificate
+  client.appendCustomCACert(root_ca);
+#endif
+
   Serial.begin(115200);
   while(!Serial) {}
+
+  // To enable AT Trace debug uncomment the following lines
+  //GSM.trace(Serial);
+  //GSM.setTraceLevel(4);
+
   Serial.println("Starting Carrier Network registration");
   if(!GSM.begin(pin, apn, username, pass, CATM1, BAND_3 | BAND_20 | BAND_19)){
     Serial.println("The board was not able to register to the network...");
@@ -33,7 +52,7 @@ void setup() {
   if (client.connect(server, port)) {
     Serial.println("connected to server");
     // Make a HTTP request:
-    client.println("GET /ip HTTP/1.1");
+    client.println("GET / HTTP/1.1");
     client.print("Host: ");
     client.println(server);
     client.println("Connection: close");

libraries/GSM/examples/GSMSSLClient/root_ca.h

@@ -0,0 +1,23 @@
+static const char root_ca[] = 
+"-----BEGIN CERTIFICATE-----\n"
+"MIIDrzCCApegAwIBAgIQCDvgVpBCRrGhdWrJWZHHSjANBgkqhkiG9w0BAQUFADBh\n"
+"MQswCQYDVQQGEwJVUzEVMBMGA1UEChMMRGlnaUNlcnQgSW5jMRkwFwYDVQQLExB3\n"
+"d3cuZGlnaWNlcnQuY29tMSAwHgYDVQQDExdEaWdpQ2VydCBHbG9iYWwgUm9vdCBD\n"
+"QTAeFw0wNjExMTAwMDAwMDBaFw0zMTExMTAwMDAwMDBaMGExCzAJBgNVBAYTAlVT\n"
+"MRUwEwYDVQQKEwxEaWdpQ2VydCBJbmMxGTAXBgNVBAsTEHd3dy5kaWdpY2VydC5j\n"
+"b20xIDAeBgNVBAMTF0RpZ2lDZXJ0IEdsb2JhbCBSb290IENBMIIBIjANBgkqhkiG\n"
+"9w0BAQEFAAOCAQ8AMIIBCgKCAQEA4jvhEXLeqKTTo1eqUKKPC3eQyaKl7hLOllsB\n"
+"CSDMAZOnTjC3U/dDxGkAV53ijSLdhwZAAIEJzs4bg7/fzTtxRuLWZscFs3YnFo97\n"
+"nh6Vfe63SKMI2tavegw5BmV/Sl0fvBf4q77uKNd0f3p4mVmFaG5cIzJLv07A6Fpt\n"
+"43C/dxC//AH2hdmoRBBYMql1GNXRor5H4idq9Joz+EkIYIvUX7Q6hL+hqkpMfT7P\n"
+"T19sdl6gSzeRntwi5m3OFBqOasv+zbMUZBfHWymeMr/y7vrTC0LUq7dBMtoM1O/4\n"
+"gdW7jVg/tRvoSSiicNoxBN33shbyTApOB6jtSj1etX+jkMOvJwIDAQABo2MwYTAO\n"
+"BgNVHQ8BAf8EBAMCAYYwDwYDVR0TAQH/BAUwAwEB/zAdBgNVHQ4EFgQUA95QNVbR\n"
+"TLtm8KPiGxvDl7I90VUwHwYDVR0jBBgwFoAUA95QNVbRTLtm8KPiGxvDl7I90VUw\n"
+"DQYJKoZIhvcNAQEFBQADggEBAMucN6pIExIK+t1EnE9SsPTfrgT1eXkIoyQY/Esr\n"
+"hMAtudXH/vTBH1jLuG2cenTnmCmrEbXjcKChzUyImZOMkXDiqw8cvpOp/2PV5Adg\n"
+"06O/nVsJ8dWO41P0jmP6P6fbtGbfYmbW0W5BjfIttep3Sp+dWOIrWcBAI+0tKIJF\n"
+"PnlUkiaY4IBIqDfv8NZ5YBberOgOzW6sRBc4L0na4UU+Krk2U886UAb3LujEV0ls\n"
+"YSEY1QSteDwsOoBrp+uvFRTp2InBuThs4pFsiv9kuXclVzDAGySj4dzp30d8tbQk\n"
+"CAUw7C29C79Fv1C5qfPrmAESrciIxpg0X40KPMbp1ZWVbd4=\n"
+"-----END CERTIFICATE-----\n";