OP-423: Clean startup and add weak function call for stack swap.

flight/CopterControl/System/coptercontrol.c

@@ -44,6 +44,7 @@
 
 /* Prototype of PIOS_Board_Init() function */
 extern void PIOS_Board_Init(void);
+extern void Stack_Change(void);
 
 /**
 * OpenPilot Main function:
@@ -68,14 +69,24 @@ int main()
 	 * */
 	PIOS_Board_Init();
 
-#ifdef ERASE_FLASH
-	PIOS_Flash_W25X_EraseChip();
-	while(TRUE){};
-#endif
-
 	/* Initialize modules */
 	MODULE_INITIALISE_ALL
 
+	/* swap the stack to use the IRQ stack */
+	Stack_Change();
+
+	/* Start the FreeRTOS scheduler which should never returns.*/
+	vTaskStartScheduler();
+
+	/* If all is well we will never reach here as the scheduler will now be running. */
+
+	/* Do some indication to user that something bad just happened */
+	PIOS_LED_Off(LED1); \
+	for(;;) { \
+		PIOS_LED_Toggle(LED1); \
+		PIOS_DELAY_WaitmS(100); \
+	};
+
     return 0;
 }
 

flight/OpenPilot/System/openpilot.c

@@ -64,6 +64,8 @@ void OP_ADC_NotifyChange(uint32_t pin, uint32_t pin_value);
 
 /* Prototype of PIOS_Board_Init() function */
 extern void PIOS_Board_Init(void);
+extern void Stack_Change(void);
+static void Stack_Change_Weak () __attribute__ ((weakref ("Stack_Change")));
 
 /**
 * OpenPilot Main function:
@@ -99,6 +101,21 @@ int main()
 	xTaskCreate(TaskSDCard, (signed portCHAR *)"SDCard", configMINIMAL_STACK_SIZE, NULL, (tskIDLE_PRIORITY + 2), NULL);
 #endif
 
+	/* swap the stack to use the IRQ stack (does nothing in sim mode) */
+	Stack_Change_Weak();
+
+	/* Start the FreeRTOS scheduler which should never returns.*/
+	vTaskStartScheduler();
+
+	/* If all is well we will never reach here as the scheduler will now be running. */
+
+	/* Do some indication to user that something bad just happened */
+	PIOS_LED_Off(LED1); \
+	for(;;) { \
+		PIOS_LED_Toggle(LED1); \
+		PIOS_DELAY_WaitmS(100); \
+	};
+
 	return 0;
 }
 

flight/PiOS.posix/inc/pios_initcall.h

@@ -53,18 +53,7 @@ extern void StartModules();
 	/* Initialize modules */ \
 	InitModules(); \
 	/* Initialize the system thread */ \
-	SystemModInitialize(); \
+	SystemModInitialize();}
-	/* Start the FreeRTOS scheduler which never returns.*/ \
-	vTaskStartScheduler(); \
-	/* If all is well we will never reach here as the scheduler will now be running. */ \
-	/* If we do get here, it will most likely be because we ran out of heap space. */ \
-	PIOS_LED_Off(LED1); \
-	PIOS_LED_Off(LED2); \
-	for(;;) { \
-		PIOS_LED_Toggle(LED1); \
-		PIOS_LED_Toggle(LED2); \
-		PIOS_DELAY_WaitmS(100); \
-	}; }
 
 #endif	/* PIOS_INITCALL_H */
 

flight/PiOS.win32/inc/pios_initcall.h

@@ -47,16 +47,8 @@
 	/* Initialize modules */ \
 	InitModules(); \
 	/* Start the FreeRTOS scheduler which never returns.*/ \
-	vTaskStartScheduler(); \
+	/* Initialize the system thread */ \
-	/* If all is well we will never reach here as the scheduler will now be running. */ \
+	SystemModInitialize();}
-	/* If we do get here, it will most likely be because we ran out of heap space. */ \
-	PIOS_LED_Off(LED1); \
-	PIOS_LED_Off(LED2); \
-	for(;;) { \
-		PIOS_LED_Toggle(LED1); \
-		PIOS_LED_Toggle(LED2); \
-		PIOS_DELAY_WaitmS(100); \
-	}; }
 
 
 #endif	/* PIOS_INITCALL_H */

flight/PiOS/STM32F10x/startup_stm32f10x_HD_OP.S

@@ -124,9 +124,25 @@ LoopFillZerobss:
   bcc  FillZerobss
 /* Call the application's entry point.*/
   bl  main
+/* will never return here */
+  bx  lr
+.size  Reset_Handler, .-Reset_Handler
+
+/**
+ * @brief  This is the code that swaps stack (from end of heap to irq_stack).
+ *         Also reclaim the heap that was used as a stack.
+ * @param  None
+ * @retval : None
+*/
+  .section  .text.Stack_Change
+  .weak  Stack_Change
+  .type  Stack_Change, %function
+Stack_Change:
+  mov r4, lr
 /* Switches stack back momentarily to MSP */
-  add r0, #0
+  movs r0, #0
   msr control, r0
+Heap_Reclaim:
 /* add heap_post_rtos to the heap (if the capability/function exist) */
 /* Also claim the unused memory (between end of heap to end of memory */
 /* CAREFULL: the heap section must be the last section in RAM in order this to work */
@@ -136,15 +152,8 @@ LoopFillZerobss:
   subs r2, r2, r1
   adds r0, r0, r2
   bl xPortIncreaseHeapSize
-/* Start the FreeRTOS scheduler which never returns.*/
+  bx r4
-  bl vTaskStartScheduler
+  .size  Stack_Change, .-Stack_Change
-/* If all is well we will never reach here as the scheduler will now be running. */
-/* If we do get here, it will most likely be because we ran out of heap space. */
-/* let the Memory manager handler figure it out (at least for now, it will make it more obvious */
-  bl  MemManage_Handler
-/* will never return here for now until mem manager give up */
-  bx  lr
-.size  Reset_Handler, .-Reset_Handler
 
 /**
  * @brief  Dummy SystemInit_ExtMemCtl function 

flight/PiOS/STM32F10x/startup_stm32f10x_MD_CC.S

@@ -35,6 +35,7 @@
 .global  Default_Handler
 .global  vTaskStartScheduler
 .global  xPortIncreaseHeapSize
+.global  Stack_Change
 
 /* start address for the initialization values of the .data section. 
 defined in linker script */
@@ -49,6 +50,7 @@ defined in linker script */
 .word  _ebss
 
 .equ  BootRAM, 0xF108F85F
+.equ  WWDG, 0x40002C00
 
 /**
  * @brief  This is the code that gets called when the processor first
@@ -127,9 +129,25 @@ LoopFillZerobss:
   bcc  FillZerobss
 /* Call the application's entry point.*/
   bl  main
+/* will never return here */
+  bx  lr
+.size  Reset_Handler, .-Reset_Handler
+
+/**
+ * @brief  This is the code that swaps stack (from end of heap to irq_stack).
+ *         Also reclaim the heap that was used as a stack.
+ * @param  None     
+ * @retval : None       
+*/
+  .section  .text.Stack_Change
+  .weak  Stack_Change
+  .type  Stack_Change, %function
+Stack_Change:
+  mov r4, lr
 /* Switches stack back momentarily to MSP */
-  add r0, #0
+  movs r0, #0
   msr control, r0
+Heap_Reclaim:
 /* add heap_post_rtos to the heap (if the capability/function exist) */
 /* Also claim the unused memory (between end of heap to end of memory */
 /* CAREFULL: the heap section must be the last section in RAM in order this to work */
@@ -139,15 +157,9 @@ LoopFillZerobss:
   subs r2, r2, r1
   adds r0, r0, r2
   bl xPortIncreaseHeapSize
-/* Start the FreeRTOS scheduler which never returns.*/
+  bx r4
-  bl vTaskStartScheduler
+  .size  Stack_Change, .-Stack_Change
-/* If all is well we will never reach here as the scheduler will now be running. */
+
-/* If we do get here, it will most likely be because we ran out of heap space. */
-/* let the Memory manager handler figure it out (at least for now, it will make it more obvious */
-  bl  MemManage_Handler
-/* will never return here for now until mem manager give up */
-  bx  lr
-.size  Reset_Handler, .-Reset_Handler
 
 /**
  * @brief  This is the code that gets called when the processor receives an