From mboxrd@z Thu Jan 1 00:00:00 1970 From: w.sang@pengutronix.de (Wolfram Sang) Date: Thu, 1 Nov 2012 17:08:28 +0100 Subject: [PATCH] i2c: mxs: remove broken PIOQUEUE support In-Reply-To: <201211011528.17596.marex@denx.de> References: <1351771003-6071-1-git-send-email-w.sang@pengutronix.de> <201211011528.17596.marex@denx.de> Message-ID: <20121101160828.GA18425@pengutronix.de> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Thu, Nov 01, 2012 at 03:28:17PM +0100, Marek Vasut wrote: > Dear Wolfram Sang, > > > This I2C master can do DMA and PIOQUEUE (PIO with FIFO). Originally, > > only PIOQEUE > > PIOQUEUE ;-) Yup, right! > > > was supported, then DMA support was added. The original > > intention was to keep PIOQUEUE since it has less overhead what is nice > > for small transfers. However, runtime switching between PIOQEUE and DMA > > depending on the transfer size never worked despite a lot of trying. > > Since PIOQUEUE mode itself was flaky (polling at places where interrupts > > failed to work) and the implementation also imposed a size limit for > > transfers, it is best to remove the support altogether which makes the > > driver a lot cleaner and more robust. If somebody really wants less > > overhead, plain PIO mode could still be implemented with the addidtional > > advantage that this mode is also available on MX23, too. > > Yes, looks to be the way to go. > > Reviewed-by: Marek Vasut Thanks. BTW have you tried combining all i2c-messages (msgs[]) of the transfer into one DMA chain? That would reduce overhead, too, no? Regards, Wolfram -- Pengutronix e.K. | Wolfram Sang | Industrial Linux Solutions | http://www.pengutronix.de/ | -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 198 bytes Desc: Digital signature URL: