Re: [PATCH 0/2] i2c: tegra: add some new features for tegra i2c

[Marc Dietrich <Marc.Dietrich@ap.physik.uni-giessen.de>]

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@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 <wni@nvidia.com>
> > 
> > *** 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@ap.physik.uni-giessen.de
www: http://www.uni-giessen.de/cms/iap/