Changeset a1732929 in mainline for uspace/drv/hid/usbhid/usbhid.c


Ignore:
Timestamp:
2018-01-15T17:04:34Z (7 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:
1 edited

Legend:

Unmodified
Added
Removed

  • uspace/drv/hid/usbhid/usbhid.c

    rc1a966e ra1732929  
    134134        usb_hid_report_path_t *usage_path = usb_hid_report_path();
    135135        if (usage_path == NULL) {
    136                 usb_log_debug("Failed to create usage path.\n");
     136                usb_log_debug("Failed to create usage path.");
    137137                return false;
    138138        }
    … …  
    143143                    mapping->usage_path[i].usage_page,
    144144                    mapping->usage_path[i].usage) != EOK) {
    145                         usb_log_debug("Failed to append to usage path.\n");
     145                        usb_log_debug("Failed to append to usage path.");
    146146                        usb_hid_report_path_free(usage_path);
    147147                        return false;
    … …  
    149149        }
    150150
    151         usb_log_debug("Compare flags: %d\n", mapping->compare);
     151        usb_log_debug("Compare flags: %d", mapping->compare);
    152152
    153153        bool matches = false;
    … …  
    155155
    156156        do {
    157                 usb_log_debug("Trying report id %u\n", report_id);
     157                usb_log_debug("Trying report id %u", report_id);
    158158                if (report_id != 0) {
    159159                        usb_hid_report_path_set_report_id(usage_path,
    … …  
    166166                        USB_HID_REPORT_TYPE_INPUT);
    167167
    168                 usb_log_debug("Field: %p\n", field);
     168                usb_log_debug("Field: %p", field);
    169169
    170170                if (field != NULL) {
    … …  
    243243                            mapping->product_id);
    244244                        if (usb_hid_ids_match(hid_dev, mapping)) {
    245                                 usb_log_debug("IDs matched.\n");
     245                                usb_log_debug("IDs matched.");
    246246                                matched = true;
    247247                        }
    … …  
    250250                /* Check usage match. */
    251251                if (mapping->usage_path != NULL) {
    252                         usb_log_debug("Comparing device against usage path.\n");
     252                        usb_log_debug("Comparing device against usage path.");
    253253                        if (usb_hid_path_matches(hid_dev, mapping)) {
    254254                                /* Does not matter if IDs were matched. */
    … …  
    258258
    259259                if (matched) {
    260                         usb_log_debug("Subdriver matched.\n");
     260                        usb_log_debug("Subdriver matched.");
    261261                        subdrivers[count++] = &mapping->subdriver;
    262262                }
    … …  
    285285                    usb_device_get_mapped_ep_desc(dev, endpoints[i].desc);
    286286                if (epm && epm->present) {
    287                         usb_log_debug("Found: %s.\n", endpoints[i].description);
     287                        usb_log_debug("Found: %s.", endpoints[i].description);
    288288                        hid_dev->poll_pipe_mapping = epm;
    289289                        return EOK;
    … …  
    301301
    302302        do {
    303                 usb_log_debug("Getting size of the report.\n");
     303                usb_log_debug("Getting size of the report.");
    304304                const size_t size =
    305305                    usb_hid_report_byte_size(&hid_dev->report, report_id,
    306306                        USB_HID_REPORT_TYPE_INPUT);
    307                 usb_log_debug("Report ID: %u, size: %zu\n", report_id, size);
     307                usb_log_debug("Report ID: %u, size: %zu", report_id, size);
    308308                max_size = (size > max_size) ? size : max_size;
    309                 usb_log_debug("Getting next report ID\n");
     309                usb_log_debug("Getting next report ID");
    310310                report_id = usb_hid_get_next_report_id(&hid_dev->report,
    311311                    report_id, USB_HID_REPORT_TYPE_INPUT);
    312312        } while (report_id != 0);
    313313
    314         usb_log_debug("Max size of input report: %zu\n", max_size);
     314        usb_log_debug("Max size of input report: %zu", max_size);
    315315
    316316        assert(hid_dev->input_report == NULL);
    … …  
    329329{
    330330        if (dev == NULL || arg == NULL || buffer == NULL) {
    331                 usb_log_error("Missing arguments to polling callback.\n");
     331                usb_log_error("Missing arguments to polling callback.");
    332332                return false;
    333333        }
    … …  
    336336        assert(hid_dev->input_report != NULL);
    337337
    338         usb_log_debug("New data [%zu/%zu]: %s\n", buffer_size,
     338        usb_log_debug("New data [%zu/%zu]: %s", buffer_size,
    339339            hid_dev->max_input_report_size,
    340340            usb_debug_str_buffer(buffer, buffer_size, 0));
    … …  
    419419        assert(dev);
    420420
    421         usb_log_debug("Initializing HID structure...\n");
     421        usb_log_debug("Initializing HID structure...");
    422422
    423423        usb_hid_report_init(&hid_dev->report);
    … …  
    441441                usb_hid_find_subdrivers(hid_dev);
    442442        } else {
    443                 usb_log_error("Failed to parse report descriptor: fallback.\n");
     443                usb_log_error("Failed to parse report descriptor: fallback.");
    444444                hid_dev->subdrivers = NULL;
    445445                hid_dev->subdriver_count = 0;
    446446        }
    447447
    448         usb_log_debug("Subdriver count(before trying boot protocol): %d\n",
     448        usb_log_debug("Subdriver count(before trying boot protocol): %d",
    449449            hid_dev->subdriver_count);
    450450
    … …  
    457457                switch (hid_dev->poll_pipe_mapping->interface->interface_protocol) {
    458458                case USB_HID_PROTOCOL_KEYBOARD:
    459                         usb_log_info("Falling back to kbd boot protocol.\n");
     459                        usb_log_info("Falling back to kbd boot protocol.");
    460460                        rc = usb_kbd_set_boot_protocol(hid_dev);
    461461                        if (rc == EOK) {
    … …  
    464464                        break;
    465465                case USB_HID_PROTOCOL_MOUSE:
    466                         usb_log_info("Falling back to mouse boot protocol.\n");
     466                        usb_log_info("Falling back to mouse boot protocol.");
    467467                        rc = usb_mouse_set_boot_protocol(hid_dev);
    468468                        if (rc == EOK) {
    … …  
    471471                        break;
    472472                default:
    473                         usb_log_info("Falling back to generic HID driver.\n");
     473                        usb_log_info("Falling back to generic HID driver.");
    474474                        usb_hid_set_generic_hid_subdriver(hid_dev);
    475475                }
    476476        }
    477477
    478         usb_log_debug("Subdriver count(after trying boot protocol): %d\n",
     478        usb_log_debug("Subdriver count(after trying boot protocol): %d",
    479479            hid_dev->subdriver_count);
    480480
    … …  
    491491        for (unsigned i = 0; i < hid_dev->subdriver_count; ++i) {
    492492                if (hid_dev->subdrivers[i].init != NULL) {
    493                         usb_log_debug("Initializing subdriver %d.\n",i);
     493                        usb_log_debug("Initializing subdriver %d.",i);
    494494                        const int pret = hid_dev->subdrivers[i].init(hid_dev,
    495495                            &hid_dev->subdrivers[i].data);
    … …  
    514514                rc = usb_hid_init_report(hid_dev);
    515515                if (rc != EOK) {
    516                         usb_log_error("Failed to initialize input report buffer: %s\n", str_error(rc));
     516                        usb_log_error("Failed to initialize input report buffer: %s", str_error(rc));
    517517                        // FIXME: What happens now?
    518518                }
    … …  
    520520                usb_polling_t *polling = &hid_dev->polling;
    521521                if ((rc = usb_polling_init(polling))) {
    522                         usb_log_error("Failed to initialize polling: %s\n", str_error(rc));
     522                        usb_log_error("Failed to initialize polling: %s", str_error(rc));
    523523                        // FIXME: What happens now?
    524524                }
    … …  
    555555        usb_polling_fini(&hid_dev->polling);
    556556
    557         usb_log_debug("Subdrivers: %p, subdriver count: %d\n",
     557        usb_log_debug("Subdrivers: %p, subdriver count: %d",
    558558            hid_dev->subdrivers, hid_dev->subdriver_count);
    559559
Note: See TracChangeset for help on using the changeset viewer.