<br><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Nov 5, 2012 at 2:34 PM, Felipe Balbi <span dir="ltr"><<a href="mailto:balbi@ti.com" target="_blank">balbi@ti.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

Hi,<br>
<div><br>
On Mon, Nov 05, 2012 at 02:04:56PM +0530, Shubhrajyoti wrote:<br>
> >> @@ -1155,7 +1187,7 @@ omap_i2c_probe(struct platform_device *pdev)<br>
> >><br>
> >>            dev->fifo_size = (dev->fifo_size / 2);<br>
> >><br>
> >> -          if (dev->rev < OMAP_I2C_REV_ON_3630_4430)<br>
> >> +          if (dev->rev < OMAP_I2C_REV_ON_3630)<br>
> >>                    dev->b_hw = 1; /* Enable hardware fixes */<br>
> > looks like this was applicable to 4430 too, what happened ?<br>
> No actually this can be deleted completely once the<br>
> start -> transaction -> stop sequence recommendation is followed.<br>
<br>
</div>yes, but we're not there just yet and this patch is changing the<br>
behavior of the driver</blockquote><div><br></div><div>No , earlier we were truncating the register for omap4 so </div><div>OMAP_I2C_REV_ON_3630_4430 was there if we read both hi and lo for </div><div>omap4 then we donot find 3630 and 4430 value to be similar.</div>
<div><br></div><div>In this case the behavior is same as earlier also it enabled this for lower than 3630 and </div><div>the same holds good even now.</div><div><br></div><div>So in essence,</div><div>Earlier  OMAP_I2C_REV_ON_3630_4430 is named to OMAP_I2C_REV_ON_3630 </div>
<div>and omap4 rev will have a 32bit value.</div><div><br></div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"> in ways which don't belong to $SUBJECT.<br>

<span><font color="#888888"><br>
--<br>
balbi<br>
</font></span></blockquote></div><br></div>