lkml.org 
[lkml]   [2019]   [Jan]   [29]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRE: [PATCH V5 3/5] i2c: tegra: Add DMA Support
Date
> > Could you please tell whether you missed my comments to V3 [0] or chose to ignore them? If the former, then I'd want to get answers to those questions and comments. I'll stop here for now.
> >
> > [0] https://patchwork.ozlabs.org/patch/1031379/
>
> Somehow missed those from multiple comments. Will go thru and respond back.

V6 includes feedback changes. Want to clarify on few feedback points

- ALIGN is used for 4 byte boundary to use with DMA but extra bytes doesn’t get transferred over I2C as I2C controller transfer bytes based on size specified in the packet header.
DMA length and memory address need to be 4 byte boundary.

- RX channel releasing when TX init fails?
For I2C both TX and RX doesn’t happen in same transaction and no dependency. So if channel request & buffer allocation fails other channel if successfully created can still be used for transfer.

- dma_burst < 8 negatively affects transfer efficiency? Performance stats for DMA Vs PIO mode.
Tested with 256 bytes of transfer and DMA Vs PIO mode transfer rate is almost same.
But the main reason for adding DMA mode is to address couple of cases mentioned earlier and not mainly from the transfer performance perspective.


\
 
 \ /
  Last update: 2019-01-30 05:17    [W:0.057 / U:0.324 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site