You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@nuttx.apache.org by li...@apache.org on 2020/09/07 03:54:22 UTC

[incubator-nuttx] 01/02: Move note driver from drivers/syslog to drivers/note

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

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

commit f99719e2605921c714130fc38dc891cf97b9c7b2
Author: Xiang Xiao <xi...@xiaomi.com>
AuthorDate: Thu Aug 6 01:43:13 2020 +0800

    Move note driver from drivers/syslog to drivers/note
    
    it's better to put the note transport layer into a common folder
    
    Signed-off-by: Xiang Xiao <xi...@xiaomi.com>
---
 arch/arm/src/common/arm_initialize.c         |  2 +-
 arch/avr/src/common/up_initialize.c          |  2 +-
 arch/hc/src/common/up_initialize.c           |  2 +-
 arch/mips/src/common/mips_initialize.c       |  2 +-
 arch/or1k/src/common/up_initialize.c         |  2 +-
 arch/renesas/src/common/up_initialize.c      |  2 +-
 arch/risc-v/src/common/riscv_initialize.c    |  2 +-
 arch/sim/src/sim/up_initialize.c             |  2 +-
 arch/x86/src/common/up_initialize.c          |  2 +-
 arch/x86_64/src/common/up_initialize.c       |  2 +-
 arch/xtensa/src/common/xtensa_initialize.c   |  2 +-
 arch/z16/src/common/z16_initialize.c         |  2 +-
 arch/z80/src/common/z80_initialize.c         |  2 +-
 drivers/Kconfig                              |  1 +
 drivers/Makefile                             |  1 +
 drivers/note/Kconfig                         | 17 +++++++++++++++++
 drivers/note/Make.defs                       | 26 ++++++++++++++++++++++++++
 drivers/{syslog => note}/note_driver.c       |  2 +-
 drivers/syslog/Kconfig                       |  9 ---------
 drivers/syslog/Make.defs                     |  7 -------
 include/nuttx/{syslog => note}/note_driver.h |  8 ++++----
 21 files changed, 63 insertions(+), 34 deletions(-)

diff --git a/arch/arm/src/common/arm_initialize.c b/arch/arm/src/common/arm_initialize.c
index 95b8b92..e73e3e9 100644
--- a/arch/arm/src/common/arm_initialize.c
+++ b/arch/arm/src/common/arm_initialize.c
@@ -33,7 +33,7 @@
 #include <nuttx/net/loopback.h>
 #include <nuttx/net/tun.h>
 #include <nuttx/net/telnet.h>
-#include <nuttx/syslog/note_driver.h>
+#include <nuttx/note/note_driver.h>
 #include <nuttx/syslog/syslog_console.h>
 #include <nuttx/serial/pty.h>
 #include <nuttx/crypto/crypto.h>
diff --git a/arch/avr/src/common/up_initialize.c b/arch/avr/src/common/up_initialize.c
index 99c82cc..e3dd482 100644
--- a/arch/avr/src/common/up_initialize.c
+++ b/arch/avr/src/common/up_initialize.c
@@ -48,7 +48,7 @@
 #include <nuttx/net/loopback.h>
 #include <nuttx/net/tun.h>
 #include <nuttx/net/telnet.h>
-#include <nuttx/syslog/note_driver.h>
+#include <nuttx/note/note_driver.h>
 #include <nuttx/syslog/syslog_console.h>
 #include <nuttx/serial/pty.h>
 #include <nuttx/crypto/crypto.h>
diff --git a/arch/hc/src/common/up_initialize.c b/arch/hc/src/common/up_initialize.c
index 2e2ceb4..0343ad2 100644
--- a/arch/hc/src/common/up_initialize.c
+++ b/arch/hc/src/common/up_initialize.c
@@ -49,7 +49,7 @@
 #include <nuttx/net/loopback.h>
 #include <nuttx/net/tun.h>
 #include <nuttx/net/telnet.h>
-#include <nuttx/syslog/note_driver.h>
+#include <nuttx/note/note_driver.h>
 #include <nuttx/syslog/syslog_console.h>
 #include <nuttx/serial/pty.h>
 #include <nuttx/crypto/crypto.h>
diff --git a/arch/mips/src/common/mips_initialize.c b/arch/mips/src/common/mips_initialize.c
index 087e554..60d8001 100644
--- a/arch/mips/src/common/mips_initialize.c
+++ b/arch/mips/src/common/mips_initialize.c
@@ -48,7 +48,7 @@
 #include <nuttx/net/loopback.h>
 #include <nuttx/net/tun.h>
 #include <nuttx/net/telnet.h>
-#include <nuttx/syslog/note_driver.h>
+#include <nuttx/note/note_driver.h>
 #include <nuttx/syslog/syslog_console.h>
 #include <nuttx/serial/pty.h>
 #include <nuttx/crypto/crypto.h>
diff --git a/arch/or1k/src/common/up_initialize.c b/arch/or1k/src/common/up_initialize.c
index 78ae0e4..8547b4d 100644
--- a/arch/or1k/src/common/up_initialize.c
+++ b/arch/or1k/src/common/up_initialize.c
@@ -48,7 +48,7 @@
 #include <nuttx/net/loopback.h>
 #include <nuttx/net/tun.h>
 #include <nuttx/net/telnet.h>
-#include <nuttx/syslog/note_driver.h>
+#include <nuttx/note/note_driver.h>
 #include <nuttx/syslog/syslog_console.h>
 #include <nuttx/serial/pty.h>
 #include <nuttx/crypto/crypto.h>
diff --git a/arch/renesas/src/common/up_initialize.c b/arch/renesas/src/common/up_initialize.c
index af4c66d..eb5b57f 100644
--- a/arch/renesas/src/common/up_initialize.c
+++ b/arch/renesas/src/common/up_initialize.c
@@ -49,7 +49,7 @@
 #include <nuttx/net/loopback.h>
 #include <nuttx/net/tun.h>
 #include <nuttx/net/telnet.h>
-#include <nuttx/syslog/note_driver.h>
+#include <nuttx/note/note_driver.h>
 #include <nuttx/syslog/syslog_console.h>
 #include <nuttx/serial/pty.h>
 #include <nuttx/crypto/crypto.h>
diff --git a/arch/risc-v/src/common/riscv_initialize.c b/arch/risc-v/src/common/riscv_initialize.c
index 0028036..aefee0c 100644
--- a/arch/risc-v/src/common/riscv_initialize.c
+++ b/arch/risc-v/src/common/riscv_initialize.c
@@ -45,7 +45,7 @@
 #include <nuttx/arch.h>
 #include <nuttx/board.h>
 #include <nuttx/serial/pty.h>
-#include <nuttx/syslog/note_driver.h>
+#include <nuttx/note/note_driver.h>
 #include <nuttx/syslog/syslog_console.h>
 #include <nuttx/drivers/drivers.h>
 
diff --git a/arch/sim/src/sim/up_initialize.c b/arch/sim/src/sim/up_initialize.c
index a512ce1..c226bff 100644
--- a/arch/sim/src/sim/up_initialize.c
+++ b/arch/sim/src/sim/up_initialize.c
@@ -49,7 +49,7 @@
 #include <nuttx/net/tun.h>
 #include <nuttx/net/telnet.h>
 #include <nuttx/mtd/mtd.h>
-#include <nuttx/syslog/note_driver.h>
+#include <nuttx/note/note_driver.h>
 #include <nuttx/syslog/syslog_console.h>
 #include <nuttx/serial/pty.h>
 #include <nuttx/crypto/crypto.h>
diff --git a/arch/x86/src/common/up_initialize.c b/arch/x86/src/common/up_initialize.c
index c1de39f7..0c2ace6 100644
--- a/arch/x86/src/common/up_initialize.c
+++ b/arch/x86/src/common/up_initialize.c
@@ -48,7 +48,7 @@
 #include <nuttx/net/loopback.h>
 #include <nuttx/net/tun.h>
 #include <nuttx/net/telnet.h>
-#include <nuttx/syslog/note_driver.h>
+#include <nuttx/note/note_driver.h>
 #include <nuttx/syslog/syslog_console.h>
 #include <nuttx/serial/pty.h>
 #include <nuttx/crypto/crypto.h>
diff --git a/arch/x86_64/src/common/up_initialize.c b/arch/x86_64/src/common/up_initialize.c
index e4968b6..6e0759c 100644
--- a/arch/x86_64/src/common/up_initialize.c
+++ b/arch/x86_64/src/common/up_initialize.c
@@ -35,7 +35,7 @@
 #include <nuttx/net/tun.h>
 #include <nuttx/net/telnet.h>
 #include <nuttx/syslog/syslog.h>
-#include <nuttx/syslog/note_driver.h>
+#include <nuttx/note/note_driver.h>
 #include <nuttx/syslog/syslog_console.h>
 #include <nuttx/serial/pty.h>
 #include <nuttx/crypto/crypto.h>
diff --git a/arch/xtensa/src/common/xtensa_initialize.c b/arch/xtensa/src/common/xtensa_initialize.c
index bfac801..edf474d 100644
--- a/arch/xtensa/src/common/xtensa_initialize.c
+++ b/arch/xtensa/src/common/xtensa_initialize.c
@@ -48,7 +48,7 @@
 #include <nuttx/net/loopback.h>
 #include <nuttx/net/tun.h>
 #include <nuttx/net/telnet.h>
-#include <nuttx/syslog/note_driver.h>
+#include <nuttx/note/note_driver.h>
 #include <nuttx/syslog/syslog_console.h>
 #include <nuttx/serial/pty.h>
 #include <nuttx/crypto/crypto.h>
diff --git a/arch/z16/src/common/z16_initialize.c b/arch/z16/src/common/z16_initialize.c
index 2e6a0e1..08406f4 100644
--- a/arch/z16/src/common/z16_initialize.c
+++ b/arch/z16/src/common/z16_initialize.c
@@ -33,7 +33,7 @@
 #include <nuttx/net/loopback.h>
 #include <nuttx/net/tun.h>
 #include <nuttx/net/telnet.h>
-#include <nuttx/syslog/note_driver.h>
+#include <nuttx/note/note_driver.h>
 #include <nuttx/syslog/syslog_console.h>
 #include <nuttx/serial/pty.h>
 #include <nuttx/crypto/crypto.h>
diff --git a/arch/z80/src/common/z80_initialize.c b/arch/z80/src/common/z80_initialize.c
index 692bde1..ba560d9 100644
--- a/arch/z80/src/common/z80_initialize.c
+++ b/arch/z80/src/common/z80_initialize.c
@@ -34,7 +34,7 @@
 #include <nuttx/net/loopback.h>
 #include <nuttx/net/telnet.h>
 #include <nuttx/net/tun.h>
-#include <nuttx/syslog/note_driver.h>
+#include <nuttx/note/note_driver.h>
 #include <nuttx/syslog/syslog_console.h>
 #include <nuttx/serial/pty.h>
 #include <nuttx/crypto/crypto.h>
diff --git a/drivers/Kconfig b/drivers/Kconfig
index 5ddf385..f5a057d 100644
--- a/drivers/Kconfig
+++ b/drivers/Kconfig
@@ -96,6 +96,7 @@ source drivers/modem/Kconfig
 source drivers/mtd/Kconfig
 source drivers/eeprom/Kconfig
 source drivers/net/Kconfig
+source drivers/note/Kconfig
 source drivers/pipes/Kconfig
 source drivers/power/Kconfig
 source drivers/rptun/Kconfig
diff --git a/drivers/Makefile b/drivers/Makefile
index dd695b8..15fdaa2 100644
--- a/drivers/Makefile
+++ b/drivers/Makefile
@@ -56,6 +56,7 @@ include modem/Make.defs
 include mtd/Make.defs
 include eeprom/Make.defs
 include net/Make.defs
+include note/Make.defs
 include pipes/Make.defs
 include power/Make.defs
 include rptun/Make.defs
diff --git a/drivers/note/Kconfig b/drivers/note/Kconfig
new file mode 100644
index 0000000..6865682
--- /dev/null
+++ b/drivers/note/Kconfig
@@ -0,0 +1,17 @@
+#
+# For a description of the syntax of this configuration file,
+# see the file kconfig-language.txt in the NuttX tools repository.
+#
+
+menu "Note Driver Support"
+
+config DRIVER_NOTE
+	bool "Scheduler instrumentation driver"
+	default n
+	depends on SCHED_INSTRUMENTATION_BUFFER && SCHED_NOTE_GET
+	---help---
+		Enable building a serial driver that can be used by an application
+		to read data from the in-memory, scheduler instrumentation "note"
+		buffer.
+
+endmenu
diff --git a/drivers/note/Make.defs b/drivers/note/Make.defs
new file mode 100644
index 0000000..91c0dc3
--- /dev/null
+++ b/drivers/note/Make.defs
@@ -0,0 +1,26 @@
+############################################################################
+# drivers/note/Make.defs
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# contributor license agreements.  See the NOTICE file distributed with
+# this work for additional information regarding copyright ownership.  The
+# ASF licenses this file to you under the Apache License, Version 2.0 (the
+# "License"); you may not use this file except in compliance with the
+# License.  You may obtain a copy of the License at
+#
+#   http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
+# WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.  See the
+# License for the specific language governing permissions and limitations
+# under the License.
+#
+############################################################################
+
+ifeq ($(CONFIG_DRIVER_NOTE),y)
+  CSRCS += note_driver.c
+endif
+
+DEPPATH += --dep-path note
+VPATH += :note
diff --git a/drivers/syslog/note_driver.c b/drivers/note/note_driver.c
similarity index 99%
rename from drivers/syslog/note_driver.c
rename to drivers/note/note_driver.c
index bade29a..afd37c2 100644
--- a/drivers/syslog/note_driver.c
+++ b/drivers/note/note_driver.c
@@ -1,5 +1,5 @@
 /****************************************************************************
- * drivers/syslog/note_driver.c
+ * drivers/note/note_driver.c
  *
  *   Copyright (C) 2016 Gregory Nutt. All rights reserved.
  *   Author: Gregory Nutt <gn...@nuttx.org>
diff --git a/drivers/syslog/Kconfig b/drivers/syslog/Kconfig
index b502261..b79fdd3 100644
--- a/drivers/syslog/Kconfig
+++ b/drivers/syslog/Kconfig
@@ -68,15 +68,6 @@ config RAMLOG_OVERWRITE
 
 endif
 
-config DRIVER_NOTE
-	bool "Scheduler instrumentation driver"
-	default n
-	depends on SCHED_INSTRUMENTATION_BUFFER && SCHED_NOTE_GET
-	---help---
-		Enable building a serial driver that can be used by an application
-		to read data from the in-memory, scheduler instrumentation "note"
-		buffer.
-
 config SYSLOG_BUFFER
 	bool "Use buffered output"
 	default n
diff --git a/drivers/syslog/Make.defs b/drivers/syslog/Make.defs
index c34a3f2..d4678c0 100644
--- a/drivers/syslog/Make.defs
+++ b/drivers/syslog/Make.defs
@@ -48,13 +48,6 @@ ifneq ($(CONFIG_ARCH_SYSLOG),y)
   CSRCS += syslog_initialize.c
 endif
 
-# The note driver is hosted in this directory, but is not associated with
-# SYSLOGging
-
-ifeq ($(CONFIG_DRIVER_NOTE),y)
-  CSRCS += note_driver.c
-endif
-
 # The RAMLOG device is usable as a system logging device or standalone
 
 ifeq ($(CONFIG_RAMLOG),y)
diff --git a/include/nuttx/syslog/note_driver.h b/include/nuttx/note/note_driver.h
similarity index 93%
rename from include/nuttx/syslog/note_driver.h
rename to include/nuttx/note/note_driver.h
index 49b3075..62df063 100644
--- a/include/nuttx/syslog/note_driver.h
+++ b/include/nuttx/note/note_driver.h
@@ -1,5 +1,5 @@
 /****************************************************************************
- * include/nuttx/syslog/note_driver.h
+ * include/nuttx/note/note_driver.h
  *
  * Licensed to the Apache Software Foundation (ASF) under one or more
  * contributor license agreements.  See the NOTICE file distributed with
@@ -18,8 +18,8 @@
  *
  ****************************************************************************/
 
-#ifndef __INCLUDE_NUTTX_SYSLOG_NOTE_DRIVER_H
-#define __INCLUDE_NUTTX_SYSLOG_NOTE_DRIVER_H
+#ifndef __INCLUDE_NUTTX_NOTE_NOTE_DRIVER_H
+#define __INCLUDE_NUTTX_NOTE_NOTE_DRIVER_H
 
 /****************************************************************************
  * Included Files
@@ -66,4 +66,4 @@ int note_register(void);
 
 #endif /* CONFIG_SCHED_INSTRUMENTATION */
 
-#endif /* __INCLUDE_NUTTX_SYSLOG_NOTE_DRIVER_H */
+#endif /* __INCLUDE_NUTTX_NOTE_NOTE_DRIVER_H */