From 7557d5679d608cc1611e22f30f068132a8d864a8 Mon Sep 17 00:00:00 2001 From: Frank Gevaerts Date: Sat, 23 May 2009 14:50:30 +0000 Subject: Clarify comment and clean up a bit (FS#10227 by Tomer Shalev) git-svn-id: svn://svn.rockbox.org/rockbox/trunk@21055 a1c6a512-1295-4272-9138-f99709370657 --- firmware/target/arm/usb-drv-arc.c | 13 ++++++------- 1 file changed, 6 insertions(+), 7 deletions(-) (limited to 'firmware/target') diff --git a/firmware/target/arm/usb-drv-arc.c b/firmware/target/arm/usb-drv-arc.c index 8a10c5fbd6..3dc7b21151 100644 --- a/firmware/target/arm/usb-drv-arc.c +++ b/firmware/target/arm/usb-drv-arc.c @@ -829,12 +829,11 @@ int usb_drv_request_endpoint(int type, int dir) return -1; } - log_ep(ep_num, ep_dir, "add"); endpoint->allocated[ep_dir] = 1; endpoint->type[ep_dir] = ep_type; - log_ep(ep_num, ep_dir, "got"); + log_ep(ep_num, ep_dir, "add"); return (ep_num | (dir & USB_ENDPOINT_DIR_MASK)); } @@ -998,13 +997,13 @@ static void init_endpoints(void) for(ep_num=1;ep_numallocated[DIR_OUT]) { + /* manual: 32.9.5.18 (Caution): Leaving an unconfigured endpoint control + * will cause undefined behavior for the data pid tracking on the active + * endpoint/direction. */ + if (!endpoint->allocated[DIR_OUT]) endpoint->type[DIR_OUT] = USB_ENDPOINT_XFER_BULK; - } - if (!endpoint->allocated[DIR_IN]) { + if (!endpoint->allocated[DIR_IN]) endpoint->type[DIR_IN] = USB_ENDPOINT_XFER_BULK; - } REG_ENDPTCTRL(ep_num) = EPCTRL_RX_DATA_TOGGLE_RST | EPCTRL_RX_ENABLE | -- cgit v1.2.3