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 Fri, May 20, 2011 at 08:32:17PM +0530, Avinash.H.M. wrote:
> On Wed, May 18, 2011 at 09:55:42PM +0530, Avinash.H.M. wrote:
> > On Wed, May 18, 2011 at 10:22:28AM -0600, Paul Walmsley wrote:
> > > On Mon, 9 May 2011, Avinash.H.M. wrote:
> > > 
> > > > On Thu, May 05, 2011 at 03:58:56PM -0600, Paul Walmsley wrote:
> > > > 
> > > > > > 
> > > > > > The patch is based on
> > > > > >  * git://git.kernel.org/pub/scm/linux/kernel/git/tmlind/linux-omap-2.6.git
> > > > > >  * master branch.
> > > > > >  * 9be20f0 commit. ( Linux-omap rebuilt: Updated to -rc5 )
> > > > > 
> > > > > Please base all your patches against mainline commits.  I suggest using 
> > > > > v2.6.39-rc6.
> > > > 
> > > > OK Paul. I will rebase against mainline, -rc6.
> > > 
> > > Ping.  Any progress?
> > 
> > Hi Paul ,
> > 
> > I am having the patch ready. Haven't had a chance to test. I will test
> > and send by tomorrow.
> 
> Hi Paul ,
> 
> I tested the patch on 3430 sdp today with the patch against 2.6.39-rc7.
> 
> I am seeing that the timeout is there during bootup event with this
> patch. Tried with the previous version (v3 , which was tested against
> the older commit and was confirmed to be working). With this also, i am
> seeing that the reset timeouts are still present. 
> 
> So, looks like there is something broken. First glance, i suspect that
> the register access may not be proper. Need to debug more. I ll debug

Hi Paul ,

This turned out to be the issue. The register access weren't proper. I2C
needs 16bit access. We need a flag 'HWMOD_16BIT_REG' added to i2c1,2,3
hwmods.

Andy Green has already submitted a patch for this and looks like it is
not merged. After adding this patch, now no more timeouts are seen. I ll
mention this dependency and submit v4.

thanks ,

- Avinash


> this and send the fix together.
> 
> Incase, if you want me to send this patch now and then send a seperate
> patch for the recent problem, do let me know.
> 
> thanks ,
> 
> - Avinash
> 
> > thanks ,
> > 
> > - Avinash
> > 
> > > 
> > > 
> > > - 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


[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