[PATCH 1/1] ARM: OMAP4: PM: Add check for avoiding dependency related update.

[Paul Walmsley]

Hello Abhijit,

On Wed, 23 Dec 2009, Abhijit Pagare wrote:

> A check is added for avoiding the sleep/wakeup dependency updates
> for OMAP4 as the structures for the dependencies are currently absent.

As a threshold matter, code and data added in temporary hack patches like 
this need to be marked with XXX or FIXME or something similar to indicate 
that these are not intended features, but rather, temporary lines that 
need to be removed in the near term.

But really, I'd rather not apply this patch.  The best way forward is to 
apply a patch that adds the OMAP4 wakeup and sleep dependencies 
themselves.  Have you had a chance to look at the series that I sent you a 
few weeks ago off-list that reorganizes the wakeup and sleep dependencies 
to determine if it is adequate for you to add the OMAP4 wakeup/sleep 
dependencies afterwards?


- Paul  

> 
> Signed-off-by: Abhijit Pagare <abhijitpagare at ti.com>
> Cc: Paul Walmsley <paul at pwsan.com>
> ---
> 
> Compiled and Boot Tested on OMAP4430 simulator, ES1 Chip & OMAP3430 SDP.
> And also Compiled for OMAP2430 and OMAP2420
> 
> This patch has a dependency on the earlier set of OMAP4 clock-domain patches.
> 
>  arch/arm/mach-omap2/clockdomain.c |   10 ++++++----
>  1 files changed, 6 insertions(+), 4 deletions(-)
> 
> diff --git a/arch/arm/mach-omap2/clockdomain.c b/arch/arm/mach-omap2/clockdomain.c
> index 8c9e78c..6ec25ee 100644
> --- a/arch/arm/mach-omap2/clockdomain.c
> +++ b/arch/arm/mach-omap2/clockdomain.c
> @@ -507,8 +507,9 @@ void omap2_clkdm_allow_idle(struct clockdomain *clkdm)
>  	pr_debug("clockdomain: enabling automatic idle transitions for %s\n",
>  		 clkdm->name);
>  
> -	if (atomic_read(&clkdm->usecount) > 0)
> -		_clkdm_add_autodeps(clkdm);
> +	if (!cpu_is_omap44xx())
> +		if (atomic_read(&clkdm->usecount) > 0)
> +			_clkdm_add_autodeps(clkdm);
>  
>  	_omap2_clkdm_set_hwsup(clkdm, 1);
>  
> @@ -540,8 +541,9 @@ void omap2_clkdm_deny_idle(struct clockdomain *clkdm)
>  
>  	_omap2_clkdm_set_hwsup(clkdm, 0);
>  
> -	if (atomic_read(&clkdm->usecount) > 0)
> -		_clkdm_del_autodeps(clkdm);
> +	if (!cpu_is_omap44xx())
> +		if (atomic_read(&clkdm->usecount) > 0)
> +			_clkdm_del_autodeps(clkdm);
>  }
>  
>  
> -- 
> 1.5.4.7
> 


- Paul