[ECOS] Re: STM32F4 timers test problem

["Morris, Richard" <Richard.Morris@lifetech.com>]

Additional information:

Just enabling the clocks in the APB for the timers doesn't resolve the issue as it appeared below.  I commented out the init_timer() function that actually starts the timers and the exception still occurs.

It seems to have something to do with some of the timers running.  If I start only timer 8 in the timers.c test, the exception still occurs.  However, if I start timer 7, or 6, or 5 the exception seems to not occur, or at least the frequency is low enough that it isn't seen during the interval of the test.

Curiously if I build the tests using the JTAG startup type rather than ROM, the test will fail after 10+ loops with a"Stack base corrupt" exception.  When built for ROM startup type, the test will run for at least 500 loops.


<<<<<<<<<<<<<<<<<<<<<<<<<<<< PREVIOUS POST BEGIN >>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>

I have been having trouble running the STM32F4 timers test on the STM3240G-Eval board:

   ecos/packages/hal/cortexm/stm32/var/current/tests/timers.c

The test would SIGTRAP in GDB when executing:

          cyg_thread_delay( 5*100 );

in the while loop in "timers_test()" in timers.c.

The issue seems to be that the clocks for the timers are not enabled in the RCC_APB1ENR and 
RCC_APB2ENR registers, so when the timers are attempted to be initialized, the initialization is not successful and the timers don't run.

I am not exactly sure why this would relate to the SIGTRAP, but when I resolved the timer initialization, the SIGTRAP went away.  If anyone could explain this, I would be interested in better understanding.

I enabled these clocks (now the test seems to run reliably) by adding code shown below at the end of "hal_variant_init()" in stm32_misc.c to initialize the TIMer clocks.

I suspect this is not the correct place for this change, but wasn't sure where to make it.  There probably also needs to be some conditional compilation to take care of the various variants of the processor.

#######################################################################################
void hal_variant_init( void )
{

#if 1 //!defined(CYG_HAL_STARTUP_RAM)
   hal_start_clocks();
#endif

   // Attach EXTI springboard to interrupt vectors
   HAL_INTERRUPT_ATTACH( CYGNUM_HAL_INTERRUPT_EXTI9_5,   hal_exti_isr, 0, 0 );
   HAL_INTERRUPT_ATTACH( CYGNUM_HAL_INTERRUPT_EXTI15_10, hal_exti_isr, 0, 0 );

#ifdef CYGSEM_HAL_VIRTUAL_VECTOR_SUPPORT
   hal_if_init();
#endif

#if (1)
   // A hack for now, enable timer clocks for STM32F407IG
   unsigned int reg;
   HAL_READ_UINT32((CYGHWR_HAL_STM32_RCC + CYGHWR_HAL_STM32_RCC_APB1ENR), reg );
   HAL_WRITE_UINT32((CYGHWR_HAL_STM32_RCC + CYGHWR_HAL_STM32_RCC_APB1ENR),
                    (reg
                     | (1u << CYGHWR_HAL_STM32_RCC_APB1ENR_TIM2) 
                     | (1u << CYGHWR_HAL_STM32_RCC_APB1ENR_TIM3)
                     | (1u << CYGHWR_HAL_STM32_RCC_APB1ENR_TIM4)
                     | (1u << CYGHWR_HAL_STM32_RCC_APB1ENR_TIM5)
                     | (1u << CYGHWR_HAL_STM32_RCC_APB1ENR_TIM6)
                     | (1u << CYGHWR_HAL_STM32_RCC_APB1ENR_TIM7)
                    )
                   );
   HAL_READ_UINT32((CYGHWR_HAL_STM32_RCC + CYGHWR_HAL_STM32_RCC_APB2ENR), reg );
   HAL_WRITE_UINT32((CYGHWR_HAL_STM32_RCC + CYGHWR_HAL_STM32_RCC_APB2ENR),
                    (reg
                     | (1u << CYGHWR_HAL_STM32_RCC_APB2ENR_TIM1)
                     | (1u << CYGHWR_HAL_STM32_RCC_APB2ENR_TIM8)
                     | (1u << CYGHWR_HAL_STM32_RCC_APB2ENR_TIM9)
                     | (1u << CYGHWR_HAL_STM32_RCC_APB2ENR_TIM10)
                     | (1u << CYGHWR_HAL_STM32_RCC_APB2ENR_TIM11)
                    )
                   );
#endif
}
#######################################################################################

This seems to resolve the issue and now the timer test runs reliably and displays reasonable looking results, whereas before the results were always 0.

Advice on how this initialization would be more correctly done would be appreciated.

<<<<<<<<<<<<<<<<<<<<<<<<<<<< PREVIOUS POST END >>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>


Richard Morris
Senior Staff Firmware Engineer
Richard Morris
Senior Staff Firmware Engineer

T  650.638.6883
850 Lincoln Centre Dr
MS 407-1
Foster City, CA 94404
USA
http://www.lifetechnologies.com



--
Before posting, please read the FAQ: http://ecos.sourceware.org/fom/ecos
and search the list archive: http://ecos.sourceware.org/ml/ecos-discuss