Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755056Ab1EDDDs (ORCPT ); Tue, 3 May 2011 23:03:48 -0400 Received: from hqemgate03.nvidia.com ([216.228.121.140]:9463 "EHLO hqemgate03.nvidia.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754742Ab1EDDDp convert rfc822-to-8bit (ORCPT ); Tue, 3 May 2011 23:03:45 -0400 X-PGP-Universal: processed; by hqnvupgp06.nvidia.com on Tue, 03 May 2011 20:03:45 -0700 From: Wei Ni To: Stephen Warren CC: "khali@linux-fr.org" , "ben-linux@fluff.org" , "lucas.demarchi@profusion.mobi" , "ccross@android.com" , "linux-i2c@vger.kernel.org" , "linux-kernel@vger.kernel.org" , "linux-tegra@vger.kernel.org" , "Olof Johansson (olof@lixom.net)" Date: Wed, 4 May 2011 11:03:41 +0800 Subject: RE: [PATCH 0/2] i2c: tegra: add some new features for tegra i2c Thread-Topic: [PATCH 0/2] i2c: tegra: add some new features for tegra i2c Thread-Index: Acv/U9peluMUOVgAQtuc8GIYvXxc8QFcZJpQAAyVSsAATfGVkAAUIY8QAOHp64A= Message-ID: <6B4D417B830BC44B8026029FD256F7F1C2CB3DD4E6@HKMAIL01.nvidia.com> References: <1303301312-15302-1-git-send-email-wni@nvidia.com> <6B4D417B830BC44B8026029FD256F7F1C2CB3DD4DB@HKMAIL01.nvidia.com> <74CDBE0F657A3D45AFBB94109FB122FF0497F1AAC4@HQMAIL01.nvidia.com> <6B4D417B830BC44B8026029FD256F7F1C2CB3DD4E2@HKMAIL01.nvidia.com> <74CDBE0F657A3D45AFBB94109FB122FF0497F1AEEF@HQMAIL01.nvidia.com> In-Reply-To: <74CDBE0F657A3D45AFBB94109FB122FF0497F1AEEF@HQMAIL01.nvidia.com> Accept-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: acceptlanguage: en-US MIME-Version: 1.0 Content-Language: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 8BIT Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2638 Lines: 71 Hi, Stephen I think if doesn't need to apply the slave address codes, we can use your patch. Thanks Wei. -----Original Message----- From: Stephen Warren Sent: Friday, April 29, 2011 11:20 PM To: Wei Ni Cc: khali@linux-fr.org; ben-linux@fluff.org; lucas.demarchi@profusion.mobi; ccross@android.com; linux-i2c@vger.kernel.org; linux-kernel@vger.kernel.org; linux-tegra@vger.kernel.org; Olof Johansson (olof@lixom.net) Subject: RE: [PATCH 0/2] i2c: tegra: add some new features for tegra i2c Wei Ni wrote at Friday, April 29, 2011 12:08 AM: > Stephen Warren wrote at Thursday, April 28, 2011 12:29 AM: > > > > Wei Ni wrote at Wednesday, April 27, 2011 4:26 AM: > > > Hi, all > > > Could anyone review these changes? > > > > Wei, > > > > It looks like some of the patches I recently posted address the same > > issues as the patches you had already posted. Sorry for the potential > > conflicts. > > Hi, Stephen > I think your patches address the different issues with my patches. > My patches are used for retry transfer and new slave controller. Wei, aren't the following two patches basically the same thing, except that yours also sets the slave address register, and is a bit more wordy: http://www.spinics.net/lists/linux-i2c/msg05464.html http://www.spinics.net/lists/linux-i2c/msg05437.html Upon further inspection, I don't think any of the other patches I posted conflict with the other patch you posted. > > Can you please take a look at the patches I posted and comment on which > > of those are still relevant given your patches. Even for the cases where > > we both posted patches to address the same issue (e.g. use new slave > > mode), there are differences in the patches. I simply took the changes > > from the chromeos-2.6.37 kernel and cherry-picked them into the mainline > > kernel. Are the changes you posted re-written to address issues that > > weren't solved in the ChromeOS kernel? > > > > Thanks for any feedback on this. Wei Ni wrote at Wednesday, April 20, 2011 8:09 PM: > From: Wei Ni > > *** BLURB HERE *** > > Wei Ni (2): > i2c: tegra: Retry transfer when unexpected/no_ack status is detected > i2c: tegra: use new i2c slave controller > > drivers/i2c/busses/i2c-tegra.c | 57 +++++++++++++++++++++++++++++++++++++-- > include/linux/i2c-tegra.h | 3 ++ > 2 files changed, 57 insertions(+), 3 deletions(-) -- nvpublic -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/