Changes in / [dd6f59f:df949c5] in mainline

Location:

uspace/drv

Files:

• uhci-hcd/batch.c (modified) (1 diff)
• uhci-hcd/uhci.c (modified) (2 diffs)
• uhci-hcd/uhci_struct/transfer_descriptor.h (modified) (1 diff)
• uhci-rhd/port.c (modified) (3 diffs)

uspace/drv/uhci-hcd/batch.c

@@ -318 +318 @@
                 ++packet;
         }
-        instance->tds[packet - 1].status |= TD_STATUS_COMPLETE_INTERRUPT_FLAG;
         device_keeper_set_toggle(instance->manager, instance->target, toggle);
 }

uspace/drv/uhci-hcd/uhci.c

@@ -372 +372 @@
         assert(instance);
         /* TODO: Check interrupt cause here */
-        /* Lower 2 bits are transaction error and transaction complete */
-        if (status & 0x3) {
-                transfer_list_remove_finished(&instance->transfers_interrupt);
-                transfer_list_remove_finished(&instance->transfers_control_slow);
-                transfer_list_remove_finished(&instance->transfers_control_full);
-                transfer_list_remove_finished(&instance->transfers_bulk_full);
-        }
+        transfer_list_remove_finished(&instance->transfers_interrupt);
+        transfer_list_remove_finished(&instance->transfers_control_slow);
+        transfer_list_remove_finished(&instance->transfers_control_full);
+        transfer_list_remove_finished(&instance->transfers_bulk_full);
 }
 /*----------------------------------------------------------------------------*/
@@ -393 +390 @@
 
         while (1) {
-                /* read and ack interrupts */
                 uint16_t status = pio_read_16(&instance->registers->usbsts);
-                pio_write_16(&instance->registers->usbsts, 0x1f);
                 if (status != 0)
                         usb_log_debug2("UHCI status: %x.\n", status);
+                status |= 1;
                 uhci_interrupt(instance, status);
+                pio_write_16(&instance->registers->usbsts, 0x1f);
                 async_usleep(UHCI_CLEANER_TIMEOUT);
         }

uspace/drv/uhci-hcd/uhci_struct/transfer_descriptor.h

@@ -91 +91 @@
 
 
-void td_init(td_t *instance, int error_count, size_t size, bool toggle,
-    bool iso, bool low_speed, usb_target_t target, usb_packet_id pid,
-    void *buffer, td_t *next);
+void td_init(td_t *instance, int error_count, size_t size, bool toggle, bool iso,
+    bool low_speed, usb_target_t target, usb_packet_id pid, void *buffer,
+    td_t *next);
 
 int td_status(td_t *instance);

uspace/drv/uhci-rhd/port.c

@@ -181 +181 @@
         uhci_port_t *port = (uhci_port_t *) arg;
 
-        usb_log_debug2("%s: new_device_enable_port.\n", port->id_string);
+        usb_log_debug2("%s: new_device_enable_port.\n",
+            port->id_string);
 
         /*
@@ -189 +190 @@
         async_usleep(100000);
 
-        /*
-         * Resets from root ports should be nominally 50ms
+
+        /* The hub maintains the reset signal to that port for 10 ms
+         * (See Section 11.5.1.5)
          */
         {
-                usb_log_debug("%s: Reset Signal start.\n", port->id_string);
+                usb_log_debug("%s: Reset Signal start.\n",
+                    port->id_string);
                 port_status_t port_status = uhci_port_read_status(port);
                 port_status |= STATUS_IN_RESET;
                 uhci_port_write_status(port, port_status);
-                async_usleep(50000);
+                async_usleep(10000);
                 port_status = uhci_port_read_status(port);
                 port_status &= ~STATUS_IN_RESET;
                 uhci_port_write_status(port, port_status);
-                usb_log_debug("%s: Reset Signal stop.\n", port->id_string);
-        }
-
-        /* the reset recovery time 10ms */
-        async_usleep(10000);
+                usb_log_debug("%s: Reset Signal stop.\n",
+                    port->id_string);
+        }
 
         /* Enable the port. */
         uhci_port_set_enabled(port, true);
-
         return EOK;
 }
@@ -226 +226 @@
         assert(usb_hc_connection_is_opened(&port->hc_connection));
 
-        usb_log_info("%s: Detected new device.\n", port->id_string);
+        usb_log_info("%s: Detected new device.\n",
+            port->id_string);
 
         usb_address_t dev_addr;