From mboxrd@z Thu Jan 1 00:00:00 1970 From: =?UTF-8?B?SsO2cmcgS3JhdXNl?= Date: Tue, 01 Jul 2014 23:25:42 +0200 Subject: [U-Boot] [PATCH 0/6] usb: ci_udc: fixes and cleanups In-Reply-To: <1404236478-22666-1-git-send-email-swarren@wwwdotorg.org> References: <1404236478-22666-1-git-send-email-swarren@wwwdotorg.org> Message-ID: <53B32756.6020407@posteo.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On 07/01/2014 07:41 PM, Stephen Warren wrote: > From: Stephen Warren > > This is a series of small fixes and cleanups either required by those > fixes, or enabled now that the fixes are made. > > I hope that either patch 1 or 4 might fix the issues J?rg is seeing, but > I'm not sure that will happen. The other patches shouldn't change any > behaviour. > > Stephen Warren (6): > usb: ci_udc: fix ci_flush_{qh,qtd} calls in ci_udc_probe() > usb: ci_udc: don't assume QTDs are adjacent when transmitting ZLPs > usb: ci_udc: lift ilist size calculations to global scope > usb: ci_udc: fix items array size/stride calculation > usb: ci_udc: remove controller.items array > usb: ci_udc: don't memalign() struct ci_req allocations > > drivers/usb/gadget/ci_udc.c | 62 ++++++++++++++++++++++----------------------- > drivers/usb/gadget/ci_udc.h | 1 - > 2 files changed, 30 insertions(+), 33 deletions(-) > Good news! The last patch usb: ci_udc: don't memalign() struct ci_req allocations removes the timeout error after starting the fourth run of tftp in a row. This is how I tested. Checked out u-boot-usb/master branch. Applied the necessary patches to support our board. Applied the patches step after step. After applying a patch reset the board and run tftp from console until an error occured, which is always the fourth run. This is the case until applying patch usb: ci_udc: don't memalign() struct ci_req allocations, which throws no timeout error within running tftp about 60 times in a row.