linux-kernel - RE: [PATCH 0/2] i2c: tegra: add some new features for tegra i2c

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <6B4D417B830BC44B8026029FD256F7F1C2CB3DD4EA@HKMAIL01.nvidia.com>
Date:	Wed, 4 May 2011 17:19:41 +0800
From:	Wei Ni <wni@...dia.com>
To:	'Marc Dietrich' <Marc.Dietrich@...physik.uni-giessen.de>
CC:	Stephen Warren <swarren@...dia.com>,
	"khali@...ux-fr.org" <khali@...ux-fr.org>,
	"ben-linux@...ff.org" <ben-linux@...ff.org>,
	"lucas.demarchi@...fusion.mobi" <lucas.demarchi@...fusion.mobi>,
	"ccross@...roid.com" <ccross@...roid.com>,
	"linux-i2c@...r.kernel.org" <linux-i2c@...r.kernel.org>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	"linux-tegra@...r.kernel.org" <linux-tegra@...r.kernel.org>,
	"Olof Johansson (olof@...om.net)" <olof@...om.net>
Subject: RE: [PATCH 0/2] i2c: tegra: add some new features for tegra i2c

Hi, Marc
I think no other implementation.

Thanks
Wei.

-----Original Message-----
From: Marc Dietrich [mailto:Marc.Dietrich@...physik.uni-giessen.de] 
Sent: Wednesday, May 04, 2011 4:28 PM
To: Wei Ni
Cc: Stephen Warren; khali@...ux-fr.org; ben-linux@...ff.org; lucas.demarchi@...fusion.mobi; ccross@...roid.com; linux-i2c@...r.kernel.org; linux-kernel@...r.kernel.org; linux-tegra@...r.kernel.org; Olof Johansson (olof@...om.net)
Subject: Re: [PATCH 0/2] i2c: tegra: add some new features for tegra i2c


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@...ux-fr.org; ben-linux@...ff.org; lucas.demarchi@...fusion.mobi;
> ccross@...roid.com; linux-i2c@...r.kernel.org;
> linux-kernel@...r.kernel.org; linux-tegra@...r.kernel.org; Olof Johansson
> (olof@...om.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@...dia.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@...physik.uni-giessen.de
www: http://www.uni-giessen.de/cms/iap/
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/