Set serial connection parameters immediately on connect.

device/serial/serial_io_handler_posix.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "device/serial/serial_io_handler_posix.h"
 
 #include <sys/ioctl.h>
 #include <termios.h>
 
 #include "base/posix/eintr_wrapper.h"
@@ skipping 178 matching lines @@
   QueueReadCompleted(0, read_cancel_reason());
 }
 
 void SerialIoHandlerPosix::CancelWriteImpl() {
   DCHECK(CalledOnValidThread());
   is_watching_writes_ = false;
   file_write_watcher_.StopWatchingFileDescriptor();
   QueueWriteCompleted(0, write_cancel_reason());
 }
 
+bool SerialIoHandlerPosix::ConfigurePortImpl() {
+  struct termios config;
+  if (tcgetattr(file().GetPlatformFile(), &config) != 0) {
+    VPLOG(1) << "Failed to get port attributes";
+    return false;
+  }
+
+  // Set flags for 'raw' operation
+  config.c_lflag &= ~(ICANON | ECHO | ECHOE | ECHONL | ISIG);
+  config.c_iflag &=
+      ~(IGNBRK | BRKINT | PARMRK | ISTRIP | INLCR | IGNCR | ICRNL | IXON);
+  config.c_oflag &= ~OPOST;
+
+  // CLOCAL causes the system to disregard the DCD signal state.
+  // CREAD enables reading from the port.
+  config.c_cflag |= (CLOCAL | CREAD);
+
+  DCHECK(options().bitrate);
+  speed_t bitrate_opt = B0;
+  if (BitrateToSpeedConstant(options().bitrate, &bitrate_opt)) {
+    cfsetispeed(&config, bitrate_opt);
+    cfsetospeed(&config, bitrate_opt);
+  } else {
+    // Attempt to set a custom speed.
+    if (!SetCustomBitrate(file().GetPlatformFile(), &config,
+                          options().bitrate)) {
+      return false;
+    }
+  }
+
+  DCHECK(options().data_bits != serial::DATA_BITS_NONE);
+  config.c_cflag &= ~CSIZE;
+  switch (options().data_bits) {
+    case serial::DATA_BITS_SEVEN:
+      config.c_cflag |= CS7;
+      break;
+    case serial::DATA_BITS_EIGHT:
+    default:
+      config.c_cflag |= CS8;
+      break;
+  }
+
+  DCHECK(options().parity_bit != serial::PARITY_BIT_NONE);
+  switch (options().parity_bit) {
+    case serial::PARITY_BIT_EVEN:
+      config.c_cflag |= PARENB;
+      config.c_cflag &= ~PARODD;
+      break;
+    case serial::PARITY_BIT_ODD:
+      config.c_cflag |= (PARODD | PARENB);
+      break;
+    case serial::PARITY_BIT_NO:
+    default:
+      config.c_cflag &= ~(PARODD | PARENB);
+      break;
+  }
+
+  DCHECK(options().stop_bits != serial::STOP_BITS_NONE);
+  switch (options().stop_bits) {
+    case serial::STOP_BITS_TWO:
+      config.c_cflag |= CSTOPB;
+      break;
+    case serial::STOP_BITS_ONE:
+    default:
+      config.c_cflag &= ~CSTOPB;
+      break;
+  }
+
+  DCHECK(options().has_cts_flow_control);
+  if (options().cts_flow_control) {
+    config.c_cflag |= CRTSCTS;
+  } else {
+    config.c_cflag &= ~CRTSCTS;
+  }
+
+  if (tcsetattr(file().GetPlatformFile(), TCSANOW, &config) != 0) {
+    VPLOG(1) << "Failed to set port attributes";
+    return false;
+  }
+  return true;
+}
+
 SerialIoHandlerPosix::SerialIoHandlerPosix(
     scoped_refptr<base::MessageLoopProxy> file_thread_message_loop,
     scoped_refptr<base::MessageLoopProxy> ui_thread_message_loop)
     : SerialIoHandler(file_thread_message_loop, ui_thread_message_loop),
       is_watching_reads_(false),
       is_watching_writes_(false) {
 }
 
 SerialIoHandlerPosix::~SerialIoHandlerPosix() {
 }
@@ skipping 66 matching lines @@
     is_watching_writes_ =
         base::MessageLoopForIO::current()->WatchFileDescriptor(
             file().GetPlatformFile(),
             true,
             base::MessageLoopForIO::WATCH_WRITE,
             &file_write_watcher_,
             this);
   }
 }
 
-bool SerialIoHandlerPosix::ConfigurePort(
-    const serial::ConnectionOptions& options) {
-  struct termios config;
-  tcgetattr(file().GetPlatformFile(), &config);
-  if (options.bitrate) {
-    speed_t bitrate_opt = B0;
-    if (BitrateToSpeedConstant(options.bitrate, &bitrate_opt)) {
-      cfsetispeed(&config, bitrate_opt);
-      cfsetospeed(&config, bitrate_opt);
-    } else {
-      // Attempt to set a custom speed.
-      if (!SetCustomBitrate(
-              file().GetPlatformFile(), &config, options.bitrate)) {
-        return false;
-      }
-    }
+bool SerialIoHandlerPosix::Flush() const {
+  if (tcflush(file().GetPlatformFile(), TCIOFLUSH) != 0) {
+    VPLOG(1) << "Failed to flush port";
+    return false;
   }
-  if (options.data_bits != serial::DATA_BITS_NONE) {
-    config.c_cflag &= ~CSIZE;
-    switch (options.data_bits) {
-      case serial::DATA_BITS_SEVEN:
-        config.c_cflag |= CS7;
-        break;
-      case serial::DATA_BITS_EIGHT:
-      default:
-        config.c_cflag |= CS8;
-        break;
-    }
-  }
-  if (options.parity_bit != serial::PARITY_BIT_NONE) {
-    switch (options.parity_bit) {
-      case serial::PARITY_BIT_EVEN:
-        config.c_cflag |= PARENB;
-        config.c_cflag &= ~PARODD;
-        break;
-      case serial::PARITY_BIT_ODD:
-        config.c_cflag |= (PARODD | PARENB);
-        break;
-      case serial::PARITY_BIT_NO:
-      default:
-        config.c_cflag &= ~(PARODD | PARENB);
-        break;
-    }
-  }
-  if (options.stop_bits != serial::STOP_BITS_NONE) {
-    switch (options.stop_bits) {
-      case serial::STOP_BITS_TWO:
-        config.c_cflag |= CSTOPB;
-        break;
-      case serial::STOP_BITS_ONE:
-      default:
-        config.c_cflag &= ~CSTOPB;
-        break;
-    }
-  }
-  if (options.has_cts_flow_control) {
-    if (options.cts_flow_control) {
-      config.c_cflag |= CRTSCTS;
-    } else {
-      config.c_cflag &= ~CRTSCTS;
-    }
-  }
-  return tcsetattr(file().GetPlatformFile(), TCSANOW, &config) == 0;
-}
-
-bool SerialIoHandlerPosix::PostOpen() {
-  struct termios config;
-  tcgetattr(file().GetPlatformFile(), &config);
-
-  // Set flags for 'raw' operation
-  config.c_lflag &= ~(ICANON | ECHO | ECHOE | ECHONL | ISIG);
-  config.c_iflag &=
-      ~(IGNBRK | BRKINT | PARMRK | ISTRIP | INLCR | IGNCR | ICRNL | IXON);
-  config.c_oflag &= ~OPOST;
-
-  // CLOCAL causes the system to disregard the DCD signal state.
-  // CREAD enables reading from the port.
-  config.c_cflag |= (CLOCAL | CREAD);
-
-  return tcsetattr(file().GetPlatformFile(), TCSANOW, &config) == 0;
-}
-
-bool SerialIoHandlerPosix::Flush() const {
-  return tcflush(file().GetPlatformFile(), TCIOFLUSH) == 0;
+  return true;
 }
 
 serial::DeviceControlSignalsPtr SerialIoHandlerPosix::GetControlSignals()
     const {
   int status;
   if (ioctl(file().GetPlatformFile(), TIOCMGET, &status) == -1) {
+    VPLOG(1) << "Failed to get port control signals";
     return serial::DeviceControlSignalsPtr();
   }
 
   serial::DeviceControlSignalsPtr signals(serial::DeviceControlSignals::New());
   signals->dcd = (status & TIOCM_CAR) != 0;
   signals->cts = (status & TIOCM_CTS) != 0;
   signals->dsr = (status & TIOCM_DSR) != 0;
   signals->ri = (status & TIOCM_RI) != 0;
   return signals.Pass();
 }
 
 bool SerialIoHandlerPosix::SetControlSignals(
     const serial::HostControlSignals& signals) {
   int status;
 
   if (ioctl(file().GetPlatformFile(), TIOCMGET, &status) == -1) {
+    VPLOG(1) << "Failed to get port control signals";
     return false;
   }
 
   if (signals.has_dtr) {
     if (signals.dtr) {
       status |= TIOCM_DTR;
     } else {
       status &= ~TIOCM_DTR;
     }
   }
 
   if (signals.has_rts) {
     if (signals.rts) {
       status |= TIOCM_RTS;
     } else {
       status &= ~TIOCM_RTS;
     }
   }
 
-  return ioctl(file().GetPlatformFile(), TIOCMSET, &status) == 0;
+  if (ioctl(file().GetPlatformFile(), TIOCMSET, &status) != 0) {
+    VPLOG(1) << "Failed to set port control signals";
+    return false;
+  }
+  return true;
 }
 
 serial::ConnectionInfoPtr SerialIoHandlerPosix::GetPortInfo() const {
   struct termios config;
   if (tcgetattr(file().GetPlatformFile(), &config) == -1) {
+    VPLOG(1) << "Failed to get port info";
     return serial::ConnectionInfoPtr();
   }
   serial::ConnectionInfoPtr info(serial::ConnectionInfo::New());
   speed_t ispeed = cfgetispeed(&config);
   speed_t ospeed = cfgetospeed(&config);
   if (ispeed == ospeed) {
     int bitrate = 0;
     if (SpeedConstantToBitrate(ispeed, &bitrate)) {
       info->bitrate = bitrate;
     } else if (ispeed > 0) {
@@ skipping 17 matching lines @@
       (config.c_cflag & CSTOPB) ? serial::STOP_BITS_TWO : serial::STOP_BITS_ONE;
   info->cts_flow_control = (config.c_cflag & CRTSCTS) != 0;
   return info.Pass();
 }
 
 std::string SerialIoHandler::MaybeFixUpPortName(const std::string& port_name) {
   return port_name;
 }
 
 }  // namespace device