Author: mmlr Date: 2011-06-10 19:18:06 +0200 (Fri, 10 Jun 2011) New Revision: 42087 Changeset: https://dev.haiku-os.org/changeset/42087 Modified: haiku/trunk/src/add-ons/kernel/drivers/ports/usb_serial/Driver.cpp haiku/trunk/src/add-ons/kernel/drivers/ports/usb_serial/Prolific.cpp haiku/trunk/src/add-ons/kernel/drivers/ports/usb_serial/Tracing.cpp Log: * Print the speeds even though they are marked as not used (as the baud index is inside the cflags) since stty for example sets them. * Turn off debug output by default and fix a warning due to that. * Some minor cleanup. Modified: haiku/trunk/src/add-ons/kernel/drivers/ports/usb_serial/Driver.cpp =================================================================== --- haiku/trunk/src/add-ons/kernel/drivers/ports/usb_serial/Driver.cpp 2011-06-10 17:12:51 UTC (rev 42086) +++ haiku/trunk/src/add-ons/kernel/drivers/ports/usb_serial/Driver.cpp 2011-06-10 17:18:06 UTC (rev 42087) @@ -43,13 +43,14 @@ const usb_configuration_info *configuration; for (int i = 0; i < descriptor->num_configurations; i++) { configuration = gUSBModule->get_nth_configuration(device, i); - if (!configuration) + if (configuration == NULL) continue; status = serialDevice->AddDevice(configuration); - if (status == B_OK) + if (status == B_OK) { // Found! break; + } } if (status < B_OK) { Modified: haiku/trunk/src/add-ons/kernel/drivers/ports/usb_serial/Prolific.cpp =================================================================== --- haiku/trunk/src/add-ons/kernel/drivers/ports/usb_serial/Prolific.cpp 2011-06-10 17:12:51 UTC (rev 42086) +++ haiku/trunk/src/add-ons/kernel/drivers/ports/usb_serial/Prolific.cpp 2011-06-10 17:18:06 UTC (rev 42087) @@ -117,6 +117,9 @@ list[i].out ? NULL : buffer, &bufferLength); TRACE(" ProlificDevice::SendRequestList(): request[%d]: 0x%08lx\n", i, status); + if (status != B_OK) { + TRACE_ALWAYS("sending request list failed:0x%08lx\n", status); + } } return B_OK; Modified: haiku/trunk/src/add-ons/kernel/drivers/ports/usb_serial/Tracing.cpp =================================================================== --- haiku/trunk/src/add-ons/kernel/drivers/ports/usb_serial/Tracing.cpp 2011-06-10 17:12:51 UTC (rev 42086) +++ haiku/trunk/src/add-ons/kernel/drivers/ports/usb_serial/Tracing.cpp 2011-06-10 17:18:06 UTC (rev 42087) @@ -18,7 +18,7 @@ #if DEBUG bool gLogEnabled = true; #else -bool gLogEnabled = true; +bool gLogEnabled = false; #endif bool gLogToFile = false; @@ -108,12 +108,12 @@ "\tc_cflag: 0x%08x\n" "\tc_lflag: 0x%08x\n" "\tc_line: 0x%08x\n" -// "\tc_ixxxxx: 0x%08x\n" -// "\tc_oxxxxx: 0x%08x\n" + "\tc_ispeed: 0x%08x\n" + "\tc_ospeed: 0x%08x\n" "\tc_cc[0x%02x, 0x%02x, 0x%02x, 0x%02x, 0x%02x, 0x%02x, 0x%02x, 0x%02x, 0x%02x, 0x%02x, 0x%02x]\n", tios->c_iflag, tios->c_oflag, tios->c_cflag, tios->c_lflag, tios->c_line, -// tios->c_ixxxxx, tios->c_oxxxxx, + tios->c_ispeed, tios->c_ospeed, tios->c_cc[0], tios->c_cc[1], tios->c_cc[2], tios->c_cc[3], tios->c_cc[4], tios->c_cc[5], tios->c_cc[6], tios->c_cc[7], tios->c_cc[8], tios->c_cc[9], tios->c_cc[10]);