Re: [PATCH v3 0/4] i2c : i2c-ibm-iic : use interrupts to perform the data transfer

[Jean-Jacques Hiblot]

Hi Wolfram,

There were not many reviewers for this patch series. Time permitting, 
could you have a look at it?


Thanks,

Jean-Jacques

Le 29/11/2013 17:53, jean-jacques hiblot a écrit :

This patch set aims to improve the performance of the driver for the IBM iic
controller by implementing the data transfer in the interrupt handler.
Using interrupts to trigger the data transfer reduces and make more
deterministic the latencies between indivdual bytes, and consequently reduces
the total transfer time,

In our test environement with multiple masters, this significantly reduces the
rate of i2c errors.

Changes since v2:
  - cosmectics change (comments style, removed a hard-coded value)
  - moved some parts from one patch to another

Changes since v1:
  - split the patch in 4 individual patches. The code has been refactored a bit
to make the diff easier to read.
  - changed some dev_dbg in dev_err or dev_warn when more appropriate


jean-jacques hiblot (4):
   i2c: i2c-ibm-iic: cleanup.
   i2c: i2c-ibm-iic: perform the transfer in the interrupt handler
   i2c: i2c-ibm-iic: Implements transfer abortion
   i2c: i2c-ibm-iic: Implements a polling mode

  drivers/i2c/busses/i2c-ibm_iic.c | 490 +++
  drivers/i2c/busses/i2c-ibm_iic.h |  20 +-
  2 files changed, 309 insertions(+), 201 deletions(-)


--
To unsubscribe from this list: send the line unsubscribe linux-i2c in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH 1/2] I2C: EMMA Mobile I2C master driver

[Ian Molton]

On 25/09/13 05:45, Simon Horman wrote:


Hi Ian,

I spoke with Magnus and in turn Ben about this at LinuxCon in New Orleans
last week.

Basically the position of Magnus and I is that any support for this
hardware is an incremental improvement on the current situation: no
support.

With this in mind from an shmobile point of view I am happy for this code.
And there is no need to wait for a review from Magnus.


Has anyone merged this for upstream yet? If not, where should I send it?

-Ian
--
To unsubscribe from this list: send the line unsubscribe linux-i2c in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html