Changeset 540abef in mainline

Timestamp:

2011-04-14T07:51:13Z (13 years ago)

Author:

Jan Vesely <jano.vesely@…>

Branches:

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

Children:

c0cea918

Parents:

27205841

Message:

Root hub driver code cleanup

Location:

uspace/drv/uhci-rhd

Files:

• port.c (modified) (4 diffs)
• port.h (modified) (1 diff)
• root_hub.c (modified) (4 diffs)
• root_hub.h (modified) (1 diff)

uspace/drv/uhci-rhd/port.c

@@ -68 +68 @@
  * @return Error code. (Always EOK)
  */
-static inline void uhci_port_write_status(
-    uhci_port_t *port, port_status_t value)
-{
-        assert(port);
-        pio_write_16(port->address, value);
+static inline void uhci_port_write_status(uhci_port_t *port, port_status_t val)
+{
+        assert(port);
+        pio_write_16(port->address, val);
 }
 /*----------------------------------------------------------------------------*/
@@ -101 +100 @@
         port->rh = rh;
 
-        int rc = usb_hc_connection_initialize_from_device(
-            &port->hc_connection, rh);
-        if (rc != EOK) {
+        int ret =
+            usb_hc_connection_initialize_from_device(&port->hc_connection, rh);
+        if (ret != EOK) {
                 usb_log_error("Failed to initialize connection to HC.");
-                return rc;
+                return ret;
         }
 
@@ -238 +237 @@
         /* Enable the port. */
         uhci_port_set_enabled(port, true);
-
         return EOK;
 }
@@ -271 +269 @@
         usb_log_info("New device at port %u, address %d (handle %llu).\n",
             port->number, dev_addr, port->attached_device);
-
         return EOK;
 }

uspace/drv/uhci-rhd/port.h

@@ -54 +54 @@
 #define STATUS_SUSPEND   (1 << 12)
 
+/** UHCI port structure */
 typedef struct uhci_port
 {

uspace/drv/uhci-rhd/root_hub.c

@@ -33 +33 @@
  */
 #include <errno.h>
+#include <str_error.h>
 #include <ddi.h>
 #include <usb/debug.h>
@@ -43 +44 @@
  * @param[in] addr Address of I/O registers.
  * @param[in] size Size of available I/O space.
- * @param[in] rh Pointer to ddf instance of the root hub driver.
+ * @param[in] rh Pointer to DDF instance of the root hub driver.
  * @return Error code.
  */
@@ -58 +59 @@
         if (ret < 0) {
                 usb_log_error(
-                    "Failed(%d) to gain access to port registers at %p\n",
-                    ret, regs);
+                    "Failed(%d) to gain access to port registers at %p: %s.\n",
+                    ret, regs, str_error(ret));
                 return ret;
         }
@@ -66 +67 @@
         unsigned i = 0;
         for (; i < UHCI_ROOT_HUB_PORT_COUNT; ++i) {
-                /* NOTE: mind pointer arithmetics here */
                 ret = uhci_port_init(
-                    &instance->ports[i], regs + i, i, ROOT_HUB_WAIT_USEC, rh);
+                    &instance->ports[i], &regs[i], i, ROOT_HUB_WAIT_USEC, rh);
                 if (ret != EOK) {
                         unsigned j = 0;

uspace/drv/uhci-rhd/root_hub.h

@@ -42 +42 @@
 #define ROOT_HUB_WAIT_USEC 250000 /* 250 miliseconds */
 
+/** UHCI root hub drvier structure */
 typedef struct root_hub {
+        /** Ports provided by the hub */
         uhci_port_t ports[UHCI_ROOT_HUB_PORT_COUNT];
-        devman_handle_t hc_handle;
 } uhci_root_hub_t;
 
 int uhci_root_hub_init(
-  uhci_root_hub_t *instance, void *addr, size_t size, ddf_dev_t *rh);
+    uhci_root_hub_t *instance, void *addr, size_t size, ddf_dev_t *rh);
 
 void uhci_root_hub_fini(uhci_root_hub_t *instance);