apps: Remove the check of CONFIG_SERIAL_TERMIOS

If the code only change c_oflag, c_iflag and c_lflag, not c_cflag in termios.
Follow up the change from kernel: apache/nuttx#8843

Signed-off-by: Xiang Xiao <[email protected]>

Author: xiaoxiang781216

examples/modbus/modbus_main.c

@@ -49,7 +49,6 @@
 #include <string.h>
 #include <unistd.h>
 #include <pthread.h>
-#include <termios.h>
 #include <signal.h>
 #include <errno.h>
 
@@ -67,11 +66,8 @@
 /****************************************************************************
  * Pre-processor Definitions
  ****************************************************************************/
-/* Configuration ************************************************************/
 
-#ifndef CONFIG_SERIAL_TERMIOS
-#  error "CONFIG_SERIAL_TERMIOS is needed by modbus example"
-#endif
+/* Configuration ************************************************************/
 
 #ifndef CONFIG_EXAMPLES_MODBUS_PORT
 #  define CONFIG_EXAMPLES_MODBUS_PORT 0

examples/modbusmaster/mbmaster_main.c

@@ -50,10 +50,6 @@
 
 /* modbus master port */
 
-#ifndef CONFIG_SERIAL_TERMIOS
-#  error "CONFIG_SERIAL_TERMIOS is needed by modbus example"
-#endif
-
 #ifdef CONFIG_EXAMPLES_MODBUSMASTER_PORT
 #  define MBMASTER_PORT CONFIG_EXAMPLES_MODBUSMASTER_PORT
 #else

examples/pty_test/pty_test.c

@@ -330,7 +330,6 @@ int main(int argc, FAR char *argv[])
 #endif
     }
 
-#ifdef CONFIG_SERIAL_TERMIOS
   /* Enable \n -> \r\n conversion during write */
 
   ret = tcgetattr(termpair.fd_uart, &tio);
@@ -347,7 +346,6 @@ int main(int argc, FAR char *argv[])
       fprintf(stderr, "ERROR: tcsetattr() failed: %d\n", errno);
       goto error_serial;
     }
-#endif
 
   printf("Starting a new NSH Session using %s\n", buffer);
 

interpreters/bas/bas_fs.c

@@ -388,11 +388,7 @@ int FS_opendev(int chn, int infd, int outfd)
 
   g_file[chn] = malloc(sizeof(struct FileStream));
   g_file[chn]->dev = 1;
-#ifdef CONFIG_SERIAL_TERMIOS
   g_file[chn]->tty = (infd == 0 ? isatty(infd) && isatty(outfd) : 0);
-#else
-  g_file[chn]->tty = 1;
-#endif
   g_file[chn]->recLength = 1;
   g_file[chn]->infd = infd;
   g_file[chn]->inSize = 0;
@@ -781,12 +777,10 @@ int FS_close(int dev)
   return 0;
 }
 
-#ifdef CONFIG_SERIAL_TERMIOS
 int FS_istty(int chn)
 {
   return (g_file[chn] && g_file[chn]->tty);
 }
-#endif
 
 int FS_lock(int chn, off_t offset, off_t length, int mode, int w)
 {

interpreters/bas/bas_fs.h

@@ -112,13 +112,7 @@ int FS_openbinaryChn(int chn, const char *name, int mode);
 int FS_freechn(void);
 int FS_flush(int dev);
 int FS_close(int dev);
-
-#ifdef CONFIG_SERIAL_TERMIOS
 int FS_istty(int chn);
-#else
-#  define FS_istty(chn) (1)
-#endif
-
 int FS_lock(int chn, off_t offset, off_t length, int mode, int w);
 int FS_truncate(int chn);
 void FS_shellmode(int chn);

interpreters/bas/bas_token.c

@@ -4163,11 +4163,7 @@ static void yy_init_buffer  (YY_BUFFER_STATE  b, FILE * file)
       b->yy_bs_column = 0;
     }
 
-#ifdef CONFIG_SERIAL_TERMIOS
   b->yy_is_interactive = file ? (isatty(fileno(file)) > 0) : 0;
-#else
-  b->yy_is_interactive = 1;
-#endif
 
   errno = oerrno;
 }

logging/embedlog/Makefile

@@ -130,12 +130,6 @@ else
 	CFLAGS += -DENABLE_PTHREAD=0
 endif
 
-ifeq ($(CONFIG_SERIAL_TERMIOS),y)
-	CFLAGS += -DHAVE_TERMIOS_H
-else
-	CFLAGS += -DHAVE_TERMIOS_H=0
-endif
-
 CFLAGS += -DEL_LOG_MAX=$(CONFIG_EMBEDLOG_LOG_MAX)
 CFLAGS += -DEL_MEM_LINE_SIZE=$(CONFIG_EMBEDLOG_MEM_LINE_SIZE)
 
@@ -151,6 +145,7 @@ CFLAGS += -DHAVE_ACCESS=0
 CFLAGS += -DENABLE_REENTRANT
 CFLAGS += -DENABLE_OUT_SYSLOG
 CFLAGS += -DHAVE_STAT
+CFLAGS += -DHAVE_TERMIOS_H
 CFLAGS += -DHAVE_UNISTD_H
 CFLAGS += -DHAVE_FSYNC
 CFLAGS += -DHAVE_FILENO

logging/nxscope/Kconfig

@@ -11,7 +11,6 @@ if LOGGING_NXSCOPE
 
 config LOGGING_NXSCOPE_INTF_SERIAL
 	bool "NxScope serial port interface support"
-	select SERIAL_TERMIOS
 	default n
 	---help---
 		For details, see logging/nxscope/nxscope_iserial.c

logging/nxscope/nxscope_iser.c

@@ -33,14 +33,6 @@
 
 #include <logging/nxscope/nxscope.h>
 
-/****************************************************************************
- * Pre-processor Definitions
- ****************************************************************************/
-
-#ifndef CONFIG_SERIAL_TERMIOS
-#  error Termios support must be enabled
-#endif
-
 /****************************************************************************
  * Private Type Definition
  ****************************************************************************/
@@ -214,6 +206,7 @@ int nxscope_ser_init(FAR struct nxscope_intf_s *intf,
 
   tcgetattr(priv->fd, &tio);
 
+#ifdef CONFIG_SERIAL_TERMIOS
   /* Configure a baud rate */
 
   DEBUGASSERT(priv->cfg->baud > 0);
@@ -224,6 +217,7 @@ int nxscope_ser_init(FAR struct nxscope_intf_s *intf,
       _err("ERROR: failed to set baud rate %d\n", errno);
       goto errout;
     }
+#endif
 
   /* Configure RAW mode */
 

modbus/README.md

@@ -110,9 +110,8 @@ The NuttX-named configuration options that are available include:
 See also other serial settings, in particular:
 
 - `CONFIG_SERIAL_TERMIOS` – Serial driver supports `termios.h` interfaces
-  (`tcsetattr`, `tcflush`, etc.). If this is not defined, then the terminal
-  settings (baud, parity, etc.) are not configurable at runtime; serial streams
-  will not be flushed when closed.
+  If this is not defined, then the terminal settings (baud, parity, etc.)
+  are not configurable at runtime.
 
 ## Note