From mboxrd@z Thu Jan 1 00:00:00 1970 From: Santosh Shilimkar Subject: RE: Integration branch base switchover to Tony's omap-for-linus branch Date: Wed, 9 Mar 2011 10:38:02 +0530 Message-ID: <1f74aa62f12f632acff9d31082a1ec9e@mail.gmail.com> References: <3f332cbd75f835ca119bdeccd72c4bb2@mail.gmail.com> <4D6F89FB.7080201@ti.com> <4D70F25C.4040206@ti.com> <58ec7480fe87b4d64c33cb2c4454a805@mail.gmail.com> <02124a94c82909eff5a589819ec86357@mail.gmail.com> <4D76591A.7060508@ti.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Return-path: Received: from na3sys009aog104.obsmtp.com ([74.125.149.73]:48497 "EHLO na3sys009aog104.obsmtp.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750788Ab1CIFIG (ORCPT ); Wed, 9 Mar 2011 00:08:06 -0500 Received: by mail-qy0-f182.google.com with SMTP id 27so149232qyk.20 for ; Tue, 08 Mar 2011 21:08:04 -0800 (PST) In-Reply-To: <4D76591A.7060508@ti.com> Sender: linux-omap-owner@vger.kernel.org List-Id: linux-omap@vger.kernel.org To: Benoit Cousson Cc: Paul Walmsley , linux-omap@vger.kernel.org, Rajendra Nayak , Kevin Hilman > -----Original Message----- > From: Cousson, Benoit [mailto:b-cousson@ti.com] > Sent: Tuesday, March 08, 2011 9:58 PM > To: Shilimkar, Santosh > Cc: Paul Walmsley; linux-omap@vger.kernel.org; Nayak, Rajendra; > Hilman, Kevin > Subject: Re: Integration branch base switchover to Tony's omap-for- > linus branch > > On 3/8/2011 4:16 PM, Shilimkar, Santosh wrote: > > Paul, > >> -----Original Message----- > >> From: Santosh Shilimkar [mailto:santosh.shilimkar@ti.com] > >> Sent: Friday, March 04, 2011 10:14 PM > >> To: Rajendra Nayak > >> Cc: linux-omap@vger.kernel.org; Benoit Cousson; Paul Walmsley > >> Subject: RE: Integration branch base switchover to Tony's omap- > for- > >> linus branch > >> > > > > [....] > > > >>> Some more debugging by the Hardware team and analyzing > >>> the register dumps showed that the I2C_WE register of the i2c > >>> modules needs to be programmed correctly for i2c wakeups to > >>> function as expected. > >>> This turned out to be the root cause for the i2c issues observed > >>> by clearing the staticdeps and a patch has been posted > >>> to address this... > >>> http://marc.info/?l=linux-omap&m=129924557219813&w=2 > >>> > >>>> > >>>> Also some more testing showed up a lockup in suspend on OMAP4 > >>> which I > >>>> could narrow down to a similar case with GPT1. Either keeping > >> the > >>>> staticdep between MPU and L4_WKUP _or_ forcing GPT1 to use > >>> software > >>>> idle control seems to help. > >>> > >>> This however is still not rootcaused and is not the same as the > >>> issue > >>> seen with i2c as the WE for GPT1 is already programmed for > >> enabling > >>> wakeup. > >>> > >>> The one way to fix this for now is to put GPT1 block in software > >>> controlled idle as was done by your test patch for i2c. > >>> > >> I tried all the floating patches for static dependency. It did > >> worked when CPU RET was tried but with MPU OFF I see the hang. > >> > >> Below is the global hack I have which works as expected for > >> all cases. > >> > > Thanks to Rajendra for isolating the OMAP4 MPU OFF issue > > with static dependency series. The issue is isolated > > to MPU<-> EMIF clock dependency. There issue appears > > if we clear this static dependency. > > > > I have posted a patch to work-around this issue till its > > being root-caused with help of hardware team so that > > OMAP4 PM series continue to work. > > > > 'OMAP4: PM: Set static dependency between MPUSS and EMIF' > > http://www.listware.net/201103/linux-omap/2628-patch-omap4-pm-set- > static-d > > ependency-between-mpuss-and-emif.html > > Cool, so the timer1 fix is not longer needed? > We still need timer fix :( In summary we need below fixes which came into light with Static depdency series. 1. I2C: driver fix posted by Rajendra to enable I2C_WE. 2. Timer1: disable hardware-controlled idle 3. clockdomain: Follow PRCM recommended enable sequence 4. MPUSS<-> EMIF static dependency fix. With above 4 patches instead of global hack of not clearing static dependency, I tested OMAP4 PM series and it works as expected. Regards, Santosh