Greg Kroah-Hartman | ba4468d4 | 2014-08-30 17:06:54 -0700 | [diff] [blame] | 1 | /* |
| 2 | * Greybus "AP" USB driver |
| 3 | * |
| 4 | * Copyright 2014 Google Inc. |
| 5 | * |
| 6 | * Released under the GPLv2 only. |
| 7 | */ |
| 8 | #include <linux/kernel.h> |
| 9 | #include <linux/module.h> |
| 10 | #include <linux/slab.h> |
| 11 | #include <linux/errno.h> |
Matt Porter | f0f7091 | 2014-09-30 16:01:40 -0400 | [diff] [blame] | 12 | #include <linux/sizes.h> |
Greg Kroah-Hartman | ba4468d4 | 2014-08-30 17:06:54 -0700 | [diff] [blame] | 13 | #include <linux/usb.h> |
Alex Elder | 1cfc667 | 2014-09-30 19:25:21 -0500 | [diff] [blame] | 14 | |
Greg Kroah-Hartman | de536e3 | 2014-08-31 16:17:04 -0700 | [diff] [blame] | 15 | #include "greybus.h" |
Greg Kroah-Hartman | 43cc32a | 2014-09-07 13:51:12 -0700 | [diff] [blame] | 16 | #include "svc_msg.h" |
Alex Elder | 1cfc667 | 2014-09-30 19:25:21 -0500 | [diff] [blame] | 17 | #include "kernel_ver.h" |
Greg Kroah-Hartman | ba4468d4 | 2014-08-30 17:06:54 -0700 | [diff] [blame] | 18 | |
Greg Kroah-Hartman | 29f000f | 2014-09-19 18:37:44 -0700 | [diff] [blame] | 19 | /* Memory sizes for the buffers sent to/from the ES1 controller */ |
Matt Porter | f0f7091 | 2014-09-30 16:01:40 -0400 | [diff] [blame] | 20 | #define ES1_SVC_MSG_SIZE (sizeof(struct svc_msg) + SZ_64K) |
Greg Kroah-Hartman | 29f000f | 2014-09-19 18:37:44 -0700 | [diff] [blame] | 21 | #define ES1_GBUF_MSG_SIZE PAGE_SIZE |
| 22 | |
Greg Kroah-Hartman | ba4468d4 | 2014-08-30 17:06:54 -0700 | [diff] [blame] | 23 | static const struct usb_device_id id_table[] = { |
Greg Kroah-Hartman | 7f9e05e | 2014-09-13 17:28:33 -0700 | [diff] [blame] | 24 | /* Made up numbers for the SVC USB Bridge in ES1 */ |
| 25 | { USB_DEVICE(0xffff, 0x0001) }, |
Greg Kroah-Hartman | ba4468d4 | 2014-08-30 17:06:54 -0700 | [diff] [blame] | 26 | { }, |
| 27 | }; |
| 28 | MODULE_DEVICE_TABLE(usb, id_table); |
| 29 | |
Greg Kroah-Hartman | 1129973 | 2014-09-12 21:17:37 -0700 | [diff] [blame] | 30 | /* |
| 31 | * Number of CPort IN urbs in flight at any point in time. |
| 32 | * Adjust if we are having stalls in the USB buffer due to not enough urbs in |
| 33 | * flight. |
| 34 | */ |
| 35 | #define NUM_CPORT_IN_URB 4 |
| 36 | |
| 37 | /* Number of CPort OUT urbs in flight at any point in time. |
| 38 | * Adjust if we get messages saying we are out of urbs in the system log. |
| 39 | */ |
| 40 | #define NUM_CPORT_OUT_URB 8 |
| 41 | |
Greg Kroah-Hartman | 7f9e05e | 2014-09-13 17:28:33 -0700 | [diff] [blame] | 42 | /** |
| 43 | * es1_ap_dev - ES1 USB Bridge to AP structure |
| 44 | * @usb_dev: pointer to the USB device we are. |
| 45 | * @usb_intf: pointer to the USB interface we are bound to. |
| 46 | * @hd: pointer to our greybus_host_device structure |
| 47 | * @control_endpoint: endpoint to send data to SVC |
| 48 | * @svc_endpoint: endpoint for SVC data in |
| 49 | * @cport_in_endpoint: bulk in endpoint for CPort data |
| 50 | * @cport-out_endpoint: bulk out endpoint for CPort data |
| 51 | * @svc_buffer: buffer for SVC messages coming in on @svc_endpoint |
| 52 | * @svc_urb: urb for SVC messages coming in on @svc_endpoint |
| 53 | * @cport_in_urb: array of urbs for the CPort in messages |
| 54 | * @cport_in_buffer: array of buffers for the @cport_in_urb urbs |
| 55 | * @cport_out_urb: array of urbs for the CPort out messages |
| 56 | * @cport_out_urb_busy: array of flags to see if the @cport_out_urb is busy or |
| 57 | * not. |
| 58 | * @cport_out_urb_lock: locks the @cport_out_urb_busy "list" |
| 59 | */ |
Greg Kroah-Hartman | 6f83ab7 | 2014-08-30 17:30:04 -0700 | [diff] [blame] | 60 | struct es1_ap_dev { |
| 61 | struct usb_device *usb_dev; |
| 62 | struct usb_interface *usb_intf; |
Greg Kroah-Hartman | a39879f | 2014-09-06 16:57:36 -0700 | [diff] [blame] | 63 | struct greybus_host_device *hd; |
Greg Kroah-Hartman | 6f83ab7 | 2014-08-30 17:30:04 -0700 | [diff] [blame] | 64 | |
Greg Kroah-Hartman | 7f9e05e | 2014-09-13 17:28:33 -0700 | [diff] [blame] | 65 | __u8 control_endpoint; |
| 66 | __u8 svc_endpoint; |
| 67 | __u8 cport_in_endpoint; |
| 68 | __u8 cport_out_endpoint; |
| 69 | |
| 70 | u8 *svc_buffer; |
| 71 | struct urb *svc_urb; |
| 72 | |
| 73 | struct urb *cport_in_urb[NUM_CPORT_IN_URB]; |
| 74 | u8 *cport_in_buffer[NUM_CPORT_IN_URB]; |
| 75 | struct urb *cport_out_urb[NUM_CPORT_OUT_URB]; |
| 76 | bool cport_out_urb_busy[NUM_CPORT_OUT_URB]; |
| 77 | spinlock_t cport_out_urb_lock; |
Greg Kroah-Hartman | 6f83ab7 | 2014-08-30 17:30:04 -0700 | [diff] [blame] | 78 | }; |
| 79 | |
Greg Kroah-Hartman | a39879f | 2014-09-06 16:57:36 -0700 | [diff] [blame] | 80 | static inline struct es1_ap_dev *hd_to_es1(struct greybus_host_device *hd) |
| 81 | { |
Alex Elder | 69f93ab | 2014-09-22 18:53:02 -0500 | [diff] [blame] | 82 | return (struct es1_ap_dev *)&hd->hd_priv; |
Greg Kroah-Hartman | a39879f | 2014-09-06 16:57:36 -0700 | [diff] [blame] | 83 | } |
| 84 | |
Greg Kroah-Hartman | 1129973 | 2014-09-12 21:17:37 -0700 | [diff] [blame] | 85 | static void cport_out_callback(struct urb *urb); |
| 86 | |
Greg Kroah-Hartman | f1eec30 | 2014-08-30 17:18:14 -0700 | [diff] [blame] | 87 | /* |
Greg Kroah-Hartman | a39879f | 2014-09-06 16:57:36 -0700 | [diff] [blame] | 88 | * Allocate the actual buffer for this gbuf and device and cport |
| 89 | * |
| 90 | * We are responsible for setting the following fields in a struct gbuf: |
| 91 | * void *hcpriv; |
| 92 | * void *transfer_buffer; |
| 93 | * u32 transfer_buffer_length; |
Greg Kroah-Hartman | f1eec30 | 2014-08-30 17:18:14 -0700 | [diff] [blame] | 94 | */ |
Alex Elder | 9a6f631 | 2014-10-06 06:53:11 -0500 | [diff] [blame] | 95 | static int alloc_gbuf_data(struct gbuf *gbuf, unsigned int size, |
| 96 | gfp_t gfp_mask) |
Greg Kroah-Hartman | a39879f | 2014-09-06 16:57:36 -0700 | [diff] [blame] | 97 | { |
Greg Kroah-Hartman | a39879f | 2014-09-06 16:57:36 -0700 | [diff] [blame] | 98 | u8 *buffer; |
| 99 | |
Alex Elder | f7935e3 | 2014-11-17 18:08:34 -0600 | [diff] [blame] | 100 | if (gbuf->transfer_buffer) |
| 101 | return -EALREADY; |
| 102 | |
Greg Kroah-Hartman | 29f000f | 2014-09-19 18:37:44 -0700 | [diff] [blame] | 103 | if (size > ES1_GBUF_MSG_SIZE) { |
Greg Kroah-Hartman | f036e05 | 2014-09-19 19:13:33 -0700 | [diff] [blame] | 104 | pr_err("guf was asked to be bigger than %ld!\n", |
Greg Kroah-Hartman | 29f000f | 2014-09-19 18:37:44 -0700 | [diff] [blame] | 105 | ES1_GBUF_MSG_SIZE); |
| 106 | } |
| 107 | |
Alex Elder | 5259ef1 | 2014-11-18 13:26:39 -0600 | [diff] [blame] | 108 | /* |
| 109 | * For ES1 we need to insert a byte at the front of the data |
Alex Elder | 06a4a06 | 2014-11-18 13:26:40 -0600 | [diff] [blame] | 110 | * to indicate the destination CPort id. We only need one |
| 111 | * extra byte, but we allocate four extra bytes to allow the |
| 112 | * buffer returned to be aligned on a four-byte boundary. |
Greg Kroah-Hartman | a39879f | 2014-09-06 16:57:36 -0700 | [diff] [blame] | 113 | * |
Alex Elder | 5259ef1 | 2014-11-18 13:26:39 -0600 | [diff] [blame] | 114 | * This is only needed for outbound data, but we handle |
| 115 | * buffers for inbound data the same way for consistency. |
Alex Elder | 9a6f631 | 2014-10-06 06:53:11 -0500 | [diff] [blame] | 116 | * |
Alex Elder | 5259ef1 | 2014-11-18 13:26:39 -0600 | [diff] [blame] | 117 | * XXX Do we need to indicate the destination device id too? |
Greg Kroah-Hartman | a39879f | 2014-09-06 16:57:36 -0700 | [diff] [blame] | 118 | */ |
Alex Elder | 06a4a06 | 2014-11-18 13:26:40 -0600 | [diff] [blame] | 119 | buffer = kzalloc(GB_BUFFER_ALIGN + size, gfp_mask); |
Greg Kroah-Hartman | a39879f | 2014-09-06 16:57:36 -0700 | [diff] [blame] | 120 | if (!buffer) |
| 121 | return -ENOMEM; |
Alex Elder | 06a4a06 | 2014-11-18 13:26:40 -0600 | [diff] [blame] | 122 | buffer += GB_BUFFER_ALIGN; |
Greg Kroah-Hartman | a39879f | 2014-09-06 16:57:36 -0700 | [diff] [blame] | 123 | |
Alex Elder | 9a6f631 | 2014-10-06 06:53:11 -0500 | [diff] [blame] | 124 | gbuf->transfer_buffer = buffer; |
Greg Kroah-Hartman | a39879f | 2014-09-06 16:57:36 -0700 | [diff] [blame] | 125 | gbuf->transfer_buffer_length = size; |
| 126 | |
Greg Kroah-Hartman | a39879f | 2014-09-06 16:57:36 -0700 | [diff] [blame] | 127 | return 0; |
| 128 | } |
| 129 | |
| 130 | /* Free the memory we allocated with a gbuf */ |
Greg Kroah-Hartman | 3e7736e | 2014-09-21 17:34:28 -0700 | [diff] [blame] | 131 | static void free_gbuf_data(struct gbuf *gbuf) |
Greg Kroah-Hartman | a39879f | 2014-09-06 16:57:36 -0700 | [diff] [blame] | 132 | { |
Alex Elder | 9a6f631 | 2014-10-06 06:53:11 -0500 | [diff] [blame] | 133 | u8 *transfer_buffer = gbuf->transfer_buffer; |
Greg Kroah-Hartman | a39879f | 2014-09-06 16:57:36 -0700 | [diff] [blame] | 134 | |
Greg Kroah-Hartman | 3e7736e | 2014-09-21 17:34:28 -0700 | [diff] [blame] | 135 | /* Can be called with a NULL transfer_buffer on some error paths */ |
Alex Elder | 9a6f631 | 2014-10-06 06:53:11 -0500 | [diff] [blame] | 136 | if (!transfer_buffer) |
| 137 | return; |
| 138 | |
Alex Elder | 06a4a06 | 2014-11-18 13:26:40 -0600 | [diff] [blame] | 139 | /* Account for the space set aside for the prepended cport id */ |
| 140 | transfer_buffer -= GB_BUFFER_ALIGN; |
Alex Elder | 9a6f631 | 2014-10-06 06:53:11 -0500 | [diff] [blame] | 141 | kfree(transfer_buffer); |
Alex Elder | f7935e3 | 2014-11-17 18:08:34 -0600 | [diff] [blame] | 142 | gbuf->transfer_buffer = NULL; |
Greg Kroah-Hartman | a39879f | 2014-09-06 16:57:36 -0700 | [diff] [blame] | 143 | } |
| 144 | |
Greg Kroah-Hartman | 8c53e07 | 2014-09-12 20:47:11 -0700 | [diff] [blame] | 145 | #define ES1_TIMEOUT 500 /* 500 ms for the SVC to do something */ |
Alex Elder | 0db32a6 | 2014-09-24 05:16:14 -0500 | [diff] [blame] | 146 | static int submit_svc(struct svc_msg *svc_msg, struct greybus_host_device *hd) |
Greg Kroah-Hartman | 68f1fc4 | 2014-09-07 13:12:11 -0700 | [diff] [blame] | 147 | { |
| 148 | struct es1_ap_dev *es1 = hd_to_es1(hd); |
Greg Kroah-Hartman | 8c53e07 | 2014-09-12 20:47:11 -0700 | [diff] [blame] | 149 | int retval; |
Greg Kroah-Hartman | 68f1fc4 | 2014-09-07 13:12:11 -0700 | [diff] [blame] | 150 | |
Greg Kroah-Hartman | 8c53e07 | 2014-09-12 20:47:11 -0700 | [diff] [blame] | 151 | /* SVC messages go down our control pipe */ |
| 152 | retval = usb_control_msg(es1->usb_dev, |
| 153 | usb_sndctrlpipe(es1->usb_dev, |
| 154 | es1->control_endpoint), |
| 155 | 0x01, /* vendor request AP message */ |
Matt Porter | 648cb6c | 2014-09-22 15:51:48 -0400 | [diff] [blame] | 156 | USB_DIR_OUT | USB_TYPE_VENDOR | USB_RECIP_INTERFACE, |
Greg Kroah-Hartman | 8c53e07 | 2014-09-12 20:47:11 -0700 | [diff] [blame] | 157 | 0x00, 0x00, |
| 158 | (char *)svc_msg, |
| 159 | sizeof(*svc_msg), |
| 160 | ES1_TIMEOUT); |
| 161 | if (retval != sizeof(*svc_msg)) |
| 162 | return retval; |
| 163 | |
| 164 | return 0; |
Greg Kroah-Hartman | 68f1fc4 | 2014-09-07 13:12:11 -0700 | [diff] [blame] | 165 | } |
| 166 | |
Greg Kroah-Hartman | 0dad95d | 2014-09-13 09:54:35 -0700 | [diff] [blame] | 167 | static struct urb *next_free_urb(struct es1_ap_dev *es1, gfp_t gfp_mask) |
Greg Kroah-Hartman | 1129973 | 2014-09-12 21:17:37 -0700 | [diff] [blame] | 168 | { |
Greg Kroah-Hartman | 0dad95d | 2014-09-13 09:54:35 -0700 | [diff] [blame] | 169 | struct urb *urb = NULL; |
| 170 | unsigned long flags; |
| 171 | int i; |
| 172 | |
| 173 | spin_lock_irqsave(&es1->cport_out_urb_lock, flags); |
| 174 | |
| 175 | /* Look in our pool of allocated urbs first, as that's the "fastest" */ |
| 176 | for (i = 0; i < NUM_CPORT_OUT_URB; ++i) { |
| 177 | if (es1->cport_out_urb_busy[i] == false) { |
| 178 | es1->cport_out_urb_busy[i] = true; |
| 179 | urb = es1->cport_out_urb[i]; |
| 180 | break; |
| 181 | } |
| 182 | } |
| 183 | spin_unlock_irqrestore(&es1->cport_out_urb_lock, flags); |
| 184 | if (urb) |
| 185 | return urb; |
| 186 | |
| 187 | /* |
| 188 | * Crap, pool is empty, complain to the syslog and go allocate one |
| 189 | * dynamically as we have to succeed. |
| 190 | */ |
| 191 | dev_err(&es1->usb_dev->dev, |
| 192 | "No free CPort OUT urbs, having to dynamically allocate one!\n"); |
| 193 | urb = usb_alloc_urb(0, gfp_mask); |
| 194 | if (!urb) |
| 195 | return NULL; |
| 196 | |
| 197 | return urb; |
Greg Kroah-Hartman | 1129973 | 2014-09-12 21:17:37 -0700 | [diff] [blame] | 198 | } |
| 199 | |
Alex Elder | 61418b9 | 2014-10-16 06:35:29 -0500 | [diff] [blame] | 200 | static int submit_gbuf(struct gbuf *gbuf, gfp_t gfp_mask) |
Greg Kroah-Hartman | 1129973 | 2014-09-12 21:17:37 -0700 | [diff] [blame] | 201 | { |
Alex Elder | ba99346 | 2014-11-17 08:08:43 -0600 | [diff] [blame] | 202 | struct greybus_host_device *hd = gbuf->hd; |
Greg Kroah-Hartman | 1129973 | 2014-09-12 21:17:37 -0700 | [diff] [blame] | 203 | struct es1_ap_dev *es1 = hd_to_es1(hd); |
| 204 | struct usb_device *udev = es1->usb_dev; |
Alex Elder | 0f4c808 | 2014-11-18 13:26:41 -0600 | [diff] [blame^] | 205 | u16 dest_cport_id = gbuf->dest_cport_id; |
Greg Kroah-Hartman | 1129973 | 2014-09-12 21:17:37 -0700 | [diff] [blame] | 206 | int retval; |
| 207 | u8 *transfer_buffer; |
| 208 | u8 *buffer; |
| 209 | struct urb *urb; |
| 210 | |
| 211 | transfer_buffer = gbuf->transfer_buffer; |
Alex Elder | f7935e3 | 2014-11-17 18:08:34 -0600 | [diff] [blame] | 212 | if (!transfer_buffer) |
| 213 | return -EINVAL; |
Greg Kroah-Hartman | 1129973 | 2014-09-12 21:17:37 -0700 | [diff] [blame] | 214 | buffer = &transfer_buffer[-1]; /* yes, we mean -1 */ |
| 215 | |
Alex Elder | 0f4c808 | 2014-11-18 13:26:41 -0600 | [diff] [blame^] | 216 | /* Do one last check of the target CPort id before filling it in */ |
| 217 | if (dest_cport_id == CPORT_ID_BAD) { |
| 218 | pr_err("request to send inbound data buffer\n"); |
Alex Elder | 5259ef1 | 2014-11-18 13:26:39 -0600 | [diff] [blame] | 219 | return -EINVAL; |
| 220 | } |
Alex Elder | 0f4c808 | 2014-11-18 13:26:41 -0600 | [diff] [blame^] | 221 | if (dest_cport_id > (u16)U8_MAX) { |
| 222 | pr_err("dest_cport_id (%hd) is out of range for ES1\n", |
| 223 | dest_cport_id); |
| 224 | return -EINVAL; |
| 225 | } |
| 226 | *buffer = dest_cport_id; |
Alex Elder | 5259ef1 | 2014-11-18 13:26:39 -0600 | [diff] [blame] | 227 | |
Greg Kroah-Hartman | 1129973 | 2014-09-12 21:17:37 -0700 | [diff] [blame] | 228 | /* Find a free urb */ |
Greg Kroah-Hartman | 0dad95d | 2014-09-13 09:54:35 -0700 | [diff] [blame] | 229 | urb = next_free_urb(es1, gfp_mask); |
Greg Kroah-Hartman | 1129973 | 2014-09-12 21:17:37 -0700 | [diff] [blame] | 230 | if (!urb) |
| 231 | return -ENOMEM; |
| 232 | |
Greg Kroah-Hartman | d814488 | 2014-10-27 13:31:01 +0800 | [diff] [blame] | 233 | gbuf->hcd_data = urb; |
| 234 | |
Greg Kroah-Hartman | 1129973 | 2014-09-12 21:17:37 -0700 | [diff] [blame] | 235 | usb_fill_bulk_urb(urb, udev, |
| 236 | usb_sndbulkpipe(udev, es1->cport_out_endpoint), |
| 237 | buffer, gbuf->transfer_buffer_length + 1, |
| 238 | cport_out_callback, gbuf); |
| 239 | retval = usb_submit_urb(urb, gfp_mask); |
| 240 | return retval; |
| 241 | } |
| 242 | |
Greg Kroah-Hartman | 4afbba0 | 2014-10-27 14:01:06 +0800 | [diff] [blame] | 243 | static void kill_gbuf(struct gbuf *gbuf) |
Greg Kroah-Hartman | d814488 | 2014-10-27 13:31:01 +0800 | [diff] [blame] | 244 | { |
| 245 | struct urb *urb = gbuf->hcd_data; |
| 246 | |
| 247 | if (!urb) |
Greg Kroah-Hartman | 4afbba0 | 2014-10-27 14:01:06 +0800 | [diff] [blame] | 248 | return; |
Greg Kroah-Hartman | d814488 | 2014-10-27 13:31:01 +0800 | [diff] [blame] | 249 | |
| 250 | usb_kill_urb(urb); |
Greg Kroah-Hartman | d814488 | 2014-10-27 13:31:01 +0800 | [diff] [blame] | 251 | } |
| 252 | |
Greg Kroah-Hartman | a39879f | 2014-09-06 16:57:36 -0700 | [diff] [blame] | 253 | static struct greybus_host_driver es1_driver = { |
Greg Kroah-Hartman | 3e7736e | 2014-09-21 17:34:28 -0700 | [diff] [blame] | 254 | .hd_priv_size = sizeof(struct es1_ap_dev), |
| 255 | .alloc_gbuf_data = alloc_gbuf_data, |
| 256 | .free_gbuf_data = free_gbuf_data, |
Alex Elder | 0db32a6 | 2014-09-24 05:16:14 -0500 | [diff] [blame] | 257 | .submit_svc = submit_svc, |
Greg Kroah-Hartman | 3e7736e | 2014-09-21 17:34:28 -0700 | [diff] [blame] | 258 | .submit_gbuf = submit_gbuf, |
Greg Kroah-Hartman | 4afbba0 | 2014-10-27 14:01:06 +0800 | [diff] [blame] | 259 | .kill_gbuf = kill_gbuf, |
Greg Kroah-Hartman | a39879f | 2014-09-06 16:57:36 -0700 | [diff] [blame] | 260 | }; |
| 261 | |
Alex Elder | 877b1ee | 2014-09-24 05:16:13 -0500 | [diff] [blame] | 262 | /* Common function to report consistent warnings based on URB status */ |
| 263 | static int check_urb_status(struct urb *urb) |
Greg Kroah-Hartman | de536e3 | 2014-08-31 16:17:04 -0700 | [diff] [blame] | 264 | { |
| 265 | struct device *dev = &urb->dev->dev; |
| 266 | int status = urb->status; |
Greg Kroah-Hartman | de536e3 | 2014-08-31 16:17:04 -0700 | [diff] [blame] | 267 | |
| 268 | switch (status) { |
| 269 | case 0: |
Alex Elder | 877b1ee | 2014-09-24 05:16:13 -0500 | [diff] [blame] | 270 | return 0; |
| 271 | |
Greg Kroah-Hartman | de536e3 | 2014-08-31 16:17:04 -0700 | [diff] [blame] | 272 | case -EOVERFLOW: |
| 273 | dev_err(dev, "%s: overflow actual length is %d\n", |
| 274 | __func__, urb->actual_length); |
| 275 | case -ECONNRESET: |
| 276 | case -ENOENT: |
| 277 | case -ESHUTDOWN: |
| 278 | case -EILSEQ: |
Matt Porter | 8fd39e3 | 2014-10-13 03:00:53 -0400 | [diff] [blame] | 279 | case -EPROTO: |
Greg Kroah-Hartman | de536e3 | 2014-08-31 16:17:04 -0700 | [diff] [blame] | 280 | /* device is gone, stop sending */ |
Alex Elder | 877b1ee | 2014-09-24 05:16:13 -0500 | [diff] [blame] | 281 | return status; |
Greg Kroah-Hartman | de536e3 | 2014-08-31 16:17:04 -0700 | [diff] [blame] | 282 | } |
Alex Elder | 877b1ee | 2014-09-24 05:16:13 -0500 | [diff] [blame] | 283 | dev_err(dev, "%s: unknown status %d\n", __func__, status); |
| 284 | |
| 285 | return -EAGAIN; |
| 286 | } |
| 287 | |
Alex Elder | 6892537 | 2014-10-20 10:27:59 -0500 | [diff] [blame] | 288 | static void ap_disconnect(struct usb_interface *interface) |
| 289 | { |
| 290 | struct es1_ap_dev *es1; |
| 291 | int i; |
| 292 | |
| 293 | es1 = usb_get_intfdata(interface); |
| 294 | if (!es1) |
| 295 | return; |
| 296 | |
| 297 | /* Tear down everything! */ |
| 298 | for (i = 0; i < NUM_CPORT_OUT_URB; ++i) { |
Alex Elder | 1ec1d6d | 2014-10-20 10:28:00 -0500 | [diff] [blame] | 299 | struct urb *urb = es1->cport_out_urb[i]; |
| 300 | |
| 301 | if (!urb) |
| 302 | break; |
| 303 | usb_kill_urb(urb); |
| 304 | usb_free_urb(urb); |
| 305 | es1->cport_out_urb[i] = NULL; |
| 306 | es1->cport_out_urb_busy[i] = false; /* just to be anal */ |
Alex Elder | 6892537 | 2014-10-20 10:27:59 -0500 | [diff] [blame] | 307 | } |
| 308 | |
| 309 | for (i = 0; i < NUM_CPORT_IN_URB; ++i) { |
Alex Elder | 1ec1d6d | 2014-10-20 10:28:00 -0500 | [diff] [blame] | 310 | struct urb *urb = es1->cport_in_urb[i]; |
| 311 | |
| 312 | if (!urb) |
| 313 | break; |
| 314 | usb_kill_urb(urb); |
| 315 | usb_free_urb(urb); |
Alex Elder | 6892537 | 2014-10-20 10:27:59 -0500 | [diff] [blame] | 316 | kfree(es1->cport_in_buffer[i]); |
Alex Elder | 1ec1d6d | 2014-10-20 10:28:00 -0500 | [diff] [blame] | 317 | es1->cport_in_buffer[i] = NULL; |
Alex Elder | 6892537 | 2014-10-20 10:27:59 -0500 | [diff] [blame] | 318 | } |
| 319 | |
| 320 | usb_kill_urb(es1->svc_urb); |
| 321 | usb_free_urb(es1->svc_urb); |
Alex Elder | 1ec1d6d | 2014-10-20 10:28:00 -0500 | [diff] [blame] | 322 | es1->svc_urb = NULL; |
Alex Elder | 6892537 | 2014-10-20 10:27:59 -0500 | [diff] [blame] | 323 | kfree(es1->svc_buffer); |
Alex Elder | 1ec1d6d | 2014-10-20 10:28:00 -0500 | [diff] [blame] | 324 | es1->svc_buffer = NULL; |
| 325 | |
Alex Elder | 6892537 | 2014-10-20 10:27:59 -0500 | [diff] [blame] | 326 | usb_set_intfdata(interface, NULL); |
Alex Elder | 1ec1d6d | 2014-10-20 10:28:00 -0500 | [diff] [blame] | 327 | greybus_remove_hd(es1->hd); |
| 328 | |
| 329 | usb_put_dev(es1->usb_dev); |
Alex Elder | 6892537 | 2014-10-20 10:27:59 -0500 | [diff] [blame] | 330 | } |
| 331 | |
Alex Elder | 877b1ee | 2014-09-24 05:16:13 -0500 | [diff] [blame] | 332 | /* Callback for when we get a SVC message */ |
| 333 | static void svc_in_callback(struct urb *urb) |
| 334 | { |
| 335 | struct es1_ap_dev *es1 = urb->context; |
| 336 | struct device *dev = &urb->dev->dev; |
| 337 | int status = check_urb_status(urb); |
| 338 | int retval; |
| 339 | |
Alex Elder | bedfdf3 | 2014-10-17 05:18:22 -0500 | [diff] [blame] | 340 | if (status) { |
Alex Elder | c4a432d | 2014-10-24 05:02:02 -0500 | [diff] [blame] | 341 | if (status == -EAGAIN) |
| 342 | goto exit; |
| 343 | dev_err(dev, "urb svc in error %d (dropped)\n", status); |
| 344 | return; |
Alex Elder | bedfdf3 | 2014-10-17 05:18:22 -0500 | [diff] [blame] | 345 | } |
Greg Kroah-Hartman | de536e3 | 2014-08-31 16:17:04 -0700 | [diff] [blame] | 346 | |
| 347 | /* We have a message, create a new message structure, add it to the |
| 348 | * list, and wake up our thread that will process the messages. |
| 349 | */ |
Alex Elder | 51c75fd | 2014-09-26 20:55:35 -0500 | [diff] [blame] | 350 | greybus_svc_in(es1->hd, urb->transfer_buffer, urb->actual_length); |
Greg Kroah-Hartman | de536e3 | 2014-08-31 16:17:04 -0700 | [diff] [blame] | 351 | |
| 352 | exit: |
| 353 | /* resubmit the urb to get more messages */ |
| 354 | retval = usb_submit_urb(urb, GFP_ATOMIC); |
| 355 | if (retval) |
| 356 | dev_err(dev, "Can not submit urb for AP data: %d\n", retval); |
| 357 | } |
| 358 | |
Greg Kroah-Hartman | 1129973 | 2014-09-12 21:17:37 -0700 | [diff] [blame] | 359 | static void cport_in_callback(struct urb *urb) |
Greg Kroah-Hartman | de536e3 | 2014-08-31 16:17:04 -0700 | [diff] [blame] | 360 | { |
| 361 | struct device *dev = &urb->dev->dev; |
Greg Kroah-Hartman | 9c8d3af | 2014-09-13 11:09:35 -0700 | [diff] [blame] | 362 | struct es1_ap_dev *es1 = urb->context; |
Alex Elder | 877b1ee | 2014-09-24 05:16:13 -0500 | [diff] [blame] | 363 | int status = check_urb_status(urb); |
Greg Kroah-Hartman | 9c8d3af | 2014-09-13 11:09:35 -0700 | [diff] [blame] | 364 | int retval; |
| 365 | u8 cport; |
| 366 | u8 *data; |
Greg Kroah-Hartman | de536e3 | 2014-08-31 16:17:04 -0700 | [diff] [blame] | 367 | |
Alex Elder | bedfdf3 | 2014-10-17 05:18:22 -0500 | [diff] [blame] | 368 | if (status) { |
Alex Elder | c4a432d | 2014-10-24 05:02:02 -0500 | [diff] [blame] | 369 | if (status == -EAGAIN) |
| 370 | goto exit; |
| 371 | dev_err(dev, "urb cport in error %d (dropped)\n", status); |
| 372 | return; |
Alex Elder | bedfdf3 | 2014-10-17 05:18:22 -0500 | [diff] [blame] | 373 | } |
Greg Kroah-Hartman | de536e3 | 2014-08-31 16:17:04 -0700 | [diff] [blame] | 374 | |
Alex Elder | 170229d | 2014-10-16 06:35:27 -0500 | [diff] [blame] | 375 | /* The size has to be at least one, for the cport id */ |
| 376 | if (!urb->actual_length) { |
| 377 | dev_err(dev, "%s: no cport id in input buffer?\n", __func__); |
Greg Kroah-Hartman | 9c8d3af | 2014-09-13 11:09:35 -0700 | [diff] [blame] | 378 | goto exit; |
| 379 | } |
| 380 | |
| 381 | /* |
| 382 | * The CPort number is the first byte of the data stream, the rest of |
| 383 | * the stream is "real" data |
| 384 | */ |
| 385 | data = urb->transfer_buffer; |
| 386 | cport = data[0]; |
| 387 | data = &data[1]; |
| 388 | |
| 389 | /* Pass this data to the greybus core */ |
Alex Elder | 0db32a6 | 2014-09-24 05:16:14 -0500 | [diff] [blame] | 390 | greybus_cport_in(es1->hd, cport, data, urb->actual_length - 1); |
Greg Kroah-Hartman | 9c8d3af | 2014-09-13 11:09:35 -0700 | [diff] [blame] | 391 | |
Greg Kroah-Hartman | de536e3 | 2014-08-31 16:17:04 -0700 | [diff] [blame] | 392 | exit: |
Greg Kroah-Hartman | 9c8d3af | 2014-09-13 11:09:35 -0700 | [diff] [blame] | 393 | /* put our urb back in the request pool */ |
| 394 | retval = usb_submit_urb(urb, GFP_ATOMIC); |
| 395 | if (retval) |
| 396 | dev_err(dev, "%s: error %d in submitting urb.\n", |
| 397 | __func__, retval); |
Greg Kroah-Hartman | de536e3 | 2014-08-31 16:17:04 -0700 | [diff] [blame] | 398 | } |
| 399 | |
Greg Kroah-Hartman | 1129973 | 2014-09-12 21:17:37 -0700 | [diff] [blame] | 400 | static void cport_out_callback(struct urb *urb) |
Greg Kroah-Hartman | 47f6ef1 | 2014-09-08 20:09:08 -0700 | [diff] [blame] | 401 | { |
Greg Kroah-Hartman | 9c8d3af | 2014-09-13 11:09:35 -0700 | [diff] [blame] | 402 | struct gbuf *gbuf = urb->context; |
Alex Elder | ba99346 | 2014-11-17 08:08:43 -0600 | [diff] [blame] | 403 | struct es1_ap_dev *es1 = hd_to_es1(gbuf->hd); |
Greg Kroah-Hartman | 9c8d3af | 2014-09-13 11:09:35 -0700 | [diff] [blame] | 404 | unsigned long flags; |
Greg Kroah-Hartman | 9c8d3af | 2014-09-13 11:09:35 -0700 | [diff] [blame] | 405 | int i; |
Greg Kroah-Hartman | ba4468d4 | 2014-08-30 17:06:54 -0700 | [diff] [blame] | 406 | |
Alex Elder | bedfdf3 | 2014-10-17 05:18:22 -0500 | [diff] [blame] | 407 | /* Record whether the transfer was successful */ |
| 408 | gbuf->status = check_urb_status(urb); |
Greg Kroah-Hartman | d814488 | 2014-10-27 13:31:01 +0800 | [diff] [blame] | 409 | gbuf->hcd_data = NULL; |
Alex Elder | bedfdf3 | 2014-10-17 05:18:22 -0500 | [diff] [blame] | 410 | |
Greg Kroah-Hartman | 9c8d3af | 2014-09-13 11:09:35 -0700 | [diff] [blame] | 411 | /* |
Greg Kroah-Hartman | 7f9e05e | 2014-09-13 17:28:33 -0700 | [diff] [blame] | 412 | * See if this was an urb in our pool, if so mark it "free", otherwise |
| 413 | * we need to free it ourselves. |
Greg Kroah-Hartman | 9c8d3af | 2014-09-13 11:09:35 -0700 | [diff] [blame] | 414 | */ |
| 415 | spin_lock_irqsave(&es1->cport_out_urb_lock, flags); |
| 416 | for (i = 0; i < NUM_CPORT_OUT_URB; ++i) { |
| 417 | if (urb == es1->cport_out_urb[i]) { |
| 418 | es1->cport_out_urb_busy[i] = false; |
| 419 | urb = NULL; |
| 420 | break; |
| 421 | } |
| 422 | } |
| 423 | spin_unlock_irqrestore(&es1->cport_out_urb_lock, flags); |
Greg Kroah-Hartman | 9c8d3af | 2014-09-13 11:09:35 -0700 | [diff] [blame] | 424 | |
Greg Kroah-Hartman | 7f9e05e | 2014-09-13 17:28:33 -0700 | [diff] [blame] | 425 | /* If urb is not NULL, then we need to free this urb */ |
| 426 | usb_free_urb(urb); |
Greg Kroah-Hartman | f9ab34c | 2014-10-28 18:20:24 +0800 | [diff] [blame] | 427 | |
| 428 | /* |
| 429 | * Yes, you are right, we aren't telling anyone that the urb finished. |
| 430 | * "That's crazy! How does this all even work?" you might be saying. |
| 431 | * The "magic" is the idea that greybus works on the "operation" level, |
| 432 | * not the "send a buffer" level. All operations are "round-trip" with |
| 433 | * a response from the device that the operation finished, or it will |
| 434 | * time out. Because of that, we don't care that this urb finished, or |
| 435 | * failed, or did anything else, as higher levels of the protocol stack |
| 436 | * will handle completions and timeouts and the rest. |
| 437 | * |
| 438 | * This protocol is "needed" due to some hardware restrictions on the |
| 439 | * current generation of Unipro controllers. Think about it for a |
| 440 | * minute, this is a USB driver, talking to a Unipro bridge, impediance |
| 441 | * mismatch is huge, yet the Unipro controller are even more |
| 442 | * underpowered than this little USB controller. We rely on the round |
| 443 | * trip to keep stalls in the Unipro controllers from happening so that |
| 444 | * we can keep data flowing properly, no matter how slow it might be. |
| 445 | * |
| 446 | * Once again, a wonderful bus protocol cut down in its prime by a naive |
| 447 | * controller chip. We dream of the day we have a "real" HCD for |
| 448 | * Unipro. Until then, we suck it up and make the hardware work, as |
| 449 | * that's the job of the firmware and kernel. |
| 450 | * </rant> |
| 451 | */ |
Greg Kroah-Hartman | 47f6ef1 | 2014-09-08 20:09:08 -0700 | [diff] [blame] | 452 | } |
| 453 | |
| 454 | /* |
| 455 | * The ES1 USB Bridge device contains 4 endpoints |
| 456 | * 1 Control - usual USB stuff + AP -> SVC messages |
| 457 | * 1 Interrupt IN - SVC -> AP messages |
| 458 | * 1 Bulk IN - CPort data in |
Alex Elder | 69f93ab | 2014-09-22 18:53:02 -0500 | [diff] [blame] | 459 | * 1 Bulk OUT - CPort data out |
Greg Kroah-Hartman | 47f6ef1 | 2014-09-08 20:09:08 -0700 | [diff] [blame] | 460 | */ |
Greg Kroah-Hartman | ba4468d4 | 2014-08-30 17:06:54 -0700 | [diff] [blame] | 461 | static int ap_probe(struct usb_interface *interface, |
| 462 | const struct usb_device_id *id) |
| 463 | { |
Greg Kroah-Hartman | a39879f | 2014-09-06 16:57:36 -0700 | [diff] [blame] | 464 | struct es1_ap_dev *es1; |
| 465 | struct greybus_host_device *hd; |
| 466 | struct usb_device *udev; |
Greg Kroah-Hartman | 6f83ab7 | 2014-08-30 17:30:04 -0700 | [diff] [blame] | 467 | struct usb_host_interface *iface_desc; |
| 468 | struct usb_endpoint_descriptor *endpoint; |
Greg Kroah-Hartman | 47f6ef1 | 2014-09-08 20:09:08 -0700 | [diff] [blame] | 469 | bool int_in_found = false; |
| 470 | bool bulk_in_found = false; |
| 471 | bool bulk_out_found = false; |
| 472 | int retval = -ENOMEM; |
Greg Kroah-Hartman | 6f83ab7 | 2014-08-30 17:30:04 -0700 | [diff] [blame] | 473 | int i; |
Greg Kroah-Hartman | 47f6ef1 | 2014-09-08 20:09:08 -0700 | [diff] [blame] | 474 | u8 svc_interval = 0; |
Greg Kroah-Hartman | 6f83ab7 | 2014-08-30 17:30:04 -0700 | [diff] [blame] | 475 | |
Greg Kroah-Hartman | a39879f | 2014-09-06 16:57:36 -0700 | [diff] [blame] | 476 | udev = usb_get_dev(interface_to_usbdev(interface)); |
| 477 | |
| 478 | hd = greybus_create_hd(&es1_driver, &udev->dev); |
Alex Elder | 599dc6a | 2014-10-20 10:27:55 -0500 | [diff] [blame] | 479 | if (!hd) { |
| 480 | usb_put_dev(udev); |
Greg Kroah-Hartman | f1eec30 | 2014-08-30 17:18:14 -0700 | [diff] [blame] | 481 | return -ENOMEM; |
Alex Elder | 599dc6a | 2014-10-20 10:27:55 -0500 | [diff] [blame] | 482 | } |
Greg Kroah-Hartman | ba4468d4 | 2014-08-30 17:06:54 -0700 | [diff] [blame] | 483 | |
Greg Kroah-Hartman | a39879f | 2014-09-06 16:57:36 -0700 | [diff] [blame] | 484 | es1 = hd_to_es1(hd); |
| 485 | es1->hd = hd; |
Greg Kroah-Hartman | 47f6ef1 | 2014-09-08 20:09:08 -0700 | [diff] [blame] | 486 | es1->usb_intf = interface; |
| 487 | es1->usb_dev = udev; |
Greg Kroah-Hartman | 0dad95d | 2014-09-13 09:54:35 -0700 | [diff] [blame] | 488 | spin_lock_init(&es1->cport_out_urb_lock); |
Greg Kroah-Hartman | 47f6ef1 | 2014-09-08 20:09:08 -0700 | [diff] [blame] | 489 | usb_set_intfdata(interface, es1); |
Greg Kroah-Hartman | a39879f | 2014-09-06 16:57:36 -0700 | [diff] [blame] | 490 | |
| 491 | /* Control endpoint is the pipe to talk to this AP, so save it off */ |
| 492 | endpoint = &udev->ep0.desc; |
Greg Kroah-Hartman | 47f6ef1 | 2014-09-08 20:09:08 -0700 | [diff] [blame] | 493 | es1->control_endpoint = endpoint->bEndpointAddress; |
Greg Kroah-Hartman | a39879f | 2014-09-06 16:57:36 -0700 | [diff] [blame] | 494 | |
Greg Kroah-Hartman | 47f6ef1 | 2014-09-08 20:09:08 -0700 | [diff] [blame] | 495 | /* find all 3 of our endpoints */ |
Greg Kroah-Hartman | 6f83ab7 | 2014-08-30 17:30:04 -0700 | [diff] [blame] | 496 | iface_desc = interface->cur_altsetting; |
| 497 | for (i = 0; i < iface_desc->desc.bNumEndpoints; ++i) { |
| 498 | endpoint = &iface_desc->endpoint[i].desc; |
| 499 | |
Greg Kroah-Hartman | 47f6ef1 | 2014-09-08 20:09:08 -0700 | [diff] [blame] | 500 | if (usb_endpoint_is_int_in(endpoint)) { |
| 501 | es1->svc_endpoint = endpoint->bEndpointAddress; |
Greg Kroah-Hartman | 47f6ef1 | 2014-09-08 20:09:08 -0700 | [diff] [blame] | 502 | svc_interval = endpoint->bInterval; |
| 503 | int_in_found = true; |
| 504 | } else if (usb_endpoint_is_bulk_in(endpoint)) { |
| 505 | es1->cport_in_endpoint = endpoint->bEndpointAddress; |
| 506 | bulk_in_found = true; |
| 507 | } else if (usb_endpoint_is_bulk_out(endpoint)) { |
| 508 | es1->cport_out_endpoint = endpoint->bEndpointAddress; |
| 509 | bulk_out_found = true; |
| 510 | } else { |
| 511 | dev_err(&udev->dev, |
| 512 | "Unknown endpoint type found, address %x\n", |
| 513 | endpoint->bEndpointAddress); |
Greg Kroah-Hartman | 6f83ab7 | 2014-08-30 17:30:04 -0700 | [diff] [blame] | 514 | } |
Greg Kroah-Hartman | 47f6ef1 | 2014-09-08 20:09:08 -0700 | [diff] [blame] | 515 | } |
| 516 | if ((int_in_found == false) || |
| 517 | (bulk_in_found == false) || |
| 518 | (bulk_out_found == false)) { |
| 519 | dev_err(&udev->dev, "Not enough endpoints found in device, aborting!\n"); |
| 520 | goto error; |
Greg Kroah-Hartman | 6f83ab7 | 2014-08-30 17:30:04 -0700 | [diff] [blame] | 521 | } |
| 522 | |
Greg Kroah-Hartman | 47f6ef1 | 2014-09-08 20:09:08 -0700 | [diff] [blame] | 523 | /* Create our buffer and URB to get SVC messages, and start it up */ |
Greg Kroah-Hartman | 29f000f | 2014-09-19 18:37:44 -0700 | [diff] [blame] | 524 | es1->svc_buffer = kmalloc(ES1_SVC_MSG_SIZE, GFP_KERNEL); |
Greg Kroah-Hartman | 47f6ef1 | 2014-09-08 20:09:08 -0700 | [diff] [blame] | 525 | if (!es1->svc_buffer) |
| 526 | goto error; |
Greg Kroah-Hartman | 6f83ab7 | 2014-08-30 17:30:04 -0700 | [diff] [blame] | 527 | |
Greg Kroah-Hartman | 47f6ef1 | 2014-09-08 20:09:08 -0700 | [diff] [blame] | 528 | es1->svc_urb = usb_alloc_urb(0, GFP_KERNEL); |
| 529 | if (!es1->svc_urb) |
Alex Elder | 1ec1d6d | 2014-10-20 10:28:00 -0500 | [diff] [blame] | 530 | goto error; |
Greg Kroah-Hartman | 47f6ef1 | 2014-09-08 20:09:08 -0700 | [diff] [blame] | 531 | |
| 532 | usb_fill_int_urb(es1->svc_urb, udev, |
| 533 | usb_rcvintpipe(udev, es1->svc_endpoint), |
Alex Elder | 877b1ee | 2014-09-24 05:16:13 -0500 | [diff] [blame] | 534 | es1->svc_buffer, ES1_SVC_MSG_SIZE, svc_in_callback, |
Greg Kroah-Hartman | 47f6ef1 | 2014-09-08 20:09:08 -0700 | [diff] [blame] | 535 | es1, svc_interval); |
| 536 | retval = usb_submit_urb(es1->svc_urb, GFP_KERNEL); |
| 537 | if (retval) |
Alex Elder | 1ec1d6d | 2014-10-20 10:28:00 -0500 | [diff] [blame] | 538 | goto error; |
Greg Kroah-Hartman | 47f6ef1 | 2014-09-08 20:09:08 -0700 | [diff] [blame] | 539 | |
Greg Kroah-Hartman | 1129973 | 2014-09-12 21:17:37 -0700 | [diff] [blame] | 540 | /* Allocate buffers for our cport in messages and start them up */ |
| 541 | for (i = 0; i < NUM_CPORT_IN_URB; ++i) { |
| 542 | struct urb *urb; |
| 543 | u8 *buffer; |
| 544 | |
| 545 | urb = usb_alloc_urb(0, GFP_KERNEL); |
| 546 | if (!urb) |
Alex Elder | 1ec1d6d | 2014-10-20 10:28:00 -0500 | [diff] [blame] | 547 | goto error; |
Greg Kroah-Hartman | 29f000f | 2014-09-19 18:37:44 -0700 | [diff] [blame] | 548 | buffer = kmalloc(ES1_GBUF_MSG_SIZE, GFP_KERNEL); |
Greg Kroah-Hartman | 1129973 | 2014-09-12 21:17:37 -0700 | [diff] [blame] | 549 | if (!buffer) |
Alex Elder | 1ec1d6d | 2014-10-20 10:28:00 -0500 | [diff] [blame] | 550 | goto error; |
Greg Kroah-Hartman | 1129973 | 2014-09-12 21:17:37 -0700 | [diff] [blame] | 551 | |
| 552 | usb_fill_bulk_urb(urb, udev, |
| 553 | usb_rcvbulkpipe(udev, es1->cport_in_endpoint), |
Alex Elder | 3e9cb4a | 2014-11-12 15:17:50 -0600 | [diff] [blame] | 554 | buffer, ES1_GBUF_MSG_SIZE, cport_in_callback, |
| 555 | es1); |
Greg Kroah-Hartman | 1129973 | 2014-09-12 21:17:37 -0700 | [diff] [blame] | 556 | es1->cport_in_urb[i] = urb; |
| 557 | es1->cport_in_buffer[i] = buffer; |
| 558 | retval = usb_submit_urb(urb, GFP_KERNEL); |
| 559 | if (retval) |
Alex Elder | 1ec1d6d | 2014-10-20 10:28:00 -0500 | [diff] [blame] | 560 | goto error; |
Greg Kroah-Hartman | 1129973 | 2014-09-12 21:17:37 -0700 | [diff] [blame] | 561 | } |
| 562 | |
| 563 | /* Allocate urbs for our CPort OUT messages */ |
| 564 | for (i = 0; i < NUM_CPORT_OUT_URB; ++i) { |
| 565 | struct urb *urb; |
| 566 | |
| 567 | urb = usb_alloc_urb(0, GFP_KERNEL); |
| 568 | if (!urb) |
Alex Elder | 1ec1d6d | 2014-10-20 10:28:00 -0500 | [diff] [blame] | 569 | goto error; |
Greg Kroah-Hartman | 1129973 | 2014-09-12 21:17:37 -0700 | [diff] [blame] | 570 | |
| 571 | es1->cport_out_urb[i] = urb; |
| 572 | es1->cport_out_urb_busy[i] = false; /* just to be anal */ |
| 573 | } |
| 574 | |
Greg Kroah-Hartman | ba4468d4 | 2014-08-30 17:06:54 -0700 | [diff] [blame] | 575 | return 0; |
Greg Kroah-Hartman | 47f6ef1 | 2014-09-08 20:09:08 -0700 | [diff] [blame] | 576 | error: |
Alex Elder | 1ec1d6d | 2014-10-20 10:28:00 -0500 | [diff] [blame] | 577 | ap_disconnect(interface); |
| 578 | |
Greg Kroah-Hartman | 47f6ef1 | 2014-09-08 20:09:08 -0700 | [diff] [blame] | 579 | return retval; |
Greg Kroah-Hartman | ba4468d4 | 2014-08-30 17:06:54 -0700 | [diff] [blame] | 580 | } |
| 581 | |
Greg Kroah-Hartman | ba4468d4 | 2014-08-30 17:06:54 -0700 | [diff] [blame] | 582 | static struct usb_driver es1_ap_driver = { |
| 583 | .name = "es1_ap_driver", |
| 584 | .probe = ap_probe, |
| 585 | .disconnect = ap_disconnect, |
| 586 | .id_table = id_table, |
| 587 | }; |
| 588 | |
| 589 | module_usb_driver(es1_ap_driver); |
| 590 | |
| 591 | MODULE_LICENSE("GPL"); |
| 592 | MODULE_AUTHOR("Greg Kroah-Hartman <gregkh@linuxfoundation.org>"); |