Changeset e1dbcbc in mainline for uspace/lib/usb/src

Timestamp:

2011-04-29T13:43:01Z (15 years ago)

Author:

Vojtech Horky <vojtechhorky@…>

Branches:

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

Children:

a81a1d09

Parents:

380e0364 (diff), f19f1b7 (diff)
Note: this is a merge changeset, the changes displayed below correspond to the merge itself.
Use the (diff) links above to see all the changes relative to each parent.

Message:

Merge development/ changes

Location:

uspace/lib/usb/src

Files:

• debug.c (modified) (6 diffs)
• devdrv.c (modified) (1 diff)
• devpoll.c (modified) (10 diffs)
• hidiface.c (added)
• hidparser.c (modified) (57 diffs)
• hidreport.c (modified) (6 diffs)
• hidreq.c (modified) (17 diffs)
• host/batch.c (modified) (5 diffs)
• host/device_keeper.c (modified) (7 diffs)
• host/endpoint.c (modified) (5 diffs)
• host/usb_endpoint_manager.c (modified) (1 diff)
• pipepriv.c (modified) (3 diffs)
• pipepriv.h (modified) (2 diffs)
• pipes.c (modified) (2 diffs)
• pipesinit.c (modified) (3 diffs)
• pipesio.c (modified) (8 diffs)

uspace/lib/usb/src/debug.c

@@ -158 +158 @@
 
 /** Fibril local storage for the dumped buffer. */
-static fibril_local char buffer_dump[BUFFER_DUMP_LEN];
+static fibril_local char buffer_dump[2][BUFFER_DUMP_LEN];
+/** Fibril local storage for buffer switching. */
+static fibril_local int buffer_dump_index = 0;
 
 /** Dump buffer into string.
@@ -167 +169 @@
  * can not do that) and you do not have to guard it against concurrent
  * calls to it.
- * The only limitation is that each call rewrites the buffer again.
+ * The only limitation is that each second call rewrites the buffer again
+ * (internally, two buffer are used in cyclic manner).
  * Thus, it is necessary to copy the buffer elsewhere (that includes printing
  * to screen or writing to file).
@@ -173 +176 @@
  * that is not a big limitation.
  *
- * @warning You cannot use this function twice in the same printf
+ * @warning You cannot use this function more than twice in the same printf
  * (see detailed explanation).
  *
@@ -185 +188 @@
 {
         /*
-         * Remove previous string (that might also reveal double usage of
-         * this function).
+         * Remove previous string.
          */
-        bzero(buffer_dump, BUFFER_DUMP_LEN);
+        bzero(buffer_dump[buffer_dump_index], BUFFER_DUMP_LEN);
 
         if (buffer == NULL) {
@@ -202 +204 @@
         /* How many bytes are available in the output buffer. */
         size_t buffer_remaining_size = BUFFER_DUMP_LEN - 1 - REMAINDER_STR_LEN;
-        char *it = buffer_dump;
+        char *it = buffer_dump[buffer_dump_index];
 
         size_t index = 0;
@@ -253 +255 @@
         }
 
-        return buffer_dump;
+        /* Next time, use the other buffer. */
+        buffer_dump_index = 1 - buffer_dump_index;
+
+        /* Need to take the old one due to previous line. */
+        return buffer_dump[1 - buffer_dump_index];
 }
 

uspace/lib/usb/src/devdrv.c

@@ -236 +236 @@
 
         /* It is worth to start a long transfer. */
-        rc = usb_pipe_start_long_transfer(ctrl_pipe);
-        if (rc != EOK) {
-                return rc;
-        }
+        usb_pipe_start_long_transfer(ctrl_pipe);
 
         /* Get the device descriptor. */

uspace/lib/usb/src/devpoll.c

@@ -33 +33 @@
  * USB device driver framework - automatic interrupt polling.
  */
-#include <usb/devdrv.h>
+#include <usb/devpoll.h>
 #include <usb/request.h>
 #include <usb/debug.h>
+#include <usb/classes/classes.h>
 #include <errno.h>
 #include <str_error.h>
@@ -45 +46 @@
 /** Data needed for polling. */
 typedef struct {
+        int debug;
+        size_t max_failures;
+        useconds_t delay;
+        bool auto_clear_halt;
+        bool (*on_data)(usb_device_t *, uint8_t *, size_t, void *);
+        void (*on_polling_end)(usb_device_t *, bool, void *);
+        bool (*on_error)(usb_device_t *, int, void *);
+
         usb_device_t *dev;
         size_t pipe_index;
-        usb_polling_callback_t callback;
-        usb_polling_terminted_callback_t terminated_callback;
         size_t request_size;
         uint8_t *buffer;
@@ -54 +61 @@
 } polling_data_t;
 
+
 /** Polling fibril.
  *
@@ -67 +75 @@
             = polling_data->dev->pipes[polling_data->pipe_index].pipe;
         
-        usb_log_debug("Pipe interface number: %d, protocol: %d, subclass: %d, max packet size: %d\n", 
-            polling_data->dev->pipes[polling_data->pipe_index].interface_no,
-            polling_data->dev->pipes[polling_data->pipe_index].description->interface_protocol,
-            polling_data->dev->pipes[polling_data->pipe_index].description->interface_subclass,
-            pipe->max_packet_size);
+        if (polling_data->debug > 0) {
+                usb_endpoint_mapping_t *mapping
+                    = &polling_data->dev->pipes[polling_data->pipe_index];
+                usb_log_debug("Poll%p: started polling of `%s' - " \
+                    "interface %d (%s,%d,%d), %zuB/%zu.\n",
+                    polling_data,
+                    polling_data->dev->ddf_dev->name,
+                    (int) mapping->interface->interface_number,
+                    usb_str_class(mapping->interface->interface_class),
+                    (int) mapping->interface->interface_subclass,
+                    (int) mapping->interface->interface_protocol,
+                    polling_data->request_size, pipe->max_packet_size);
+        }
 
         size_t failed_attempts = 0;
-        while (failed_attempts < MAX_FAILED_ATTEMPTS) {
+        while (failed_attempts <= polling_data->max_failures) {
                 int rc;
 
@@ -81 +97 @@
                     polling_data->request_size, &actual_size);
 
-                
-//              if (rc == ESTALL) {
-//                      usb_log_debug("Seding clear feature...\n");
-//                      usb_request_clear_feature(pipe, USB_REQUEST_TYPE_STANDARD,
-//                        USB_REQUEST_RECIPIENT_ENDPOINT, 0, pipe->endpoint_no);
-//                      continue;
-//              }
+                if (polling_data->debug > 1) {
+                        if (rc == EOK) {
+                                usb_log_debug(
+                                    "Poll%p: received: '%s' (%zuB).\n",
+                                    polling_data,
+                                    usb_debug_str_buffer(polling_data->buffer,
+                                        actual_size, 16),
+                                    actual_size);
+                        } else {
+                                usb_log_debug(
+                                    "Poll%p: polling failed: %s.\n",
+                                    polling_data, str_error(rc));
+                        }
+                }
+
+                /* If the pipe stalled, we can try to reset the stall. */
+                if ((rc == ESTALL) && (polling_data->auto_clear_halt)) {
+                        /*
+                         * We ignore error here as this is usually a futile
+                         * attempt anyway.
+                         */
+                        usb_request_clear_endpoint_halt(
+                            &polling_data->dev->ctrl_pipe,
+                            pipe->endpoint_no);
+                }
 
                 if (rc != EOK) {
+                        if (polling_data->on_error != NULL) {
+                                bool cont = polling_data->on_error(
+                                    polling_data->dev, rc,
+                                    polling_data->custom_arg);
+                                if (!cont) {
+                                        failed_attempts
+                                            = polling_data->max_failures;
+                                }
+                        }
                         failed_attempts++;
                         continue;
@@ -95 +138 @@
 
                 /* We have the data, execute the callback now. */
-                bool carry_on = polling_data->callback(polling_data->dev,
+                bool carry_on = polling_data->on_data(polling_data->dev,
                     polling_data->buffer, actual_size,
                     polling_data->custom_arg);
@@ -106 +149 @@
                 /* Reset as something might be only a temporary problem. */
                 failed_attempts = 0;
-        }
-
-        if (failed_attempts > 0) {
-                usb_log_error(
-                    "Polling of device `%s' terminated: recurring failures.\n",
-                    polling_data->dev->ddf_dev->name);
-        }
-
-        if (polling_data->terminated_callback != NULL) {
-                polling_data->terminated_callback(polling_data->dev,
+
+                /* Take a rest before next request. */
+                async_usleep(polling_data->delay);
+        }
+
+        if (polling_data->on_polling_end != NULL) {
+                polling_data->on_polling_end(polling_data->dev,
                     failed_attempts > 0, polling_data->custom_arg);
+        }
+
+        if (polling_data->debug > 0) {
+                if (failed_attempts > 0) {
+                        usb_log_error(
+                            "Polling of device `%s' terminated: %s.\n",
+                            polling_data->dev->ddf_dev->name,
+                            "recurring failures");
+                } else {
+                        usb_log_debug(
+                            "Polling of device `%s' terminated by user.\n",
+                            polling_data->dev->ddf_dev->name
+                        );
+                }
         }
 
@@ -159 +213 @@
         }
 
+        usb_device_auto_polling_t *auto_polling
+            = malloc(sizeof(usb_device_auto_polling_t));
+        if (auto_polling == NULL) {
+                return ENOMEM;
+        }
+
+        auto_polling->debug = 1;
+        auto_polling->auto_clear_halt = true;
+        auto_polling->delay = 0;
+        auto_polling->max_failures = MAX_FAILED_ATTEMPTS;
+        auto_polling->on_data = callback;
+        auto_polling->on_polling_end = terminated_callback;
+        auto_polling->on_error = NULL;
+
+        int rc = usb_device_auto_polling(dev, pipe_index, auto_polling,
+           request_size, arg);
+
+        free(auto_polling);
+
+        return rc;
+}
+
+/** Start automatic device polling over interrupt in pipe.
+ *
+ * The polling settings is copied thus it is okay to destroy the structure
+ * after this function returns.
+ *
+ * @warning There is no guarantee when the request to the device
+ * will be sent for the first time (it is possible that this
+ * first request would be executed prior to return from this function).
+ *
+ * @param dev Device to be periodically polled.
+ * @param pipe_index Index of the endpoint pipe used for polling.
+ * @param polling Polling settings.
+ * @param request_size How many bytes to ask for in each request.
+ * @param arg Custom argument (passed as is to the callbacks).
+ * @return Error code.
+ * @retval EOK New fibril polling the device was already started.
+ */
+int usb_device_auto_polling(usb_device_t *dev, size_t pipe_index,
+    usb_device_auto_polling_t *polling,
+    size_t request_size, void *arg)
+{
+        if (dev == NULL) {
+                return EBADMEM;
+        }
+        if (pipe_index >= dev->pipes_count) {
+                return EINVAL;
+        }
+        if ((dev->pipes[pipe_index].pipe->transfer_type != USB_TRANSFER_INTERRUPT)
+            || (dev->pipes[pipe_index].pipe->direction != USB_DIRECTION_IN)) {
+                return EINVAL;
+        }
+        if ((polling == NULL) || (polling->on_data == NULL)) {
+                return EBADMEM;
+        }
+
         polling_data_t *polling_data = malloc(sizeof(polling_data_t));
         if (polling_data == NULL) {
@@ -164 +275 @@
         }
 
-        /* Allocate now to prevent immediate failure in the polling fibril. */
-        polling_data->buffer = malloc(request_size);
+        /* Fill-in the data. */
+        polling_data->buffer = malloc(sizeof(request_size));
         if (polling_data->buffer == NULL) {
                 free(polling_data);
                 return ENOMEM;
         }
+        polling_data->request_size = request_size;
         polling_data->dev = dev;
         polling_data->pipe_index = pipe_index;
-        polling_data->callback = callback;
-        polling_data->terminated_callback = terminated_callback;
-        polling_data->request_size = request_size;
         polling_data->custom_arg = arg;
+
+        polling_data->debug = polling->debug;
+        polling_data->max_failures = polling->max_failures;
+        if (polling->delay >= 0) {
+                polling_data->delay = (useconds_t) polling->delay;
+        } else {
+                polling_data->delay = (useconds_t) dev->pipes[pipe_index]
+                    .descriptor->poll_interval;
+        }
+        polling_data->auto_clear_halt = polling->auto_clear_halt;
+
+        polling_data->on_data = polling->on_data;
+        polling_data->on_polling_end = polling->on_polling_end;
+        polling_data->on_error = polling->on_error;
 
         fid_t fibril = fibril_create(polling_fibril, polling_data);
@@ -181 +304 @@
                 free(polling_data->buffer);
                 free(polling_data);
-                /* FIXME: better error code. */
                 return ENOMEM;
         }
         fibril_add_ready(fibril);
 
-        /* The allocated buffer etc. will be freed by the fibril. */
+        /* Fibril launched. That fibril will free the allocated data. */
 
         return EOK;

uspace/lib/usb/src/hidparser.c

@@ -39 +39 @@
 #include <mem.h>
 #include <usb/debug.h>
-
-/** */
+#include <assert.h>
+
+/** The new report item flag. Used to determine when the item is completly
+ * configured and should be added to the report structure
+ */
 #define USB_HID_NEW_REPORT_ITEM 1
 
-/** */
-#define USB_HID_NO_ACTION               2
-
-/** */
+/** No special action after the report descriptor tag is processed should be
+ * done
+ */
+#define USB_HID_NO_ACTION       2
+
+#define USB_HID_RESET_OFFSET    3
+
+/** Unknown tag was founded in report descriptor data*/
 #define USB_HID_UNKNOWN_TAG             -99
 
@@ -52 +59 @@
  * Private descriptor parser functions
  */
+int usb_hid_report_init(usb_hid_report_t *report);
+int usb_hid_report_append_fields(usb_hid_report_t *report, usb_hid_report_item_t *report_item);
+usb_hid_report_description_t * usb_hid_report_find_description(const usb_hid_report_t *report, uint8_t report_id, usb_hid_report_type_t type);
 int usb_hid_report_parse_tag(uint8_t tag, uint8_t class, const uint8_t *data, size_t item_size,
                              usb_hid_report_item_t *report_item, usb_hid_report_path_t *usage_path);
@@ -61 +71 @@
                              usb_hid_report_item_t *report_item, usb_hid_report_path_t *usage_path);
 
+void usb_hid_print_usage_path(usb_hid_report_path_t *path);
 void usb_hid_descriptor_print_list(link_t *head);
-int usb_hid_report_reset_local_items();
+void usb_hid_report_reset_local_items(usb_hid_report_item_t *report_item);
 void usb_hid_free_report_list(link_t *head);
 usb_hid_report_item_t *usb_hid_report_item_clone(const usb_hid_report_item_t *item);
@@ -68 +79 @@
  * Data translation private functions
  */
-int32_t usb_hid_report_tag_data_int32(const uint8_t *data, size_t size);
+uint32_t usb_hid_report_tag_data_uint32(const uint8_t *data, size_t size);
 inline size_t usb_hid_count_item_offset(usb_hid_report_item_t * report_item, size_t offset);
-int usb_hid_translate_data(usb_hid_report_item_t *item, const uint8_t *data, size_t j);
-int32_t usb_hid_translate_data_reverse(usb_hid_report_item_t *item, int32_t value);
+int usb_hid_translate_data(usb_hid_report_field_t *item, const uint8_t *data);
+uint32_t usb_hid_translate_data_reverse(usb_hid_report_field_t *item, int32_t value);
 int usb_pow(int a, int b);
 
+#define USB_HID_UINT32_TO_INT32(x, size)        ((((x) & (1 << ((size) - 1))) != 0) ? -(~(x - 1) & ((1 << size) - 1)) : (x)) //(-(~((x) - 1)))
+#define USB_HID_INT32_TO_UINT32(x, size)        (((x) < 0 ) ? ((1 << (size)) + (x)) : (x))
 // TODO: tohle ma bejt asi jinde
 int usb_pow(int a, int b)
@@ -96 +109 @@
  * @return Error code
  */
-int usb_hid_parser_init(usb_hid_report_parser_t *parser)
-{
-        if(parser == NULL) {
+int usb_hid_report_init(usb_hid_report_t *report)
+{
+        if(report == NULL) {
                 return EINVAL;
         }
 
-        list_initialize(&(parser->input));
-    list_initialize(&(parser->output));
-    list_initialize(&(parser->feature));
-
-        list_initialize(&(parser->stack));
-
-        parser->use_report_id = 0;
+        memset(report, 0, sizeof(usb_hid_report_t));
+        list_initialize(&report->reports);
+        list_initialize(&report->collection_paths);
+
+        report->use_report_ids = 0;
     return EOK;   
 }
 
+int usb_hid_report_append_fields(usb_hid_report_t *report, usb_hid_report_item_t *report_item)
+{
+        usb_hid_report_field_t *field;
+        int i;
+
+
+        /* find or append current collection path to the list */
+        link_t *path_it = report->collection_paths.next;
+        usb_hid_report_path_t *path = NULL;
+        while(path_it != &report->collection_paths) {
+                path = list_get_instance(path_it, usb_hid_report_path_t, link);
+                
+                if(usb_hid_report_compare_usage_path(path, report_item->usage_path, USB_HID_PATH_COMPARE_STRICT) == EOK){
+                        break;
+                }                       
+                path_it = path_it->next;
+        }
+        if(path_it == &report->collection_paths) {
+                path = usb_hid_report_path_clone(report_item->usage_path);                      
+                list_append(&path->link, &report->collection_paths);                                    
+                report->collection_paths_count++;
+        }
+
+        for(i=0; i<report_item->usages_count; i++){
+                usb_log_debug("usages (%d) - %x\n", i, report_item->usages[i]);
+        }
+
+        
+        for(i=0; i<report_item->count; i++){
+
+                field = malloc(sizeof(usb_hid_report_field_t));
+                memset(field, 0, sizeof(usb_hid_report_field_t));
+                list_initialize(&field->link);
+
+                /* fill the attributes */               
+                field->collection_path = path;
+                field->logical_minimum = report_item->logical_minimum;
+                field->logical_maximum = report_item->logical_maximum;
+                field->physical_minimum = report_item->physical_minimum;
+                field->physical_maximum = report_item->physical_maximum;
+
+                field->usage_minimum = report_item->usage_minimum;
+                field->usage_maximum = report_item->usage_maximum;
+                if(report_item->extended_usage_page != 0){
+                        field->usage_page = report_item->extended_usage_page;
+                }
+                else {
+                        field->usage_page = report_item->usage_page;
+                }
+
+                if(report_item->usages_count > 0 && ((report_item->usage_minimum == 0) && (report_item->usage_maximum == 0))) {
+                        uint32_t usage;
+                        if(report_item->type != USB_HID_REPORT_TYPE_INPUT) {
+                                if(i < report_item->usages_count){
+                                        usage = report_item->usages[i];
+                                }
+                                else {
+                                        usage = report_item->usages[report_item->usages_count - 1];
+                                }
+                        }
+                        else {
+                                if((report_item->count - i - 1) < report_item->usages_count){
+                                        usage = report_item->usages[(report_item->count - i - 1)];
+                                }
+                                else {
+                                        usage = report_item->usages[report_item->usages_count - 1];
+                                }
+                        }
+
+                                                
+                        if((usage & 0xFFFF0000) != 0){
+                                field->usage_page = (usage >> 16);                                      
+                                field->usage = (usage & 0xFFFF);
+                        }
+                        else {
+                                field->usage = usage;
+                        }
+
+                        
+                }       
+
+                if((USB_HID_ITEM_FLAG_VARIABLE(report_item->item_flags) != 0) && (!((report_item->usage_minimum == 0) && (report_item->usage_maximum == 0)))) {
+                        if(report_item->type == USB_HID_REPORT_TYPE_INPUT) {
+                                field->usage = report_item->usage_maximum - i;
+                        }
+                        else {
+                                field->usage = report_item->usage_minimum + i;                                  
+                        }
+
+                }
+                
+                field->size = report_item->size;
+                field->offset = report_item->offset + (i * report_item->size);
+                if(report_item->id != 0) {
+                        field->offset += 8;
+                        report->use_report_ids = 1;
+                }
+                field->item_flags = report_item->item_flags;
+
+                /* find the right report list*/
+                usb_hid_report_description_t *report_des;
+                report_des = usb_hid_report_find_description(report, report_item->id, report_item->type);
+                if(report_des == NULL){
+                        report_des = malloc(sizeof(usb_hid_report_description_t));
+                        memset(report_des, 0, sizeof(usb_hid_report_description_t));
+
+                        report_des->type = report_item->type;
+                        report_des->report_id = report_item->id;
+                        list_initialize (&report_des->link);
+                        list_initialize (&report_des->report_items);
+
+                        list_append(&report_des->link, &report->reports);
+                        report->report_count++;
+                }
+
+                /* append this field to the end of founded report list */
+                list_append (&field->link, &report_des->report_items);
+                
+                /* update the sizes */
+                report_des->bit_length += field->size;
+                report_des->item_length++;
+
+        }
+
+
+        return EOK;
+}
+
+usb_hid_report_description_t * usb_hid_report_find_description(const usb_hid_report_t *report, uint8_t report_id, usb_hid_report_type_t type)
+{
+        link_t *report_it = report->reports.next;
+        usb_hid_report_description_t *report_des = NULL;
+        
+        while(report_it != &report->reports) {
+                report_des = list_get_instance(report_it, usb_hid_report_description_t, link);
+
+                if((report_des->report_id == report_id) && (report_des->type == type)){
+                        return report_des;
+                }
+                
+                report_it = report_it->next;
+        }
+
+        return NULL;
+}
 
 /** Parse HID report descriptor.
@@ -119 +275 @@
  * @return Error code.
  */
-int usb_hid_parse_report_descriptor(usb_hid_report_parser_t *parser, 
+int usb_hid_parse_report_descriptor(usb_hid_report_t *report, 
     const uint8_t *data, size_t size)
 {
@@ -130 +286 @@
         usb_hid_report_item_t *new_report_item; 
         usb_hid_report_path_t *usage_path;
-        usb_hid_report_path_t *tmp_usage_path;
 
         size_t offset_input=0;
         size_t offset_output=0;
         size_t offset_feature=0;
-        
+
+        link_t stack;
+        list_initialize(&stack);        
 
         /* parser structure initialization*/
-        if(usb_hid_parser_init(parser) != EOK) {
+        if(usb_hid_report_init(report) != EOK) {
                 return EINVAL;
         }
         
-
         /*report item initialization*/
         if(!(report_item=malloc(sizeof(usb_hid_report_item_t)))){
@@ -159 +315 @@
 
                         if((i+USB_HID_ITEM_SIZE(data[i]))>= size){
-                                return EINVAL; // TODO ERROR CODE
+                                return EINVAL;
                         }
                         
@@ -165 +321 @@
                         item_size = USB_HID_ITEM_SIZE(data[i]);
                         class = USB_HID_ITEM_TAG_CLASS(data[i]);
-
-                        usb_log_debug2(
-                                "i(%u) data(%X) value(%X): TAG %u, class %u, size %u - ", i, 
-                            data[i], usb_hid_report_tag_data_int32(data+i+1,item_size), 
-                            tag, class, item_size);
                         
                         ret = usb_hid_report_parse_tag(tag,class,data+i+1,
                                                        item_size,report_item, usage_path);
-                        usb_log_debug2("ret: %u\n", ret);
                         switch(ret){
                                 case USB_HID_NEW_REPORT_ITEM:
                                         // store report item to report and create the new one
-                                        usb_log_debug("\nNEW REPORT ITEM: %X",ret);
-
-                                        // store current usage path
+                                        // store current collection path
                                         report_item->usage_path = usage_path;
                                         
-                                        // clone path to the new one
-                                        tmp_usage_path = usb_hid_report_path_clone(usage_path);
-
-                                        // swap
-                                        usage_path = tmp_usage_path;
-                                        tmp_usage_path = NULL;
-
                                         usb_hid_report_path_set_report_id(report_item->usage_path, report_item->id);    
                                         if(report_item->id != 0){
-                                                parser->use_report_id = 1;
+                                                report->use_report_ids = 1;
                                         }
                                         
                                         switch(tag) {
                                                 case USB_HID_REPORT_TAG_INPUT:
+                                                        report_item->type = USB_HID_REPORT_TYPE_INPUT;
                                                         report_item->offset = offset_input;
                                                         offset_input += report_item->count * report_item->size;
-                                                        usb_log_debug(" - INPUT\n");
-                                                        list_append(&(report_item->link), &(parser->input));
                                                         break;
                                                 case USB_HID_REPORT_TAG_OUTPUT:
+                                                        report_item->type = USB_HID_REPORT_TYPE_OUTPUT;
                                                         report_item->offset = offset_output;
                                                         offset_output += report_item->count * report_item->size;
-                                                        usb_log_debug(" - OUTPUT\n");
-                                                                list_append(&(report_item->link), &(parser->output));
 
                                                         break;
                                                 case USB_HID_REPORT_TAG_FEATURE:
+                                                        report_item->type = USB_HID_REPORT_TYPE_FEATURE;
                                                         report_item->offset = offset_feature;
                                                         offset_feature += report_item->count * report_item->size;
-                                                        usb_log_debug(" - FEATURE\n");
-                                                                list_append(&(report_item->link), &(parser->feature));
                                                         break;
                                                 default:
@@ -218 +356 @@
                                                     break;
                                         }
-
-                                        /* clone current state table to the new item */
-                                        if(!(new_report_item = malloc(sizeof(usb_hid_report_item_t)))) {
-                                                return ENOMEM;
-                                        }                                       
-                                        memcpy(new_report_item,report_item, sizeof(usb_hid_report_item_t));
-                                        link_initialize(&(new_report_item->link));
                                         
+                                        /* 
+                                         * append new fields to the report
+                                         * structure                                     
+                                         */
+                                        usb_hid_report_append_fields(report, report_item);
+
                                         /* reset local items */
-                                        new_report_item->usage_minimum = 0;
-                                        new_report_item->usage_maximum = 0;
-                                        new_report_item->designator_index = 0;
-                                        new_report_item->designator_minimum = 0;
-                                        new_report_item->designator_maximum = 0;
-                                        new_report_item->string_index = 0;
-                                        new_report_item->string_minimum = 0;
-                                        new_report_item->string_maximum = 0;
-
-                                        /* reset usage from current usage path */
-                                        usb_hid_report_usage_path_t *path = list_get_instance(&usage_path->link, usb_hid_report_usage_path_t, link);
-                                        path->usage = 0;
-                                        
-                                        report_item = new_report_item;
-                                                                                
+                                        usb_hid_report_reset_local_items (report_item);
+
                                         break;
+
+                                case USB_HID_RESET_OFFSET:
+                                        offset_input = 0;
+                                        offset_output = 0;
+                                        offset_feature = 0;
+                                        usb_hid_report_path_set_report_id (usage_path, report_item->id);
+                                        break;
+
                                 case USB_HID_REPORT_TAG_PUSH:
                                         // push current state to stack
                                         new_report_item = usb_hid_report_item_clone(report_item);
-                                        list_prepend (&parser->stack, &new_report_item->link);
-                                        
+                                        usb_hid_report_path_t *tmp_path = usb_hid_report_path_clone(usage_path);
+                                        new_report_item->usage_path = tmp_path; 
+
+                                        list_prepend (&new_report_item->link, &stack);
                                         break;
                                 case USB_HID_REPORT_TAG_POP:
                                         // restore current state from stack
-                                        if(list_empty (&parser->stack)) {
+                                        if(list_empty (&stack)) {
                                                 return EINVAL;
                                         }
+                                        free(report_item);
+                                                
+                                        report_item = list_get_instance(stack.next, usb_hid_report_item_t, link);
                                         
-                                        report_item = list_get_instance(&parser->stack, usb_hid_report_item_t, link);
-                                        list_remove (parser->stack.next);
+                                        usb_hid_report_usage_path_t *tmp_usage_path;
+                                        tmp_usage_path = list_get_instance(report_item->usage_path->link.prev, usb_hid_report_usage_path_t, link);
+                                        
+                                        usb_hid_report_set_last_item(usage_path, tmp_usage_path->usage_page, tmp_usage_path->usage);
+
+                                        usb_hid_report_path_free(report_item->usage_path);
+                                        list_initialize(&report_item->usage_path->link);
+                                        list_remove (stack.next);
                                         
                                         break;
@@ -279 +422 @@
 }
 
-
-/**
- * Parse input report.
- *
- * @param data Data for report
- * @param size Size of report
- * @param callbacks Callbacks for report actions
- * @param arg Custom arguments
- *
- * @return Error code
- */
-int usb_hid_boot_keyboard_input_report(const uint8_t *data, size_t size,
-        const usb_hid_report_in_callbacks_t *callbacks, void *arg)
-{
-        int i;
-        usb_hid_report_item_t item;
-
-        /* fill item due to the boot protocol report descriptor */
-        // modifier keys are in the first byte
-        uint8_t modifiers = data[0];
-
-        item.offset = 2; /* second byte is reserved */
-        item.size = 8;
-        item.count = 6;
-        item.usage_minimum = 0;
-        item.usage_maximum = 255;
-        item.logical_minimum = 0;
-        item.logical_maximum = 255;
-
-        if (size != 8) {
-                return -1; //ERANGE;
-        }
-
-        uint8_t keys[6];
-        for (i = 0; i < item.count; i++) {
-                keys[i] = data[i + item.offset];
-        }
-
-        callbacks->keyboard(keys, 6, modifiers, arg);
-        return EOK;
-}
-
-/**
- * Makes output report for keyboard boot protocol
- *
- * @param leds
- * @param output Output report data buffer
- * @param size Size of the output buffer
- * @return Error code
- */
-int usb_hid_boot_keyboard_output_report(uint8_t leds, uint8_t *data, size_t size)
-{
-        if(size != 1){
-                return -1;
-        }
-
-        /* used only first five bits, others are only padding*/
-        *data = leds;
-        return EOK;
-}
 
 /**
@@ -401 +484 @@
                         
                 case USB_HID_REPORT_TAG_COLLECTION:
-                        usb_hid_report_path_append_item(usage_path, 0, 0);
-                                                
+                        // TODO usage_path->flags = *data;
+                        usb_hid_report_path_append_item(usage_path, report_item->usage_page, report_item->usages[report_item->usages_count-1]);                                         
+                        usb_hid_report_reset_local_items (report_item);
                         return USB_HID_NO_ACTION;
                         break;
                         
                 case USB_HID_REPORT_TAG_END_COLLECTION:
-                        // TODO
-                        // znici posledni uroven ve vsech usage paths
-                        // otazka jestli nema nicit dve, respektive novou posledni vynulovat?
                         usb_hid_report_remove_last_item(usage_path);
                         return USB_HID_NO_ACTION;
@@ -436 +517 @@
         {
                 case USB_HID_REPORT_TAG_USAGE_PAGE:
-                        // zmeni to jenom v poslednim poli aktualni usage path
-                        usb_hid_report_set_last_item(usage_path, USB_HID_TAG_CLASS_GLOBAL,
-                                usb_hid_report_tag_data_int32(data,item_size));
+                        report_item->usage_page = usb_hid_report_tag_data_uint32(data, item_size);
                         break;
                 case USB_HID_REPORT_TAG_LOGICAL_MINIMUM:
-                        report_item->logical_minimum = usb_hid_report_tag_data_int32(data,item_size);
+                        report_item->logical_minimum = USB_HID_UINT32_TO_INT32(usb_hid_report_tag_data_uint32(data,item_size), item_size * 8);
                         break;
                 case USB_HID_REPORT_TAG_LOGICAL_MAXIMUM:
-                        report_item->logical_maximum = usb_hid_report_tag_data_int32(data,item_size);
+                        report_item->logical_maximum = USB_HID_UINT32_TO_INT32(usb_hid_report_tag_data_uint32(data,item_size), item_size * 8);
                         break;
                 case USB_HID_REPORT_TAG_PHYSICAL_MINIMUM:
-                        report_item->physical_minimum = usb_hid_report_tag_data_int32(data,item_size);
+                        report_item->physical_minimum = USB_HID_UINT32_TO_INT32(usb_hid_report_tag_data_uint32(data,item_size), item_size * 8);
                         break;                  
                 case USB_HID_REPORT_TAG_PHYSICAL_MAXIMUM:
-                        report_item->physical_maximum = usb_hid_report_tag_data_int32(data,item_size);
+                        report_item->physical_maximum = USB_HID_UINT32_TO_INT32(usb_hid_report_tag_data_uint32(data,item_size), item_size * 8);
+
                         break;
                 case USB_HID_REPORT_TAG_UNIT_EXPONENT:
-                        report_item->unit_exponent = usb_hid_report_tag_data_int32(data,item_size);
+                        report_item->unit_exponent = usb_hid_report_tag_data_uint32(data,item_size);
                         break;
                 case USB_HID_REPORT_TAG_UNIT:
-                        report_item->unit = usb_hid_report_tag_data_int32(data,item_size);
+                        report_item->unit = usb_hid_report_tag_data_uint32(data,item_size);
                         break;
                 case USB_HID_REPORT_TAG_REPORT_SIZE:
-                        report_item->size = usb_hid_report_tag_data_int32(data,item_size);
+                        report_item->size = usb_hid_report_tag_data_uint32(data,item_size);
                         break;
                 case USB_HID_REPORT_TAG_REPORT_COUNT:
-                        report_item->count = usb_hid_report_tag_data_int32(data,item_size);
+                        report_item->count = usb_hid_report_tag_data_uint32(data,item_size);
                         break;
                 case USB_HID_REPORT_TAG_REPORT_ID:
-                        report_item->id = usb_hid_report_tag_data_int32(data,item_size);
+                        report_item->id = usb_hid_report_tag_data_uint32(data,item_size);
+                        return USB_HID_RESET_OFFSET;
                         break;
                 case USB_HID_REPORT_TAG_PUSH:
                 case USB_HID_REPORT_TAG_POP:
+                        /* 
+                         * stack operations are done in top level parsing
+                         * function
+                         */
                         return tag;
                         break;
@@ -475 +560 @@
                         return USB_HID_NO_ACTION;
         }
-        
+
         return EOK;
 }
@@ -494 +579 @@
         {
                 case USB_HID_REPORT_TAG_USAGE:
-                        usb_hid_report_set_last_item(usage_path, USB_HID_TAG_CLASS_LOCAL,
-                                usb_hid_report_tag_data_int32(data,item_size));
+                        report_item->usages[report_item->usages_count] = usb_hid_report_tag_data_uint32(data,item_size);
+                        report_item->usages_count++;
                         break;
                 case USB_HID_REPORT_TAG_USAGE_MINIMUM:
-                        report_item->usage_minimum = usb_hid_report_tag_data_int32(data,item_size);
+                        if (item_size == 3) {
+                                // usage extended usages
+                                report_item->extended_usage_page = (usb_hid_report_tag_data_uint32(data,item_size) & 0xFF00) >> 16; 
+                                report_item->usage_minimum = usb_hid_report_tag_data_uint32(data,item_size) & 0xFF;
+                        }
+                        else {
+                                report_item->usage_minimum = usb_hid_report_tag_data_uint32(data,item_size);
+                        }
                         break;
                 case USB_HID_REPORT_TAG_USAGE_MAXIMUM:
-                        report_item->usage_maximum = usb_hid_report_tag_data_int32(data,item_size);
+                        if (item_size == 3) {
+                                // usage extended usages
+                                report_item->extended_usage_page = (usb_hid_report_tag_data_uint32(data,item_size) & 0xFF00) >> 16; 
+                                report_item->usage_maximum = usb_hid_report_tag_data_uint32(data,item_size) & 0xFF;
+                        }
+                        else {
+                                report_item->usage_maximum = usb_hid_report_tag_data_uint32(data,item_size);
+                        }
                         break;
                 case USB_HID_REPORT_TAG_DESIGNATOR_INDEX:
-                        report_item->designator_index = usb_hid_report_tag_data_int32(data,item_size);
+                        report_item->designator_index = usb_hid_report_tag_data_uint32(data,item_size);
                         break;
                 case USB_HID_REPORT_TAG_DESIGNATOR_MINIMUM:
-                        report_item->designator_minimum = usb_hid_report_tag_data_int32(data,item_size);
+                        report_item->designator_minimum = usb_hid_report_tag_data_uint32(data,item_size);
                         break;
                 case USB_HID_REPORT_TAG_DESIGNATOR_MAXIMUM:
-                        report_item->designator_maximum = usb_hid_report_tag_data_int32(data,item_size);
+                        report_item->designator_maximum = usb_hid_report_tag_data_uint32(data,item_size);
                         break;
                 case USB_HID_REPORT_TAG_STRING_INDEX:
-                        report_item->string_index = usb_hid_report_tag_data_int32(data,item_size);
+                        report_item->string_index = usb_hid_report_tag_data_uint32(data,item_size);
                         break;
                 case USB_HID_REPORT_TAG_STRING_MINIMUM:
-                        report_item->string_minimum = usb_hid_report_tag_data_int32(data,item_size);
+                        report_item->string_minimum = usb_hid_report_tag_data_uint32(data,item_size);
                         break;
                 case USB_HID_REPORT_TAG_STRING_MAXIMUM:
-                        report_item->string_maximum = usb_hid_report_tag_data_int32(data,item_size);
+                        report_item->string_maximum = usb_hid_report_tag_data_uint32(data,item_size);
                         break;                  
                 case USB_HID_REPORT_TAG_DELIMITER:
-                        report_item->delimiter = usb_hid_report_tag_data_int32(data,item_size);
+                        //report_item->delimiter = usb_hid_report_tag_data_uint32(data,item_size);
+                        //TODO: 
+                        //      DELIMITER STUFF
                         break;
                 
@@ -533 +634 @@
 
 /**
- * Converts raw data to int32 (thats the maximum length of short item data)
+ * Converts raw data to uint32 (thats the maximum length of short item data)
  *
  * @param Data buffer
@@ -539 +640 @@
  * @return Converted int32 number
  */
-int32_t usb_hid_report_tag_data_int32(const uint8_t *data, size_t size)
+uint32_t usb_hid_report_tag_data_uint32(const uint8_t *data, size_t size)
 {
         unsigned int i;
-        int32_t result;
+        uint32_t result;
 
         result = 0;
@@ -552 +653 @@
 }
 
-
-
 /**
  * Prints content of given list of report items.
@@ -562 +661 @@
 void usb_hid_descriptor_print_list(link_t *head)
 {
-        usb_hid_report_item_t *report_item;
-        usb_hid_report_usage_path_t *path_item;
-        link_t *path;
+        usb_hid_report_field_t *report_item;
         link_t *item;
-        
+
+
         if(head == NULL || list_empty(head)) {
             usb_log_debug("\tempty\n");
@@ -574 +672 @@
         for(item = head->next; item != head; item = item->next) {
                 
-                report_item = list_get_instance(item, usb_hid_report_item_t, link);
-
-                usb_log_debug("\tOFFSET: %X\n", report_item->offset);
-                usb_log_debug("\tCOUNT: %X\n", report_item->count);
-                usb_log_debug("\tSIZE: %X\n", report_item->size);
-                usb_log_debug("\tCONSTANT/VAR: %X\n", USB_HID_ITEM_FLAG_CONSTANT(report_item->item_flags));
-                usb_log_debug("\tVARIABLE/ARRAY: %X\n", USB_HID_ITEM_FLAG_VARIABLE(report_item->item_flags));
-                usb_log_debug("\tUSAGE PATH:\n");
-
-                path = report_item->usage_path->link.next;
-                while(path != &report_item->usage_path->link)   {
-                        path_item = list_get_instance(path, usb_hid_report_usage_path_t, link);
-                        usb_log_debug("\t\tUSAGE PAGE: %X, USAGE: %X\n", path_item->usage_page, path_item->usage);
-                        path = path->next;
-                }
-                                
-                usb_log_debug("\tLOGMIN: %X\n", report_item->logical_minimum);
-                usb_log_debug("\tLOGMAX: %X\n", report_item->logical_maximum);          
-                usb_log_debug("\tPHYMIN: %X\n", report_item->physical_minimum);         
-                usb_log_debug("\tPHYMAX: %X\n", report_item->physical_maximum);                         
-                usb_log_debug("\tUSAGEMIN: %X\n", report_item->usage_minimum);
-                usb_log_debug("\tUSAGEMAX: %X\n", report_item->usage_maximum);
-                
-                usb_log_debug("\n");            
+                report_item = list_get_instance(item, usb_hid_report_field_t, link);
+
+                usb_log_debug("\t\tOFFSET: %X\n", report_item->offset);
+                usb_log_debug("\t\tSIZE: %X\n", report_item->size);                             
+                usb_log_debug("\t\tLOGMIN: %d\n", report_item->logical_minimum);
+                usb_log_debug("\t\tLOGMAX: %d\n", report_item->logical_maximum);                
+                usb_log_debug("\t\tPHYMIN: %d\n", report_item->physical_minimum);               
+                usb_log_debug("\t\tPHYMAX: %d\n", report_item->physical_maximum);                               
+                usb_log_debug("\t\ttUSAGEMIN: %X\n", report_item->usage_minimum);
+                usb_log_debug("\t\tUSAGEMAX: %X\n", report_item->usage_maximum);
+
+                usb_log_debug("\t\tVALUE: %X\n", report_item->value);
+                usb_log_debug("\t\ttUSAGE: %X\n", report_item->usage);
+                usb_log_debug("\t\tUSAGE PAGE: %X\n", report_item->usage_page);
+                                                
+//              usb_log_debug("\n");            
 
         }
@@ -609 +699 @@
  * @return void
  */
-void usb_hid_descriptor_print(usb_hid_report_parser_t *parser)
-{
-        if(parser == NULL) {
+void usb_hid_descriptor_print(usb_hid_report_t *report)
+{
+        if(report == NULL) {
                 return;
         }
-        
-        usb_log_debug("INPUT:\n");
-        usb_hid_descriptor_print_list(&parser->input);
-        
-        usb_log_debug("OUTPUT: \n");
-        usb_hid_descriptor_print_list(&parser->output);
-        
-        usb_log_debug("FEATURE:\n");    
-        usb_hid_descriptor_print_list(&parser->feature);
-
+
+        link_t *report_it = report->reports.next;
+        usb_hid_report_description_t *report_des;
+
+        while(report_it != &report->reports) {
+                report_des = list_get_instance(report_it, usb_hid_report_description_t, link);
+                usb_log_debug("Report ID: %d\n", report_des->report_id);
+                usb_log_debug("\tType: %d\n", report_des->type);
+                usb_log_debug("\tLength: %d\n", report_des->bit_length);                
+                usb_log_debug("\tItems: %d\n", report_des->item_length);                
+
+                usb_hid_descriptor_print_list(&report_des->report_items);
+
+
+                link_t *path_it = report->collection_paths.next;
+                while(path_it != &report->collection_paths) {
+                        usb_hid_print_usage_path (list_get_instance(path_it, usb_hid_report_path_t, link));
+                        path_it = path_it->next;
+                }
+                
+                report_it = report_it->next;
+        }
 }
 
@@ -667 +769 @@
  * @return void
  */
-void usb_hid_free_report_parser(usb_hid_report_parser_t *parser)
-{
-        if(parser == NULL){
+void usb_hid_free_report(usb_hid_report_t *report)
+{
+        if(report == NULL){
                 return;
         }
 
-        parser->use_report_id = 0;
-
-        usb_hid_free_report_list(&parser->input);
-        usb_hid_free_report_list(&parser->output);
-        usb_hid_free_report_list(&parser->feature);
-
+        // free collection paths
+        usb_hid_report_path_t *path;
+        while(!list_empty(&report->collection_paths)) {
+                path = list_get_instance(report->collection_paths.next, usb_hid_report_path_t, link);
+                usb_hid_report_path_free(path);         
+        }
+        
+        // free report items
+        usb_hid_report_description_t *report_des;
+        usb_hid_report_field_t *field;
+        while(!list_empty(&report->reports)) {
+                report_des = list_get_instance(report->reports.next, usb_hid_report_description_t, link);
+                list_remove(&report_des->link);
+                
+                while(!list_empty(&report_des->report_items)) {
+                        field = list_get_instance(report_des->report_items.next, usb_hid_report_field_t, link);
+                        list_remove(&field->link);
+
+                        free(field);
+                }
+                
+                free(report_des);
+        }
+        
         return;
 }
@@ -688 +808 @@
  * @param parser Opaque HID report parser structure.
  * @param data Data for the report.
- * @param callbacks Callbacks for report actions.
- * @param arg Custom argument (passed through to the callbacks).
  * @return Error code.
  */ 
-int usb_hid_parse_report(const usb_hid_report_parser_t *parser,  
-    const uint8_t *data, size_t size,
-    usb_hid_report_path_t *path, int flags,
-    const usb_hid_report_in_callbacks_t *callbacks, void *arg)
+int usb_hid_parse_report(const usb_hid_report_t *report,  
+    const uint8_t *data, size_t size, uint8_t *report_id)
 {
         link_t *list_item;
-        usb_hid_report_item_t *item;
-        uint8_t *keys;
-        uint8_t item_value;
-        size_t key_count=0;
-        size_t i=0;
-        size_t j=0;
-        uint8_t report_id = 0;
-
-        if(parser == NULL) {
+        usb_hid_report_field_t *item;
+
+        usb_hid_report_description_t *report_des;
+        usb_hid_report_type_t type = USB_HID_REPORT_TYPE_INPUT;
+
+        if(report == NULL) {
                 return EINVAL;
         }
-        
-        /* get the size of result array */
-        key_count = usb_hid_report_input_length(parser, path, flags);
-
-        if(!(keys = malloc(sizeof(uint8_t) * key_count))){
-                return ENOMEM;
-        }
-
-        if(parser->use_report_id != 0) {
-                report_id = data[0];
-                usb_hid_report_path_set_report_id(path, report_id);
-        }
+
+        if(report->use_report_ids != 0) {
+                *report_id = data[0];
+        }       
+        else {
+                *report_id = 0;
+        }
+
+
+        report_des = usb_hid_report_find_description(report, *report_id, type);
 
         /* read data */
-        list_item = parser->input.next;    
-        while(list_item != &(parser->input)) {
-
-                item = list_get_instance(list_item, usb_hid_report_item_t, link);
-
-                if(!USB_HID_ITEM_FLAG_CONSTANT(item->item_flags) && 
-                   (usb_hid_report_compare_usage_path(item->usage_path, path, flags) == EOK)) {
-                        for(j=0; j<(size_t)(item->count); j++) {
-                                if((USB_HID_ITEM_FLAG_VARIABLE(item->item_flags) == 0) ||
-                                   ((item->usage_minimum == 0) && (item->usage_maximum == 0))) {
-                                        // variable item
-                                        keys[i++] = usb_hid_translate_data(item, data,j);
-                                }
-                                else {
-                                        // bitmapa
-                                        if((item_value = usb_hid_translate_data(item, data, j)) != 0) {
-                                                keys[i++] = (item->count - 1 - j) + item->usage_minimum;
-                                        }
-                                        else {
-                                                keys[i++] = 0;
-                                        }
-                                }
-                        }
+        list_item = report_des->report_items.next;         
+        while(list_item != &(report_des->report_items)) {
+
+                item = list_get_instance(list_item, usb_hid_report_field_t, link);
+
+                if(USB_HID_ITEM_FLAG_CONSTANT(item->item_flags) == 0) {
+                        
+                        if(USB_HID_ITEM_FLAG_VARIABLE(item->item_flags) == 0) {
+
+                                // array
+                                item->value = usb_hid_translate_data(item, data);
+                            item->usage = (item->value - item->physical_minimum) + item->usage_minimum;
+                        }
+                        else {
+                                // variable item
+                                item->value = usb_hid_translate_data(item, data);                               
+                        }                               
                 }
                 list_item = list_item->next;
         }
-
-        callbacks->keyboard(keys, key_count, report_id, arg);
            
-        free(keys);     
         return EOK;
         
@@ -758 +860 @@
 
 /**
- * Translate data from the report as specified in report descriptor
+ * Translate data from the report as specified in report descriptor item
  *
  * @param item Report descriptor item with definition of translation
@@ -765 +867 @@
  * @return Translated data
  */
-int usb_hid_translate_data(usb_hid_report_item_t *item, const uint8_t *data, size_t j)
+int usb_hid_translate_data(usb_hid_report_field_t *item, const uint8_t *data)
 {
         int resolution;
@@ -771 +873 @@
         int part_size;
         
-        int32_t value;
+        int32_t value=0;
         int32_t mask;
         const uint8_t *foo;
 
-        // now only common numbers llowed
+        // now only shot tags are allowed
         if(item->size > 32) {
                 return 0;
         }
 
-        if((item->physical_minimum == 0) && (item->physical_maximum == 0)) {
+        if((item->physical_minimum == 0) && (item->physical_maximum == 0)){
                 item->physical_minimum = item->logical_minimum;
-                item->physical_maximum = item->logical_maximum;         
-        }
+                item->physical_maximum = item->logical_maximum;                 
+        }
+        
 
         if(item->physical_maximum == item->physical_minimum){
@@ -794 +897 @@
         }
 
-        offset = item->offset + (j * item->size);
-        if(item->id != 0) {
-                offset += 8;
-                usb_log_debug("MOVED OFFSET BY 1Byte, REPORT_ID(%d)\n", item->id);
-        }
-        
+        offset = item->offset;
         // FIXME
-        if((offset/8) != ((offset+item->size)/8)) {
-                usb_log_debug2("offset %d\n", offset);
+        if((size_t)(offset/8) != (size_t)((offset+item->size-1)/8)) {
                 
                 part_size = ((offset+item->size)%8);
-                usb_log_debug2("part size %d\n",part_size);
-
-                // the higher one
-                foo = data+(offset/8);
-                mask =  ((1 << (item->size-part_size))-1);
-                value = (*foo & mask) << part_size;
-
-                usb_log_debug2("hfoo %x\n", *foo);
-                usb_log_debug2("hmaska %x\n",  mask);
-                usb_log_debug2("hval %d\n", value);             
-
-                // the lower one
-                foo = data+((offset+item->size)/8);
-                mask =  ((1 << part_size)-1) << (8-part_size);
-                value += ((*foo & mask) >> (8-part_size));
-
-                usb_log_debug2("lfoo %x\n", *foo);
-                usb_log_debug2("lmaska %x\n",  mask);
-                usb_log_debug2("lval %d\n", ((*foo & mask) >> (8-(item->size-part_size))));             
-                usb_log_debug2("val %d\n", value);
-                
-                
+
+                size_t i=0;
+                for(i=(size_t)(offset/8); i<=(size_t)(offset+item->size-1)/8; i++){
+                        if(i == (size_t)(offset/8)) {
+                                // the higher one
+                                foo = data + i;
+                                mask =  ((1 << (item->size-part_size))-1);
+                                value = (*foo & mask) << part_size;
+                        }
+                        else if(i == ((offset+item->size-1)/8)){
+                                // the lower one
+                                foo = data + i;
+                                mask =  ((1 << part_size)-1) << (8-part_size);
+                                value += ((*foo & mask) >> (8-part_size));
+                        }
+                        else {
+                                value = value << 8;
+                                value += *(data + 1);
+                        }
+                }
         }
         else {          
@@ -832 +927 @@
                 mask =  ((1 << item->size)-1) << (8-((offset%8)+item->size));
                 value = (*foo & mask) >> (8-((offset%8)+item->size));
-
-                usb_log_debug2("offset %d\n", offset);
-        
-                usb_log_debug2("foo %x\n", *foo);
-                usb_log_debug2("maska %x\n",  mask);
-                usb_log_debug2("val %d\n", value);                              
-        }
-
-        usb_log_debug2("---\n\n"); 
+        }
+
+        if((item->logical_minimum < 0) || (item->logical_maximum < 0)){
+                value = USB_HID_UINT32_TO_INT32(value, item->size);
+        }
 
         return (int)(((value - item->logical_minimum) / resolution) + item->physical_minimum);
@@ -847 +938 @@
 
 /**
- *
- *
- * @param parser
- * @param path
- * @param flags
- * @return
- */
-size_t usb_hid_report_input_length(const usb_hid_report_parser_t *parser,
+ * Returns number of items in input report which are accessible by given usage path
+ *
+ * @param parser Opaque report descriptor structure
+ * @param path Usage path specification
+ * @param flags Usage path comparison flags
+ * @return Number of items in input report
+ */
+size_t usb_hid_report_input_length(const usb_hid_report_t *report,
         usb_hid_report_path_t *path, int flags)
 {       
+        
         size_t ret = 0;
-        link_t *item;
-        usb_hid_report_item_t *report_item;
-
-        if(parser == NULL) {
+
+        if(report == NULL) {
                 return 0;
         }
-        
-        item = parser->input.next;
-        while(&parser->input != item) {
-                report_item = list_get_instance(item, usb_hid_report_item_t, link);
-                if(!USB_HID_ITEM_FLAG_CONSTANT(report_item->item_flags) &&
-                   (usb_hid_report_compare_usage_path(report_item->usage_path, path, flags) == EOK)) {
-                        ret += report_item->count;
-                }
-
-                item = item->next;
-        } 
+
+        usb_hid_report_description_t *report_des;
+        report_des = usb_hid_report_find_description (report, path->report_id, USB_HID_REPORT_TYPE_INPUT);
+        if(report_des == NULL) {
+                return 0;
+        }
+
+        link_t *field_it = report_des->report_items.next;
+        usb_hid_report_field_t *field;
+        while(field_it != &report_des->report_items) {
+
+                field = list_get_instance(field_it, usb_hid_report_field_t, link);
+                if(USB_HID_ITEM_FLAG_CONSTANT(field->item_flags) == 0) {
+                        
+                        usb_hid_report_path_append_item (field->collection_path, field->usage_page, field->usage);
+                        if(usb_hid_report_compare_usage_path (field->collection_path, path, flags) == EOK) {
+                                ret++;
+                        }
+                        usb_hid_report_remove_last_item (field->collection_path);
+                }
+                
+                field_it = field_it->next;
+        }
 
         return ret;
-}
-
-
-/**
- * 
- * @param usage_path
- * @param usage_page
- * @param usage
- * @return
+        }
+
+
+/**
+ * Appends one item (couple of usage_path and usage) into the usage path
+ * structure
+ *
+ * @param usage_path Usage path structure
+ * @param usage_page Usage page constant
+ * @param usage Usage constant
+ * @return Error code
  */
 int usb_hid_report_path_append_item(usb_hid_report_path_t *usage_path, 
@@ -899 +1003 @@
         item->usage = usage;
         item->usage_page = usage_page;
-        
-        list_append (&usage_path->link, &item->link);
+        item->flags = 0;
+        
+        list_append (&item->link, &usage_path->head);
         usage_path->depth++;
         return EOK;
@@ -906 +1011 @@
 
 /**
- *
- * @param usage_path
- * @return
+ * Removes last item from the usage path structure
+ * @param usage_path 
+ * @return void
  */
 void usb_hid_report_remove_last_item(usb_hid_report_path_t *usage_path)
@@ -914 +1019 @@
         usb_hid_report_usage_path_t *item;
         
-        if(!list_empty(&usage_path->link)){
-                item = list_get_instance(usage_path->link.prev, usb_hid_report_usage_path_t, link);             
-                list_remove(usage_path->link.prev);
+        if(!list_empty(&usage_path->head)){
+                item = list_get_instance(usage_path->head.prev, usb_hid_report_usage_path_t, link);             
+                list_remove(usage_path->head.prev);
                 usage_path->depth--;
                 free(item);
@@ -923 +1028 @@
 
 /**
+ * Nulls last item of the usage path structure.
  *
  * @param usage_path
- * @return
+ * @return void
  */
 void usb_hid_report_null_last_item(usb_hid_report_path_t *usage_path)
@@ -931 +1037 @@
         usb_hid_report_usage_path_t *item;
         
-        if(!list_empty(&usage_path->link)){     
-                item = list_get_instance(usage_path->link.prev, usb_hid_report_usage_path_t, link);
+        if(!list_empty(&usage_path->head)){     
+                item = list_get_instance(usage_path->head.prev, usb_hid_report_usage_path_t, link);
                 memset(item, 0, sizeof(usb_hid_report_usage_path_t));
         }
@@ -938 +1044 @@
 
 /**
- *
- * @param usage_path
- * @param tag
- * @param data
- * @return
+ * Modifies last item of usage path structure by given usage page or usage
+ *
+ * @param usage_path Opaque usage path structure
+ * @param tag Class of currently processed tag (Usage page tag falls into Global
+ * class but Usage tag into the Local)
+ * @param data Value of the processed tag
+ * @return void
  */
 void usb_hid_report_set_last_item(usb_hid_report_path_t *usage_path, int32_t tag, int32_t data)
@@ -948 +1056 @@
         usb_hid_report_usage_path_t *item;
         
-        if(!list_empty(&usage_path->link)){     
-                item = list_get_instance(usage_path->link.prev, usb_hid_report_usage_path_t, link);
+        if(!list_empty(&usage_path->head)){     
+                item = list_get_instance(usage_path->head.prev, usb_hid_report_usage_path_t, link);
 
                 switch(tag) {
@@ -963 +1071 @@
 }
 
-/**
- * 
- *
- * @param report_path
- * @param path
- * @param flags
- * @return
+
+void usb_hid_print_usage_path(usb_hid_report_path_t *path)
+{
+        usb_log_debug("USAGE_PATH FOR RId(%d):\n", path->report_id);
+        usb_log_debug("\tLENGTH: %d\n", path->depth);
+
+        link_t *item = path->head.next;
+        usb_hid_report_usage_path_t *path_item;
+        while(item != &path->head) {
+
+                path_item = list_get_instance(item, usb_hid_report_usage_path_t, link);
+                usb_log_debug("\tUSAGE_PAGE: %X\n", path_item->usage_page);
+                usb_log_debug("\tUSAGE: %X\n", path_item->usage);
+                usb_log_debug("\tFLAGS: %d\n", path_item->flags);               
+                
+                item = item->next;
+        }
+}
+
+/**
+ * Compares two usage paths structures
+ *
+ * If USB_HID_PATH_COMPARE_COLLECTION_ONLY flag is given, the last item in report_path structure is forgotten
+ *
+ * @param report_path usage path structure to compare
+ * @param path usage patrh structure to compare
+ * @param flags Flags determining the mode of comparison
+ * @return EOK if both paths are identical, non zero number otherwise
  */
 int usb_hid_report_compare_usage_path(usb_hid_report_path_t *report_path, 
@@ -1003 +1132 @@
                                 }
 
-                                report_link = report_path->link.next;
-                                path_link = path->link.next;
+                                report_link = report_path->head.next;
+                                path_link = path->head.next;
                         
-                                while((report_link != &report_path->link) && (path_link != &path->link)) {
+                                while((report_link != &report_path->head) && (path_link != &path->head)) {
                                         report_item = list_get_instance(report_link, usb_hid_report_usage_path_t, link);
                                         path_item = list_get_instance(path_link, usb_hid_report_usage_path_t, link);            
@@ -1020 +1149 @@
                                 }
 
-                                if((report_link == &report_path->link) && (path_link == &path->link)) {
+                                if(((report_link == &report_path->head) && (path_link == &path->head)) || 
+                                   (((flags & USB_HID_PATH_COMPARE_COLLECTION_ONLY) != 0) && (path_link = &path->head) && (report_link == report_path->head.prev))) {
                                         return EOK;
                                 }
@@ -1030 +1160 @@
                 /* compare with only the end of path*/
                 case USB_HID_PATH_COMPARE_END:
-                                report_link = report_path->link.prev;
-                                path_link = path->link.prev;
-
-                                if(list_empty(&path->link)){
+
+                                if((flags & USB_HID_PATH_COMPARE_COLLECTION_ONLY) != 0) {
+                                        report_link = report_path->head.prev->prev;
+                                }
+                                else {
+                                        report_link = report_path->head.prev;
+                                }
+                                path_link = path->head.prev;
+
+                                if(list_empty(&path->head)){
                                         return EOK;
                                 }
                         
-                                while((report_link != &report_path->link) && (path_link != &path->link)) {
+                                while((report_link != &report_path->head) && (path_link != &path->head)) {
                                         report_item = list_get_instance(report_link, usb_hid_report_usage_path_t, link);
                                         path_item = list_get_instance(path_link, usb_hid_report_usage_path_t, link);            
@@ -1051 +1187 @@
                                 }
 
-                                if(path_link == &path->link) {
+                                if(path_link == &path->head) {
                                         return EOK;
                                 }
@@ -1070 +1206 @@
 
 /**
- *
- * @return
+ * Allocates and initializes new usage path structure.
+ *
+ * @return Initialized usage path structure
  */
 usb_hid_report_path_t *usb_hid_report_path(void)
@@ -1077 +1214 @@
         usb_hid_report_path_t *path;
         path = malloc(sizeof(usb_hid_report_path_t));
-        if(!path){
+        if(path == NULL){
                 return NULL;
         }
@@ -1084 +1221 @@
                 path->report_id = 0;
                 list_initialize(&path->link);
+                list_initialize(&path->head);
                 return path;
         }
@@ -1089 +1227 @@
 
 /**
- *
- * @param path
+ * Releases given usage path structure.
+ *
+ * @param path usage path structure to release
  * @return void
  */
 void usb_hid_report_path_free(usb_hid_report_path_t *path)
 {
-        while(!list_empty(&path->link)){
+        while(!list_empty(&path->head)){
                 usb_hid_report_remove_last_item(path);
         }
+
+        list_remove(&path->link);
+        free(path);
 }
 
@@ -1104 +1246 @@
  * Clone content of given usage path to the new one
  *
- * @param usage_path
- * @return
+ * @param usage_path Usage path structure to clone
+ * @return New copy of given usage path structure
  */
 usb_hid_report_path_t *usb_hid_report_path_clone(usb_hid_report_path_t *usage_path)
 {
+        link_t *path_link;
         usb_hid_report_usage_path_t *path_item;
-        link_t *path_link;
+        usb_hid_report_usage_path_t *new_path_item;
         usb_hid_report_path_t *new_usage_path = usb_hid_report_path ();
 
@@ -1116 +1259 @@
                 return NULL;
         }
-        
-        if(list_empty(&usage_path->link)){
+
+        new_usage_path->report_id = usage_path->report_id;
+        
+        if(list_empty(&usage_path->head)){
                 return new_usage_path;
         }
 
-        path_link = usage_path->link.next;
-        while(path_link != &usage_path->link) {
+        path_link = usage_path->head.next;
+        while(path_link != &usage_path->head) {
                 path_item = list_get_instance(path_link, usb_hid_report_usage_path_t, link);
-                usb_hid_report_path_append_item (new_usage_path, path_item->usage_page, path_item->usage);
+                new_path_item = malloc(sizeof(usb_hid_report_usage_path_t));
+                if(new_path_item == NULL) {
+                        return NULL;
+                }
+                
+                list_initialize (&new_path_item->link);         
+                new_path_item->usage_page = path_item->usage_page;
+                new_path_item->usage = path_item->usage;                
+                new_path_item->flags = path_item->flags;                
+                
+                list_append(&new_path_item->link, &new_usage_path->head);
+                new_usage_path->depth++;
 
                 path_link = path_link->next;
@@ -1135 +1291 @@
 /*** OUTPUT API **/
 
-/** Allocates output report buffer
- *
- * @param parser
- * @param size
- * @return
- */
-uint8_t *usb_hid_report_output(usb_hid_report_parser_t *parser, size_t *size)
-{
-        if(parser == NULL) {
+/** 
+ * Allocates output report buffer for output report
+ *
+ * @param parser Report parsed structure
+ * @param size Size of returned buffer
+ * @param report_id Report id of created output report
+ * @return Returns allocated output buffer for specified output
+ */
+uint8_t *usb_hid_report_output(usb_hid_report_t *report, size_t *size, uint8_t report_id)
+{
+        if(report == NULL) {
                 *size = 0;
                 return NULL;
         }
-        
-        // read the last output report item
-        usb_hid_report_item_t *last;
-        link_t *link;
-
-        link = parser->output.prev;
-        if(link != &parser->output) {
-                last = list_get_instance(link, usb_hid_report_item_t, link);
-                *size = (last->offset + (last->size * last->count)) / 8;
-
-                uint8_t *buffer = malloc(sizeof(uint8_t) * (*size));
-                memset(buffer, 0, sizeof(uint8_t) * (*size));
-                usb_log_debug("output buffer: %s\n", usb_debug_str_buffer(buffer, *size, 0));
-
-                return buffer;
+
+        link_t *report_it = report->reports.next;
+        usb_hid_report_description_t *report_des = NULL;
+        while(report_it != &report->reports) {
+                report_des = list_get_instance(report_it, usb_hid_report_description_t, link);
+                if((report_des->report_id == report_id) && (report_des->type == USB_HID_REPORT_TYPE_OUTPUT)){
+                        break;
+                }
+
+                report_it = report_it->next;
+        }
+
+        if(report_des == NULL){
+                *size = 0;
+                return NULL;
         }
         else {
-                *size = 0;              
-                return NULL;
+                *size = (report_des->bit_length + (8 - 1))/8;
+                uint8_t *ret = malloc((*size) * sizeof(uint8_t));
+                memset(ret, 0, (*size) * sizeof(uint8_t));
+                return ret;
         }
 }
@@ -1173 +1333 @@
  *
  * @param output Output report buffer
- * @return
+ * @return void
  */
 void usb_hid_report_output_free(uint8_t *output)
@@ -1185 +1345 @@
 /** Returns size of output for given usage path 
  *
- * @param parser
- * @param path
- * @param flags
- * @return
- */
-size_t usb_hid_report_output_size(usb_hid_report_parser_t *parser,
+ * @param parser Opaque report parser structure
+ * @param path Usage path specified which items will be thought for the output
+ * @param flags Flags of usage path structure comparison
+ * @return Number of items matching the given usage path
+ */
+size_t usb_hid_report_output_size(usb_hid_report_t *report,
                                   usb_hid_report_path_t *path, int flags)
 {
-        size_t ret = 0;
-        link_t *item;
-        usb_hid_report_item_t *report_item;
-
-        if(parser == NULL) {
+        size_t ret = 0; 
+        usb_hid_report_description_t *report_des;
+
+        if(report == NULL) {
                 return 0;
         }
 
-        item = parser->output.next;
-        while(&parser->output != item) {
-                report_item = list_get_instance(item, usb_hid_report_item_t, link);
-                if(!USB_HID_ITEM_FLAG_CONSTANT(report_item->item_flags) &&
-                   (usb_hid_report_compare_usage_path(report_item->usage_path, path, flags) == EOK)) {
-                        ret += report_item->count;
-                }
-
-                item = item->next;
-        } 
+        report_des = usb_hid_report_find_description (report, path->report_id, USB_HID_REPORT_TYPE_OUTPUT);
+        if(report_des == NULL){
+                return 0;
+        }
+        
+        link_t *field_it = report_des->report_items.next;
+        usb_hid_report_field_t *field;
+        while(field_it != &report_des->report_items) {
+
+                field = list_get_instance(field_it, usb_hid_report_field_t, link);
+                if(USB_HID_ITEM_FLAG_CONSTANT(field->item_flags) == 0){
+                        usb_hid_report_path_append_item (field->collection_path, field->usage_page, field->usage);
+                        if(usb_hid_report_compare_usage_path (field->collection_path, path, flags) == EOK) {
+                                ret++;
+                        }
+                        usb_hid_report_remove_last_item (field->collection_path);
+                }
+                
+                field_it = field_it->next;
+        }
 
         return ret;
@@ -1216 +1385 @@
 }
 
-/** Updates the output report buffer by translated given data 
- *
- * @param parser
- * @param path
- * @param flags
- * @param buffer
- * @param size
- * @param data
- * @param data_size
- * @return
- */
-int usb_hid_report_output_translate(usb_hid_report_parser_t *parser,
-                                    usb_hid_report_path_t *path, int flags,
-                                    uint8_t *buffer, size_t size,
-                                    int32_t *data, size_t data_size)
-{
-        usb_hid_report_item_t *report_item;
+/** Makes the output report buffer for data given in the report structure
+ *
+ * @param parser Opaque report parser structure
+ * @param path Usage path specifing which parts of output will be set
+ * @param flags Usage path structure comparison flags
+ * @param buffer Output buffer
+ * @param size Size of output buffer
+ * @return Error code
+ */
+int usb_hid_report_output_translate(usb_hid_report_t *report, uint8_t report_id,
+                                    uint8_t *buffer, size_t size)
+{
         link_t *item;   
-        size_t idx=0;
-        int i=0;
         int32_t value=0;
         int offset;
         int length;
         int32_t tmp_value;
-        size_t offset_prefix = 0;
-        
-        if(parser == NULL) {
+        
+        if(report == NULL) {
                 return EINVAL;
         }
 
-        if(parser->use_report_id != 0) {
-                buffer[0] = path->report_id;
-                offset_prefix = 8;
+        if(report->use_report_ids != 0) {
+                buffer[0] = report_id;          
         }
 
         usb_log_debug("OUTPUT BUFFER: %s\n", usb_debug_str_buffer(buffer,size, 0));
-        usb_log_debug("OUTPUT DATA[0]: %d, DATA[1]: %d, DATA[2]: %d\n", data[0], data[1], data[2]);
-
-        item = parser->output.next;     
-        while(item != &parser->output) {
-                report_item = list_get_instance(item, usb_hid_report_item_t, link);
-
-                for(i=0; i<report_item->count; i++) {
-
-                        if(idx >= data_size) {
-                                break;
-                        }
-
-                        if((USB_HID_ITEM_FLAG_VARIABLE(report_item->item_flags) == 0) ||
-                                ((report_item->usage_minimum == 0) && (report_item->usage_maximum == 0))) {
+        
+        usb_hid_report_description_t *report_des;
+        report_des = usb_hid_report_find_description (report, report_id, USB_HID_REPORT_TYPE_OUTPUT);
+        if(report_des == NULL){
+                return EINVAL;
+        }
+
+        usb_hid_report_field_t *report_item;    
+        item = report_des->report_items.next;   
+        while(item != &report_des->report_items) {
+                report_item = list_get_instance(item, usb_hid_report_field_t, link);
+
+                        if(USB_HID_ITEM_FLAG_VARIABLE(report_item->item_flags) == 0) {
                                         
-//                              // variable item
-                                value = usb_hid_translate_data_reverse(report_item, data[idx++]);
-                                offset = report_item->offset + (i * report_item->size) + offset_prefix;
+                                // array
+                                value = usb_hid_translate_data_reverse(report_item, report_item->value);
+                                offset = report_item->offset;
                                 length = report_item->size;
                         }
                         else {
-                                //bitmap
-                                value += usb_hid_translate_data_reverse(report_item, data[idx++]);
-                                offset = report_item->offset + offset_prefix;
-                                length = report_item->size * report_item->count;
+                                // variable item
+                                value  = usb_hid_translate_data_reverse(report_item, report_item->value);
+                                offset = report_item->offset;
+                                length = report_item->size;
                         }
 
@@ -1295 +1454 @@
                         }
                         else {
-                                // je to ve dvou!! FIXME: melo by to umet delsi jak 2
-
-                                // konec prvniho -- dolni x bitu
-                                tmp_value = value;
-                                tmp_value = tmp_value & ((1 << (8-(offset%8)))-1);                              
-                                tmp_value = tmp_value << (offset%8);
-
+                                int i = 0;
                                 uint8_t mask = 0;
-                                mask = ~(((1 << (8-(offset%8)))-1) << (offset%8));
-                                buffer[offset/8] = (buffer[offset/8] & mask) | tmp_value;
-
-                                // a ted druhej -- hornich length-x bitu
-                                value = value >> (8 - (offset % 8));
-                                value = value & ((1 << (length - (8 - (offset % 8)))) - 1);
+                                for(i = (offset/8); i <= ((offset+length-1)/8); i++) {
+                                        if(i == (offset/8)) {
+                                                tmp_value = value;
+                                                tmp_value = tmp_value & ((1 << (8-(offset%8)))-1);                              
+                                                tmp_value = tmp_value << (offset%8);
+        
+                                                mask = ~(((1 << (8-(offset%8)))-1) << (offset%8));
+                                                buffer[i] = (buffer[i] & mask) | tmp_value;                     
+                                        }
+                                        else if (i == ((offset + length -1)/8)) {
+                                                
+                                                value = value >> (length - ((offset + length) % 8));
+                                                value = value & ((1 << (length - ((offset + length) % 8))) - 1);
                                 
-                                mask = ((1 << (length - (8 - (offset % 8)))) - 1);
-                                buffer[(offset+length-1)/8] = (buffer[(offset+length-1)/8] & mask) | value;
-                        }
-
-                }
-
+                                                mask = (1 << (length - ((offset + length) % 8))) - 1;
+                                                buffer[i] = (buffer[i] & mask) | value;
+                                        }
+                                        else {
+                                                buffer[i] = value & (0xFF << i);
+                                        }
+                                }
+                        }
+
+
+                // reset value
+                report_item->value = 0;
+                
                 item = item->next;
         }
-
+        
         usb_log_debug("OUTPUT BUFFER: %s\n", usb_debug_str_buffer(buffer,size, 0));
 
@@ -1325 +1492 @@
 
 /**
- *
- * @param item
- * @param value
- * @return
- */
-int32_t usb_hid_translate_data_reverse(usb_hid_report_item_t *item, int value)
+ * Translate given data for putting them into the outoput report
+ * @param item Report item structure
+ * @param value Value to translate
+ * @return ranslated value
+ */
+uint32_t usb_hid_translate_data_reverse(usb_hid_report_field_t *item, int value)
 {
         int ret=0;
@@ -1339 +1506 @@
         }
 
+        if((item->physical_minimum == 0) && (item->physical_maximum == 0)){
+                item->physical_minimum = item->logical_minimum;
+                item->physical_maximum = item->logical_maximum;                 
+        }
+        
+
         if((USB_HID_ITEM_FLAG_VARIABLE(item->item_flags) == 0)) {
 
                 // variable item
-                if((item->physical_minimum == 0) && (item->physical_maximum == 0)) {
-                        item->physical_minimum = item->logical_minimum;
-                        item->physical_maximum = item->logical_maximum;
-                }
-
                 if(item->physical_maximum == item->physical_minimum){
                     resolution = 1;
@@ -1369 +1537 @@
         }
 
-
-        return ret;
-}
-
-
+        if((item->logical_minimum < 0) || (item->logical_maximum < 0)){
+                return USB_HID_INT32_TO_UINT32(ret, item->size);
+        }
+        return (int32_t)ret;
+}
+
+/**
+ * Sets report id in usage path structure
+ *
+ * @param path Usage path structure
+ * @param report_id Report id to set
+ * @return Error code
+ */
 int usb_hid_report_path_set_report_id(usb_hid_report_path_t *path, uint8_t report_id)
 {
@@ -1381 +1557 @@
 
         path->report_id = report_id;
+        return EOK;
+}
+
+/**
+ *
+ *
+ *
+ *
+ *
+ */
+int usb_hid_report_output_set_data(usb_hid_report_t *report, 
+                                   usb_hid_report_path_t *path, int flags, 
+                                  int *data, size_t data_size)
+{
+        size_t data_idx = 0;
+        if(report == NULL){
+                return EINVAL;
+        }
+
+        usb_hid_report_description_t *report_des;
+        report_des = usb_hid_report_find_description (report, path->report_id, 
+                                                      USB_HID_REPORT_TYPE_OUTPUT);
+        if(report_des == NULL){
+                return EINVAL;
+        }
+
+        usb_hid_report_field_t *field;
+        link_t *field_it = report_des->report_items.next;       
+        while(field_it != &report_des->report_items){
+
+                field = list_get_instance(field_it, usb_hid_report_field_t, link);              
+                if(USB_HID_ITEM_FLAG_CONSTANT(field->item_flags) == 0) {
+                        usb_hid_report_path_append_item (field->collection_path, field->usage_page, field->usage);
+                        if(usb_hid_report_compare_usage_path (field->collection_path, path, 
+                                                      flags) == EOK) {
+                                if(data_idx < data_size) {
+                                        if((data[data_idx] >= field->physical_minimum) && (data[data_idx] >= field->physical_minimum)) {
+                                                field->value = data[data_idx];
+                                        }
+                                        else {
+                                                return ERANGE;
+                                        }
+
+                                        data_idx++;
+                                }
+                                else {
+                                        field->value = 0;
+                                }
+                        }
+                        usb_hid_report_remove_last_item (field->collection_path);
+                }
+                
+                field_it = field_it->next;
+        }
+
         return EOK;
 }
@@ -1398 +1629 @@
 }
 
+
+usb_hid_report_field_t *usb_hid_report_get_sibling(usb_hid_report_t *report, 
+                                                        usb_hid_report_field_t *field, 
+                            usb_hid_report_path_t *path, int flags, 
+                            usb_hid_report_type_t type)
+{
+        usb_hid_report_description_t *report_des = usb_hid_report_find_description (report, path->report_id, type);
+        link_t *field_it;
+        
+        if(report_des == NULL){
+                return NULL;
+        }
+
+        if(field == NULL){
+                // vezmu prvni co mathuje podle path!!
+                field_it = report_des->report_items.next;
+        }
+        else {
+                field_it = field->link.next;
+        }
+
+        while(field_it != &report_des->report_items) {
+                field = list_get_instance(field_it, usb_hid_report_field_t, link);
+
+                if(USB_HID_ITEM_FLAG_CONSTANT(field->item_flags) == 0) {
+                        usb_hid_report_path_append_item (field->collection_path, field->usage_page, field->usage);
+                        if(usb_hid_report_compare_usage_path (field->collection_path, path, flags) == EOK){
+                                usb_hid_report_remove_last_item (field->collection_path);
+                                return field;
+                        }
+                        usb_hid_report_remove_last_item (field->collection_path);
+                }
+                field_it = field_it->next;
+        }
+
+        return NULL;
+}
+
+uint8_t usb_hid_report_get_report_id(usb_hid_report_t *report, uint8_t report_id, usb_hid_report_type_t type)
+{
+        if(report == NULL){
+                return 0;
+        }
+
+        usb_hid_report_description_t *report_des;
+        link_t *report_it;
+        
+        if(report_id == 0) {
+                report_it = usb_hid_report_find_description (report, report_id, type)->link.next;               
+        }
+        else {
+                report_it = report->reports.next;
+        }
+
+        while(report_it != &report->reports) {
+                report_des = list_get_instance(report_it, usb_hid_report_description_t, link);
+                if(report_des->type == type){
+                        return report_des->report_id;
+                }
+        }
+
+        return 0;
+}
+
+void usb_hid_report_reset_local_items(usb_hid_report_item_t *report_item)
+{
+        if(report_item == NULL) {
+                return;
+        }
+        
+        report_item->usages_count = 0;
+        memset(report_item->usages, 0, USB_HID_MAX_USAGES);
+        
+        report_item->extended_usage_page = 0;
+        report_item->usage_minimum = 0;
+        report_item->usage_maximum = 0;
+        report_item->designator_index = 0;
+        report_item->designator_minimum = 0;
+        report_item->designator_maximum = 0;
+        report_item->string_index = 0;
+        report_item->string_minimum = 0;
+        report_item->string_maximum = 0;
+
+        return;
+}
 /**
  * @}

uspace/lib/usb/src/hidreport.c

@@ -109 +109 @@
         
         if (*d != sizeof(usb_standard_hid_descriptor_t)) {
-                usb_log_error("HID descriptor hass wrong size (%u, expected %u"
+                usb_log_error("HID descriptor has wrong size (%u, expected %zu"
                     ")\n", *d, sizeof(usb_standard_hid_descriptor_t));
                 return EINVAL;
@@ -119 +119 @@
         uint16_t length =  hid_desc->report_desc_info.length;
         size_t actual_size = 0;
-        
-        /*
-         * Start session for the control transfer.
-         */
-        int sess_rc = usb_pipe_start_session(&dev->ctrl_pipe);
-        if (sess_rc != EOK) {
-                usb_log_warning("Failed to start a session: %s.\n",
-                    str_error(sess_rc));
-                return sess_rc;
-        }
 
         /*
@@ -159 +149 @@
                 free(*report_desc);
                 *report_desc = NULL;
-                usb_log_error("Report descriptor has wrong size (%u, expected "
+                usb_log_error("Report descriptor has wrong size (%zu, expected "
                     "%u)\n", actual_size, length);
                 return EINVAL;
         }
         
-        /*
-         * End session for the control transfer.
-         */
-        sess_rc = usb_pipe_end_session(&dev->ctrl_pipe);
-        if (sess_rc != EOK) {
-                usb_log_warning("Failed to end a session: %s.\n",
-                    str_error(sess_rc));
-                free(*report_desc);
-                *report_desc = NULL;
-                return sess_rc;
-        }
-        
         *size = length;
         
@@ -186 +164 @@
 
 int usb_hid_process_report_descriptor(usb_device_t *dev, 
-    usb_hid_report_parser_t *parser)
+    usb_hid_report_t *report)
 {
-        if (dev == NULL || parser == NULL) {
+        if (dev == NULL || report == NULL) {
                 usb_log_error("Failed to process Report descriptor: wrong "
                     "parameters given.\n");
@@ -211 +189 @@
         assert(report_desc != NULL);
         
-        rc = usb_hid_parse_report_descriptor(parser, report_desc, report_size);
+        rc = usb_hid_parse_report_descriptor(report, report_desc, report_size);
         if (rc != EOK) {
                 usb_log_error("Problem parsing Report descriptor: %s.\n",
@@ -219 +197 @@
         }
         
-        usb_hid_descriptor_print(parser);
+        usb_hid_descriptor_print(report);
         free(report_desc);
         

uspace/lib/usb/src/hidreq.c

@@ -56 +56 @@
  * @retval EOK if successful.
  * @retval EINVAL if no HID device is given.
- * @return Other value inherited from one of functions 
- *         usb_pipe_start_session(), usb_pipe_end_session(),
- *         usb_control_request_set().
+ * @return Other value inherited from function usb_control_request_set().
  */
 int usbhid_req_set_report(usb_pipe_t *ctrl_pipe, int iface_no,
@@ -79 +77 @@
          */
         
-        int rc, sess_rc;
-        
-        sess_rc = usb_pipe_start_session(ctrl_pipe);
-        if (sess_rc != EOK) {
-                usb_log_warning("Failed to start a session: %s.\n",
-                    str_error(sess_rc));
-                return sess_rc;
-        }
+        int rc;
         
         uint16_t value = 0;
@@ -97 +88 @@
             USB_HIDREQ_SET_REPORT, value, iface_no, buffer, buf_size);
 
-        sess_rc = usb_pipe_end_session(ctrl_pipe);
-
-        if (rc != EOK) {
-                usb_log_warning("Error sending output report to the keyboard: "
-                    "%s.\n", str_error(rc));
-                return rc;
-        }
-
-        if (sess_rc != EOK) {
-                usb_log_warning("Error closing session: %s.\n",
-                    str_error(sess_rc));
-                return sess_rc;
+        if (rc != EOK) {
+                usb_log_warning("Error sending output report to the keyboard: "
+                    "%s.\n", str_error(rc));
+                return rc;
         }
         
@@ -123 +106 @@
  * @retval EOK if successful.
  * @retval EINVAL if no HID device is given.
- * @return Other value inherited from one of functions 
- *         usb_pipe_start_session(), usb_pipe_end_session(),
- *         usb_control_request_set().
+ * @return Other value inherited from function usb_control_request_set().
  */
 int usbhid_req_set_protocol(usb_pipe_t *ctrl_pipe, int iface_no,
@@ -146 +127 @@
          */
         
-        int rc, sess_rc;
-        
-        sess_rc = usb_pipe_start_session(ctrl_pipe);
-        if (sess_rc != EOK) {
-                usb_log_warning("Failed to start a session: %s.\n",
-                    str_error(sess_rc));
-                return sess_rc;
-        }
+        int rc;
 
         usb_log_debug("Sending Set_Protocol request to the device ("
@@ -162 +136 @@
             USB_HIDREQ_SET_PROTOCOL, protocol, iface_no, NULL, 0);
 
-        sess_rc = usb_pipe_end_session(ctrl_pipe);
-
-        if (rc != EOK) {
-                usb_log_warning("Error sending output report to the keyboard: "
-                    "%s.\n", str_error(rc));
-                return rc;
-        }
-
-        if (sess_rc != EOK) {
-                usb_log_warning("Error closing session: %s.\n",
-                    str_error(sess_rc));
-                return sess_rc;
+        if (rc != EOK) {
+                usb_log_warning("Error sending output report to the keyboard: "
+                    "%s.\n", str_error(rc));
+                return rc;
         }
         
@@ -189 +155 @@
  * @retval EOK if successful.
  * @retval EINVAL if no HID device is given.
- * @return Other value inherited from one of functions 
- *         usb_pipe_start_session(), usb_pipe_end_session(),
- *         usb_control_request_set().
+ * @return Other value inherited from function usb_control_request_set().
  */
 int usbhid_req_set_idle(usb_pipe_t *ctrl_pipe, int iface_no, uint8_t duration)
@@ -211 +175 @@
          */
         
-        int rc, sess_rc;
-        
-        sess_rc = usb_pipe_start_session(ctrl_pipe);
-        if (sess_rc != EOK) {
-                usb_log_warning("Failed to start a session: %s.\n",
-                    str_error(sess_rc));
-                return sess_rc;
-        }
+        int rc;
 
         usb_log_debug("Sending Set_Idle request to the device ("
@@ -229 +186 @@
             USB_HIDREQ_SET_IDLE, value, iface_no, NULL, 0);
 
-        sess_rc = usb_pipe_end_session(ctrl_pipe);
-
-        if (rc != EOK) {
-                usb_log_warning("Error sending output report to the keyboard: "
-                    "%s.\n", str_error(rc));
-                return rc;
-        }
-
-        if (sess_rc != EOK) {
-                usb_log_warning("Error closing session: %s.\n",
-                    str_error(sess_rc));
-                return sess_rc;
+        if (rc != EOK) {
+                usb_log_warning("Error sending output report to the keyboard: "
+                    "%s.\n", str_error(rc));
+                return rc;
         }
         
@@ -259 +208 @@
  * @retval EOK if successful.
  * @retval EINVAL if no HID device is given.
- * @return Other value inherited from one of functions 
- *         usb_pipe_start_session(), usb_pipe_end_session(),
- *         usb_control_request_set().
+ * @return Other value inherited from function usb_control_request_set().
  */
 int usbhid_req_get_report(usb_pipe_t *ctrl_pipe, int iface_no, 
@@ -283 +230 @@
          */
         
-        int rc, sess_rc;
-        
-        sess_rc = usb_pipe_start_session(ctrl_pipe);
-        if (sess_rc != EOK) {
-                usb_log_warning("Failed to start a session: %s.\n",
-                    str_error(sess_rc));
-                return sess_rc;
-        }
+        int rc;
 
         uint16_t value = 0;
@@ -302 +242 @@
             actual_size);
 
-        sess_rc = usb_pipe_end_session(ctrl_pipe);
-
-        if (rc != EOK) {
-                usb_log_warning("Error sending output report to the keyboard: "
-                    "%s.\n", str_error(rc));
-                return rc;
-        }
-
-        if (sess_rc != EOK) {
-                usb_log_warning("Error closing session: %s.\n",
-                    str_error(sess_rc));
-                return sess_rc;
+        if (rc != EOK) {
+                usb_log_warning("Error sending output report to the keyboard: "
+                    "%s.\n", str_error(rc));
+                return rc;
         }
         
@@ -328 +260 @@
  * @retval EOK if successful.
  * @retval EINVAL if no HID device is given.
- * @return Other value inherited from one of functions 
- *         usb_pipe_start_session(), usb_pipe_end_session(),
- *         usb_control_request_set().
+ * @return Other value inherited from function usb_control_request_set().
  */
 int usbhid_req_get_protocol(usb_pipe_t *ctrl_pipe, int iface_no, 
@@ -351 +281 @@
          */
         
-        int rc, sess_rc;
-        
-        sess_rc = usb_pipe_start_session(ctrl_pipe);
-        if (sess_rc != EOK) {
-                usb_log_warning("Failed to start a session: %s.\n",
-                    str_error(sess_rc));
-                return sess_rc;
-        }
+        int rc; 
 
         usb_log_debug("Sending Get_Protocol request to the device ("
@@ -370 +293 @@
             USB_HIDREQ_GET_PROTOCOL, 0, iface_no, buffer, 1, &actual_size);
 
-        sess_rc = usb_pipe_end_session(ctrl_pipe);
-
-        if (rc != EOK) {
-                usb_log_warning("Error sending output report to the keyboard: "
-                    "%s.\n", str_error(rc));
-                return rc;
-        }
-
-        if (sess_rc != EOK) {
-                usb_log_warning("Error closing session: %s.\n",
-                    str_error(sess_rc));
-                return sess_rc;
+        if (rc != EOK) {
+                usb_log_warning("Error sending output report to the keyboard: "
+                    "%s.\n", str_error(rc));
+                return rc;
         }
         
@@ -427 +342 @@
          */
         
-        int rc, sess_rc;
-        
-        sess_rc = usb_pipe_start_session(ctrl_pipe);
-        if (sess_rc != EOK) {
-                usb_log_warning("Failed to start a session: %s.\n",
-                    str_error(sess_rc));
-                return sess_rc;
-        }
+        int rc;
 
         usb_log_debug("Sending Get_Idle request to the device ("
@@ -448 +356 @@
             &actual_size);
 
-        sess_rc = usb_pipe_end_session(ctrl_pipe);
-
-        if (rc != EOK) {
-                usb_log_warning("Error sending output report to the keyboard: "
-                    "%s.\n", str_error(rc));
-                return rc;
-        }
-
-        if (sess_rc != EOK) {
-                usb_log_warning("Error closing session: %s.\n",
-                    str_error(sess_rc));
-                return sess_rc;
+        if (rc != EOK) {
+                usb_log_warning("Error sending output report to the keyboard: "
+                    "%s.\n", str_error(rc));
+                return rc;
         }
         

uspace/lib/usb/src/host/batch.c

@@ -39 +39 @@
 #include <usb/host/batch.h>
 
+void usb_transfer_batch_call_in(usb_transfer_batch_t *instance);
+void usb_transfer_batch_call_out(usb_transfer_batch_t *instance);
+
 void usb_transfer_batch_init(
     usb_transfer_batch_t *instance,
-    usb_target_t target,
-    usb_transfer_type_t transfer_type,
-    usb_speed_t speed,
-    size_t max_packet_size,
+    endpoint_t *ep,
     char *buffer,
-    char *transport_buffer,
+    char *data_buffer,
     size_t buffer_size,
     char *setup_buffer,
@@ -54 +54 @@
     void *arg,
     ddf_fun_t *fun,
-                endpoint_t *ep,
-    void *private_data
+    void *private_data,
+    void (*private_data_dtor)(void *p_data)
     )
 {
         assert(instance);
         link_initialize(&instance->link);
-        instance->target = target;
-        instance->transfer_type = transfer_type;
-        instance->speed = speed;
-        instance->direction = ep->direction;
+        instance->ep = ep;
         instance->callback_in = func_in;
         instance->callback_out = func_out;
         instance->arg = arg;
         instance->buffer = buffer;
-        instance->transport_buffer = transport_buffer;
+        instance->data_buffer = data_buffer;
         instance->buffer_size = buffer_size;
         instance->setup_buffer = setup_buffer;
         instance->setup_size = setup_size;
-        instance->max_packet_size = max_packet_size;
         instance->fun = fun;
         instance->private_data = private_data;
+        instance->private_data_dtor = private_data_dtor;
         instance->transfered_size = 0;
         instance->next_step = NULL;
         instance->error = EOK;
-        instance->ep = ep;
         endpoint_use(instance->ep);
+}
+/*----------------------------------------------------------------------------*/
+/** Helper function, calls callback and correctly destroys batch structure.
+ *
+ * @param[in] instance Batch structure to use.
+ */
+void usb_transfer_batch_call_in_and_dispose(usb_transfer_batch_t *instance)
+{
+        assert(instance);
+        usb_transfer_batch_call_in(instance);
+        usb_transfer_batch_dispose(instance);
+}
+/*----------------------------------------------------------------------------*/
+/** Helper function calls callback and correctly destroys batch structure.
+ *
+ * @param[in] instance Batch structure to use.
+ */
+void usb_transfer_batch_call_out_and_dispose(usb_transfer_batch_t *instance)
+{
+        assert(instance);
+        usb_transfer_batch_call_out(instance);
+        usb_transfer_batch_dispose(instance);
 }
 /*----------------------------------------------------------------------------*/
@@ -105 +123 @@
         assert(instance);
         assert(instance->callback_in);
+        assert(instance->ep);
 
         /* We are data in, we need data */
-        memcpy(instance->buffer, instance->transport_buffer,
-            instance->buffer_size);
+        memcpy(instance->buffer, instance->data_buffer, instance->buffer_size);
 
         usb_log_debug("Batch %p done (T%d.%d, %s %s in, %zuB): %s (%d).\n",
-            instance,
-            instance->target.address, instance->target.endpoint,
-            usb_str_speed(instance->speed),
-            usb_str_transfer_type_short(instance->transfer_type),
-            instance->transfered_size,
-            str_error(instance->error), instance->error);
+            instance, instance->ep->address, instance->ep->endpoint,
+            usb_str_speed(instance->ep->speed),
+            usb_str_transfer_type_short(instance->ep->transfer_type),
+            instance->transfered_size, str_error(instance->error), instance->error);
 
         instance->callback_in(instance->fun, instance->error,
@@ -132 +148 @@
 
         usb_log_debug("Batch %p done (T%d.%d, %s %s out): %s (%d).\n",
-            instance,
-            instance->target.address, instance->target.endpoint,
-            usb_str_speed(instance->speed),
-            usb_str_transfer_type_short(instance->transfer_type),
+            instance, instance->ep->address, instance->ep->endpoint,
+            usb_str_speed(instance->ep->speed),
+            usb_str_transfer_type_short(instance->ep->transfer_type),
             str_error(instance->error), instance->error);
 
@@ -141 +156 @@
             instance->error, instance->arg);
 }
+/*----------------------------------------------------------------------------*/
+/** Correctly dispose all used data structures.
+ *
+ * @param[in] instance Batch structure to use.
+ */
+void usb_transfer_batch_dispose(usb_transfer_batch_t *instance)
+{
+        assert(instance);
+        usb_log_debug("Batch(%p) disposing.\n", instance);
+        if (instance->private_data) {
+                assert(instance->private_data_dtor);
+                instance->private_data_dtor(instance->private_data);
+        }
+        free(instance);
+}
 /**
  * @}

uspace/lib/usb/src/host/device_keeper.c

@@ -48 +48 @@
 {
         assert(instance);
-        fibril_mutex_initialize(&instance->guard);
-        fibril_condvar_initialize(&instance->change);
-        instance->last_address = 0;
         unsigned i = 0;
         for (; i < USB_ADDRESS_COUNT; ++i) {
@@ -60 +57 @@
         // (it is needed to allow smooth registration at default address)
         instance->devices[0].occupied = true;
+        instance->last_address = 0;
+        fibril_mutex_initialize(&instance->guard);
 }
-/*----------------------------------------------------------------------------*/
-/** Attempt to obtain address 0, blocks.
- *
- * @param[in] instance Device keeper structure to use.
- * @param[in] speed Speed of the device requesting default address.
- */
-void usb_device_keeper_reserve_default_address(
-    usb_device_keeper_t *instance, usb_speed_t speed)
-{
-        assert(instance);
-        fibril_mutex_lock(&instance->guard);
-        while (instance->devices[USB_ADDRESS_DEFAULT].occupied) {
-                fibril_condvar_wait(&instance->change, &instance->guard);
-        }
-        instance->devices[USB_ADDRESS_DEFAULT].occupied = true;
-        instance->devices[USB_ADDRESS_DEFAULT].speed = speed;
-        fibril_mutex_unlock(&instance->guard);
-}
-/*----------------------------------------------------------------------------*/
-/** Attempt to obtain address 0, blocks.
- *
- * @param[in] instance Device keeper structure to use.
- * @param[in] speed Speed of the device requesting default address.
- */
-void usb_device_keeper_release_default_address(usb_device_keeper_t *instance)
-{
-        assert(instance);
-        fibril_mutex_lock(&instance->guard);
-        instance->devices[USB_ADDRESS_DEFAULT].occupied = false;
-        fibril_mutex_unlock(&instance->guard);
-        fibril_condvar_signal(&instance->change);
-}
-/*----------------------------------------------------------------------------*/
 /*----------------------------------------------------------------------------*/
 /** Get a free USB address
@@ -120 +86 @@
         assert(new_address != USB_ADDRESS_DEFAULT);
         assert(instance->devices[new_address].occupied == false);
+
         instance->devices[new_address].occupied = true;
         instance->devices[new_address].speed = speed;
         instance->last_address = new_address;
+
         fibril_mutex_unlock(&instance->guard);
         return new_address;
@@ -138 +106 @@
         assert(instance);
         fibril_mutex_lock(&instance->guard);
+
         assert(address > 0);
         assert(address <= USB11_ADDRESS_MAX);
         assert(instance->devices[address].occupied);
+
         instance->devices[address].handle = handle;
         fibril_mutex_unlock(&instance->guard);
@@ -159 +129 @@
         fibril_mutex_lock(&instance->guard);
         assert(instance->devices[address].occupied);
+
         instance->devices[address].occupied = false;
         fibril_mutex_unlock(&instance->guard);
@@ -177 +148 @@
         while (address <= USB11_ADDRESS_MAX) {
                 if (instance->devices[address].handle == handle) {
+                        assert(instance->devices[address].occupied);
                         fibril_mutex_unlock(&instance->guard);
                         return address;
@@ -198 +170 @@
         assert(address >= 0);
         assert(address <= USB11_ADDRESS_MAX);
+
         return instance->devices[address].speed;
 }

uspace/lib/usb/src/host/endpoint.c

@@ -53 +53 @@
         fibril_mutex_initialize(&instance->guard);
         fibril_condvar_initialize(&instance->avail);
+        endpoint_clear_hc_data(instance);
         return EOK;
 }
@@ -61 +62 @@
         assert(!instance->active);
         free(instance);
+}
+/*----------------------------------------------------------------------------*/
+void endpoint_set_hc_data(endpoint_t *instance,
+    void *data, int (*toggle_get)(void *), void (*toggle_set)(void *, int))
+{
+        assert(instance);
+        instance->hc_data.data = data;
+        instance->hc_data.toggle_get = toggle_get;
+        instance->hc_data.toggle_set = toggle_set;
+}
+/*----------------------------------------------------------------------------*/
+void endpoint_clear_hc_data(endpoint_t *instance)
+{
+        assert(instance);
+        instance->hc_data.data = NULL;
+        instance->hc_data.toggle_get = NULL;
+        instance->hc_data.toggle_set = NULL;
 }
 /*----------------------------------------------------------------------------*/
@@ -85 +103 @@
 {
         assert(instance);
+        if (instance->hc_data.toggle_get)
+                instance->toggle =
+                    instance->hc_data.toggle_get(instance->hc_data.data);
         return (int)instance->toggle;
 }
@@ -92 +113 @@
         assert(instance);
         assert(toggle == 0 || toggle == 1);
+        if (instance->hc_data.toggle_set)
+                instance->hc_data.toggle_set(instance->hc_data.data, toggle);
         instance->toggle = toggle;
 }
@@ -100 +123 @@
         if (instance->address == target.address &&
             (instance->endpoint == target.endpoint || target.endpoint == 0))
-                instance->toggle = 0;
+                endpoint_toggle_set(instance, 0);
 }
 /**

uspace/lib/usb/src/host/usb_endpoint_manager.c

@@ -211 +211 @@
 
         node_t *node = hash_table_get_instance(item, node_t, link);
+        if (node->ep->active)
+                return EBUSY;
+
         instance->free_bw += node->bw;
         hash_table_remove(&instance->ep_table, key, MAX_KEYS);

uspace/lib/usb/src/pipepriv.c

@@ -77 +77 @@
  *
  * @param pipe The USB pipe.
+ * @param hide_failure Whether to hide failure when adding reference
+ *      (use soft refcount).
  * @return Error code.
  * @retval EOK Currently always.
  */
-int pipe_add_ref(usb_pipe_t *pipe)
+int pipe_add_ref(usb_pipe_t *pipe, bool hide_failure)
 {
-another_try:
         pipe_acquire(pipe);
 
@@ -89 +90 @@
                 int phone = devman_device_connect(pipe->wire->hc_handle, 0);
                 if (phone < 0) {
-                        // TODO: treat some error as non-recoverable
-                        // and return error from here
+                        if (hide_failure) {
+                                pipe->refcount_soft++;
+                                phone = EOK;
+                        }
                         pipe_release(pipe);
-                        goto another_try;
+                        return phone;
                 }
                 /*
@@ -114 +117 @@
 {
         pipe_acquire(pipe);
+        if (pipe->refcount_soft > 0) {
+                pipe->refcount_soft--;
+                pipe_release(pipe);
+                return;
+        }
         assert(pipe->refcount > 0);
         pipe->refcount--;

uspace/lib/usb/src/pipepriv.h

@@ -37 +37 @@
 
 #include <usb/pipes.h>
+#include <bool.h>
 
 void pipe_acquire(usb_pipe_t *);
@@ -44 +45 @@
 void pipe_end_transaction(usb_pipe_t *);
 
-int pipe_add_ref(usb_pipe_t *);
+int pipe_add_ref(usb_pipe_t *, bool);
 void pipe_drop_ref(usb_pipe_t *);
 

uspace/lib/usb/src/pipes.c

@@ -229 +229 @@
 }
 
-
-/** Start a session on the endpoint pipe.
- *
- * A session is something inside what any communication occurs.
- * It is expected that sessions would be started right before the transfer
- * and ended - see usb_pipe_end_session() - after the last
- * transfer.
- * The reason for this is that session actually opens some communication
- * channel to the host controller (or to the physical hardware if you
- * wish) and thus it involves acquiring kernel resources.
- * Since they are limited, sessions shall not be longer than strictly
- * necessary.
- *
- * @deprecated
- * Obsoleted with introduction of usb_pipe_start_long_transfer
- *
- * @param pipe Endpoint pipe to start the session on.
- * @return Error code.
- */
-int usb_pipe_start_session(usb_pipe_t *pipe)
-{
-        usb_log_warning("usb_pipe_start_session() was deprecated.\n");
-        return EOK;
-}
-
-
-/** Ends a session on the endpoint pipe.
- *
- * @deprecated
- * Obsoleted with introduction of usb_pipe_end_long_transfer
- *
- * @see usb_pipe_start_session
- *
- * @param pipe Endpoint pipe to end the session on.
- * @return Error code.
- */
-int usb_pipe_end_session(usb_pipe_t *pipe)
-{
-        usb_log_warning("usb_pipe_end_session() was deprecated.\n");
-        return EOK;
-}
-
-/** Tell whether a session is started (open) on the endpoint pipe.
- *
- * The expected usage of this function is in assertions for some
- * nested functions.
- *
- * @param pipe Endpoint pipe in question.
- * @return Whether @p pipe has opened a session.
- */
-bool usb_pipe_is_session_started(usb_pipe_t *pipe)
-{
-        pipe_acquire(pipe);
-        bool started = pipe->refcount > 0;
-        pipe_release(pipe);
-        return started;
-}
-
 /** Prepare pipe for a long transfer.
  *
@@ -297 +239 @@
  * @return Error code.
  */
-int usb_pipe_start_long_transfer(usb_pipe_t *pipe)
-{
-        return pipe_add_ref(pipe);
+void usb_pipe_start_long_transfer(usb_pipe_t *pipe)
+{
+        (void) pipe_add_ref(pipe, true);
 }
 

uspace/lib/usb/src/pipesinit.c

@@ -365 +365 @@
         pipe->direction = direction;
         pipe->refcount = 0;
+        pipe->refcount_soft = 0;
+        pipe->auto_reset_halt = false;
 
         return EOK;
@@ -385 +387 @@
             0, USB_TRANSFER_CONTROL, CTRL_PIPE_MIN_PACKET_SIZE,
             USB_DIRECTION_BOTH);
+
+        pipe->auto_reset_halt = true;
 
         return rc;
@@ -416 +420 @@
         int rc;
 
-        rc = usb_pipe_start_long_transfer(pipe);
-        if (rc != EOK) {
-                return rc;
-        }
-
+        usb_pipe_start_long_transfer(pipe);
 
         uint8_t dev_descr_start[CTRL_PIPE_MIN_PACKET_SIZE];

uspace/lib/usb/src/pipesio.c

@@ -49 +49 @@
 #include <assert.h>
 #include <usbhc_iface.h>
+#include <usb/request.h>
 #include "pipepriv.h"
 
@@ -172 +173 @@
 
         int rc;
-        rc = pipe_add_ref(pipe);
+        rc = pipe_add_ref(pipe, false);
         if (rc != EOK) {
                 return rc;
@@ -295 +296 @@
         int rc;
 
-        rc = pipe_add_ref(pipe);
+        rc = pipe_add_ref(pipe, false);
         if (rc != EOK) {
                 return rc;
@@ -305 +306 @@
 
         return rc;
+}
+
+/** Try to clear endpoint halt of default control pipe.
+ *
+ * @param pipe Pipe for control endpoint zero.
+ */
+static void clear_self_endpoint_halt(usb_pipe_t *pipe)
+{
+        assert(pipe != NULL);
+
+        if (!pipe->auto_reset_halt || (pipe->endpoint_no != 0)) {
+                return;
+        }
+
+
+        /* Prevent indefinite recursion. */
+        pipe->auto_reset_halt = false;
+        usb_request_clear_endpoint_halt(pipe, 0);
+        pipe->auto_reset_halt = true;
 }
 
@@ -427 +447 @@
         int rc;
 
-        rc = pipe_add_ref(pipe);
+        rc = pipe_add_ref(pipe, false);
         if (rc != EOK) {
                 return rc;
@@ -436 +456 @@
             setup_buffer, setup_buffer_size,
             data_buffer, data_buffer_size, &act_size);
+
+        if (rc == ESTALL) {
+                clear_self_endpoint_halt(pipe);
+        }
 
         pipe_drop_ref(pipe);
@@ -555 +579 @@
         int rc;
 
-        rc = pipe_add_ref(pipe);
+        rc = pipe_add_ref(pipe, false);
         if (rc != EOK) {
                 return rc;
@@ -563 +587 @@
             setup_buffer, setup_buffer_size, data_buffer, data_buffer_size);
 
+        if (rc == ESTALL) {
+                clear_self_endpoint_halt(pipe);
+        }
+
         pipe_drop_ref(pipe);