X-Git-Url: https://sigrok.org/gitweb/?a=blobdiff_plain;f=hardware%2Fcommon%2Fserial.c;h=46d4ad0535d8c4d3b6cb65509216a800a51111cf;hb=1483577eed7ac27b8107b1b76e1adc5369745c29;hp=cebca29e418dd833007c37572a747d2c84d85cbc;hpb=e006af1116adb76014e77db718bb8673137a8bf5;p=libsigrok.git

diff --git a/hardware/common/serial.c b/hardware/common/serial.c
index cebca29e..46d4ad05 100644
--- a/hardware/common/serial.c
+++ b/hardware/common/serial.c
@@ -31,13 +31,14 @@
 #include <stdlib.h>
 #include <glib.h>
 #include <sigrok.h>
+#include <sigrok-internal.h>
 
 // FIXME: Must be moved, or rather passed as function argument.
 #ifdef _WIN32
 HANDLE hdl;
 #endif
 
-char *serial_port_glob[] = {
+const char *serial_port_glob[] = {
 	/* Linux */
 	"/dev/ttyS*",
 	"/dev/ttyUSB*",
@@ -51,11 +52,14 @@ char *serial_port_glob[] = {
 
 GSList *list_serial_ports(void)
 {
+	GSList *ports;
+
 #ifdef _WIN32
 	/* TODO */
+	ports = NULL;
+	ports = g_slist_append(ports, strdup("COM1"));
 #else
 	glob_t g;
-	GSList *ports;
 	unsigned int i, j;
 
 	ports = NULL;
@@ -66,9 +70,9 @@ GSList *list_serial_ports(void)
 			ports = g_slist_append(ports, strdup(g.gl_pathv[j]));
 		globfree(&g);
 	}
+#endif
 
 	return ports;
-#endif
 }
 
 int serial_open(const char *pathname, int flags)
@@ -86,19 +90,70 @@ int serial_open(const char *pathname, int flags)
 #endif
 }
 
+/*
+ * Close the serial port.
+ * Returns 0 upon success, -1 upon failure.
+ */
 int serial_close(int fd)
 {
 #ifdef _WIN32
-	CloseHandle(hdl);
+	/* Returns non-zero upon success, 0 upon failure. */
+	return (CloseHandle(hdl) == 0) ? -1 : 0;
 #else
+	/* Returns 0 upon success, -1 upon failure. */
 	return close(fd);
 #endif
 }
 
+/*
+ * Flush serial port buffers (if any).
+ * Returns 0 upon success, -1 upon failure.
+ */
 int serial_flush(int fd)
 {
-
+#ifdef _WIN32
+	/* Returns non-zero upon success, 0 upon failure. */
+	return (PurgeComm(hdl, PURGE_RXCLEAR | PURGE_TXCLEAR) == 0) ? -1 : 0;
+#else
+	/* Returns 0 upon success, -1 upon failure. */
 	return tcflush(fd, TCIOFLUSH);
+#endif
+}
+
+/*
+ * Write a number of bytes to the specified serial port.
+ * Returns the number of bytes written, or -1 upon failure.
+ */
+int serial_write(int fd, const void *buf, size_t count)
+{
+#ifdef _WIN32
+	DWORD tmp = 0;
+
+	/* FIXME */
+	/* Returns non-zero upon success, 0 upon failure. */
+	WriteFile(hdl, buf, count, &tmp, NULL);
+#else
+	/* Returns the number of bytes written, or -1 upon failure. */
+	return write(fd, buf, count);
+#endif
+}
+
+/*
+ * Read a number of bytes from the specified serial port.
+ * Returns the number of bytes read, or -1 upon failure.
+ */
+int serial_read(int fd, void *buf, size_t count)
+{
+#ifdef _WIN32
+	DWORD tmp = 0;
+
+	/* FIXME */
+	/* Returns non-zero upon success, 0 upon failure. */
+	return ReadFile(hdl, buf, count, &tmp, NULL);
+#else
+	/* Returns the number of bytes read, or -1 upon failure. */
+	return read(fd, buf, count);
+#endif
 }
 
 void *serial_backup_params(int fd)
@@ -125,8 +180,10 @@ void serial_restore_params(int fd, void *backup)
 }
 
 /*
- * flowcontrol 1 = rts/cts  2 = xon/xoff
- * parity 0 = none, 1 = even, 2 = odd
+ * Set serial parameters.
+ *
+ * flowcontrol: 1 = rts/cts, 2 = xon/xoff
+ * parity: 0 = none, 1 = even, 2 = odd
  */
 int serial_set_params(int fd, int speed, int bits, int parity, int stopbits,
 		      int flowcontrol)
@@ -136,10 +193,12 @@ int serial_set_params(int fd, int speed, int bits, int parity, int stopbits,
 
 	if (!GetCommState(hdl, &dcb)) {
 		/* TODO: Error handling. */
+		return SIGROK_ERR;
 	}
 
 	/* TODO: Rename 'speed' to 'baudrate'. */
 	switch(speed) {
+	/* TODO: Support for higher baud rates. */
 	case 115200:
 		dcb.BaudRate = CBR_115200;
 		break;
@@ -165,6 +224,7 @@ int serial_set_params(int fd, int speed, int bits, int parity, int stopbits,
 
 	if (!SetCommState(hdl, &dcb)) {
 		/* TODO: Error handling. */
+		return SIGROK_ERR;
 	}
 #else
 	struct termios term;
@@ -228,16 +288,17 @@ int serial_set_params(int fd, int speed, int bits, int parity, int stopbits,
 		return SIGROK_ERR;
 	}
 
-	term.c_iflag &= ~(IGNPAR | PARODD | PARENB);
+	term.c_iflag &= ~IGNPAR;
+	term.c_cflag &= ~(PARODD | PARENB);
 	switch (parity) {
 	case 0:
 		term.c_iflag |= IGNPAR;
 		break;
 	case 1:
-		term.c_iflag |= PARENB;
+		term.c_cflag |= PARENB;
 		break;
 	case 2:
-		term.c_iflag |= PARENB | PARODD;
+		term.c_cflag |= PARENB | PARODD;
 		break;
 	default:
 		return SIGROK_ERR;