Changeset a1732929 in mainline for uspace/drv/bus/usb/usbdiag/tests.c

Timestamp:

2018-01-15T17:04:34Z (6 years ago)

Author:

Ondřej Hlavatý <aearsis@…>

Branches:

lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export

Children:

9ff99e8

Parents:

c1a966e

git-author:

Ondřej Hlavatý <aearsis@…> (2018-01-15 17:04:32)

git-committer:

Ondřej Hlavatý <aearsis@…> (2018-01-15 17:04:34)

Message:

usb: unified logging

Use logger instead of printf. Logger adds newlines automatically.

File:

• uspace/drv/bus/usb/usbdiag/tests.c (modified) (5 diffs)

uspace/drv/bus/usb/usbdiag/tests.c

@@ -68 +68 @@
                 while (remaining > 0) {
                         if ((rc = usb_pipe_read(pipe, buffer + size - remaining, remaining, &transferred))) {
-                                usb_log_error("Read of %s IN endpoint failed with error: %s\n", usb_str_transfer_type(pipe->desc.transfer_type), str_error(rc));
+                                usb_log_error("Read of %s IN endpoint failed with error: %s", usb_str_transfer_type(pipe->desc.transfer_type), str_error(rc));
                                 break;
                         }
 
                         if (transferred > remaining) {
-                                usb_log_error("Read of %s IN endpoint returned more data than expected.\n", usb_str_transfer_type(pipe->desc.transfer_type));
+                                usb_log_error("Read of %s IN endpoint returned more data than expected.", usb_str_transfer_type(pipe->desc.transfer_type));
                                 rc = EINVAL;
                                 break;
@@ -121 +121 @@
                 // Write buffer to device.
                 if ((rc = usb_pipe_write(pipe, buffer, size))) {
-                        usb_log_error("Write to %s OUT endpoint failed with error: %s\n", usb_str_transfer_type(pipe->desc.transfer_type), str_error(rc));
+                        usb_log_error("Write to %s OUT endpoint failed with error: %s", usb_str_transfer_type(pipe->desc.transfer_type), str_error(rc));
                         break;
                 }
@@ -169 +169 @@
                 while (remaining > 0) {
                         if ((rc = usb_pipe_read(pipe, buffer + size - remaining, remaining, &transferred))) {
-                                usb_log_error("Read of %s IN endpoint failed with error: %s\n", usb_str_transfer_type(pipe->desc.transfer_type), str_error(rc));
+                                usb_log_error("Read of %s IN endpoint failed with error: %s", usb_str_transfer_type(pipe->desc.transfer_type), str_error(rc));
                                 break;
                         }
 
                         if (transferred > remaining) {
-                                usb_log_error("Read of %s IN endpoint returned more data than expected.\n", usb_str_transfer_type(pipe->desc.transfer_type));
+                                usb_log_error("Read of %s IN endpoint returned more data than expected.", usb_str_transfer_type(pipe->desc.transfer_type));
                                 rc = EINVAL;
                                 break;
@@ -187 +187 @@
                 for (size_t i = 0; i < size; i += sizeof(test_data)) {
                         if (*(uint32_t *)(buffer + i) != test_data) {
-                                usb_log_error("Read of %s IN endpoint returned invald data at address %lu.\n", usb_str_transfer_type(pipe->desc.transfer_type), i);
+                                usb_log_error("Read of %s IN endpoint returned invald data at address %lu.", usb_str_transfer_type(pipe->desc.transfer_type), i);
                                 rc = EINVAL;
                                 break;
@@ -238 +238 @@
                 // Write buffer to device.
                 if ((rc = usb_pipe_write(pipe, buffer, size))) {
-                        usb_log_error("Write to %s OUT endpoint failed with error: %s\n", usb_str_transfer_type(pipe->desc.transfer_type), str_error(rc));
+                        usb_log_error("Write to %s OUT endpoint failed with error: %s", usb_str_transfer_type(pipe->desc.transfer_type), str_error(rc));
                         break;
                 }