From mboxrd@z Thu Jan  1 00:00:00 1970
From: "Avinash.H.M." <avinashhm@ti.com>
Subject: Re: [PATCH v3] OMAP2/3: hwmod: fix the i2c-reset timeout during
 bootup
Date: Fri, 20 May 2011 21:08:15 +0530
Message-ID: <20110520153815.GB11915@avinash-laptop>
References: <1304328111-15999-1-git-send-email-avinashhm@ti.com>
 <alpine.DEB.2.00.1105051548210.30136@utopia.booyaka.com>
 <20110509124842.GA25211@avinash-laptop>
 <alpine.DEB.2.00.1105181021590.4550@utopia.booyaka.com>
 <20110518162542.GB3026@avinash-laptop>
 <20110520150216.GA11915@avinash-laptop>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Return-path: <linux-omap-owner@vger.kernel.org>
Received: from na3sys009aog116.obsmtp.com ([74.125.149.240]:37388 "EHLO
	na3sys009aog116.obsmtp.com" rhost-flags-OK-OK-OK-OK)
	by vger.kernel.org with ESMTP id S932752Ab1ETPi1 (ORCPT
	<rfc822;linux-omap@vger.kernel.org>);
	Fri, 20 May 2011 11:38:27 -0400
Received: by yih10 with SMTP id 10so1995885yih.39
        for <linux-omap@vger.kernel.org>; Fri, 20 May 2011 08:38:25 -0700 (PDT)
Content-Disposition: inline
In-Reply-To: <20110520150216.GA11915@avinash-laptop>
Sender: linux-omap-owner@vger.kernel.org
List-Id: linux-omap@vger.kernel.org
To: Paul Walmsley <paul@pwsan.com>, linux-omap@vger.kernel.org, linux-arm-kernel@lists.infradead.org, Rajendra Nayak <rnayak@ti.com>, Benoit Cousson <b-cousson@ti.com>, Kevin Hilman <khi>

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