RE: [PATCH v3] OMAP2/3: hwmod: fix the i2c-reset timeout during bootup

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

 



>On Thu, May 19, 2011 at 06:05:21PM +0530, Koyamangalath, Abhilash wrote:
>
> > I have tried the patch on am37x evm and the i2c soft-reset time-out issue doesn't
> > seem to go off for me. Previously I used to get a log  :
> >
> > [    0.000000] omap_hwmod: i2c1: softreset failed (waited 10000 usec)
> > [    0.000000] omap_hwmod: i2c2: softreset failed (waited 10000 usec)
> > [    0.000000] omap_hwmod: i2c3: softreset failed (waited 10000 usec)
> >
> > And now I get :
> > [    0.000000] omap_i2c_reset: i2c1: softreset failed (waited 10000 usec)
> > [    0.000000] omap_i2c_reset: i2c2: softreset failed (waited 10000 usec)
> > [    0.000000] omap_i2c_reset: i2c3: softreset failed (waited 10000 usec)
> >
> > So the function where this failure happens seems to be the only thing that has changed.
> > I've tried this patch on top of the recent PSP 04.02.00.07 release.
> > What am I missing?
>
>Abhilash ,
>
>I am not sure what is PSP 04.02.00.07 release. But you should make sure
>that the hwmod structures in omap_hwmod_3xxx_data.c should have the
>16_bit flags.
>
>Below is just a snippet in P.S.. There is already a patch submitted by
>andy green for this. Not sure, if it is merged.
>
>With this and the posted v3 patch, you should not see timeouts. Or else
>you have some other problem.
>
>thanks ,
>
>- Avinash
>
>PS:
>
> static struct omap_hwmod omap3xxx_i2c1_hwmod = {
>        .name           = "i2c1",
>+       .flags          = HWMOD_16BIT_REG,

Yes Avinash, that works , 
thanks.
>
>
> >
> > thanks,
> > Abhilash
> >
> >
> >
> > > >
> > > >
> > > > - Paul
> --
> 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