michallenc commented on issue #16004:
URL: https://github.com/apache/nuttx/issues/16004#issuecomment-2751637869

   > > If you set bytes_to_read to something a bit bigger (like 5), you don't 
get anything out any more. Also the thing keeps crashing.
   > 
   > In order to reduce the copying of data, the USB reqbuf is directly used as 
the serial port buf. When CONFIG_CDCACM_NRDREQS is 4, at most 4 packets of USB 
data are stored without reading. Minicom uses 1 byte as 1 packet of data, so it 
can only receive 4 bytes. More bytes can be received by expanding 
CONFIG_CDCACM_NRDREQS.
   > 
   > If bytes_to_read is bigger than 5, it will cause a crash?
   
   I think this explains my issues I mentioned earlier. I am not quite sure I 
like this and I agree with @jlaitine on this. I don't know if revert is 
necessary, it seems like making it a configuration option should be sufficient, 
but the default behavior HAS to be as before this patch.


-- 
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