Re: [PATCH] OMAP4: I2C: Enable FIFO usage for OMAP4

2011-11-08 Thread Kevin Hilman
+Ben Dooks

Shubhrajyoti D  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.
>
> Tested-and-Reported-by: Nishanth Menon 
> Signed-off-by: Shubhrajyoti D 

Acked-by: Kevin Hilman 

Ben, please queue as a fix for the v3.2-rc.  It applies cleanly to
v3.2-rc1.

If you prefer a branch:
https://git.kernel.org/pub/scm/linux/kernel/git/khilman/linux-omap-pm.git 
for_3.2/fixes/i2c

Thanks,

Kevin
--
To unsubscribe from this list: send the line "unsubscribe linux-i2c" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH] OMAP4: I2C: Enable FIFO usage for OMAP4

2011-11-03 Thread Shubhrajyoti
On Thursday 03 November 2011 07:51 PM, Kevin Hilman wrote:
> Shubhrajyoti  writes:
>
>> 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  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 
> Signed-off-by: Shubhrajyoti D 
> ---
> 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?
> Please update this patch against my for_3.2/i2c-cleanup branch, and add
> the Tested-by from Nishanth Menon.
>
> Then I will push to get this in as a fix for v3.2-rc.
Thanks the patch was already rebased to the 

for_3.2/i2c-cleanup branch and applies on your tree . Resent it adding 
Nishanth's Tested-by

Thanks.

> Kevin

--
To unsubscribe from this list: send the line "unsubscribe linux-i2c" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[PATCH] OMAP4: I2C: Enable FIFO usage for OMAP4

2011-11-03 Thread Shubhrajyoti D
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.

Tested-and-Reported-by: Nishanth Menon 
Signed-off-by: Shubhrajyoti D 
---
 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 a43d002..fa23faa 100644
--- a/drivers/i2c/busses/i2c-omap.c
+++ b/drivers/i2c/busses/i2c-omap.c
@@ -1047,13 +1047,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 = (100 * dev->fifo_size) /
-- 
1.7.1

--
To unsubscribe from this list: send the line "unsubscribe linux-i2c" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH] OMAP4: I2C: Enable FIFO usage for OMAP4

2011-11-03 Thread Kevin Hilman
Shubhrajyoti  writes:

> 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  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 
 Signed-off-by: Shubhrajyoti D 
 ---
 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?

Please update this patch against my for_3.2/i2c-cleanup branch, and add
the Tested-by from Nishanth Menon.

Then I will push to get this in as a fix for v3.2-rc.

Kevin
--
To unsubscribe from this list: send the line "unsubscribe linux-i2c" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH] OMAP4: I2C: Enable FIFO usage for OMAP4

2011-11-03 Thread Shubhrajyoti
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  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 
>>> Signed-off-by: Shubhrajyoti D 
>>> ---
>>> 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 = (100 * dev->fifo_size) /
>> --
>> To unsubscribe from this list: send the line "unsubscribe linux-i2c" in
>> the body of a message to majord...@vger.kernel.org
>> More majordomo info at  http://vger.kernel.org/majordomo-info.html

--
To unsubscribe from this list: send the line "unsubscribe linux-i2c" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH] OMAP4: I2C: Enable FIFO usage for OMAP4

2011-07-13 Thread Ben Dooks
On Tue, Jul 05, 2011 at 05:01:01PM -0700, Kevin Hilman wrote:
> Shubhrajyoti D  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 
> > Signed-off-by: Shubhrajyoti D 
> > ---
> > 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?
 
> 
> >  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 = (100 * dev->fifo_size) /
> --
> To unsubscribe from this list: send the line "unsubscribe linux-i2c" in
> the body of a message to majord...@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe linux-i2c" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH] OMAP4: I2C: Enable FIFO usage for OMAP4

2011-07-06 Thread Kevin Hilman
"Menon, Nishanth"  writes:

> On Tue, Jul 5, 2011 at 17:01, Kevin Hilman  wrote:
>>
>> Shubhrajyoti D  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 
>> > Signed-off-by: Shubhrajyoti D 
>> > ---
>> > 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
>
> For what it is worth: tested on SDP4460/4430 with 255 multi-byte i2c
> read operation on twl6030.
>
> Tested-by: Nishanth Menon 
>

Thanks Nishanth,

Kevin
--
To unsubscribe from this list: send the line "unsubscribe linux-i2c" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH] OMAP4: I2C: Enable FIFO usage for OMAP4

2011-07-05 Thread Menon, Nishanth
On Tue, Jul 5, 2011 at 17:01, Kevin Hilman  wrote:
>
> Shubhrajyoti D  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 
> > Signed-off-by: Shubhrajyoti D 
> > ---
> > 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

For what it is worth: tested on SDP4460/4430 with 255 multi-byte i2c
read operation on twl6030.

Tested-by: Nishanth Menon 

Regards,
Nishanth Menon



>
> Kevin
>
>
> >  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 = (100 * dev->fifo_size) /
> --
> To unsubscribe from this list: send the line "unsubscribe linux-omap" in
> the body of a message to majord...@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe linux-i2c" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH] OMAP4: I2C: Enable FIFO usage for OMAP4

2011-07-05 Thread Kevin Hilman
Shubhrajyoti D  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 
> Signed-off-by: Shubhrajyoti D 
> ---
> 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


>  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 = (100 * dev->fifo_size) /
--
To unsubscribe from this list: send the line "unsubscribe linux-i2c" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH] OMAP4: I2C: Enable FIFO usage for OMAP4

2011-07-05 Thread Shubhrajyoti

On Friday 01 July 2011 04:20 AM, Kevin Hilman wrote:

Shubhrajyoti  writes:


On Thursday 30 June 2011 04:53 AM, Kevin Hilman wrote:

Shubhrajyoti D   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.

Do you know the history of why the FIFO depth was set to zero?  A
summary of that history would greatly help reviewers.
Looks like it is disabled in the initial bring up and some how didn’t 
get enabled.

All I could find is
http://www.mail-archive.com/linux-i2c@vger.kernel.org/msg01404.html


Reported-By:Nishanth Menon

minor: missing space after ':'

Yes will fix it.

Signed-off-by: Shubhrajyoti D

Tested on what platforms?   OMAP3 also?

OMAP4 and OMAP3

Please add a brief summary of what platforms it was tested on in the
changelog.

Will update it

Could you please rebase this onto my for_3.1/i2c-andy branch[1] where we
have a large series of I2C patches queued up for the v3.1 merge window
already?

OK will rebase and resend.

Sent a rebased version.

Thanks,

Kevin


--
To unsubscribe from this list: send the line "unsubscribe linux-i2c" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[PATCH] OMAP4: I2C: Enable FIFO usage for OMAP4

2011-07-05 Thread Shubhrajyoti D
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 
Signed-off-by: Shubhrajyoti D 
---
Rebased on top of the series by Andy Green
http://www.spinics.net/lists/linux-i2c/msg05632.html

 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 = (100 * dev->fifo_size) /
-- 
1.7.1

--
To unsubscribe from this list: send the line "unsubscribe linux-i2c" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH] OMAP4: I2C: Enable FIFO usage for OMAP4

2011-06-30 Thread Kevin Hilman
Shubhrajyoti  writes:

> On Thursday 30 June 2011 04:53 AM, Kevin Hilman wrote:
>> Shubhrajyoti D  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.
>> Do you know the history of why the FIFO depth was set to zero?  A
>> summary of that history would greatly help reviewers.
>>
>>> Reported-By:Nishanth Menon
>> minor: missing space after ':'
>>
>>> Signed-off-by: Shubhrajyoti D
>> Tested on what platforms?   OMAP3 also?
> OMAP4 and OMAP3

Please add a brief summary of what platforms it was tested on in the
changelog.

>> Could you please rebase this onto my for_3.1/i2c-andy branch[1] where we
>> have a large series of I2C patches queued up for the v3.1 merge window
>> already?
> OK will rebase and resend.

Thanks,

Kevin
--
To unsubscribe from this list: send the line "unsubscribe linux-i2c" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH] OMAP4: I2C: Enable FIFO usage for OMAP4

2011-06-30 Thread Shubhrajyoti

On Thursday 30 June 2011 04:53 AM, Kevin Hilman wrote:

Shubhrajyoti D  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.

Do you know the history of why the FIFO depth was set to zero?  A
summary of that history would greatly help reviewers.


Reported-By:Nishanth Menon

minor: missing space after ':'


Signed-off-by: Shubhrajyoti D

Tested on what platforms?   OMAP3 also?

OMAP4 and OMAP3

Could you please rebase this onto my for_3.1/i2c-andy branch[1] where we
have a large series of I2C patches queued up for the v3.1 merge window
already?

OK will rebase and resend.

Thanks,

Kevin

[1] git://git.kernel.org/pub/scm/linux/kernel/git/khilman/linux-omap-pm.git



--
To unsubscribe from this list: send the line "unsubscribe linux-i2c" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH] OMAP4: I2C: Enable FIFO usage for OMAP4

2011-06-29 Thread Kevin Hilman
Shubhrajyoti D  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.

Do you know the history of why the FIFO depth was set to zero?  A
summary of that history would greatly help reviewers.

> Reported-By:Nishanth Menon 

minor: missing space after ':'

> Signed-off-by: Shubhrajyoti D 

Tested on what platforms?   OMAP3 also?

Could you please rebase this onto my for_3.1/i2c-andy branch[1] where we
have a large series of I2C patches queued up for the v3.1 merge window
already? 

Thanks,

Kevin

[1] git://git.kernel.org/pub/scm/linux/kernel/git/khilman/linux-omap-pm.git

--
To unsubscribe from this list: send the line "unsubscribe linux-i2c" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[PATCH] OMAP4: I2C: Enable FIFO usage for OMAP4

2011-06-28 Thread Shubhrajyoti D
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.

Reported-By:Nishanth Menon 
Signed-off-by: Shubhrajyoti D 
---
 drivers/i2c/busses/i2c-omap.c |9 -
 1 files changed, 4 insertions(+), 5 deletions(-)

diff --git a/drivers/i2c/busses/i2c-omap.c b/drivers/i2c/busses/i2c-omap.c
index 58a58c7..934b52c 100644
--- a/drivers/i2c/busses/i2c-omap.c
+++ b/drivers/i2c/busses/i2c-omap.c
@@ -1059,13 +1059,12 @@ 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_4430) {
-   dev->fifo_size = 0;
+   dev->fifo_size = (dev->fifo_size / 2);
+   if (dev->rev >= OMAP_I2C_REV_ON_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 = (100 * dev->fifo_size) /
-- 
1.7.1

--
To unsubscribe from this list: send the line "unsubscribe linux-i2c" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html