Changeset 7813516 in mainline for uspace/drv/bus/usb/ehci

Timestamp:

2014-01-01T01:19:10Z (12 years ago)

Author:

Jan Vesely <jano.vesely@…>

Branches:

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

Children:

7191992

Parents:

d1df381

Message:

uhci,ohci,ehci: Use all hw resources to initialize HC

Location:

uspace/drv/bus/usb/ehci

Files:

• ehci.c (modified) (2 diffs)
• hc.c (modified) (3 diffs)
• hc.h (modified) (1 diff)

uspace/drv/bus/usb/ehci/ehci.c

@@ -94 +94 @@
                 return ret;
         }
-        addr_range_t regs = hw_res.mem_ranges.ranges[0];
 
         /* Initialize generic HCD driver */
@@ -127 +126 @@
 
         /* Initialize EHCI HC */
-        ret = hc_init(hc, &regs, interrupts);
+        ret = hc_init(hc, &hw_res, interrupts);
         hw_res_list_parsed_clean(&hw_res);
         if (ret != EOK) {

uspace/drv/bus/usb/ehci/hc.c

@@ -157 +157 @@
  * @return Error code
  */
-int hc_init(hc_t *instance, addr_range_t *regs, bool interrupts)
-{
-        assert(instance);
-
-        int ret = pio_enable_range(regs, (void **) &instance->caps);
+int hc_init(hc_t *instance, const hw_res_list_parsed_t *hw_res, bool interrupts)
+{
+        assert(instance);
+        assert(hw_res);
+        if (hw_res->mem_ranges.count != 1 ||
+            hw_res->mem_ranges.ranges[0].size <
+                (sizeof(ehci_caps_regs_t) + sizeof(ehci_regs_t)))
+            return EINVAL;
+
+        int ret = pio_enable_range(&hw_res->mem_ranges.ranges[0],
+            (void **)&instance->caps);
         if (ret != EOK) {
                 usb_log_error("Failed to gain access to device registers: %s.\n",
@@ -167 +173 @@
                 return ret;
         }
+        usb_log_debug("Device registers at %" PRIx64 " (%zuB) accessible.\n",
+            hw_res->mem_ranges.ranges[0].address.absolute,
+            hw_res->mem_ranges.ranges[0].size);
         instance->registers =
             (void*)instance->caps + EHCI_RD8(instance->caps->caplength);
@@ -194 +203 @@
         return EOK;
 }
+
+/** Safely dispose host controller internal structures
+ *
+ * @param[in] instance Host controller structure to use.
+ */
+void hc_fini(hc_t *instance)
+{
+        assert(instance);
+        /* TODO: implement*/
+};
 
 void hc_enqueue_endpoint(hc_t *instance, const endpoint_t *ep)

uspace/drv/bus/usb/ehci/hc.h

@@ -76 +76 @@
 int hc_gen_irq_code(irq_code_t *code, const hw_res_list_parsed_t *hw_res);
 int hc_register_hub(hc_t *instance, ddf_fun_t *hub_fun);
-int hc_init(hc_t *instance, addr_range_t *regs, bool interrupts);
-
-/** Safely dispose host controller internal structures
- *
- * @param[in] instance Host controller structure to use.
- */
-static inline void hc_fini(hc_t *instance) { /* TODO: implement*/ };
+int hc_init(hc_t *instance, const hw_res_list_parsed_t *hw_res, bool interrupts);
+void hc_fini(hc_t *instance);
 
 void hc_enqueue_endpoint(hc_t *instance, const endpoint_t *ep);