Re: [PATCH] i2c: mxs: Add PIO and mixed-DMA support

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Dear Wolfram Sang,

> On Thu, Nov 22, 2012 at 01:50:37AM +0100, Marek Vasut wrote:
> > Dear Marek Vasut,
> > 
> > > Add support for the PIO mode and mixed PIO/DMA mode support. The mixed
> > > PIO/DMA is the default mode of operation. This shall leverage overhead
> > > that the driver creates due to setting up DMA descriptors even for very
> > > short transfers.
> > > 
> > > The current boundary between PIO/DMA 8 bytes, transfers shorter than 8
> > > bytes are transfered by PIO, longer transfers use DMA. The performance
> > > of write transfers remains unchanged, while there is a minor
> > > improvement of read performance. Reading 16KB EEPROM with DMA-only
> > > operations gives a read speed of 39.5KB/s, while with then new
> > > mixed-mode the speed is blazing 40.6KB/s.
> > > 
> > > Signed-off-by: Marek Vasut <marex@xxxxxxx>
> > > Cc: Fabio Estevam <fabio.estevam@xxxxxxxxxxxxx>
> > > Cc: Shawn Guo <shawn.guo@xxxxxxxxxx>
> > > Cc: Wolfram Sang <w.sang@xxxxxxxxxxxxxx>
> > 
> > [...]
> > 
> > btw. Wolfram, you might want to drop me some credit into the MX28 i2c
> > driver, I suspect most of the code in there is mine now anyway :-D
> 
> It's fine if you do this yourself.

I'll submit some separate patch then ... something like "i2c: mxs: hostile 
takeover of the driver, kekeke" ;-)

Anyway, please review this one as is, it should be mostly ok to apply I hope ...

Best regards,
Marek Vasut
--
To unsubscribe from this list: send the line "unsubscribe linux-i2c" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux GPIO]     [Linux SPI]     [Linux Hardward Monitoring]     [LM Sensors]     [Linux USB Devel]     [Linux Media]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux