Pulled from main. added required changes.

Author: guhansiyer

obc/drivers/rm46/obc_gio_ctrl.c

@@ -12,7 +12,6 @@ void gioNotification(gioPORT_t *port, uint32 bit) {
     switch (bit) {
       case DS3232_INT_PIN:
         alarmInterruptCallback();
-        LOG_DEBUG_FROM_ISR("Alarm handler called.");
         break;
       // See section 3.4.1.1
       // Triggered on falling edge so TX FIFO can be written to once the signal that RTX FIFO is above
@@ -33,7 +32,6 @@ void gioNotification(gioPORT_t *port, uint32 bit) {
       // is asserted to avoid a FIFO overflow
       case CC1120_RX_THR_PKT_gioPORTB_PIN:
         rxFifoReadyCallback();
-        LOG_DEBUG_FROM_ISR("RX FIFO above threshold.");
         break;
 
 #ifdef OBC_REVISION_2

obc/drivers/rm46/obc_i2c_io.c

@@ -155,9 +155,9 @@ void i2cNotification(i2cBASE_t *i2c, uint32 flags) {
     xSemaphoreGiveFromISR(i2cTransferComplete, &xHigherPriorityTaskWoken);
 
     if (xSemaphoreGiveFromISR(i2cTransferComplete, &xHigherPriorityTaskWoken) == pdTRUE) {
-      LOG_DEBUG_FROM_ISR("Semaphore successfully given.");
+      LOG_ERROR_FROM_ISR("Semaphore successfully given.");
     } else {
-      LOG_DEBUG_FROM_ISR("Semaphore cannot be given or was already given.");
+      LOG_ERROR_FROM_ISR("Semaphore cannot be given or was already given.");
     }
   }
 

obc/modules/alarm_mgr/alarm_handler.c

@@ -224,6 +224,11 @@ void alarmInterruptCallback(void) {
 
   alarm_handler_event_t event = {.id = ALARM_HANDLER_ALARM_TRIGGERED};
   xQueueSendToFrontFromISR(alarmHandlerQueueHandle, (void *)&event, &xHigherPriorityTaskWoken);
+  if (xQueueSendToFrontFromISR(alarmHandlerQueueHandle, (void *)&event, &xHigherPriorityTaskWoken) == pdTRUE) {
+    LOG_DEBUG_FROM_ISR("Alarm callback called; data sent to front of queue.");
+  } else {
+    LOG_ERROR_FROM_ISR("Alarm callback failed.");
+  }
 
   if (xHigherPriorityTaskWoken) {
     portYIELD_FROM_ISR(xHigherPriorityTaskWoken);

obc/modules/comms_link_mgr/cc1120_txrx.c

@@ -388,7 +388,7 @@ void txFifoReadyCallback(void) {
   // give semaphore and set xHigherPriorityTaskAwoken to pdTRUE if this unblocks a higher priority task than the current
   // one
   if (xSemaphoreGiveFromISR(txSemaphore, &xHigherPriorityTaskAwoken) != pdPASS) {
-    /* TODO: figure out how to log from ISR */
+    LOG_ERROR_FROM_ISR("TX FIFO below spec.");
   }
   // if xHigherPriorityTaskAwoken == pdTRUE then request a context switch since this means a higher priority task has
   // been unblocked
@@ -400,7 +400,7 @@ void rxFifoReadyCallback(void) {
   // give semaphore and set xHigherPriorityTaskAwoken to pdTRUE if this unblocks a higher priority task than the current
   // one
   if (xSemaphoreGiveFromISR(rxSemaphore, &xHigherPriorityTaskAwoken) != pdPASS) {
-    /* TODO: figure out how to log from ISR */
+    LOG_ERROR_FROM_ISR("RX FIFO above threshold.");
   }
   // if xHigherPriorityTaskAwoken == pdTRUE then request a context switch since this means a higher priority task has
   // been unblocked
@@ -412,7 +412,7 @@ void txFifoEmptyCallback(void) {
   // give semaphore and set xHigherPriorityTaskAwoken to pdTRUE if this unblocks a higher priority task than the current
   // one
   if (xSemaphoreGiveFromISR(txFifoEmptySemaphore, &xHigherPriorityTaskAwoken) != pdPASS) {
-    /* TODO: figure out how to log from ISR */
+    LOG_ERROR_FROM_ISR("TX FIFO is empty.");
   }
   // if xHigherPriorityTaskAwoken == pdTRUE then request a context switch since this means a higher priority task has
   // been unblocked
@@ -424,7 +424,7 @@ void syncEventCallback(void) {
   // give semaphore and set xHigherPriorityTaskAwoken to pdTRUE if this unblocks a higher priority task than the current
   // one
   if (xSemaphoreGiveFromISR(syncReceivedSemaphore, &xHigherPriorityTaskAwoken) != pdPASS) {
-    /* TODO: figure out how to log from ISR */
+    LOG_ERROR_FROM_ISR("Sync word receieved.");
   }
   // if xHigherPriorityTaskAwoken == pdTRUE then request a context switch since this means a higher priority task has
   // been unblocked