(nuttx) 03/06: board: There is no need to use sched_[un]lock

[archer]

This is an automated email from the ASF dual-hosted git repository.

archer pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/nuttx.git

commit a4c9268a400c78e585946d3e61372ad6e28f8201
Author: hujun5 <huj...@xiaomi.com>
AuthorDate: Mon Dec 11 16:24:17 2023 +0800

    board: There is no need to use sched_[un]lock
    
    Signed-off-by: hujun5 <huj...@xiaomi.com>
---
 boards/arm/cxd56xx/drivers/audio/cxd56_audio_dma.c | 3 ---
 boards/arm/sam34/sam4s-xplained-pro/src/sam_wdt.c  | 3 ---
 boards/sparc/bm3803/xx3803/src/bm3803_wdt.c        | 4 ----
 boards/sparc/s698pm/s698pm-dkit/src/s698pm_wdt.c   | 4 ----
 4 files changed, 14 deletions(-)

diff --git a/boards/arm/cxd56xx/drivers/audio/cxd56_audio_dma.c 
b/boards/arm/cxd56xx/drivers/audio/cxd56_audio_dma.c
index 2fde4a536c..d7722d1ba2 100644
--- a/boards/arm/cxd56xx/drivers/audio/cxd56_audio_dma.c
+++ b/boards/arm/cxd56xx/drivers/audio/cxd56_audio_dma.c
@@ -241,7 +241,6 @@ static CXD56_AUDIO_ECODE exec_dma_ch_sync_workaround(
       /* Lock interrupt */
 
       up_irq_disable();
-      sched_lock();
 
       /* Wait smp interrupt. */
 
@@ -255,7 +254,6 @@ static CXD56_AUDIO_ECODE exec_dma_ch_sync_workaround(
 
       if (timeout_cnt == DMA_TIMEOUT_CNT)
         {
-          sched_unlock();
           up_irq_enable();
           return CXD56_AUDIO_ECODE_DMA_SMP_TIMEOUT;
         }
@@ -270,7 +268,6 @@ static CXD56_AUDIO_ECODE exec_dma_ch_sync_workaround(
 
       /* Unlock interrupt */
 
-      sched_unlock();
       up_irq_enable();
 
       /* Wait for 1sample tramsfer. */
diff --git a/boards/arm/sam34/sam4s-xplained-pro/src/sam_wdt.c 
b/boards/arm/sam34/sam4s-xplained-pro/src/sam_wdt.c
index 3c45f823ed..f6a186833e 100644
--- a/boards/arm/sam34/sam4s-xplained-pro/src/sam_wdt.c
+++ b/boards/arm/sam34/sam4s-xplained-pro/src/sam_wdt.c
@@ -183,8 +183,6 @@ int sam_watchdog_initialize(void)
   /* Start Kicker task */
 
 #if defined(CONFIG_WDT_THREAD)
-  sched_lock();
-
   int taskid = kthread_create(CONFIG_WDT_THREAD_NAME,
                               CONFIG_WDT_THREAD_PRIORITY,
                               CONFIG_WDT_THREAD_STACKSIZE,
@@ -193,7 +191,6 @@ int sam_watchdog_initialize(void)
   DEBUGASSERT(taskid > 0);
   UNUSED(taskid);
 
-  sched_unlock();
 #endif
   return OK;
 errout_with_dev:
diff --git a/boards/sparc/bm3803/xx3803/src/bm3803_wdt.c 
b/boards/sparc/bm3803/xx3803/src/bm3803_wdt.c
index 39bf3c32ee..efd279aa45 100644
--- a/boards/sparc/bm3803/xx3803/src/bm3803_wdt.c
+++ b/boards/sparc/bm3803/xx3803/src/bm3803_wdt.c
@@ -142,8 +142,6 @@ int xx3803_watchdog_initialize(void)
 
 #if defined(CONFIG_XX3803_WDG_THREAD)
 
-  sched_lock();
-
   /* Spawn wdog daemon thread */
 
   int taskid = kthread_create(CONFIG_XX3803_WDG_THREAD_NAME,
@@ -154,8 +152,6 @@ int xx3803_watchdog_initialize(void)
   DEBUGASSERT(taskid > 0);
   UNUSED(taskid);
 
-  sched_unlock();
-
 #endif /* CONFIG_XX3803_WDG_THREAD */
 
   return OK;
diff --git a/boards/sparc/s698pm/s698pm-dkit/src/s698pm_wdt.c 
b/boards/sparc/s698pm/s698pm-dkit/src/s698pm_wdt.c
index b6f3536582..b5ec0a3dc7 100644
--- a/boards/sparc/s698pm/s698pm-dkit/src/s698pm_wdt.c
+++ b/boards/sparc/s698pm/s698pm-dkit/src/s698pm_wdt.c
@@ -142,8 +142,6 @@ int s698pm_dkit_watchdog_initialize(void)
 
 #if defined(CONFIG_S698PM_DKIT_WDG_THREAD)
 
-  sched_lock();
-
   /* Spawn wdog daemon thread */
 
   int taskid = kthread_create(CONFIG_S698PM_DKIT_WDG_THREAD_NAME,
@@ -154,8 +152,6 @@ int s698pm_dkit_watchdog_initialize(void)
   DEBUGASSERT(taskid > 0);
   UNUSED(taskid);
 
-  sched_unlock();
-
 #endif /* CONFIG_S698PM_DKIT_WDG_THREAD */
 
   return OK;