From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1758934AbcHYQXy convert rfc822-to-8bit (ORCPT ); Thu, 25 Aug 2016 12:23:54 -0400 Received: from nopam.alitech.com ([202.3.176.31]:52108 "EHLO nopam.alitech.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751393AbcHYQXx (ORCPT ); Thu, 25 Aug 2016 12:23:53 -0400 split_mail: 1 Subject: Re: [PATCH v4 0/4] i2c: designware: improve performance for transfers To: Lucas De Marchi , linux-i2c@vger.kernel.org References: <1471990736-30190-1-git-send-email-lucas.demarchi@intel.com> Cc: mika.westerberg@linux.intel.com, jarkko.nikula@linux.intel.com, linux-kernel@vger.kernel.org, Wolfram Sang From: Christian Ruppert Message-ID: Date: Thu, 25 Aug 2016 18:23:25 +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: <1471990736-30190-1-git-send-email-lucas.demarchi@intel.com> X-MIMETrack: =?Big5?B?SXRlbWl6ZSBieSBTTVRQIFNlcnZlciBvbiBUV0FMSU5TMi9BTElfVFBFL0FMaSg=?= =?Big5?B?UmVsZWFzZSA4LjAuMkZQNnxKdWx5IDE1LCAyMDEwKSBhdCAyMDE2LzA4LzI2IKRX?= =?Big5?B?pMggMTI6MjM6MTg=?=, =?Big5?B?U2VyaWFsaXplIGJ5IFJvdXRlciBvbiBUV0FMSU5TMi9BTElfVFBFL0FMaSg=?= =?Big5?B?UmVsZWFzZSA4LjAuMkZQNnxKdWx5IDE1LCAyMDEwKSBhdCAyMDE2LzA4LzI2IKRX?= =?Big5?B?pMggMTI6MjM6MzE=?= 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 24.08.2016 00:18, Lucas De Marchi wrote: > Diff from v3: > > - Fix over 80chars in one place > > - Move check for adapter being able to dynamically update TAR to be done > on probe time rather than init as requested by Jarkko > > For the previous version, Christian had added: > > Tested-by: Christian Ruppert > on TB101 with Linux-4.7 Tested-by: Christian Ruppert Re-tested this iteration in the same configuration (TB101, Linux-4.7). Still works perfectly. > And Jarkko added his Acked-by: Jarkko Nikula > on patches 1 and 3 (now 4). > > There's a new patch #2 as a preparatory work to move the check > mentioned above to i2c_dw_probe(). > > 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" > > José Roberto de Souza (1): > i2c: designware: wait for disable/enable only if necessary > > Lucas De Marchi (3): > i2c: designware: add common functions for locking > i2c: designware: detect when dynamic tar update is possible > i2c: designware: do not disable adapter after transfer > > drivers/i2c/busses/i2c-designware-core.c | 162 +++++++++++++++++++++---------- > drivers/i2c/busses/i2c-designware-core.h | 1 + > 2 files changed, 111 insertions(+), 52 deletions(-) >