drivers: timer: Enable STM32_RADIO_TIMER Kconfig parameter for STM32WB0x

Use radio timer as the system timer when Bluetooth is used.

Modify CMakeLists.txt to compile radio timer driver when
STM32_RADIO_TIMER is enabled.

Remove the common parts from hci_stm32wb0.c that are present
in the radio timer driver.

Set the appropriate value for SYS_CLOCK_HW_CYCLES_PER_SEC and
SYS_CLOCK_TICKS_PER_SEC.

Signed-off-by: Ali Hozhabri <[email protected]>

Author: HoZHel

drivers/bluetooth/hci/hci_stm32wb0.c

@@ -31,15 +31,8 @@ LOG_MODULE_REGISTER(bt_driver);
 #define DT_DRV_COMPAT st_hci_stm32wb0
 
 /* Max HS startup time expressed in system time (1953 us / 2.4414 us) */
-#define MAX_HS_STARTUP_TIME 320
-#define BLE_WKUP_PRIO       0
-#define BLE_WKUP_FLAGS      0
 #define BLE_TX_RX_PRIO      0
 #define BLE_TX_RX_FLAGS     0
-#define CPU_WKUP_PRIO       1
-#define CPU_WKUP_FLAGS      0
-#define BLE_ERROR_PRIO      3
-#define BLE_ERROR_FLAGS     0
 #define BLE_RXTX_SEQ_PRIO   3
 #define BLE_RXTX_SEQ_FLAGS  0
 #define PKA_PRIO	    2
@@ -58,7 +51,7 @@ LOG_MODULE_REGISTER(bt_driver);
 
 static uint32_t __noinit dyn_alloc_a[BLE_DYN_ALLOC_SIZE >> 2];
 static uint8_t buffer_out_mem[MAX_EVENT_SIZE];
-static struct k_work_delayable hal_radio_timer_work, ble_stack_work;
+static struct k_work_delayable ble_stack_work;
 
 static struct net_buf *get_rx(uint8_t *msg);
 static PKA_HandleTypeDef hpka;
@@ -77,6 +70,14 @@ int BLEPLAT_NvmGet(void)
 	return 0;
 }
 
+uint8_t BLEPLAT_SetRadioTimerValue(uint32_t Time, uint8_t EventType, uint8_t CalReq)
+{
+	uint8_t retval;
+
+	retval = HAL_RADIO_TIMER_SetRadioTimerValue(Time, EventType, CalReq);
+	return retval;
+}
+
 static void blestack_process(struct k_work *work)
 {
 	BLE_STACK_Tick();
@@ -85,11 +86,6 @@ static void blestack_process(struct k_work *work)
 	}
 }
 
-static void vtimer_process(struct k_work *work)
-{
-	HAL_RADIO_TIMER_Tick();
-}
-
 /* "If, since the last power-on or reset, the Host has ever issued a legacy
  * advertising command and then issues an extended advertising command, or
  * has ever issued an extended advertising command and then issues a legacy
@@ -210,28 +206,13 @@ void send_event(uint8_t *buffer_out, uint16_t buffer_out_length, int8_t overflow
 
 void HAL_RADIO_TIMER_TxRxWakeUpCallback(void)
 {
-	k_work_schedule(&hal_radio_timer_work, K_NO_WAIT);
-	k_work_schedule(&ble_stack_work, K_NO_WAIT);
-}
-
-void HAL_RADIO_TIMER_CpuWakeUpCallback(void)
-{
-	k_work_schedule(&hal_radio_timer_work, K_NO_WAIT);
 	k_work_schedule(&ble_stack_work, K_NO_WAIT);
 }
 
 void HAL_RADIO_TxRxCallback(uint32_t flags)
 {
 	BLE_STACK_RadioHandler(flags);
 	k_work_schedule(&ble_stack_work, K_NO_WAIT);
-	k_work_schedule(&hal_radio_timer_work, K_NO_WAIT);
-}
-
-ISR_DIRECT_DECLARE(RADIO_TIMER_TXRX_WKUP_IRQHandler)
-{
-	HAL_RADIO_TIMER_TXRX_WKUP_IRQHandler();
-	ISR_DIRECT_PM(); /* PM done after servicing interrupt for best latency */
-	return 1;
 }
 
 ISR_DIRECT_DECLARE(RADIO_TXRX_IRQHandler)
@@ -248,56 +229,24 @@ ISR_DIRECT_DECLARE(RADIO_TXRX_SEQ_IRQHandler)
 	return 1;
 }
 
-ISR_DIRECT_DECLARE(RADIO_TIMER_CPU_WKUP_IRQHandler)
-{
-	HAL_RADIO_TIMER_TimeoutCallback();
-	HAL_RADIO_TIMER_CpuWakeUpCallback();
-	ISR_DIRECT_PM(); /* PM done after servicing interrupt for best latency */
-	return 1;
-}
-
-ISR_DIRECT_DECLARE(RADIO_TIMER_ERROR_IRQHandler)
-{
-	volatile uint32_t debug_cmd;
-
-	BLUE->DEBUGCMDREG |= 1;
-	/* If the device is configured with CLK_SYS = 64MHz
-	 * and BLE clock = 16MHz, a register read is necessary
-	 * to ensure interrupt register is properly cleared
-	 * due to AHB down converter latency
-	 */
-	debug_cmd = BLUE->DEBUGCMDREG;
-	LOG_ERR("Timer error");
-	ISR_DIRECT_PM(); /* PM done after servicing interrupt for best latency */
-	return 1;
-}
-
 /* Function called from PKA_IRQHandler() context. */
 void PKAMGR_IRQCallback(void)
 {
 	k_work_schedule(&ble_stack_work, K_NO_WAIT);
 }
 
-static int _PKA_IRQHandler(void *args)
+static void _PKA_IRQHandler(void *args)
 {
 	ARG_UNUSED(args);
 
 	HAL_PKA_IRQHandler(&hpka);
-	ISR_DIRECT_PM(); /* PM done after servicing interrupt for best latency */
-	return 1;
 }
 
 static void ble_isr_installer(void)
 {
-	IRQ_DIRECT_CONNECT(RADIO_TIMER_TXRX_WKUP_IRQn, BLE_WKUP_PRIO,
-			   RADIO_TIMER_TXRX_WKUP_IRQHandler, BLE_WKUP_FLAGS);
 	IRQ_DIRECT_CONNECT(RADIO_TXRX_IRQn, BLE_TX_RX_PRIO, RADIO_TXRX_IRQHandler, BLE_TX_RX_FLAGS);
-	IRQ_DIRECT_CONNECT(RADIO_TIMER_CPU_WKUP_IRQn, CPU_WKUP_PRIO,
-			   RADIO_TIMER_CPU_WKUP_IRQHandler, CPU_WKUP_FLAGS);
 	IRQ_DIRECT_CONNECT(RADIO_TXRX_SEQ_IRQn, BLE_RXTX_SEQ_PRIO, RADIO_TXRX_SEQ_IRQHandler,
 			   BLE_RXTX_SEQ_FLAGS);
-	IRQ_DIRECT_CONNECT(RADIO_TIMER_ERROR_IRQn, BLE_ERROR_PRIO, RADIO_TIMER_ERROR_IRQHandler,
-			   BLE_ERROR_FLAGS);
 	IRQ_CONNECT(PKA_IRQn, PKA_PRIO, _PKA_IRQHandler, NULL, PKA_FLAGS);
 }
 
@@ -450,7 +399,6 @@ static int bt_hci_stm32wb0_send(const struct device *dev, struct net_buf *buf)
 static int bt_hci_stm32wb0_open(const struct device *dev, bt_hci_recv_t recv)
 {
 	struct hci_data *data = dev->data;
-	RADIO_TIMER_InitTypeDef VTIMER_InitStruct = {MAX_HS_STARTUP_TIME, 0, 0};
 	RADIO_HandleTypeDef hradio = {0};
 	BLE_STACK_InitTypeDef BLE_STACK_InitParams = {
 		.BLEStartRamAddress = (uint8_t *)dyn_alloc_a,
@@ -487,8 +435,6 @@ static int bt_hci_stm32wb0_open(const struct device *dev, bt_hci_recv_t recv)
 	ble_isr_installer();
 	hradio.Instance = RADIO;
 	HAL_RADIO_Init(&hradio);
-	HAL_RADIO_TIMER_Init(&VTIMER_InitStruct);
-
 	HW_AES_Init();
 	hpka.Instance = PKA;
 	HAL_PKA_Init(&hpka);
@@ -505,7 +451,6 @@ static int bt_hci_stm32wb0_open(const struct device *dev, bt_hci_recv_t recv)
 	aci_adv_nwk_init();
 
 	data->recv = recv;
-	k_work_init_delayable(&hal_radio_timer_work, vtimer_process);
 	k_work_init_delayable(&ble_stack_work, blestack_process);
 	k_work_schedule(&ble_stack_work, K_NO_WAIT);
 

drivers/timer/CMakeLists.txt

@@ -41,6 +41,7 @@ zephyr_library_sources_ifdef(CONFIG_REALTEK_RTS5912_RTMR realtek_rts5912_rtmr.c)
 zephyr_library_sources_ifdef(CONFIG_SAM0_RTC_TIMER sam0_rtc_timer.c)
 zephyr_library_sources_ifdef(CONFIG_SILABS_SLEEPTIMER_TIMER silabs_sleeptimer_timer.c)
 zephyr_library_sources_ifdef(CONFIG_STM32_LPTIM_TIMER stm32_lptim_timer.c)
+zephyr_library_sources_ifdef(CONFIG_STM32_RADIO_TIMER stm32_radio_timer.c)
 zephyr_library_sources_ifdef(CONFIG_TI_DM_TIMER ti_dmtimer.c)
 zephyr_library_sources_ifdef(CONFIG_XLNX_PSTTC_TIMER xlnx_psttc_timer.c)
 zephyr_library_sources_ifdef(CONFIG_XTENSA_TIMER xtensa_sys_timer.c)

drivers/timer/Kconfig

@@ -99,6 +99,7 @@ source "drivers/timer/Kconfig.sam0_rtc"
 source "drivers/timer/Kconfig.silabs"
 source "drivers/timer/Kconfig.smartbond"
 source "drivers/timer/Kconfig.stm32_lptim"
+source "drivers/timer/Kconfig.stm32_radio"
 source "drivers/timer/Kconfig.ti_dm_timer"
 source "drivers/timer/Kconfig.xlnx_psttc"
 source "drivers/timer/Kconfig.xtensa"

drivers/timer/Kconfig.stm32_radio

@@ -0,0 +1,13 @@
+# STM32WB0x radio timer configuration options
+
+# Copyright (c) 2025 STMicroelectronics
+# SPDX-License-Identifier: Apache-2.0
+
+config STM32_RADIO_TIMER
+	bool "STM32 Radio Timer for WB0x"
+	default y if BT
+	depends on SOC_SERIES_STM32WB0X
+	select USE_STM32_HAL_RADIO_TIMER
+	select TICKLESS_CAPABLE
+	select HAS_STM32LIB
+	select SYSTEM_TIMER_HAS_DISABLE_SUPPORT

soc/st/stm32/Kconfig.defconfig

@@ -35,6 +35,7 @@ DT_FLASH_PARENT_IS_XSPI := $(dt_node_has_compat,$(DT_CHOSEN_FLASH_PARENT),$(DT_C
 
 config SYS_CLOCK_HW_CYCLES_PER_SEC
 	default "$(DT_STM32_RCC_CLOCK_FREQ)" if "$(dt_nodelabel_enabled,rcc)"
+	depends on !STM32_RADIO_TIMER
 
 config LOG_BACKEND_SWO_REF_FREQ_HZ
 	default "$(DT_STM32_RCC_CLOCK_FREQ)" if "$(dt_nodelabel_enabled,rcc)"

soc/st/stm32/stm32wb0x/Kconfig.defconfig

@@ -8,6 +8,18 @@ if SOC_SERIES_STM32WB0X
 config NUM_IRQS
 	default 32
 
+config SYS_CLOCK_HW_CYCLES_PER_SEC
+	default 409600
+	depends on STM32_RADIO_TIMER
+
+config SYS_CLOCK_TICKS_PER_SEC
+	default 10240
+	depends on STM32_RADIO_TIMER
+
+config CORTEX_M_SYSTICK
+	default n
+	depends on STM32_RADIO_TIMER
+
 if BT
 
 config BT_AUTO_PHY_UPDATE