diff options
author | Tobias Diedrich <ranma+coreboot@tdiedrich.de> | 2010-06-21 06:22:36 +0000 |
---|---|---|
committer | Tobias Diedrich <ranma+coreboot@tdiedrich.de> | 2010-06-21 06:22:36 +0000 |
commit | 9297a02d4964cae7727248665e278b251cac5533 (patch) | |
tree | 92ece84ce0638521d43041799c57ff1e4415e89c /firmware/target/arm/as3525/usb-drv-as3525.c | |
parent | a398c2846aa86650aa386a68cb51929477b6aa23 (diff) | |
download | rockbox-9297a02d4964cae7727248665e278b251cac5533.tar.gz rockbox-9297a02d4964cae7727248665e278b251cac5533.zip |
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
Diffstat (limited to 'firmware/target/arm/as3525/usb-drv-as3525.c')
-rw-r--r-- | firmware/target/arm/as3525/usb-drv-as3525.c | 20 |
1 files 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 2f6fa20af8..d41acc7f81 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); |