Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932141Ab1D2PUx (ORCPT ); Fri, 29 Apr 2011 11:20:53 -0400 Received: from hqemgate03.nvidia.com ([216.228.121.140]:8062 "EHLO hqemgate03.nvidia.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753090Ab1D2PUv convert rfc822-to-8bit (ORCPT ); Fri, 29 Apr 2011 11:20:51 -0400 X-PGP-Universal: processed; by hqnvupgp06.nvidia.com on Fri, 29 Apr 2011 08:20:26 -0700 From: Stephen Warren 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)" Date: Fri, 29 Apr 2011 08:20:25 -0700 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/U9peluMUOVgAQtuc8GIYvXxc8QFcZJpQAAyVSsAATfGVkAAUIY8Q Message-ID: <74CDBE0F657A3D45AFBB94109FB122FF0497F1AEEF@HQMAIL01.nvidia.com> References: <1303301312-15302-1-git-send-email-wni@nvidia.com> <6B4D417B830BC44B8026029FD256F7F1C2CB3DD4DB@HKMAIL01.nvidia.com> <74CDBE0F657A3D45AFBB94109FB122FF0497F1AAC4@HQMAIL01.nvidia.com> <6B4D417B830BC44B8026029FD256F7F1C2CB3DD4E2@HKMAIL01.nvidia.com> In-Reply-To: <6B4D417B830BC44B8026029FD256F7F1C2CB3DD4E2@HKMAIL01.nvidia.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: acceptlanguage: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 8BIT MIME-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2145 Lines: 58 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/