From: khilman@ti.com (Kevin Hilman)
Date: Fri, 08 Jul 2011 07:19:32 -0700
Subject: [PATCH v4 3/7] OMAP2+: PM: Initialise sleep_switch to a non-valid
	value
In-Reply-To: <alpine.DEB.2.00.1107071635230.2647@utopia.booyaka.com> (Paul
	Walmsley's message of "Thu, 7 Jul 2011 16:36:50 -0600 (MDT)")
References: <1309555626-22608-1-git-send-email-b-cousson@ti.com>
	<1309555626-22608-4-git-send-email-b-cousson@ti.com>
	<alpine.DEB.2.00.1107071635230.2647@utopia.booyaka.com>
Message-ID: <87wrfsc0sr.fsf@ti.com>
To: linux-arm-kernel@lists.infradead.org
List-Id: linux-arm-kernel.lists.infradead.org

Paul Walmsley <paul@pwsan.com> writes:

> cc'ing Kevin, since this touches mach-omap2/pm.c - an ack would be great 
> if you have the chance
>
> On Fri, 1 Jul 2011, Benoit Cousson wrote:
>
>> From: Rajendra Nayak <rnayak@ti.com>
>> 
>> sleep_switch which is initialised to 0 in omap_set_pwrdm_state
>> happens to be a valid sleep_switch type (FORCEWAKEUP_SWITCH)
>> which are defined as:
>> 
>>  #define FORCEWAKEUP_SWITCH      0
>>  #define LOWPOWERSTATE_SWITCH    1
>> 
>> This causes the function to wrongly program some clock domains
>> even when the Powerdomain is in ON state.
>> 
>> Signed-off-by: Rajendra Nayak <rnayak@ti.com>
>> Cc: Paul Walmsley <paul@pwsan.com>


Acked-by: Kevin Hilman <khilman@ti.com>


>
> Thanks, this patch has been queued for 3.1 at 
> git://git.pwsan.com/linux-2.6 in the 'powerdomain_fixes_3.1' branch.
>
>
> - Paul