关于serial.c驱动DMA框架接收逻辑的问题

发布于 2018-05-23 23:48:25 浏览：3532 订阅该版

最近要用到串口DMA通讯，发现serial.c对DMA的接收实现让人匪夷所思。
读过代码的人肯定会认为这是在强行支持DMA，而且还没支持好。
下面我就提出几个问题：
1.DMA接收事，更具config.bufsz来区分不同的DMA接收数据方式，
从这里可以看出思路是当时设计时没考虑好如何从DMA buffer到serial.c的rx_fifo实现方式，所以为整个DMA实现方式埋下伏笔。
2.当config.bufsz==0时，rt_hw_serial_isr几乎不处理任何rx_fifo逻辑，所以可想而知，在rt_serial_read的时候数据只能从usart.c
底层驱动实现中去处理serial的rx_fifo的逻辑。很巧是的，确实是这样，不过又很不幸被_serial_dma_rx给半路打劫了请看下面的
代码段：```if (serial->config.bufsz == 0)
    {
        int result = RT_EOK;
        struct rt_serial_rx_dma *rx_dma;

rx_dma = (struct rt_serial_rx_dma*)serial->serial_rx;
        RT_ASSERT(rx_dma != RT_NULL);

if (rx_dma->activated != RT_TRUE)
        {
            rx_dma->activated = RT_TRUE;
            RT_ASSERT(serial->ops->dma_transmit != RT_NULL);
            serial->ops->dma_transmit(serial, data, length, RT_SERIAL_DMA_RX);
        }
        else result = -RT_EBUSY;
        rt_hw_interrupt_enable(level);

if (result == RT_EOK) return length;

rt_set_errno(result);
        return 0;
    }```
注意问题就出在serial->ops->dma_transmit(serial, data, length, RT_SERIAL_DMA_RX);这一行上。不管你底层怎么实现rx_fifo，反正我就要读取length这么多，不能少，你也不能多，不然我返回的length就是错误的。鬼知道此时底层的rx_fifo有多少？而且从rx_dma->activated的前后逻辑看像是一个块设备，不再是字符设备。
3.serial->config.bufsz != 0的实现方式，这个也很扯，从uart中断过来后，rx_fifo不做任何的数据copy操作，直接就跟新index了.
这里作者的意图肯定是认为rx_fifo又底层适配DMA驱动时自动有DMA硬件填充数据，这个设计意图就唯一限定了DAM硬件的实现方式了，因为这里的rx_fifo只有一个buffer，所以就不能实现双buffer。而且这种实现方式必须时接收完一个DMA事件后，就要应用层立马处理掉这一事件的数据，否则你告诉我怎么让DMA利用这个buffer来继续接收下一包数据？这里作者又假设了一种实现方式，那就是底层DMA必须时cycle模式的，也就是循环覆盖。可以想象，既然用DMA来做肯定数据量很大，传输速度很快，必然导致没有处理完一包数据就被新包数据给覆盖了。