Hi @Community member ,
I think that I find out what is happening here... We are doing HAL_Init, and there HAL_InitTick is called, first HAL_NVIC_EnableIRQ is done, and several lines after htim2.Instance is set to TIM2. There's a problem, because interrupts are enabled, HAL_TIM2_IRQHandler is called and there:
if (htim2.Instance != NULL)
{
HAL_TIM_IRQHandler(&htim2);
}
else
{
/* Disable the TIM2 global Interrupt */
HAL_NVIC_DisableIRQ(TIM2_IRQn);
}
So we will enter into else and TIM2_IRQn will be disabled because htim2 instance is not set. Because that if you add HAL_NVIC_EnableIRQ is working the job when placed after HAL_Init. So what is maybe better solution?
HAL_InitTick function (stm32mp1xx_hal_timebase_tim.c in OpenAMP_FreeRTOS) should be changed and htim2.Instance = TIM2; should be placed before HAL_NVIC_EnableIRQ(TIM2_IRQn).
It should look like:
HAL_StatusTypeDef HAL_InitTick(uint32_t TickPriority)
{
//RCC_ClkInitTypeDef clkconfig;
uint32_t uwTimclock = 0;
uint32_t uwPrescalerValue = 0;
//uint32_t pFLatency;
/*Configure the TIM2 IRQ priority */
HAL_NVIC_SetPriority(TIM2_IRQn, TickPriority, 0);
htim2.Instance = TIM2; //ADD THIS HERE
/* Enable the TIM2 global Interrupt */
HAL_NVIC_EnableIRQ(TIM2_IRQn);
/* Enable TIM2 clock */
__HAL_RCC_TIM2_CLK_ENABLE();
/* Get clock configuration */
//HAL_RCC_GetClockConfig(&clkconfig, &pFLatency);
__HAL_RCC_TIM2_FORCE_RESET();
__HAL_RCC_TIM2_RELEASE_RESET();
/* Compute TIM2 clock */
uwTimclock = HAL_RCC_GetPCLK1Freq();
/* Compute the prescaler value to have TIM2 counter clock equal to 1MHz */
uwPrescalerValue = (uint32_t)((uwTimclock / 1000000) - 1);
/* Initialize TIM2 */
//htim2.Instance = TIM2; //Comment this
/* Initialize TIMx peripheral as follow:
+ Period = [(TIM2CLK/1000) - 1]. to have a (1/1000) s time base.
+ Prescaler = (uwTimclock/1000000 - 1) to have a 1MHz counter clock.
+ ClockDivision = 0
+ Counter direction = Up
*/
htim2.Init.Period = (1000000 / 1000) - 1;
htim2.Init.Prescaler = uwPrescalerValue;
htim2.Init.ClockDivision = 0;
htim2.Init.CounterMode = TIM_COUNTERMODE_UP;
if (HAL_TIM_Base_Init(&htim2) == HAL_OK)
{
/* Start the TIM time Base generation in interrupt mode */
return HAL_TIM_Base_Start_IT(&htim2);
}
/* Return function status */
return HAL_ERROR;
}
Best regards,
Nenad
