Re: [U-Boot] [PATCH 1/2] fastboot: Fix wMaxPacketSize for High-Speed IN endpoint

2016-04-13 Thread Hong Chen
..@konsulko.com>; Steve Rae <s...@broadcom.com>; U-Boot Mailing List <u-boot@lists.denx.de> Subject: Re: [U-Boot] [PATCH 1/2] fastboot: Fix wMaxPacketSize for High-Speed IN endpoint oops -- my last reply was interpreted as a new patch: http://patchwork.ozlabs.org/patch/609843/ so I'l

Re: [U-Boot] [PATCH 1/2] fastboot: Fix wMaxPacketSize for High-Speed IN endpoint

2016-04-13 Thread Roger Quadros
On 13/04/16 04:42, Steve Rae wrote: > Hi Roger, > > On Tue, Apr 12, 2016 at 6:54 AM, Lukasz Majewski > wrote: >> Hi Roger, >> >>> wMaxPacketSize for IN endpoing in High-Speed must be 512 and not 64. >>> While fixing that we do some clean ups like >>> >>> - use

Re: [U-Boot] [PATCH 1/2] fastboot: Fix wMaxPacketSize for High-Speed IN endpoint

2016-04-12 Thread Steve Rae
oops -- my last reply was interpreted as a new patch: http://patchwork.ozlabs.org/patch/609843/ so I'll try again: Tested-by: Steve Rae [Test HW: bcm235xx board] On Tue, Apr 12, 2016 at 5:51 AM, Roger Quadros wrote: > wMaxPacketSize for IN endpoing in

Re: [U-Boot] [PATCH 1/2] fastboot: Fix wMaxPacketSize for High-Speed IN endpoint

2016-04-12 Thread Steve Rae
Hi Roger, On Tue, Apr 12, 2016 at 6:54 AM, Lukasz Majewski wrote: > Hi Roger, > >> wMaxPacketSize for IN endpoing in High-Speed must be 512 and not 64. >> While fixing that we do some clean ups like >> >> - use cpu_to_le16(decimal_length) instead of hexadecimal length. >>

Re: [U-Boot] [PATCH 1/2] fastboot: Fix wMaxPacketSize for High-Speed IN endpoint

2016-04-12 Thread Lukasz Majewski
Hi Roger, > wMaxPacketSize for IN endpoing in High-Speed must be 512 and not 64. > While fixing that we do some clean ups like > > - use cpu_to_le16(decimal_length) instead of hexadecimal length. > - No need to initialize bInterval to 0. Static variables are 0 > initialized. > - Move descriptor