[U-Boot] [RESEND PATCH] usb: gadget: f_thor: Allocate request up to THOR_PACKET_SIZE
Lukasz Majewski
l.majewski at samsung.com
Wed Aug 12 13:09:00 CEST 2015
Hi Michal,
> Hi Lukasz,
>
> On 04/24/2015 05:46 PM, Lukasz Majewski wrote:
> > Hi Michal,
> >
> >> From: Siva Durga Prasad Paladugu <siva.durga.paladugu at xilinx.com>
> >>
> >> 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 <sivadur at xilinx.com>
> >> Signed-off-by: Michal Simek <michal.simek at xilinx.com>
> >> ---
> >>
> >> 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?
Please look into u-boot-dfu/testing branch.
I've placed them there for testing. Afterwards I will put them to
master branch
>
> Thanks,
> Michal
>
--
Best regards,
Lukasz Majewski
Samsung R&D Institute Poland (SRPOL) | Linux Platform Group
More information about the U-Boot
mailing list