From a841be9b9c20e8998bdbc635080b22d6e55e2822 Mon Sep 17 00:00:00 2001 From: Xiang Xiao Date: Mon, 20 Feb 2023 17:40:20 +0800 Subject: [PATCH] drivers/note: Remove the warning of preemption and critical sections since both aren't suitable after: commit 98ab55ef6858170bccd151c56a7fd28a0a28860b Author: Xiang Xiao Date: Fri Dec 30 12:02:47 2022 +0800 drivers/note: Replace the scritical section with spin_xxx_wo_note to avoid to generate the unexpected schedule information Signed-off-by: Xiang Xiao --- drivers/note/notelog_driver.c | 14 -------------- 1 file changed, 14 deletions(-) diff --git a/drivers/note/notelog_driver.c b/drivers/note/notelog_driver.c index 01ec3d6e44..79798f8d48 100644 --- a/drivers/note/notelog_driver.c +++ b/drivers/note/notelog_driver.c @@ -308,13 +308,6 @@ static void notelog_cpu_resumed(FAR struct note_driver_s *drv, #endif #ifdef CONFIG_SCHED_INSTRUMENTATION_PREEMPTION -/* This does not work well... it interferes with the operation of the - * simulated /dev/console device which, of course, does disable preemption - * and does use critical sections. - */ - -#warning CONFIG_SCHED_INSTRUMENTATION_PREEMPTION is a bad idea - static void notelog_premption(FAR struct note_driver_s *drv, FAR struct tcb_s *tcb, bool locked) { @@ -339,13 +332,6 @@ static void notelog_premption(FAR struct note_driver_s *drv, #endif #ifdef CONFIG_SCHED_INSTRUMENTATION_CSECTION -/* This does not work well... it interferes with the operation of the - * simulated /dev/console device which, of course, does disable preemption - * and does use critical sections. - */ - -#warning CONFIG_SCHED_INSTRUMENTATION_CSECTION is a bad idea - static void notelog_csection(FAR struct note_driver_s *drv, FAR struct tcb_s *tcb, bool enter) {