Re: [PATCH v3] i2c: mediatek: i2c multi transfer optimization

From: Wolfram Sang
Date: Tue Apr 12 2016 - 17:14:15 EST


Hi,

thanks for the submission!

On Tue, Mar 08, 2016 at 02:23:51AM +0800, Liguo Zhang wrote:
> Signal complete() in the i2c irq handler after one transfer done,
> and then wait_for_completion_timeout() will return, this procedure
> may cost much time, so only signal complete() when the entire
> transaction has been completed, it will reduce the entire transaction
> time.
>
> Signed-off-by: Liguo Zhang <liguo.zhang@xxxxxxxxxxxx>

I wonder. You have less context switches, yes. On the other hand, you
likely have bigger interrupt latency because you do more stuff in the
interrupt handler. Is it really a gain in the end?

Regards,

Wolfram

Attachment: signature.asc
Description: PGP signature