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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Jul 5, 2011 at 17:01, Kevin Hilman <khilman@xxxxxx> wrote:
>
> Shubhrajyoti D <shubhrajyoti@xxxxxx> 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 <nm@xxxxxx>
> > Signed-off-by: Shubhrajyoti D <shubhrajyoti@xxxxxx>
> > ---
> > 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 <nm@xxxxxx>

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


[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux