Changeset 90df90c in mainline for uspace/drv/bus/usb/usbhid/usbhid.c

Timestamp:

2011-11-09T13:23:03Z (13 years ago)

Author:

Jan Vesely <jano.vesely@…>

Branches:

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

Children:

07b9cbae

Parents:

6f730278

Message:

usbhid: Rework usb_hid_check_pipes. Comment fixes.

File:

• uspace/drv/bus/usb/usbhid/usbhid.c (modified) (3 diffs)

uspace/drv/bus/usb/usbhid/usbhid.c

@@ -128 +128 @@
         return (hid_dev->usb_dev->descriptors.device.vendor_id
             == mapping->vendor_id
-            && hid_dev->usb_dev->descriptors.device.product_id 
+            && hid_dev->usb_dev->descriptors.device.product_id
             == mapping->product_id);
 }
@@ -301 +301 @@
         assert(dev);
 
-        if (dev->pipes[USB_HID_KBD_POLL_EP_NO].present) {
-                usb_log_debug("Found keyboard endpoint.\n");
-                // save the pipe index
-                hid_dev->poll_pipe_index = USB_HID_KBD_POLL_EP_NO;
-        } else if (dev->pipes[USB_HID_MOUSE_POLL_EP_NO].present) {
-                usb_log_debug("Found mouse endpoint.\n");
-                // save the pipe index
-                hid_dev->poll_pipe_index = USB_HID_MOUSE_POLL_EP_NO;
-        } else if (dev->pipes[USB_HID_GENERIC_POLL_EP_NO].present) {
-                usb_log_debug("Found generic HID endpoint.\n");
-                // save the pipe index
-                hid_dev->poll_pipe_index = USB_HID_GENERIC_POLL_EP_NO;
-        } else {
-                usb_log_error("None of supported endpoints found - probably"
-                    " not a supported device.\n");
-                return ENOTSUP;
-        }
-
-        return EOK;
+        static const struct {
+                unsigned ep_number;
+                const char* description;
+        } endpoints[] = {
+                {USB_HID_KBD_POLL_EP_NO, "Keyboard endpoint"},
+                {USB_HID_MOUSE_POLL_EP_NO, "Mouse endpoint"},
+                {USB_HID_GENERIC_POLL_EP_NO, "Generic HID endpoint"},
+        };
+
+        for (unsigned i = 0; i < sizeof(endpoints)/sizeof(endpoints[0]); ++i) {
+                if (endpoints[i].ep_number >= dev->pipes_count) {
+                        return EINVAL;
+                }
+                if (dev->pipes[endpoints[i].ep_number].present) {
+                        usb_log_debug("Found: %s.\n", endpoints[i].description);
+                        hid_dev->poll_pipe_index = endpoints[i].ep_number;
+                        return EOK;
+                }
+        }
+        return ENOTSUP;
 }
 
@@ -395 +396 @@
         rc = usb_hid_process_report_descriptor(hid_dev->usb_dev,
             &hid_dev->report, &hid_dev->report_desc, &hid_dev->report_desc_size);
+
+        /*
+         * 1) subdriver vytvori vlastnu ddf_fun, vlastne ddf_dev_ops, ktore da
+         *    do nej.
+         * 2) do tych ops do .interfaces[DEV_IFACE_USBHID (asi)] priradi 
+         *    vyplnenu strukturu usbhid_iface_t.
+         * 3) klientska aplikacia - musi si rucne vytvorit telefon
+         *    (devman_device_connect() - cesta k zariadeniu (/hw/pci0/...) az 
+         *    k tej fcii.
+         *    pouzit usb/classes/hid/iface.h - prvy int je telefon
+         */
 
         bool fallback = false;