[U-Boot] NetRxPackets

[Vitaly Kuzmichev <vkuzmichev@mvista.com>]

Hi Marcel,

Marcel wrote:
>> Not sure that branch is anything up to date....
>
> hmmm.  What branch is best to work on for this code ?

If someone sends USB-CDC patches they usually are applied into master
branch of u-boot-usb.git repository:
http://git.denx.de/cgi-bin/gitweb.cgi?p=u-boot/u-boot-usb.git;a=summary

AFAIK, master branch of u-boot main repository is also working well, but
I haven't checked whether all recent patches are pulled into main
repository.
If you are using another version of u-boot (one of the previous
releases) the driver in your version may be broken.

According to the log from "Usage of DMA" thread I think your gadget
stack is up to date.

>>>>> req->buf = (u8 *) NetRxPackets[0];
>>
>> *maybe* "req" itself is not allocated?
> 
> Good question. My code that rejects the buffer is this :
> 
> [...]
>         if (!_req || !_req->complete
>                         || !_req->buf || !list_empty(&req->queue)) {
>           
>                 DBG("invalid request\n");
>                 if(!_req) debug("NO REQUEST\n");
>                 if(!_req->complete) debug("NO REQUEST COMPLETE\n");
>                 if(!_req->buf) debug("NO REQUEST BUF\n");
>                 if(!list_empty(&req->queue)) debug("NO LIST EMPTY\n");
>                 return -EINVAL;
>         }
> 
> It just keeps failing with "NO REQUEST BUF". I feel like I'm staring at a typo 
> for days and just don't see it.

I suppose that usage of kzalloc in usba_ep_alloc_request (if you haven't
changed it) may be not well. at91_udc driver (for example) uses static
pool for requests allocation.

Anyway it's strange. I have a suspicion that buf was set to NULL
somewhere inside atmel_usba_udc.


____
With best regards,
Vitaly.