Wei, Stephen, > Hi, Stephen > I think if doesn't need to apply the slave address codes, we can use your > patch. we use the slave address as part of our nvec mfd driver in the ac100 project, so it would be nice to have this merged. Is there any other implemenation for a driver which actually use "is_slave"? (beside the one from the older android kernels)? Regards, Marc > Thanks > Wei. > > -----Original Message----- > From: Stephen Warren > Sent: Friday, April 29, 2011 11:20 PM > To: Wei Ni > Cc: khali@xxxxxxxxxxxx; ben-linux@xxxxxxxxx; lucas.demarchi@xxxxxxxxxxxxxx; > ccross@xxxxxxxxxxx; linux-i2c@xxxxxxxxxxxxxxx; > linux-kernel@xxxxxxxxxxxxxxx; linux-tegra@xxxxxxxxxxxxxxx; Olof Johansson > (olof@xxxxxxxxx) 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 <wni@xxxxxxxxxx> > > > > *** 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(-) -- Dipl. Phys. Marc Dietrich Institut für Angewandte Physik - AG Kleinkühler - Justus-Liebig-Universität Gießen Heinrich-Buff-Ring 16 D-35392 Gießen Germany Tel. +49 641 99 33462 Fax. +49 641 99 33409 email: marc.dietrich@xxxxxxxxxxxxxxxxxxxxxxxx www: http://www.uni-giessen.de/cms/iap/ -- To unsubscribe from this list: send the line "unsubscribe linux-i2c" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html