Re: [PATCH 05/10] ARM: OMAP2+: PM/powerdomain: move omap_set_pwrdm_state() to powerdomain code

2013-01-29 Thread Paul Walmsley
Hi, (redacted irrelevant code sections) On Wed, 12 Dec 2012, Jean Pihet wrote: > On Sun, Dec 9, 2012 at 2:23 AM, Paul Walmsley wrote: > > > -/** > > - * pwrdm_set_lowpwrstchange - Request a low power state change > > - * @pwrdm: struct powerdomain * > > - * > > - * Allows a powerdomain to tran

Re: [PATCH 05/10] ARM: OMAP2+: PM/powerdomain: move omap_set_pwrdm_state() to powerdomain code

2013-01-09 Thread Russell King - ARM Linux
On Wed, Dec 12, 2012 at 03:51:49PM +0530, Vaibhav Hiremath wrote: > > -int omap_set_pwrdm_state(struct powerdomain *pwrdm, u32 pwrst) > > -{ > > - u8 curr_pwrst, next_pwrst; > > - int sleep_switch = -1, ret = 0, hwsup = 0; > > - > > - if (!pwrdm || IS_ERR(pwrdm)) > > You can use IS_ERR_OR_NU

Re: [PATCH 05/10] ARM: OMAP2+: PM/powerdomain: move omap_set_pwrdm_state() to powerdomain code

2012-12-12 Thread Jean Pihet
Hi Paul, -resending in plain text only, sorry about that- On Sun, Dec 9, 2012 at 2:23 AM, Paul Walmsley wrote: > > Move omap_set_pwrdm_state() from the PM code to the powerdomain code, > and refactor it to split it up into several functions. A subsequent patch > will rename it to conform with t

Re: [PATCH 05/10] ARM: OMAP2+: PM/powerdomain: move omap_set_pwrdm_state() to powerdomain code

2012-12-12 Thread Vaibhav Hiremath
Minor comment below, On 12/9/2012 6:53 AM, Paul Walmsley wrote: > Move omap_set_pwrdm_state() from the PM code to the powerdomain code, > and refactor it to split it up into several functions. A subsequent patch > will rename it to conform with the existing powerdomain function names. > > Signed

[PATCH 05/10] ARM: OMAP2+: PM/powerdomain: move omap_set_pwrdm_state() to powerdomain code

2012-12-08 Thread Paul Walmsley
Move omap_set_pwrdm_state() from the PM code to the powerdomain code, and refactor it to split it up into several functions. A subsequent patch will rename it to conform with the existing powerdomain function names. Signed-off-by: Paul Walmsley Cc: Jean Pihet Cc: Kevin Hilman --- arch/arm/mac