From mboxrd@z Thu Jan 1 00:00:00 1970 From: Laxman Dewangan Subject: Re: [PATCH V1 2/2] i2c: tegra: support for I2C_M_NOSTART protocol mangling Date: Thu, 10 May 2012 17:50:32 +0530 Message-ID: <4FABB290.40609@nvidia.com> References: <1335251976-31925-1-git-send-email-ldewangan@nvidia.com> <1335251976-31925-3-git-send-email-ldewangan@nvidia.com> <4FABA585.7090600@nvidia.com> <20120510135411.3ba5bb6d@endymion.delvare> Mime-Version: 1.0 Content-Type: text/plain; charset="ISO-8859-1"; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20120510135411.3ba5bb6d-R0o5gVi9kd7kN2dkZ6Wm7A@public.gmane.org> Sender: linux-i2c-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Jean Delvare Cc: "w.sang-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org" , "ben-linux-elnMNo+KYs3YtjvyW6yDsg@public.gmane.org" , "swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org" , "olof-nZhT3qVonbNeoWH0uzbU5w@public.gmane.org" , "linux-i2c-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , "linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" List-Id: linux-i2c@vger.kernel.org On Thursday 10 May 2012 05:24 PM, Jean Delvare wrote: > Hi Laxman, > > On Thu, 10 May 2012 16:54:53 +0530, Laxman Dewangan wrote: >> Jean, >> On Tuesday 24 April 2012 12:49 PM, Laxman Dewangan wrote: >>> Adding support for protocol mangling I2C_M_NOSTART. >>> When multiple message transfer request made through i2c >>> and if any message is flagged with I2C_M_NOSTART then >>> it will not send the start/repeat start/address of that >>> message i.e. send the data directly. >> Now the framework/core changes for making M_NOSTART as a separate flag >> is already part of your tree. >> So do I need to re-send the patch for having that change? >> 1/1 is already part of Wolfram's tree and so not sure that how do I go >> i.e. from where I take the base i2c-tegra driver. >> It may create the problem in integration on main if they are not sync. >> Also I have some couple of changes which will be on top of this. > I'm not sure I read you correctly so, for clarity: the patch > introducing I2C_FUNC_NOSTART finally landed in my tree [1], not > Wolfram's. It will be merged in kernel 3.5. > > [1] http://khali.linux-fr.org/devel/linux-3/jdelvare-i2c/ > > If you don't want to depend on that, then just use > I2C_FUNC_PROTOCOL_MANGLING for now and you can switch to > I2C_FUNC_NOSTART later. > It is fine. So this change can go in to Wolfram's tree as it is and after k3.5, I will post another change which will use I2C_FUNC_NOSTART. Thanks, Laxman