Hi Lukasz, On 04/24/2015 05:46 PM, Lukasz Majewski wrote: > Hi Michal, > >> From: Siva Durga Prasad Paladugu <[email protected]> >> >> Allocate request up to THOR_PACKET_SIZE not the ep0->maxpacket >> as the descriptors data depend on the number of descriptors >> and this 64 bytes were not enough and the buffer might overflow >> which results in memalign failures later. >> >> Signed-off-by: Siva Durga Prasad Paladugu <[email protected]> >> Signed-off-by: Michal Simek <[email protected]> >> --- >> >> drivers/usb/gadget/f_thor.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/drivers/usb/gadget/f_thor.c b/drivers/usb/gadget/f_thor.c >> index 31eec36be876..5c8f6768519c 100644 >> --- a/drivers/usb/gadget/f_thor.c >> +++ b/drivers/usb/gadget/f_thor.c >> @@ -775,7 +775,7 @@ static int thor_func_bind(struct >> usb_configuration *c, struct usb_function *f) goto fail; >> } >> dev->req->buf = memalign(CONFIG_SYS_CACHELINE_SIZE, >> - gadget->ep0->maxpacket); >> + THOR_PACKET_SIZE); >> if (!dev->req->buf) { >> status = -ENOMEM; >> goto fail; > > Applied to u-boot-dfu. Thanks! >
I can't see this patch in your branch. Did you add it? Thanks, Michal -- Michal Simek, Ing. (M.Eng), OpenPGP -> KeyID: FE3D1F91 w: www.monstr.eu p: +42-0-721842854 Maintainer of Linux kernel - Microblaze cpu - http://www.monstr.eu/fdt/ Maintainer of Linux kernel - Xilinx Zynq ARM architecture Microblaze U-BOOT custodian and responsible for u-boot arm zynq platform
signature.asc
Description: OpenPGP digital signature
_______________________________________________ U-Boot mailing list [email protected] http://lists.denx.de/mailman/listinfo/u-boot
