Changes in / [8357fc9:b9e3aa3] in mainline


Ignore:
Location:
uspace
Files:
2 edited

Legend:

Unmodified
Added
Removed

  • uspace/app/mkbd/main.c

    r8357fc9 rb9e3aa3  
    7171                usb_hid_free_report(*report);
    7272                *report = NULL;
    73                 printf("usb_hid_report_init() failed.\n");
     73                //printf("usb_hid_report_init() failed.\n");
    7474                return rc;
    7575        }
    … …  
    8282                usb_hid_free_report(*report);
    8383                *report = NULL;
    84                 printf("usbhid_dev_get_report_descriptor_length() failed.\n");
     84                //printf("usbhid_dev_get_report_descriptor_length() failed.\n");
    8585                return rc;
    8686        }
    … …  
    8989                usb_hid_free_report(*report);
    9090                *report = NULL;
    91                 printf("usbhid_dev_get_report_descriptor_length() returned 0.\n");
     91                //printf("usbhid_dev_get_report_descriptor_length() returned 0.\n");
    9292                return EINVAL;  // TODO: other error code?
    9393        }
    … …  
    108108                *report = NULL;
    109109                free(desc);
    110                 printf("usbhid_dev_get_report_descriptor() failed.\n");
     110                //printf("usbhid_dev_get_report_descriptor() failed.\n");
    111111                return rc;
    112112        }
    … …  
    116116                *report = NULL;
    117117                free(desc);
    118                 printf("usbhid_dev_get_report_descriptor() returned wrong size:"
    119                     " %zu, expected: %zu.\n", actual_size, report_desc_size);
     118//              printf("usbhid_dev_get_report_descriptor() returned wrong size:"
     119//                  " %zu, expected: %zu.\n", actual_size, report_desc_size);
    120120                return EINVAL;  // TODO: other error code?
    121121        }
    … …  
    128128        if (rc != EOK) {
    129129                free(desc);
    130                 printf("usb_hid_parse_report_descriptor() failed.\n");
     130//              printf("usb_hid_parse_report_descriptor() failed.\n");
    131131                return rc;
    132132        }

  • uspace/drv/usbhid/usbhid.c

    r8357fc9 rb9e3aa3  
    7878        }
    7979       
     80        assert(hid_dev->subdriver_count >= 0);
     81       
    8082        // set the init callback
    81         hid_dev->subdrivers[0].init = usb_kbd_init;
     83        hid_dev->subdrivers[hid_dev->subdriver_count].init = usb_kbd_init;
    8284       
    8385        // set the polling callback
    84         hid_dev->subdrivers[0].poll = usb_kbd_polling_callback;
     86        hid_dev->subdrivers[hid_dev->subdriver_count].poll =
     87            usb_kbd_polling_callback;
    8588       
    8689        // set the polling ended callback
    87         hid_dev->subdrivers[0].poll_end = NULL;
     90        hid_dev->subdrivers[hid_dev->subdriver_count].poll_end = NULL;
    8891       
    8992        // set the deinit callback
    90         hid_dev->subdrivers[0].deinit = usb_kbd_deinit;
     93        hid_dev->subdrivers[hid_dev->subdriver_count].deinit = usb_kbd_deinit;
    9194       
    9295        // set subdriver count
    93         hid_dev->subdriver_count = 1;
     96        ++hid_dev->subdriver_count;
    9497       
    9598        return EOK;
    … …  
    108111        }
    109112       
     113        assert(hid_dev->subdriver_count >= 0);
     114       
    110115        // set the init callback
    111         hid_dev->subdrivers[0].init = usb_mouse_init;
     116        hid_dev->subdrivers[hid_dev->subdriver_count].init = usb_mouse_init;
    112117       
    113118        // set the polling callback
    114         hid_dev->subdrivers[0].poll = usb_mouse_polling_callback;
     119        hid_dev->subdrivers[hid_dev->subdriver_count].poll =
     120            usb_mouse_polling_callback;
    115121       
    116122        // set the polling ended callback
    117         hid_dev->subdrivers[0].poll_end = NULL;
     123        hid_dev->subdrivers[hid_dev->subdriver_count].poll_end = NULL;
    118124       
    119125        // set the deinit callback
    120         hid_dev->subdrivers[0].deinit = usb_mouse_deinit;
     126        hid_dev->subdrivers[hid_dev->subdriver_count].deinit = usb_mouse_deinit;
    121127       
    122128        // set subdriver count
    123         hid_dev->subdriver_count = 1;
     129        ++hid_dev->subdriver_count;
    124130       
    125131        return EOK;
    … …  
    138144        }
    139145       
     146        assert(hid_dev->subdriver_count >= 0);
     147       
    140148        // set the init callback
    141         hid_dev->subdrivers[0].init = usb_generic_hid_init;
     149        hid_dev->subdrivers[hid_dev->subdriver_count].init =
     150            usb_generic_hid_init;
    142151       
    143152        // set the polling callback
    144         hid_dev->subdrivers[0].poll = usb_generic_hid_polling_callback;
     153        hid_dev->subdrivers[hid_dev->subdriver_count].poll =
     154            usb_generic_hid_polling_callback;
    145155       
    146156        // set the polling ended callback
    147         hid_dev->subdrivers[0].poll_end = NULL;
     157        hid_dev->subdrivers[hid_dev->subdriver_count].poll_end = NULL;
    148158       
    149159        // set the deinit callback
    150         hid_dev->subdrivers[0].deinit = NULL;
     160        hid_dev->subdrivers[hid_dev->subdriver_count].deinit = NULL;
    151161       
    152162        // set subdriver count
    153         hid_dev->subdriver_count = 1;
     163        ++hid_dev->subdriver_count;
    154164       
    155165        return EOK;
    … …  
    249259        }
    250260       
    251         usb_log_debug("Size of the input report: %zuB\n", size);
     261        usb_log_debug("Size of the input report: %zu\n", size);
    252262        usb_hid_report_path_free(usage_path);
    253263       
    … …  
    401411       
    402412        do {
     413                usb_log_debug("Getting size of the report.\n");
    403414                size = usb_hid_report_byte_size(hid_dev->report, report_id,
    404415                    USB_HID_REPORT_TYPE_INPUT);
    405416                usb_log_debug("Report ID: %u, size: %zu\n", report_id, size);
    406417                max_size = (size > max_size) ? size : max_size;
     418                usb_log_debug("Getting next report ID\n");
    407419                report_id = usb_hid_get_next_report_id(hid_dev->report,
    408420                    report_id, USB_HID_REPORT_TYPE_INPUT);
    … …  
    534546                    hid_dev->subdriver_count);
    535547                //usb_hid_free(&hid_dev);
     548               
    536549        } else {
    537550                bool ok = false;
    … …  
    560573        }
    561574       
    562         // save max input report size and allocate space for the report
    563         rc = usb_hid_init_report(hid_dev);
    564         if (rc != EOK) {
    565                 usb_log_error("Failed to initialize input report buffer.\n");
    566         }
     575       
     576        if (rc == EOK) {
     577                // save max input report size and allocate space for the report
     578                rc = usb_hid_init_report(hid_dev);
     579                if (rc != EOK) {
     580                        usb_log_error("Failed to initialize input report buffer"
     581                            ".\n");
     582                }
     583        }
     584       
    567585       
    568586        return rc;
    … …  
    587605        usb_log_debug("Max input report size: %zu, buffer size: %zu\n",
    588606            hid_dev->max_input_report_size, buffer_size);
    589         assert(hid_dev->max_input_report_size >= buffer_size);
    590        
    591 //      if (/*!allocated*/
    592 //          /*|| *//*hid_dev->input_report_size < buffer_size*/) {
    593 //              uint8_t *input_old = hid_dev->input_report;
    594 //              uint8_t *input_new = (uint8_t *)malloc(buffer_size);
    595                
    596 //              if (input_new == NULL) {
    597 //                      usb_log_error("Failed to allocate space for input "
    598 //                          "buffer. This event may not be reported\n");
    599 //                      memset(hid_dev->input_report, 0,
    600 //                          hid_dev->input_report_size);
    601 //              } else {
    602 //                      memcpy(input_new, input_old,
    603 //                          hid_dev->input_report_size);
    604 //                      hid_dev->input_report = input_new;
    605 //                      if (allocated) {
    606 //                              free(input_old);
    607 //                      }
    608 //                      usb_hid_new_report();
    609 //              }
    610 //      }
    611        
    612         /*! @todo This should probably be atomic. */
    613         memcpy(hid_dev->input_report, buffer, buffer_size);
    614         hid_dev->input_report_size = buffer_size;
    615         usb_hid_new_report(hid_dev);
     607        //assert(hid_dev->max_input_report_size >= buffer_size);
     608        if (hid_dev->max_input_report_size >= buffer_size) {
     609                /*! @todo This should probably be atomic. */
     610                memcpy(hid_dev->input_report, buffer, buffer_size);
     611                hid_dev->input_report_size = buffer_size;
     612                usb_hid_new_report(hid_dev);
     613        }
    616614       
    617615        bool cont = false;
Note: See TracChangeset for help on using the changeset viewer.