Changes in uspace/app/usbinfo/main.c [a12917e:632ed68] in mainline

File:

• uspace/app/usbinfo/main.c (modified) (5 diffs)

uspace/app/usbinfo/main.c

@@ -77 +77 @@
 }
 
-static int get_host_controller_handle(const char *path,
-    devman_handle_t *hc_handle)
+static int set_new_host_controller(int *phone, const char *path)
 {
         int rc;
-
-        devman_handle_t handle;
-        rc = devman_device_get_handle(path, &handle, 0);
-        if (rc != EOK) {
-                fprintf(stderr,
-                    NAME ": failed getting handle of `devman::/%s'.\n",
-                    path);
-                return rc;
-        }
-        *hc_handle = handle;
+        int tmp_phone;
+
+        if (path[0] != '/') {
+                int hc_class_index = (int) strtol(path, NULL, 10);
+                char *dev_path;
+                rc = asprintf(&dev_path, "class/usbhc\\%d", hc_class_index);
+                if (rc < 0) {
+                        internal_error(rc);
+                        return rc;
+                }
+                devmap_handle_t handle;
+                rc = devmap_device_get_handle(dev_path, &handle, 0);
+                if (rc < 0) {
+                        fprintf(stderr,
+                            NAME ": failed getting handle of `devman://%s'.\n",
+                            dev_path);
+                        free(dev_path);
+                        return rc;
+                }
+                tmp_phone = devmap_device_connect(handle, 0);
+                if (tmp_phone < 0) {
+                        fprintf(stderr,
+                            NAME ": could not connect to `%s'.\n",
+                            dev_path);
+                        free(dev_path);
+                        return tmp_phone;
+                }
+                free(dev_path);
+        } else {
+                devman_handle_t handle;
+                rc = devman_device_get_handle(path, &handle, 0);
+                if (rc != EOK) {
+                        fprintf(stderr,
+                            NAME ": failed getting handle of `devmap::/%s'.\n",
+                            path);
+                        return rc;
+                }
+                tmp_phone = devman_device_connect(handle, 0);
+                if (tmp_phone < 0) {
+                        fprintf(stderr,
+                            NAME ": could not connect to `%s'.\n",
+                            path);
+                        return tmp_phone;
+                }
+        }
+
+        *phone = tmp_phone;
 
         return EOK;
 }
 
-static int get_device_address(const char *str_address, usb_address_t *address)
-{
-        usb_address_t addr = (usb_address_t) strtol(str_address, NULL, 0);
-        if ((addr < 0) || (addr >= USB11_ADDRESS_MAX)) {
+static int connect_with_address(int hc_phone, const char *str_address)
+{
+        usb_address_t address = (usb_address_t) strtol(str_address, NULL, 0);
+        if ((address < 0) || (address >= USB11_ADDRESS_MAX)) {
                 fprintf(stderr, NAME ": USB address out of range.\n");
                 return ERANGE;
         }
 
-        *address = addr;
-        return EOK;
+        if (hc_phone < 0) {
+                fprintf(stderr, NAME ": no active host controller.\n");
+                return ENOENT;
+        }
+
+        return dump_device(hc_phone, address);
 }
 
@@ -110 +150 @@
 int main(int argc, char *argv[])
 {
-        devman_handle_t hc_handle = (devman_handle_t) -1;
-        usb_address_t device_address = (usb_address_t) -1;
+        int hc_phone = -1;
 
         if (argc <= 1) {
@@ -136 +175 @@
                         case 'a':
                         case ACTION_DEVICE_ADDRESS: {
-                                int rc = get_device_address(optarg,
-                                    &device_address);
+                                int rc = connect_with_address(hc_phone, optarg);
                                 if (rc != EOK) {
                                         return rc;
@@ -146 +184 @@
                         case 't':
                         case ACTION_HOST_CONTROLLER: {
-                                int rc = get_host_controller_handle(optarg,
-                                   &hc_handle);
+                                int rc = set_new_host_controller(&hc_phone,
+                                    optarg);
                                 if (rc != EOK) {
                                         return rc;
@@ -164 +202 @@
         } while (i != -1);
 
-        if ((hc_handle == (devman_handle_t) -1)
-            || (device_address == (usb_address_t) -1)) {
-                fprintf(stderr, NAME ": no target specified.\n");
-                return EINVAL;
-        }
-
-        dump_device(hc_handle, device_address);
-
         return 0;
 }