From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752535AbcHLPYW convert rfc822-to-8bit (ORCPT ); Fri, 12 Aug 2016 11:24:22 -0400 Received: from nopam.alitech.com ([202.3.176.31]:62191 "EHLO nopam.alitech.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752124AbcHLPYV (ORCPT ); Fri, 12 Aug 2016 11:24:21 -0400 X-Greylist: delayed 1475 seconds by postgrey-1.27 at vger.kernel.org; Fri, 12 Aug 2016 11:24:20 EDT split_mail: 1 Subject: Re: [PATCH v3 0/3] i2c: designware: improve performance for transfers To: Lucas De Marchi , linux-i2c@vger.kernel.org References: <1469743424-2480-1-git-send-email-lucas.demarchi@intel.com> Cc: linux-kernel@vger.kernel.org, mika.westerberg@linux.intel.com, jarkko.nikula@linux.intel.com From: Christian Ruppert Message-ID: <28c393bf-416b-b406-ff39-a2d85317a102@alitech.com> Date: Fri, 12 Aug 2016 16:59:32 +0200 User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:45.0) Gecko/20100101 Thunderbird/45.2.0 MIME-Version: 1.0 In-Reply-To: <1469743424-2480-1-git-send-email-lucas.demarchi@intel.com> X-MIMETrack: =?Big5?B?SXRlbWl6ZSBieSBTTVRQIFNlcnZlciBvbiBUV0FMSU5TMi9BTElfVFBFL0FMaSg=?= =?Big5?B?UmVsZWFzZSA4LjAuMkZQNnxKdWx5IDE1LCAyMDEwKSBhdCAyMDE2LzA4LzEyIKRV?= =?Big5?B?pMggMTE6MDI6NTU=?=, =?Big5?B?U2VyaWFsaXplIGJ5IFJvdXRlciBvbiBUV0FMSU5TMi9BTElfVFBFL0FMaSg=?= =?Big5?B?UmVsZWFzZSA4LjAuMkZQNnxKdWx5IDE1LCAyMDEwKSBhdCAyMDE2LzA4LzEyIKRV?= =?Big5?B?pMggMTE6MDI6NTY=?= X-TNEFEvaluated: 1 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8BIT Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 29.07.2016 00:03, Lucas De Marchi wrote: > This can be considered a v3 of "i2c: designware: do not disable adapter after > transfer". Differences are: > > - Now there's a first patch that does not depend on IC_TAR being dynamically > enabled/disabled: it just doesn't wait for the state change when not needed. > > - We added a patch that allows detecting if HW supports the dynamic TAR updates > > - In the last patch the bits were changed as suggested by Jarkko. > > - This is tested on BayTrail and CherryTrail, both of them returning true for > "dynamically update TAR" Tested-by: Christian Ruppert on TB101 with Linux-4.7 Seems to work perfectly now, congratulations and thanks for your patience with our platform... This was a hard one! > José Roberto de Souza (1): > i2c: designware: wait for disable/enable only if necessary > > Lucas De Marchi (2): > i2c: designware: detect when dynamic tar update is possible > i2c: designware: do not disable adapter after transfer > > drivers/i2c/busses/i2c-designware-core.c | 103 +++++++++++++++++++++---------- > drivers/i2c/busses/i2c-designware-core.h | 1 + > 2 files changed, 72 insertions(+), 32 deletions(-) >