Hi
I have tried to test CMSIS DSP FIR library in my STM32G474. Unfortunately after writing a simple test code, where i have prepared input signal and fir coefficients (by Matlab, verified) processor after executing arm_fir_q15 instruction goes to the HardFault. Im using newest DSP library from the Github. arm_mat.h 1.7.0, amr_fir_init_q15.c 1.6.0 and arm_fir_q15.c 1.6.0.
Its hard to catch where exacly processor goes to the debug (lots of clicking in debug step by step instruction execution) but what is strange for me that processor executes a code an function arm_fir_q15_1_8_mve from arm_fir_q15.c whith is not defined! (my IDE shoudnt compiled this). However when i removed this function and forced software to use standard arm_fir_q15 function , HardFault appears also. I have also tried older 1.5.1 version of amr_fir_q15.c but result is the same.
/* USER CODE BEGIN Header */ /** ****************************************************************************** * @file : main.c * @brief : Main program body ****************************************************************************** * @attention * * <h2><center>© Copyright (c) 2020 STMicroelectronics. * All rights reserved.</center></h2> * * This software component is licensed by ST under BSD 3-Clause license, * the "License"; You may not use this file except in compliance with the * License. You may obtain a copy of the License at: * opensource.org/licenses/BSD-3-Clause * ****************************************************************************** */ /* USER CODE END Header */ /* Includes ------------------------------------------------------------------*/ #include "main.h" /* Private includes ----------------------------------------------------------*/ /* USER CODE BEGIN Includes */ //#define ARM_MATH_CM4 /* USER CODE END Includes */ /* Private typedef -----------------------------------------------------------*/ /* USER CODE BEGIN PTD */ /* USER CODE END PTD */ /* Private define ------------------------------------------------------------*/ /* USER CODE BEGIN PD */ #include <math.h> #include "arm_math.h" #define ARM_MATH_DSP #define TEST_LENGTH_SAMPLES 320 #define BLOCK_SIZE 32 #define NUM_TAPS 29 static q15_t signal_q15_in[TEST_LENGTH_SAMPLES] = {0,16436,19259,12778,12465,22935,31161,24333,7704,-2261 ,-0,2261,-7704,-24333,-31161,-22936,-12465,-12778,-19259, -16436,-0,16436,19259,12778,12465,22935,31161,24333,7704, -2261,-0,2261,-7704,-24333,-31161,-22936,-12465,-12778, -19259,-16436,-0,16436,19259,12778,12465,22935,31161, 24333,7704,-2261,-0,2261,-7704,-24333,-31161,-22935, -12465,-12778,-19259,-16436,-0,16436,19259,12778,12465, 22936,31161,24333,7704,-2261,-0,2261,-7704,-24333,-31161, -22935,-12465,-12778,-19259,-16436,-0,16436,19259,12778, 12465,22936,31161,24333,7704,-2261,-0,2261,-7704,-24333, -31161,-22936,-12465,-12778,-19259,-16436,-0,16436,19259, 12778,12465,22936,31161,24333,7704,-2261,0,2261,-7704, -24333,-31161,-22936,-12465,-12778,-19259,-16436,-0, 16436,19259,12778,12465,22936,31161,24333,7704,-2261, 0,2261,-7704,-24333,-31161,-22936,-12465,-12778,-19259, -16436,-0,16436,19259,12778,12465,22936,31161,24333,7704, -2261,0,2261,-7704,-24333,-31161,-22936,-12465,-12778,-19259, -16436,-0,16436,19259,12778,12465,22935,31161,24333,7704, -2261,0,2261,-7704,-24333,-31161,-22936,-12465,-12778,-19259, -16436,-0,16436,19259,12778,12465,22935,31161,24333,7704, -2261,-0,2261,-7704,-24333,-31161,-22936,-12465,-12778,-19259, -16436,-0,16436,19259,12778,12465,22935,31161,24333,7704,-2261, 0,2261,-7704,-24333,-31161,-22936,-12465,-12778,-19259,-16436,0, 16436,19259,12778,12465,22935,31161,24333,7704,-2261,-0,2261, -7704,-24333,-31161,-22936,-12465,-12778,-19259,-16436,-0,16436, 19259,12778,12465,22935,31161,24333,7704,-2261,0,2261,-7704,-24333, -31161,-22936,-12465,-12778,-19259,-16436,0,16436,19259,12778,12465, 22935,31161,24333,7704,-2261,-0,2261,-7704,-24333,-31161,-22936,-12465, -12778,-19259,-16436,-0,16436,19259,12778,12465,22935,31161,24333, 7704,-2261,0,2261,-7704,-24333,-31161,-22936,-12465,-12778,-19259, -16436,0,16436,19259,12778,12465,22935,31161,24333,7704,-2261,-0, 2261,-7704,-24333,-31161,-22936,-12465,-12778,-19259,-16436 }; static q15_t signal_q15_out[TEST_LENGTH_SAMPLES]; static q15_t firStateQ15[BLOCK_SIZE + NUM_TAPS - 1]; static q15_t firCoeffs[NUM_TAPS] = {35, -23, -104, -138, 0, 319, 543, 249, -657, -1543, -1268, 967, 4736, 8345, 9835, 8345, 4736, 967, -1268, -1543, -657, 249, 543, 319, 0, -138, -104, -23, 35}; uint32_t blockSize = BLOCK_SIZE; uint32_t numBlocks = TEST_LENGTH_SAMPLES/BLOCK_SIZE; /* USER CODE END PD */ /* Private macro -------------------------------------------------------------*/ /* USER CODE BEGIN PM */ /* USER CODE END PM */ /* Private variables ---------------------------------------------------------*/ UART_HandleTypeDef hlpuart1; /* USER CODE BEGIN PV */ /* USER CODE END PV */ /* Private function prototypes -----------------------------------------------*/ void SystemClock_Config(void); static void MX_GPIO_Init(void); static void MX_LPUART1_UART_Init(void); /* USER CODE BEGIN PFP */ /* USER CODE END PFP */ /* Private user code ---------------------------------------------------------*/ /* USER CODE BEGIN 0 */ /* USER CODE END 0 */ /** * @brief The application entry point. * @retval int */ int main(void) { /* USER CODE BEGIN 1 */ /* USER CODE END 1 */ /* MCU Configuration--------------------------------------------------------*/ /* Reset of all peripherals, Initializes the Flash interface and the Systick. */ HAL_Init(); /* USER CODE BEGIN Init */ /* USER CODE END Init */ /* Configure the system clock */ SystemClock_Config(); /* USER CODE BEGIN SysInit */ /* USER CODE END SysInit */ /* Initialize all configured peripherals */ MX_GPIO_Init(); MX_LPUART1_UART_Init(); /* USER CODE BEGIN 2 */ uint32_t i; arm_fir_instance_q15 S; q15_t* inbufPtr_q15; q15_t* outbufPtr_q15; inbufPtr_q15 = &signal_q15_in[0]; outbufPtr_q15 = &signal_q15_out[0]; arm_fir_init_q15(&S, NUM_TAPS, (q15_t *)&firCoeffs[0], (q15_t *)&firStateQ15[0], blockSize); for(i=0; i < numBlocks; i++) { arm_fir_q15(&S, inbufPtr_q15 + (i*blockSize), outbufPtr_q15+ (i*blockSize), blockSize); } /* USER CODE END 2 */ /* Infinite loop */ /* USER CODE BEGIN WHILE */ while (1) { /* USER CODE END WHILE */ /* USER CODE BEGIN 3 */ } /* USER CODE END 3 */ } /** * @brief System Clock Configuration * @retval None */ void SystemClock_Config(void) { RCC_OscInitTypeDef RCC_OscInitStruct = {0}; RCC_ClkInitTypeDef RCC_ClkInitStruct = {0}; RCC_PeriphCLKInitTypeDef PeriphClkInit = {0}; /** Configure the main internal regulator output voltage */ HAL_PWREx_ControlVoltageScaling(PWR_REGULATOR_VOLTAGE_SCALE1_BOOST); /** Initializes the CPU, AHB and APB busses clocks */ RCC_OscInitStruct.OscillatorType = RCC_OSCILLATORTYPE_HSI; RCC_OscInitStruct.HSIState = RCC_HSI_ON; RCC_OscInitStruct.HSICalibrationValue = RCC_HSICALIBRATION_DEFAULT; RCC_OscInitStruct.PLL.PLLState = RCC_PLL_ON; RCC_OscInitStruct.PLL.PLLSource = RCC_PLLSOURCE_HSI; RCC_OscInitStruct.PLL.PLLM = RCC_PLLM_DIV4; RCC_OscInitStruct.PLL.PLLN = 85; RCC_OscInitStruct.PLL.PLLP = RCC_PLLP_DIV2; RCC_OscInitStruct.PLL.PLLQ = RCC_PLLQ_DIV2; RCC_OscInitStruct.PLL.PLLR = RCC_PLLR_DIV2; if (HAL_RCC_OscConfig(&RCC_OscInitStruct) != HAL_OK) { Error_Handler(); } /** Initializes the CPU, AHB and APB busses clocks */ RCC_ClkInitStruct.ClockType = RCC_CLOCKTYPE_HCLK|RCC_CLOCKTYPE_SYSCLK |RCC_CLOCKTYPE_PCLK1|RCC_CLOCKTYPE_PCLK2; RCC_ClkInitStruct.SYSCLKSource = RCC_SYSCLKSOURCE_PLLCLK; RCC_ClkInitStruct.AHBCLKDivider = RCC_SYSCLK_DIV1; RCC_ClkInitStruct.APB1CLKDivider = RCC_HCLK_DIV1; RCC_ClkInitStruct.APB2CLKDivider = RCC_HCLK_DIV1; if (HAL_RCC_ClockConfig(&RCC_ClkInitStruct, FLASH_LATENCY_8) != HAL_OK) { Error_Handler(); } /** Initializes the peripherals clocks */ PeriphClkInit.PeriphClockSelection = RCC_PERIPHCLK_LPUART1; PeriphClkInit.Lpuart1ClockSelection = RCC_LPUART1CLKSOURCE_PCLK1; if (HAL_RCCEx_PeriphCLKConfig(&PeriphClkInit) != HAL_OK) { Error_Handler(); } } /** * @brief LPUART1 Initialization Function * @param None * @retval None */ static void MX_LPUART1_UART_Init(void) { /* USER CODE BEGIN LPUART1_Init 0 */ /* USER CODE END LPUART1_Init 0 */ /* USER CODE BEGIN LPUART1_Init 1 */ /* USER CODE END LPUART1_Init 1 */ hlpuart1.Instance = LPUART1; hlpuart1.Init.BaudRate = 115200; hlpuart1.Init.WordLength = UART_WORDLENGTH_8B; hlpuart1.Init.StopBits = UART_STOPBITS_1; hlpuart1.Init.Parity = UART_PARITY_NONE; hlpuart1.Init.Mode = UART_MODE_TX_RX; hlpuart1.Init.HwFlowCtl = UART_HWCONTROL_NONE; hlpuart1.Init.OneBitSampling = UART_ONE_BIT_SAMPLE_DISABLE; hlpuart1.AdvancedInit.AdvFeatureInit = UART_ADVFEATURE_NO_INIT; if (HAL_UART_Init(&hlpuart1) != HAL_OK) { Error_Handler(); } if (HAL_UARTEx_SetTxFifoThreshold(&hlpuart1, UART_TXFIFO_THRESHOLD_1_8) != HAL_OK) { Error_Handler(); } if (HAL_UARTEx_SetRxFifoThreshold(&hlpuart1, UART_RXFIFO_THRESHOLD_1_8) != HAL_OK) { Error_Handler(); } if (HAL_UARTEx_DisableFifoMode(&hlpuart1) != HAL_OK) { Error_Handler(); } /* USER CODE BEGIN LPUART1_Init 2 */ /* USER CODE END LPUART1_Init 2 */ } /** * @brief GPIO Initialization Function * @param None * @retval None */ static void MX_GPIO_Init(void) { GPIO_InitTypeDef GPIO_InitStruct = {0}; /* GPIO Ports Clock Enable */ __HAL_RCC_GPIOC_CLK_ENABLE(); __HAL_RCC_GPIOF_CLK_ENABLE(); __HAL_RCC_GPIOA_CLK_ENABLE(); __HAL_RCC_GPIOB_CLK_ENABLE(); /*Configure GPIO pin Output Level */ HAL_GPIO_WritePin(LD2_GPIO_Port, LD2_Pin, GPIO_PIN_RESET); /*Configure GPIO pin : B1_Pin */ GPIO_InitStruct.Pin = B1_Pin; GPIO_InitStruct.Mode = GPIO_MODE_IT_RISING; GPIO_InitStruct.Pull = GPIO_NOPULL; HAL_GPIO_Init(B1_GPIO_Port, &GPIO_InitStruct); /*Configure GPIO pin : LD2_Pin */ GPIO_InitStruct.Pin = LD2_Pin; GPIO_InitStruct.Mode = GPIO_MODE_OUTPUT_PP; GPIO_InitStruct.Pull = GPIO_NOPULL; GPIO_InitStruct.Speed = GPIO_SPEED_FREQ_LOW; HAL_GPIO_Init(LD2_GPIO_Port, &GPIO_InitStruct); /* EXTI interrupt init*/ HAL_NVIC_SetPriority(EXTI15_10_IRQn, 0, 0); HAL_NVIC_EnableIRQ(EXTI15_10_IRQn); } /* USER CODE BEGIN 4 */ /* USER CODE END 4 */ /** * @brief This function is executed in case of error occurrence. * @retval None */ void Error_Handler(void) { /* USER CODE BEGIN Error_Handler_Debug */ /* User can add his own implementation to report the HAL error return state */ /* USER CODE END Error_Handler_Debug */ } #ifdef USE_FULL_ASSERT /** * @brief Reports the name of the source file and the source line number * where the assert_param error has occurred. * @param file: pointer to the source file name * @param line: assert_param error line source number * @retval None */ void assert_failed(uint8_t *file, uint32_t line) { /* USER CODE BEGIN 6 */ /* User can add his own implementation to report the file name and line number, tex: printf("Wrong parameters value: file %s on line %d\r\n", file, line) */ /* USER CODE END 6 */ } #endif /* USE_FULL_ASSERT */ /************************ (C) COPYRIGHT STMicroelectronics *****END OF FILE****/
HardFault problem solved, the number of taps was not a even number (29 insteed of 28).