From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jean Delvare Subject: Re: [RFC/PATCH 3/3] i2c: omap: Add support for I2C_M_STOP message flag Date: Tue, 17 Jul 2012 17:29:35 +0200 Message-ID: <20120717172935.45ff210f@endymion.delvare> References: <1340720229-30356-1-git-send-email-laurent.pinchart@ideasonboard.com> <1340720229-30356-4-git-send-email-laurent.pinchart@ideasonboard.com> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1340720229-30356-4-git-send-email-laurent.pinchart-ryLnwIuWjnjg/C1BVhZhaw@public.gmane.org> Sender: linux-i2c-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Laurent Pinchart Cc: linux-i2c-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-omap-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: linux-i2c@vger.kernel.org On Tue, 26 Jun 2012 16:17:09 +0200, Laurent Pinchart wrote: > Generate a stop condition after each message marked with I2C_M_STOP. > > Signed-off-by: Laurent Pinchart > --- > drivers/i2c/busses/i2c-omap.c | 2 ++ > 1 files changed, 2 insertions(+), 0 deletions(-) > > diff --git a/drivers/i2c/busses/i2c-omap.c b/drivers/i2c/busses/i2c-omap.c > index 801df60..cf1bda0 100644 > --- a/drivers/i2c/busses/i2c-omap.c > +++ b/drivers/i2c/busses/i2c-omap.c > @@ -545,6 +545,8 @@ static int omap_i2c_xfer_msg(struct i2c_adapter *adap, > if (dev->speed > 400) > w |= OMAP_I2C_CON_OPMODE_HS; > > + if (msg->flags & I2C_M_STOP) > + stop = 1; > if (msg->flags & I2C_M_TEN) > w |= OMAP_I2C_CON_XA; > if (!(msg->flags & I2C_M_RD)) Looks OK to me, but I can't test it, and I also don't know a thing about OMAP. Maybe it would be a good idea to let omap_i2c_func() return I2C_FUNC_PROTOCOL_MANGLING now? -- Jean Delvare