Merge remote-tracking branch 'origin' into low-power

Author: M65649

.gitignore

@@ -10,4 +10,5 @@ libcryptoauth.a
 .ccls-cache
 .vscode/c_cpp_properties.json
 .vscode/arduino.json
-deploy_sandbox
+deploy_sandbox
+builds

Jenkinsfilek8s

@@ -19,7 +19,7 @@ pipeline {
         GITHUB_REPO = 'https://github.com/mchpTestArea/avr-iot-test'
         GITHUB_REPO_SHORT = 'avr-iot-test'
         GITHUB_ORG = 'mchpTestArea'
-        TARGET_BRANCH = 'main'
+        TARGET_BRANCH = 'sandbox'
 	}
 
     stages {
@@ -81,7 +81,6 @@ pipeline {
                     steps {
                         sh "git clone ${env.DEPLOY_TOOL_URL}"
                         sh "pip3 install PyGithub atlassian-python-api jsonschema packaging lxml"
-                        sh "pwd"
                     }   
                 }
 
@@ -101,7 +100,7 @@ pipeline {
                     steps {
                         script {
                             withCredentials([usernamePassword(credentialsId: 'github_mchptest', usernameVariable: 'USER_NAME', passwordVariable:'PASS' )]) {
-                                sh "python3 ${env.DEPLOY_SCRIPT_FILE} -rlo=true -gpat=${PASS} -dgid=${USER_NAME} -rpn=${GITHUB_REPO_SHORT} -rporg=${GITHUB_ORG} -rltt=\"${env.GIT_TAG}\" -rltv=\"${env.GIT_TAG}\" -rlua=\"avr-iot-cellular-${env.GIT_TAG}.zip\""
+                                sh "python3 ${env.DEPLOY_SCRIPT_FILE} -rlo=true -gpat=${PASS} -dgid=${USER_NAME} -rpn=${GITHUB_REPO_SHORT} -rporg=${GITHUB_ORG} -rltt=\"${env.GIT_TAG}\" -rltv=\"${env.GIT_TAG}\" -rlua=\"avr-iot-cellular-${env.GIT_TAG}.zip, builds/mini/sandbox.ino.hex\""
                             }
                         }
                     }

README.md

@@ -4,7 +4,7 @@
 <p align="center">
 <img width="50%" style={{paddingTop: "10px", paddingBottom: "10px"}} src="./readme_images/mini-board-2.png" />
 </p>
-The AVR-IoT Cellular Mini is a development board from Microchip to develop cellular IoT applications. 
+The AVR-IoT Cellular Mini is a development board from Microchip to develop cellular IoT applications.
 
 📓 Full Arduino support through a library built on top of the open-source [DxCore](https://github.com/SpenceKonde/DxCore)
 

scripts/compile_examples.sh

@@ -9,5 +9,5 @@ done
 # Test for AVR-IoT Cellular Mini
 for d in src/examples/*/ ; do
 	echo "Compiling $d...";
-	arduino-cli compile -b DxCore:megaavr:avrdb:appspm=no,chip=avr128db48,clock=24internal,bodvoltage=1v9,bodmode=disabled,eesave=enable,resetpin=reset,millis=tcb2,startuptime=8 --libraries=".." "$d";
+	arduino-cli compile -b DxCore:megaavr:avrdb:appspm=no,chip=avr128db48,clock=24internal,bodvoltage=1v9,bodmode=disabled,eesave=enable,resetpin=reset,millis=tcb2,startuptime=8,wiremode=mors2 --libraries=".." "$d"  --output-dir "builds/mini/$(basename $d)";
 done

scripts/flash.sh

@@ -2,7 +2,7 @@
 
 SCRIPTPATH="$( cd "$(dirname "$0")" ; pwd -P )"
 
-PORT=COM6
+PORT=COM7
 BUILD_DIR=$SCRIPTPATH/../build
 
 # Do path conversion for WSL

scripts/make.sh

@@ -2,7 +2,7 @@
 
 SCRIPTPATH="$( cd "$(dirname "$0")" ; pwd -P )"
 
-TARGET=$SCRIPTPATH/../src/examples/low_power/low_power.ino
+TARGET=$SCRIPTPATH/../src/examples/sandbox/sandbox.ino
 EXTRA_ARGS=--clean
 
 if [ "$1" = "clean" ]; then

src/ecc608.cpp

@@ -92,4 +92,4 @@ uint8_t ECC608Class::readProvisionItem(enum ecc_data_types type,
 
     *length = 0;
     return this->ERR_NOTFOUND;
-}
+}

src/ecc608.h

@@ -1,4 +1,3 @@
-
 #ifndef ECC608_H
 #define ECC608_H
 
@@ -58,22 +57,21 @@ class ECC608Class {
     ECC608Class(){};
 
     /*
-     * Extract item with given type from ECC slot.
+     * @brief Extract item with given type from ECC slot.
      *
      * This code traverses the list until matching type is found,
      * assuming there is only one item of each type in the slot.
      * Notice: A \0 terminator is added to the returned item for
      *         easier string processing. This is not included in returned
      *         length, but there must be space for it in the buffer.
      *
-     * Parameters:
-     *   type: Type of requested item
-     *   buffer: Buffer to store item in
-     *   length: Pointer to length of buffer (== max length of returned data),
-     *           set to actual item length on return
-     * Returns:
-     *   Error code: 0 => OK, 1-0xff => cryptoauth error code, >= 0x100: see
-     * ecc_provision.h
+     * @param type Type of requested item
+     * @param buffer Buffer to store item in
+     * @param length Pointer to length of buffer (== max length of returned
+     * data), set to actual item length on return
+     *
+     * @return Error code: 0 => OK, 1-0xff => cryptoauth error code, >= 0x100:
+     * see ecc_provision.h
      */
     uint8_t readProvisionItem(enum ecc_data_types type,
                               uint8_t *buffer,
@@ -99,11 +97,12 @@ class ECC608Class {
     };
 
     uint8_t getEndpoint(uint8_t *endpoint, uint8_t *length);
+
     uint8_t getThingName(uint8_t *thingName, uint8_t *length);
 
     uint8_t begin();
 };
 
 extern ECC608Class ECC608;
 
-#endif
+#endif

src/examples/mqtt_interrupt/mqtt_interrupt.ino

@@ -63,7 +63,12 @@ void disconnectedFromNetwork(void) { callback_flags |= NETWORK_DISCONN_FLAG; }
 void connectedToBroker(void) { callback_flags |= BROKER_CONN_FLAG; }
 void disconnectedFromBroker(void) { callback_flags |= BROKER_DISCONN_FLAG; }
 
-void receive(char *topic, uint16_t msg_length) {
+void receive(const char *topic,
+             const uint16_t msg_length,
+             const int32_t message_id) {
+    // Message ID is not used here, as we don't specify a MQTT Quality of
+    // Service different from the default one. Read more about this in the
+    // documentation for the onReceive() function in MqttClient
     memcpy(topic_buffer, topic, MQTT_TOPIC_MAX_LENGTH);
     message_length = msg_length;