[U-Boot] [PATCH] USB-CDC: wrong ep status used

Stefano Babic sbabic at denx.de
Thu Aug 12 14:28:40 CEST 2010


Vitaly Kuzmichev wrote:
> Hi Stefano,
> 
> 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).

Best regards,
Stefano

-- 
=====================================================================
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80  Email: office at denx.de
=====================================================================


More information about the U-Boot mailing list