Compile with -pedantic.

[libserialport.git] / linux_termios.c

diff --git a/linux_termios.c b/linux_termios.c
index 874889aba4641c7ed268923f1eeb32d4908b5d47..7c1540a2ddb4e0450a4d1e4c5b66b985a263fb6c 100644 (file)
--- a/linux_termios.c
+++ b/linux_termios.c
@@ -17,10 +17,26 @@
  * along with this program.  If not, see <http://www.gnu.org/licenses/>.
  */
 
+/*
+ * At the time of writing, glibc does not support the Linux kernel interfaces
+ * for setting non-standard baud rates and flow control. We therefore have to
+ * prepare the correct ioctls ourselves, for which we need the declarations in
+ * linux/termios.h.
+ *
+ * We can't include linux/termios.h in serialport.c however, because its
+ * contents conflict with the termios.h provided by glibc. So this file exists
+ * to isolate the bits of code which use the kernel termios declarations.
+ *
+ * The details vary by architecture. Some architectures have c_ispeed/c_ospeed
+ * in struct termios, accessed with TCSETS/TCGETS. Others have these fields in
+ * struct termios2, accessed with TCSETS2/TCGETS2. Some architectures have the
+ * TCSETX/TCGETX ioctls used with struct termiox, others do not.
+ */
+
 #include <linux/termios.h>
 #include "linux_termios.h"
 
-int get_termios_get_ioctl(void)
+SP_PRIV int get_termios_get_ioctl(void)
 {
 #ifdef HAVE_TERMIOS2
        return TCGETS2;
@@ -29,7 +45,7 @@ int get_termios_get_ioctl(void)
 #endif
 }
 
-int get_termios_set_ioctl(void)
+SP_PRIV int get_termios_set_ioctl(void)
 {
 #ifdef HAVE_TERMIOS2
        return TCSETS2;
@@ -38,7 +54,7 @@ int get_termios_set_ioctl(void)
 #endif
 }
 
-int get_termios_size(void)
+SP_PRIV int get_termios_size(void)
 {
 #ifdef HAVE_TERMIOS2
        return sizeof(struct termios2);
@@ -47,7 +63,8 @@ int get_termios_size(void)
 #endif
 }
 
-int get_termios_speed(void *data)
+#if defined(HAVE_TERMIOS_SPEED) || defined(HAVE_TERMIOS2_SPEED)
+SP_PRIV int get_termios_speed(void *data)
 {
 #ifdef HAVE_TERMIOS2
        struct termios2 *term = (struct termios2 *) data;
@@ -60,7 +77,7 @@ int get_termios_speed(void *data)
                return term->c_ispeed;
 }
 
-void set_termios_speed(void *data, int speed)
+SP_PRIV void set_termios_speed(void *data, int speed)
 {
 #ifdef HAVE_TERMIOS2
        struct termios2 *term = (struct termios2 *) data;
@@ -71,43 +88,40 @@ void set_termios_speed(void *data, int speed)
        term->c_cflag |= BOTHER;
        term->c_ispeed = term->c_ospeed = speed;
 }
+#endif
 
 #ifdef HAVE_TERMIOX
-int get_termiox_size(void)
+SP_PRIV int get_termiox_size(void)
 {
        return sizeof(struct termiox);
 }
 
-int get_termiox_flow(void *data)
+SP_PRIV int get_termiox_flow(void *data, int *rts, int *cts, int *dtr, int *dsr)
 {
        struct termiox *termx = (struct termiox *) data;
-
        int flags = 0;
-       if (termx->x_cflag & RTSXOFF)
-               flags |= RTS_FLOW;
-       if (termx->x_cflag & CTSXON)
-               flags |= CTS_FLOW;
-       if (termx->x_cflag & DTRXOFF)
-               flags |= DTR_FLOW;
-       if (termx->x_cflag & DSRXON)
-               flags |= DSR_FLOW;
+
+       *rts = (termx->x_cflag & RTSXOFF);
+       *cts = (termx->x_cflag & CTSXON);
+       *dtr = (termx->x_cflag & DTRXOFF);
+       *dsr = (termx->x_cflag & DSRXON);
 
        return flags;
 }
 
-void set_termiox_flow(void *data, int flags)
+SP_PRIV void set_termiox_flow(void *data, int rts, int cts, int dtr, int dsr)
 {
        struct termiox *termx = (struct termiox *) data;
 
        termx->x_cflag &= ~(RTSXOFF | CTSXON | DTRXOFF | DSRXON);
 
-       if (flags & RTS_FLOW)
+       if (rts)
                termx->x_cflag |= RTSXOFF;
-       if (flags & CTS_FLOW)
+       if (cts)
                termx->x_cflag |= CTSXON;
-       if (flags & DTR_FLOW)
+       if (dtr)
                termx->x_cflag |= DTRXOFF;
-       if (flags & DSR_FLOW)
+       if (dsr)
                termx->x_cflag |= DSRXON;
 }
 #endif