On Thursday 14 July 2011 12:50 AM, Ben Dooks wrote:
> On Tue, Jul 05, 2011 at 05:01:01PM -0700, Kevin Hilman wrote:
>> Shubhrajyoti D <[email protected]> writes:
>>
>>> Currently the fifo depth is set to zero for OMAP4 which disables
>>> the FIFO usage. This patch enables the FIFO usage for I2C transactions
>>> on OMAP4 also.
>>>
>>> Tested on omap4430 and 3430.
>>>
>>> Reported-By: Nishanth Menon <[email protected]>
>>> Signed-off-by: Shubhrajyoti D <[email protected]>
>>> ---
>>> Rebased on top of the series by Andy Green
>>> http://www.spinics.net/lists/linux-i2c/msg05632.html
>> Thanks.
>>
>> This is v3.1 material, but would be nice to see a couple tested-by or
>> acked-by tags from folks that are more actively using the I2C driver
>> before merging
>>
>> Kevin
> guess you'll be picking these up?

Ben Could this patch be lined up as well?
>  
>>>  drivers/i2c/busses/i2c-omap.c |   11 ++++++-----
>>>  1 files changed, 6 insertions(+), 5 deletions(-)
>>>
>>> diff --git a/drivers/i2c/busses/i2c-omap.c b/drivers/i2c/busses/i2c-omap.c
>>> index d53cd61..8f87a37 100644
>>> --- a/drivers/i2c/busses/i2c-omap.c
>>> +++ b/drivers/i2c/busses/i2c-omap.c
>>> @@ -1068,13 +1068,14 @@ omap_i2c_probe(struct platform_device *pdev)
>>>              * size. This is to ensure that we can handle the status on int
>>>              * call back latencies.
>>>              */
>>> -           if (dev->rev >= OMAP_I2C_REV_ON_3530_4430) {
>>> -                   dev->fifo_size = 0;
>>> +
>>> +           dev->fifo_size = (dev->fifo_size / 2);
>>> +
>>> +           if (dev->rev >= OMAP_I2C_REV_ON_3530_4430)
>>>                     dev->b_hw = 0; /* Disable hardware fixes */
>>> -           } else {
>>> -                   dev->fifo_size = (dev->fifo_size / 2);
>>> +           else
>>>                     dev->b_hw = 1; /* Enable hardware fixes */
>>> -           }
>>> +
>>>             /* calculate wakeup latency constraint for MPU */
>>>             if (dev->set_mpu_wkup_lat != NULL)
>>>                     dev->latency = (1000000 * dev->fifo_size) /
>> --
>> To unsubscribe from this list: send the line "unsubscribe linux-i2c" in
>> the body of a message to [email protected]
>> More majordomo info at  http://vger.kernel.org/majordomo-info.html

--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to