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

[Wei Ni]

Hi, Stephen
I think your patches address the different issues with my patches.
My patches are used for retry transfer and new slave controller.

Thanks
Wei.


-----Original Message-----
From: Stephen Warren 
Sent: Thursday, April 28, 2011 12:29 AM
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 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.

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.

> 
> Thanks
> Wei.
> 
> -----Original Message-----
> From: Wei Ni
> Sent: Wednesday, April 20, 2011 8:09 PM
> To: khali@xxxxxxxxxxxx; ben-linux@xxxxxxxxx;
> lucas.demarchi@xxxxxxxxxxxxxx; ccross@xxxxxxxxxxx; linux-
> i2c@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx
> Cc: Wei Ni
> Subject: [PATCH 0/2] i2c: tegra: add some new features for tegra i2c
> 
> 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(-)

-- 
nvpublic

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/