Commit 0367eef2 authored by Thomas Pugliese's avatar Thomas Pugliese Committed by Greg Kroah-Hartman

usb: wusbcore: rename fields in struct wahc

Rename xfer_result to dti_buf and xfer_result_size to dti_buf_size in
struct wahc.  The dti buffer will also be used for isochronous status
packets once isochronous transfers are supported.
Signed-off-by: default avatarThomas Pugliese <thomas.pugliese@gmail.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 09d94cbd
...@@ -44,11 +44,11 @@ int wa_create(struct wahc *wa, struct usb_interface *iface) ...@@ -44,11 +44,11 @@ int wa_create(struct wahc *wa, struct usb_interface *iface)
/* Fill up Data Transfer EP pointers */ /* Fill up Data Transfer EP pointers */
wa->dti_epd = &iface->cur_altsetting->endpoint[1].desc; wa->dti_epd = &iface->cur_altsetting->endpoint[1].desc;
wa->dto_epd = &iface->cur_altsetting->endpoint[2].desc; wa->dto_epd = &iface->cur_altsetting->endpoint[2].desc;
wa->xfer_result_size = usb_endpoint_maxp(wa->dti_epd); wa->dti_buf_size = usb_endpoint_maxp(wa->dti_epd);
wa->xfer_result = kmalloc(wa->xfer_result_size, GFP_KERNEL); wa->dti_buf = kmalloc(wa->dti_buf_size, GFP_KERNEL);
if (wa->xfer_result == NULL) { if (wa->dti_buf == NULL) {
result = -ENOMEM; result = -ENOMEM;
goto error_xfer_result_alloc; goto error_dti_buf_alloc;
} }
result = wa_nep_create(wa, iface); result = wa_nep_create(wa, iface);
if (result < 0) { if (result < 0) {
...@@ -59,8 +59,8 @@ int wa_create(struct wahc *wa, struct usb_interface *iface) ...@@ -59,8 +59,8 @@ int wa_create(struct wahc *wa, struct usb_interface *iface)
return 0; return 0;
error_nep_create: error_nep_create:
kfree(wa->xfer_result); kfree(wa->dti_buf);
error_xfer_result_alloc: error_dti_buf_alloc:
wa_rpipes_destroy(wa); wa_rpipes_destroy(wa);
error_rpipes_create: error_rpipes_create:
return result; return result;
...@@ -76,7 +76,7 @@ void __wa_destroy(struct wahc *wa) ...@@ -76,7 +76,7 @@ void __wa_destroy(struct wahc *wa)
usb_kill_urb(wa->buf_in_urb); usb_kill_urb(wa->buf_in_urb);
usb_put_urb(wa->buf_in_urb); usb_put_urb(wa->buf_in_urb);
} }
kfree(wa->xfer_result); kfree(wa->dti_buf);
wa_nep_destroy(wa); wa_nep_destroy(wa);
wa_rpipes_destroy(wa); wa_rpipes_destroy(wa);
} }
......
...@@ -184,8 +184,8 @@ struct wahc { ...@@ -184,8 +184,8 @@ struct wahc {
struct urb *dti_urb; /* URB for reading xfer results */ struct urb *dti_urb; /* URB for reading xfer results */
struct urb *buf_in_urb; /* URB for reading data in */ struct urb *buf_in_urb; /* URB for reading data in */
struct edc dti_edc; /* DTI error density counter */ struct edc dti_edc; /* DTI error density counter */
struct wa_xfer_result *xfer_result; /* real size = dti_ep maxpktsize */ void *dti_buf;
size_t xfer_result_size; size_t dti_buf_size;
s32 status; /* For reading status */ s32 status; /* For reading status */
......
...@@ -1418,7 +1418,8 @@ static int wa_xfer_status_to_errno(u8 status) ...@@ -1418,7 +1418,8 @@ static int wa_xfer_status_to_errno(u8 status)
* *
* FIXME: this function needs to be broken up in parts * FIXME: this function needs to be broken up in parts
*/ */
static void wa_xfer_result_chew(struct wahc *wa, struct wa_xfer *xfer) static void wa_xfer_result_chew(struct wahc *wa, struct wa_xfer *xfer,
struct wa_xfer_result *xfer_result)
{ {
int result; int result;
struct device *dev = &wa->usb_iface->dev; struct device *dev = &wa->usb_iface->dev;
...@@ -1426,8 +1427,7 @@ static void wa_xfer_result_chew(struct wahc *wa, struct wa_xfer *xfer) ...@@ -1426,8 +1427,7 @@ static void wa_xfer_result_chew(struct wahc *wa, struct wa_xfer *xfer)
u8 seg_idx; u8 seg_idx;
struct wa_seg *seg; struct wa_seg *seg;
struct wa_rpipe *rpipe; struct wa_rpipe *rpipe;
struct wa_xfer_result *xfer_result = wa->xfer_result; unsigned done = 0;
u8 done = 0;
u8 usb_status; u8 usb_status;
unsigned rpipe_ready = 0; unsigned rpipe_ready = 0;
...@@ -1687,7 +1687,7 @@ static void wa_buf_in_cb(struct urb *urb) ...@@ -1687,7 +1687,7 @@ static void wa_buf_in_cb(struct urb *urb)
* We go back to OFF when we detect a ENOENT or ESHUTDOWN (or too many * We go back to OFF when we detect a ENOENT or ESHUTDOWN (or too many
* errors) in the URBs. * errors) in the URBs.
*/ */
static void wa_xfer_result_cb(struct urb *urb) static void wa_dti_cb(struct urb *urb)
{ {
int result; int result;
struct wahc *wa = urb->context; struct wahc *wa = urb->context;
...@@ -1709,7 +1709,7 @@ static void wa_xfer_result_cb(struct urb *urb) ...@@ -1709,7 +1709,7 @@ static void wa_xfer_result_cb(struct urb *urb)
urb->actual_length, sizeof(*xfer_result)); urb->actual_length, sizeof(*xfer_result));
break; break;
} }
xfer_result = wa->xfer_result; xfer_result = (struct wa_xfer_result *)(wa->dti_buf);
if (xfer_result->hdr.bLength != sizeof(*xfer_result)) { if (xfer_result->hdr.bLength != sizeof(*xfer_result)) {
dev_err(dev, "DTI Error: xfer result--" dev_err(dev, "DTI Error: xfer result--"
"bad header length %u\n", "bad header length %u\n",
...@@ -1735,7 +1735,7 @@ static void wa_xfer_result_cb(struct urb *urb) ...@@ -1735,7 +1735,7 @@ static void wa_xfer_result_cb(struct urb *urb)
xfer_id, usb_status); xfer_id, usb_status);
break; break;
} }
wa_xfer_result_chew(wa, xfer); wa_xfer_result_chew(wa, xfer, xfer_result);
wa_xfer_put(xfer); wa_xfer_put(xfer);
break; break;
case -ENOENT: /* (we killed the URB)...so, no broadcast */ case -ENOENT: /* (we killed the URB)...so, no broadcast */
...@@ -1777,7 +1777,7 @@ static void wa_xfer_result_cb(struct urb *urb) ...@@ -1777,7 +1777,7 @@ static void wa_xfer_result_cb(struct urb *urb)
* don't really set it up and start it until the first xfer complete * don't really set it up and start it until the first xfer complete
* notification arrives, which is what we do here. * notification arrives, which is what we do here.
* *
* Follow up in wa_xfer_result_cb(), as that's where the whole state * Follow up in wa_dti_cb(), as that's where the whole state
* machine starts. * machine starts.
* *
* So here we just initialize the DTI URB for reading transfer result * So here we just initialize the DTI URB for reading transfer result
...@@ -1813,8 +1813,8 @@ void wa_handle_notif_xfer(struct wahc *wa, struct wa_notif_hdr *notif_hdr) ...@@ -1813,8 +1813,8 @@ void wa_handle_notif_xfer(struct wahc *wa, struct wa_notif_hdr *notif_hdr)
usb_fill_bulk_urb( usb_fill_bulk_urb(
wa->dti_urb, wa->usb_dev, wa->dti_urb, wa->usb_dev,
usb_rcvbulkpipe(wa->usb_dev, 0x80 | notif_xfer->bEndpoint), usb_rcvbulkpipe(wa->usb_dev, 0x80 | notif_xfer->bEndpoint),
wa->xfer_result, wa->xfer_result_size, wa->dti_buf, wa->dti_buf_size,
wa_xfer_result_cb, wa); wa_dti_cb, wa);
wa->buf_in_urb = usb_alloc_urb(0, GFP_KERNEL); wa->buf_in_urb = usb_alloc_urb(0, GFP_KERNEL);
if (wa->buf_in_urb == NULL) { if (wa->buf_in_urb == NULL) {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment