From mboxrd@z Thu Jan 1 00:00:00 1970 From: t-kristo@ti.com (Tero Kristo) Date: Thu, 16 Feb 2012 15:15:59 +0200 Subject: [PATCHv2 8/8] arm: omap3: prevent per_clkdm from attempting manual domain transitions In-Reply-To: References: <1329320274-481-1-git-send-email-t-kristo@ti.com> <1329320274-481-9-git-send-email-t-kristo@ti.com> <87pqdf99c2.fsf@ti.com> <1329382652.4102.386.camel@sokoban> Message-ID: <1329398159.4102.391.camel@sokoban> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Thu, 2012-02-16 at 15:27 +0530, Shilimkar, Santosh wrote: > On Thu, Feb 16, 2012 at 2:27 PM, Tero Kristo wrote: > > On Wed, 2012-02-15 at 11:37 -0800, Kevin Hilman wrote: > >> Tero Kristo writes: > >> > >> > Attempting this will cause problems especially with off-mode enabled. > >> > >> Please be more verbose about the problems seen, and the root cause(s). > >> > > > > I was actually looking forward for some help with this commit message, > > as I am still not quite sure what is going on in here. :) Here is the > > log for suspend (btw, cam_pwrdm does not go to off in mainline yet, but > > I think that is probably fixed by the patch from Paul, > > omap_set_pwrdm_state() does not work properly.) The warning comes out > > after wakeup from off-mode, and it is triggered by the disabling of > > autodeps before off-mode entry. > > > This mostly indicates that one of the per clock-domain module > clock turning ON seems to be not working well with auto deps > disabled. This leads to interconnect violation. > > if not tried already, can you put the per_clockdomain in SW_WKUP > in the low power code early resume path and see if this > error goes away. This seems to get rid of the dump also. It looks like some driver resume is not behaving nicely, I am trying to pinpoint the culprit currently and see whether it can provide more info. -Tero