From 4386c9138bc8386da176e81a05b3c3f24c0cbd23 Mon Sep 17 00:00:00 2001 From: ranma Date: Mon, 21 Jun 2010 06:22:36 +0000 Subject: [PATCH] mps depends on endpoint, I wonder why gcc didn't warn about i being used uninitialized... git-svn-id: svn://svn.rockbox.org/rockbox/trunk@27005 a1c6a512-1295-4272-9138-f99709370657 --- firmware/target/arm/as3525/usb-drv-as3525.c | 20 +++++++++++--------- 1 file changed, 11 insertions(+), 9 deletions(-) diff --git a/firmware/target/arm/as3525/usb-drv-as3525.c b/firmware/target/arm/as3525/usb-drv-as3525.c index 2f6fa20af..d41acc7f8 100644 --- a/firmware/target/arm/as3525/usb-drv-as3525.c +++ b/firmware/target/arm/as3525/usb-drv-as3525.c @@ -150,15 +150,6 @@ static void dma_desc_init(int ep, int dir) static void reset_endpoints(int init) { int i; - /* - * MPS sizes depending on speed: - * LS: 8 (control), no bulk available - * FS: 64 (control), 64 (bulk) - * HS: 64 (control), 512 (bulk) - * - * We don't need to handle LS since there is no low-speed only host AFAIK. - */ - int mps = i == 0 ? 64 : (usb_drv_port_speed() ? 512 : 64); /* * OUT EP 2 is an alias for OUT EP 0 on this HW! @@ -171,6 +162,17 @@ static void reset_endpoints(int init) endpoints[2][1].state |= EP_STATE_ALLOCATED; for(i = 0; i < USB_NUM_EPS; i++) { + /* + * MPS sizes depending on speed: + * LS: 8 (control), no bulk available + * FS: 64 (control), 64 (bulk) + * HS: 64 (control), 512 (bulk) + * + * We don't need to handle LS since there is no low-speed only + * host AFAIK. + */ + int mps = i == 0 ? 64 : (usb_drv_port_speed() ? 512 : 64); + if (init) { endpoints[i][0].state = 0; wakeup_init(&endpoints[i][0].complete); -- 2.11.4.GIT