pkarashchenko commented on issue #10912:
URL: https://github.com/apache/nuttx/issues/10912#issuecomment-1764416504

   > I have a [PR](https://github.com/apache/nuttx/pull/10911) up which fixes 
the overrun in `imxrt_dma_nextrx()`. The other half of the fix which I haven't 
included in the PR is to place the `g_lpuart{1-8}rxfifo` into TCM. It fixes the 
issue but what I don't understand is why our call to `up_invalidate_dcache()` 
[here](https://github.com/apache/nuttx/blob/master/arch/arm/src/imxrt/imxrt_serial.c#L2724)
 isn't sufficient.
   > 
   > @davids5 @xiaoxiang781216 Do either of u have any thoughts on what's going 
on?
   
   Maybe you can move
   ```
     up_invalidate_dcache((uintptr_t)priv->rxfifo,
                          (uintptr_t)priv->rxfifo + RXDMA_BUFFER_SIZE);
   ```
   from `imxrt_dma_rxcallback` to `imxrt_dma_setup` before 
`imxrt_dmach_xfrsetup(priv->rxdma , &config);` call? Not sure if that will 
work, but worth of trying


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