[RFC PATCH 06/14] usb: core: hcd: Introduce USB HCD APIs for interrupter management

Wesley Cheng
2022-12-23T23:33:23Z

For USB HCDs that can support multiple USB interrupters, expose functions
that class drivers can utilize for setting up secondary interrupters.
Class drivers can pass this information to its respective clients, i.e.
a dedicated DSP.

Signed-off-by: Wesley Cheng <quic_wcheng@quicinc.com>
---
drivers/usb/core/hcd.c | 86 +++++++++++++++++++++++++++++++++++++++++
include/linux/usb.h | 7 ++++
include/linux/usb/hcd.h | 16 +++++++-
3 files changed, 108 insertions(+), 1 deletion(-)

diff --git a/drivers/usb/core/hcd.c b/drivers/usb/core/hcd.c
index 8300baedafd2..90ead90faf1d 100644
--- a/drivers/usb/core/hcd.c
+++ b/drivers/usb/core/hcd.c
@@ -1579,6 +1579,92 @@ int usb_hcd_submit_urb (struct urb *urb, gfp_t mem_flags)
return status;
}

+/**
+ * usb_free_interrupter - free an interrupter
+ * @udev: usb device which requested the interrupter
+ * @intr_num: interrupter number to free
+ *
+ * Release the USB HCD interrupter that was reserved by a USB class driver.
+ **/
+int usb_free_interrupter(struct usb_device *udev, int intr_num)
+{
+ struct usb_hcd *hcd = bus_to_hcd(udev->bus);
+ int ret = 0;
+
+ if (hcd->driver->free_interrupter)
+ ret = hcd->driver->free_interrupter(hcd, intr_num);
+
+ return ret;
+}
+EXPORT_SYMBOL_GPL(usb_free_interrupter);
+
+/**
+ * usb_set_interruper - Reserve an interrupter
+ * @udev: usb device which requested the interrupter
+ * @intr_num: interrupter number to reserve
+ * @dma: iova address to event ring
+ *
+ * Request for a specific interrupter to be reserved for a USB class driver.
+ * This will return the base address to the event ring that was allocated to
+ * the specific interrupter.
+ **/
+phys_addr_t usb_set_interruper(struct usb_device *udev, int intr_num,
+ dma_addr_t *dma)
+{
+ struct usb_hcd *hcd;
+ phys_addr_t pa = 0;
+
+ hcd = bus_to_hcd(udev->bus);
+ if (hcd->driver->update_interrupter)
+ pa = hcd->driver->update_interrupter(hcd, intr_num, dma);
+
+ return pa;
+}
+EXPORT_SYMBOL_GPL(usb_set_interruper);
+
+/**
+ * usb_hcd_get_transfer_resource - Retrieve USB EP resource information
+ * @udev: usb device
+ * @ep: usb ep to retrieve resource information
+ * @dma: iova address to transfer resource
+ *
+ * Request for USB EP transfer resource which is used for submitting
+ * transfers to the USB HCD.
+ **/
+phys_addr_t usb_hcd_get_transfer_resource(struct usb_device *udev,
+ struct usb_host_endpoint *ep, dma_addr_t *dma)
+{
+ struct usb_hcd *hcd;
+ phys_addr_t pa = 0;
+
+ hcd = bus_to_hcd(udev->bus);
+ if (hcd->driver->get_transfer_resource)
+ pa = hcd->driver->get_transfer_resource(udev, ep, dma);
+
+ return pa;
+}
+EXPORT_SYMBOL_GPL(usb_hcd_get_transfer_resource);
+
+/**
+ * usb_hcd_stop_endpoint - Halt USB EP transfers
+ * @udev: usb device
+ * @ep: usb ep to stop
+ *
+ * Stop pending transfers on a specific USB endpoint.
+ **/
+int usb_hcd_stop_endpoint(struct usb_device *udev,
+ struct usb_host_endpoint *ep)
+{
+ struct usb_hcd *hcd = bus_to_hcd(udev->bus);
+ int ret = 0;
+
+ if (hcd->driver->stop_endpoint)
+ ret = hcd->driver->stop_endpoint(hcd, udev, ep);
+
+ return ret;
+}
+EXPORT_SYMBOL_GPL(usb_hcd_stop_endpoint);
+
/*-------------------------------------------------------------------------*/

/* this makes the hcd giveback() the urb more quickly, by kicking it
diff --git a/include/linux/usb.h b/include/linux/usb.h
index d4afeeec1e1a..2f71cd4fb6e0 100644
--- a/include/linux/usb.h
+++ b/include/linux/usb.h
@@ -1724,6 +1724,13 @@ static inline void usb_fill_int_urb(struct urb *urb,
urb->start_frame = -1;
}

+extern int usb_free_interrupter(struct usb_device *udev, int intr_num);
+extern phys_addr_t usb_set_interruper(struct usb_device *udev,
+ int intr_num, dma_addr_t *dma);
+extern int usb_hcd_stop_endpoint(struct usb_device *udev,
+ struct usb_host_endpoint *ep);
+extern phys_addr_t usb_hcd_get_transfer_resource(struct usb_device *udev,
+ struct usb_host_endpoint *ep, dma_addr_t *dma);
extern void usb_init_urb(struct urb *urb);
extern struct urb *usb_alloc_urb(int iso_packets, gfp_t mem_flags);
extern void usb_free_urb(struct urb *urb);
diff --git a/include/linux/usb/hcd.h b/include/linux/usb/hcd.h
index b51c07111729..f5bce80b2e40 100644
--- a/include/linux/usb/hcd.h
+++ b/include/linux/usb/hcd.h
@@ -343,7 +343,21 @@ struct hc_driver {
int (*free_streams)(struct usb_hcd *hcd, struct usb_device *udev,
struct usb_host_endpoint **eps, unsigned int num_eps,
gfp_t mem_flags);
-
+ /* Frees an interrupter from the current client, and makes it available
+ * for use.
+ */
+ int (*free_interrupter)(struct usb_hcd *hcd, int intr_num);
+ /* Request an interrupter from the current allocated pool. Will provide
+ * the address to the allocated ring.
+ */
+ phys_addr_t (*update_interrupter)(struct usb_hcd *hcd, int intr_num,
+ dma_addr_t *dma);
+ /* Fetch transfer/ep ring address */
+ phys_addr_t (*get_transfer_resource)(struct usb_device *udev,
+ struct usb_host_endpoint *ep, dma_addr_t *dma);
+ /* Stop transfers on a particular endpoint */
+ int (*stop_endpoint)(struct usb_hcd *hcd, struct usb_device *udev,
+ struct usb_host_endpoint *ep);
/* Bandwidth computation functions */
/* Note that add_endpoint() can only be called once per endpoint before
* check_bandwidth() or reset_bandwidth() must be called.