davids5 commented on code in PR #11020:
URL: https://github.com/apache/nuttx/pull/11020#discussion_r1374408499
##########
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:
@danielappiagyei-bc - I finally was able to force the condition. PR is
https://github.com/apache/nuttx/pull/11070
There is a race, which I would have expected to see more on the faster
CPU...but I forced it....While sending data to the UART. I added a loop in
imxrt_dma_nextrx to re-call `imxrt_dmach_getcount()` N times and check for 0. I
then could break on it and see the DMA as done, until I stepped and it
reloaded. Thank you for reporting it and you patience!
--
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]
