按照GitHub AI 的PR建议

1. 增加关闭定时器实现
2. 去掉多余的break入口
3. KConfig缩进保持与当前已有的一致

Author: oxlm

bsp/novosns/ns800/libraries/HAL_Drivers/drivers/drv_timer.c

@@ -84,17 +84,26 @@ void TIM2_IRQHandler(void) { ns800_clock_timer_isr(&ns800_timers[TIM2_INDEX]); }
 static void ns800_clock_timer_init(rt_clock_timer_t *timer, rt_uint32_t state)
 {
     struct ns800_clock_timer *tim = (struct ns800_clock_timer *)timer->parent.user_data;
+    TIM_TypeDef *htim = (TIM_TypeDef *)tim->instance;
 
-    __IO uint32_t cfg =
-        TIM_PWMMODE_ONEPOINT |
-        TIM_CLOCKDIVISION_DIV1 |
-        TIM_AUTORELOADPRELOAD_ENABLE |
-        TIM_COUNTERMODE_UP |
-        TIM_ONEPULSEMODE_REPETITIVE;
-
-    TIM_configTimeBase((TIM_TypeDef *)tim->instance, 200-1, 100-1, cfg);
-    TIM_clearFlags((TIM_TypeDef *)tim->instance, TIM_FLAG_UPDATE);
-    TIM_enableInterruptSource((TIM_TypeDef *)tim->instance, TIM_IT_UPDATE);
+    if(state)
+    {
+        __IO uint32_t cfg =
+            TIM_PWMMODE_ONEPOINT |
+            TIM_CLOCKDIVISION_DIV1 |
+            TIM_AUTORELOADPRELOAD_ENABLE |
+            TIM_COUNTERMODE_UP |
+            TIM_ONEPULSEMODE_REPETITIVE;
+
+        TIM_configTimeBase(htim, 200-1, 100-1, cfg);
+        TIM_clearFlags(htim, TIM_FLAG_UPDATE);
+        TIM_enableInterruptSource(htim, TIM_IT_UPDATE);
+    }
+    else
+    {
+        TIM_disableInterruptSource(htim, TIM_IT_UPDATE);
+        TIM_disableCounter(htim, TIM_FLAG_UPDATE);
+    }
 }
 
 static rt_err_t ns800_clock_timer_start(rt_clock_timer_t *timer, rt_uint32_t cnt, rt_clock_timer_mode_t mode)
@@ -146,7 +155,6 @@ static rt_err_t ns800_clock_timer_control(rt_clock_timer_t *timer, rt_uint32_t c
         case CLOCK_TIMER_CTRL_FREQ_SET:
             freq = *(rt_uint32_t *)args;
             __set_timerx_freq(timer, freq);
-             break;
             break;
         case CLOCK_TIMER_CTRL_INFO_GET:
             *(struct rt_clock_timer_info*)args = *timer->info;

bsp/novosns/ns800/ns800rt7p65-nssinepad/board/Kconfig

@@ -105,23 +105,23 @@ menu "On-chip Peripheral Drivers"
 
         endif
 
-	menuconfig BSP_USING_TIM
-		bool "Enable TIM (Hardware Timer)"
-		select RT_USING_CLOCK_TIME
-		default n
-		help
-			Enable hardware timer TIM driver
-
-	if BSP_USING_TIM
-    menu "TIM Timer Configuration"
-        config BSP_USING_TIM1
-            bool "Enable TIM1"
-            default n
-        config BSP_USING_TIM2
-            bool "Enable TIM2"
-            default n
-    endmenu
-
-	endif
+    menuconfig BSP_USING_TIM
+        bool "Enable TIM (Hardware Timer)"
+        select RT_USING_CLOCK_TIME
+        default n
+        help
+            Enable hardware timer TIM driver
+
+    if BSP_USING_TIM
+        menu "TIM Timer Configuration"
+            config BSP_USING_TIM1
+                bool "Enable TIM1"
+                default n
+            config BSP_USING_TIM2
+                bool "Enable TIM2"
+                default n
+        endmenu
+
+    endif
 
 endmenu