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. _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
