Changeset a1732929 in mainline for uspace/drv/hid/usbhid/generic/hiddev.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/hid/usbhid/generic/hiddev.c (modified) (11 diffs)

uspace/drv/hid/usbhid/generic/hiddev.c

@@ -96 +96 @@
         const usb_hid_dev_t *hid_dev = fun_hid_dev(fun);
 
-        usb_log_debug2("hid_dev: %p, Max input report size (%zu).\n",
+        usb_log_debug2("hid_dev: %p, Max input report size (%zu).",
             hid_dev, hid_dev->max_input_report_size);
 
@@ -105 +105 @@
     size_t size, size_t *act_size, int *event_nr, unsigned int flags)
 {
-        usb_log_debug2("Generic HID: Get event.\n");
+        usb_log_debug2("Generic HID: Get event.");
 
         if (buffer == NULL || act_size == NULL || event_nr == NULL) {
@@ -115 +115 @@
 
         if (hid_dev->input_report_size > size) {
-                usb_log_debug("input_report_size > size (%zu, %zu)\n",
+                usb_log_debug("input_report_size > size (%zu, %zu)",
                     hid_dev->input_report_size, size);
                 return EINVAL;  // TODO: other error code
@@ -126 +126 @@
         *event_nr = usb_hid_report_number(hid_dev);
 
-        usb_log_debug2("OK\n");
+        usb_log_debug2("OK");
 
         return EOK;
@@ -133 +133 @@
 static size_t usb_generic_get_report_descriptor_length(ddf_fun_t *fun)
 {
-        usb_log_debug("Generic HID: Get report descriptor length.\n");
-
-        const usb_hid_dev_t *hid_dev = fun_hid_dev(fun);
-
-        usb_log_debug2("hid_dev->report_desc_size = %zu\n",
+        usb_log_debug("Generic HID: Get report descriptor length.");
+
+        const usb_hid_dev_t *hid_dev = fun_hid_dev(fun);
+
+        usb_log_debug2("hid_dev->report_desc_size = %zu",
             hid_dev->report_desc_size);
 
@@ -146 +146 @@
     size_t size, size_t *actual_size)
 {
-        usb_log_debug2("Generic HID: Get report descriptor.\n");
+        usb_log_debug2("Generic HID: Get report descriptor.");
 
         const usb_hid_dev_t *hid_dev = fun_hid_dev(fun);
@@ -162 +162 @@
 static int usb_generic_hid_client_connected(ddf_fun_t *fun)
 {
-        usb_log_debug("Generic HID: Client connected.\n");
+        usb_log_debug("Generic HID: Client connected.");
         return EOK;
 }
@@ -173 +173 @@
 
         if (ddf_fun_unbind(fun) != EOK) {
-                usb_log_error("Failed to unbind generic hid fun.\n");
+                usb_log_error("Failed to unbind generic hid fun.");
                 return;
         }
-        usb_log_debug2("%s unbound.\n", ddf_fun_get_name(fun));
+        usb_log_debug2("%s unbound.", ddf_fun_get_name(fun));
         ddf_fun_destroy(fun);
 }
@@ -189 +189 @@
 
         /* Create the exposed function. */
-        usb_log_debug("Creating DDF function %s...\n", HID_GENERIC_FUN_NAME);
+        usb_log_debug("Creating DDF function %s...", HID_GENERIC_FUN_NAME);
         ddf_fun_t *fun = usb_device_ddf_fun_create(hid_dev->usb_dev,
             fun_exposed, HID_GENERIC_FUN_NAME);
         if (fun == NULL) {
-                usb_log_error("Could not create DDF function node.\n");
+                usb_log_error("Could not create DDF function node.");
                 return ENOMEM;
         }
@@ -204 +204 @@
         int rc = ddf_fun_bind(fun);
         if (rc != EOK) {
-                usb_log_error("Could not bind DDF function: %s.\n",
+                usb_log_error("Could not bind DDF function: %s.",
                     str_error(rc));
                 ddf_fun_destroy(fun);
@@ -210 +210 @@
         }
 
-        usb_log_debug("HID function created. Handle: %" PRIun "\n",
+        usb_log_debug("HID function created. Handle: %" PRIun "",
             ddf_fun_get_handle(fun));
         *data = fun;