TimJTi opened a new pull request, #14128:
URL: https://github.com/apache/nuttx/pull/14128

   ## Summary
   
   When trying CDC/NCM I found that a debug assert was triggered. Initially I 
commented out the offending DEBUGASSERT and found the code worked just fine.
   
   I don't know why the assert was placed there - perhaps with other uses of 
this driver there would always be data in the receive queue - so I moved the 
assignment to the places that actually needed and used the variable. 
   
   I have checked that not only CDC/NCM (now) works but also that RNDIS still 
behaves as before, so it seems like a "safe" fix.
   
   ## Impact
   
   None expected, but maybe reviewers with USB and DMA knowledge can comment.
   
   ## Testing
   
   Custom board with SAMA5D27C-D1G, using either CDC/NCM or RNDIS connectivity.
   
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscr...@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to