Hello.

Stefano Babic wrote:

>> On 08/12/2010 01:41 AM, Stefano Babic wrote:
>>>  #if defined(DEV_CONFIG_CDC)
>>>     if (dev->status_ep) {
>>> -           dev->stat_req = usb_ep_alloc_request(gadget->ep0, GFP_KERNEL);
>>> -           dev->stat_req->buf = status_req;
>>> +           dev->stat_req = usb_ep_alloc_request(dev->status_ep, 
>>> GFP_KERNEL);
>>>             if (!dev->stat_req) {
>>>                     dev->stat_req->buf=NULL;
>> We get oops here!

> Agree, and the issue is not related to this patch, I missed to correct
> it, thanks. If no one complains, I will send a single patch to fix both
> problems (wrong ep status + null pointer access).

    Looks like Vitaly has done the same already:

http://lists.denx.de/pipermail/u-boot/2010-August/075419.html

> Best regards,
> Stefano

WBR, Sergei
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to