lte/alt1250: Add error handling to ltefwupdate_execute()

Author: SPRESENSE

include/lte/lte_api.h

@@ -1513,6 +1513,16 @@ int lte_acquire_wakelock(void);
 
 int lte_release_wakelock(void);
 
+/* Get the number of wakelock counts acquired.
+ * Please call this API after calling lte_initialize().
+ * Otherwise this API will result in an error.
+ *
+ * On success, return the count of the current modem wakelock. On failure,
+ * negative value is returned according to <errno.h>.
+ */
+
+int lte_get_wakelock_count(void);
+
 /* Send AT command to the modem.
  *
  * [in] cmd: The AT command data.

include/lte/lte_fwupdate.h

@@ -126,6 +126,7 @@ int ltefwupdate_injected_datasize(void);
  * - LTEFW_RESULT_PRECHK_SIZE_ERROR
  * - LTEFW_RESULT_PRECHK_PKG_ERROR
  * - LTEFW_RESULT_PRECHK_CRC_ERROR
+ * - -EPERM
  *
  */
 

lte/alt1250/usock_handlers/alt1250_ioctl_power.c

@@ -63,6 +63,7 @@ int usockreq_ioctl_power(FAR struct alt1250_s *dev,
       case LTE_CMDID_POWERON:
       case LTE_CMDID_TAKEWLOCK:
       case LTE_CMDID_GIVEWLOCK:
+      case LTE_CMDID_COUNTWLOCK:
         *usock_result = altdevice_powercontrol(dev->altfd, ltecmd->cmdid);
         if (MODEM_STATE_IS_POFF(dev) && ltecmd->cmdid == LTE_CMDID_POWERON)
           {

lte/lapi/src/lapi_firmware.c

@@ -35,6 +35,7 @@
 #include "lte/lte_fwupdate.h"
 
 #include "lapi_util.h"
+#include "lapi_dbg.h"
 
 /****************************************************************************
  * Private Functions
@@ -106,6 +107,24 @@ int ltefwupdate_injected_datasize(void)
 
 int ltefwupdate_execute(void)
 {
+  int ret;
+
+  /* If wakelock is not acquired, firmware update
+   * cannot be performed.
+   */
+
+  ret = lte_get_wakelock_count();
+  if (ret <= 0)
+    {
+      if (ret == 0)
+        {
+          lapi_printf("wakelock is not acquired\n");
+          ret = -EPERM;
+        }
+
+      return ret;
+    }
+
   return fw_generic_request(LTE_CMDID_EXEUPDATE);
 }
 

lte/lapi/src/lapi_power.c

@@ -273,6 +273,15 @@ int lte_release_wakelock(void)
   return lapi_req(LTE_CMDID_GIVEWLOCK, NULL, 0, NULL, 0, NULL);
 }
 
+/****************************************************************************
+ * Name: lte_get_wakelock_count
+ ****************************************************************************/
+
+int lte_get_wakelock_count(void)
+{
+  return lapi_req(LTE_CMDID_COUNTWLOCK, NULL, 0, NULL, 0, NULL);
+}
+
 /****************************************************************************
  * Name: lte_set_context_save_cb
  ****************************************************************************/