davids5 commented on code in PR #11020:
URL: https://github.com/apache/nuttx/pull/11020#discussion_r1372179887


##########
arch/arm/src/imxrt/imxrt_serial.c:
##########
@@ -1373,6 +1373,7 @@ static inline void imxrt_serialout(struct imxrt_uart_s 
*priv,
 static int imxrt_dma_nextrx(struct imxrt_uart_s *priv)
 {
   int dmaresidual = imxrt_dmach_getcount(priv->rxdma);
+  DEBUGASSERT(dmaresidual <= RXDMA_BUFFER_SIZE);

Review Comment:
   Updated!
   
   > agree that this scenario can and does happen
   
   I agree - HW wise you can try this? - it is a simple statc test that should 
answer the "Do we see different" things on our HW. 
   
   Use  putty on the remotes side with an FTDI to the port.
   Place a break in imxrt_serial.c [line: 1925] the call to  ` int dmaresidual 
= imxrt_dmach_getcount(priv->rxdma);`
   reboot nuttx
   On the nuttx side `nsh> cat /dev/ttySn`  where n is your port.
   hit a 'U' on the putty side.
   the break point will not be hit until you hit a char on the putty side
   step into . 
   I get 
   <img width="1546" alt="image" 
src="https://github.com/apache/nuttx/assets/1945821/6568060d-c3a3-42ed-aceb-85bdef9b5310";>
   
    



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to