Changeset 6832245 in mainline for uspace/lib/usbhost/include/usb/host/bus.h

Timestamp:

2017-12-14T23:01:57Z (6 years ago)

Author:

Ondřej Hlavatý <aearsis@…>

Branches:

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

Children:

837d53d

Parents:

bd05140

git-author:

Ondřej Hlavatý <aearsis@…> (2017-12-14 23:01:54)

git-committer:

Ondřej Hlavatý <aearsis@…> (2017-12-14 23:01:57)

Message:

usbhost bus: refactor the bus ops

This way, method names better represent the entity it is working with.
Their semantics was shifted a bit.

Regarding the tree of structures:

bus ← device ← endpoint ← batch

Previously, devices were kept in DDF function nodes, and endpoints had
pointer to the bus and device. Now, devices have pointer to bus,
endpoints don't.

Pointer to hcd_t in bus is WIP, and will be removed.

File:

• uspace/lib/usbhost/include/usb/host/bus.h (modified) (2 diffs)

uspace/lib/usbhost/include/usb/host/bus.h

@@ -77 +77 @@
         usb_address_t address;
 
+        /* Managing bus */
+        bus_t *bus;
+
         /* This structure is meant to be extended by overriding. */
 } device_t;
 
-typedef struct {
-        int (*enumerate_device)(bus_t *, hcd_t *, device_t *);
-        int (*remove_device)(bus_t *, hcd_t *, device_t *);
+typedef struct bus_ops bus_ops_t;
 
-        int (*online_device)(bus_t *, hcd_t *, device_t *);                     /**< Optional */
-        int (*offline_device)(bus_t *, hcd_t *, device_t *);                    /**< Optional */
+/**
+ * Operations structure serving as an interface of hc driver for the library
+ * (and the rest of the system).
+ */
+struct bus_ops {
+        /* Undefined operations will be delegated to parent ops */
+        const bus_ops_t *parent;
 
-        /* The following operations are protected by a bus guard. */
-        endpoint_t *(*create_endpoint)(bus_t *);
-        int (*register_endpoint)(bus_t *, device_t *, endpoint_t *, const usb_endpoint_desc_t *);
-        int (*unregister_endpoint)(bus_t *, endpoint_t *);
-        endpoint_t *(*find_endpoint)(bus_t *, device_t*, usb_target_t, usb_direction_t);
-        void (*destroy_endpoint)(endpoint_t *);                 /**< Optional */
+        /* Global operations on the bus */
+        int (*reserve_default_address)(bus_t *, usb_speed_t);
+        int (*release_default_address)(bus_t *);
+        int (*reset_toggle)(bus_t *, usb_target_t, toggle_reset_mode_t);
+
+        /* Operations on device */
+        int (*device_enumerate)(device_t *);
+        int (*device_remove)(device_t *);
+        int (*device_online)(device_t *);                       /**< Optional */
+        int (*device_offline)(device_t *);                      /**< Optional */
+        endpoint_t *(*device_find_endpoint)(device_t*, usb_target_t, usb_direction_t);
+        endpoint_t *(*endpoint_create)(device_t *, const usb_endpoint_desc_t *);
+
+        /* Operations on endpoint */
+        int (*endpoint_register)(endpoint_t *);
+        int (*endpoint_unregister)(endpoint_t *);
+        void (*endpoint_destroy)(endpoint_t *);                 /**< Optional */
         bool (*endpoint_get_toggle)(endpoint_t *);              /**< Optional */
         void (*endpoint_set_toggle)(endpoint_t *, bool);        /**< Optional */
+        ssize_t (*endpoint_count_bw) (endpoint_t *, size_t);
+        usb_transfer_batch_t *(*batch_create)(endpoint_t *);    /**< Optional */
 
-        int (*reserve_default_address)(bus_t *, usb_speed_t);
-        int (*release_default_address)(bus_t *);
+        /* Operations on batch */
+        void (*batch_destroy)(usb_transfer_batch_t *);  /**< Optional */
+};
 
-        int (*reset_toggle)(bus_t *, usb_target_t, toggle_reset_mode_t);
-
-        size_t (*count_bw) (endpoint_t *, size_t);
-
-        usb_transfer_batch_t *(*create_batch)(bus_t *, endpoint_t *); /**< Optional */
-        void (*destroy_batch)(usb_transfer_batch_t *);  /**< Optional */
-} bus_ops_t;
+/**
+ * Use this macro to lookup virtual function.
+ */
+#define BUS_OPS_LOOKUP(start, fn) ({ bus_ops_t const * ops = (start); while (ops && ops->fn == NULL) ops = ops->parent; ops; })
 
 /** Endpoint management structure */
@@ -112 +129 @@
         fibril_mutex_t guard;
 
+        /* TODO: get rid of this one. */
+        hcd_t *hcd;
+
         size_t device_size;
 
         /* Do not call directly, ops are synchronized. */
-        bus_ops_t ops;
+        const bus_ops_t *ops;
 
         /* This structure is meant to be extended by overriding. */
 } bus_t;
 
-void bus_init(bus_t *, size_t);
-int device_init(device_t *);
+void bus_init(bus_t *, hcd_t *, size_t);
+int bus_device_init(device_t *, bus_t *);
 
-int device_set_default_name(device_t *);
+int bus_device_set_default_name(device_t *);
 
-int bus_enumerate_device(bus_t *, hcd_t *, device_t *);
-int bus_remove_device(bus_t *, hcd_t *, device_t *);
+int bus_device_enumerate(device_t *);
+int bus_device_remove(device_t *);
 
-int bus_online_device(bus_t *, hcd_t *, device_t *);
-int bus_offline_device(bus_t *, hcd_t *, device_t *);
+int bus_device_online(device_t *);
+int bus_device_offline(device_t *);
 
-int bus_add_endpoint(bus_t *, device_t *, const usb_endpoint_desc_t *, endpoint_t **);
-endpoint_t *bus_find_endpoint(bus_t *, device_t *, usb_target_t, usb_direction_t);
-int bus_remove_endpoint(bus_t *, endpoint_t *);
-
-size_t bus_count_bw(endpoint_t *, size_t);
+int bus_endpoint_add(device_t *, const usb_endpoint_desc_t *, endpoint_t **);
+endpoint_t *bus_find_endpoint(device_t *, usb_target_t, usb_direction_t);
+int bus_endpoint_remove(endpoint_t *);
 
 int bus_reserve_default_address(bus_t *, usb_speed_t);