[PATCH] ARM: OMAP4 clock domain: Add check for avoiding dependency related update.
Pagare, Abhijit
abhijitpagare at ti.com
Thu Feb 11 00:02:11 EST 2010
Paul,
>-----Original Message-----
>From: Paul Walmsley [mailto:paul at pwsan.com]
>Sent: Thursday, February 11, 2010 5:44 AM
>To: Pagare, Abhijit
>Cc: linux-omap at vger.kernel.org; linux-arm-kernel at lists.infradead.org
>Subject: Re: [PATCH] ARM: OMAP4 clock domain: Add check for avoiding
>dependency related update.
>
>Hi Abhijit,
>
>On Wed, 10 Feb 2010, 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.
>>
>> Signed-off-by: Abhijit Pagare <abhijitpagare at ti.com>
>> Cc: Paul Walmsley <paul at pwsan.com>
>
>Thanks, I've made the following changes to the patch to try to ensure that
>this doesn't slip through the cracks, and queued it for 2.6.34. If you
>have any comments on the revised patch, please let me know.
Looks neat. Thanks.
-Abhijit
>
>
>- Paul
>
>From: Abhijit Pagare <abhijitpagare at ti.com>
>Date: Wed, 10 Feb 2010 13:42:03 +0530
>Subject: [PATCH] ARM: OMAP4 clock domain: Add check for avoiding dependency
>related update.
>
>A check is added for avoiding the sleep/wakeup dependency updates
>for OMAP4 as the structures for the dependencies are currently absent.
>
>Signed-off-by: Abhijit Pagare <abhijitpagare at ti.com>
>[paul at pwsan.com: added warnings, explanatory comment, copyright update]
>Signed-off-by: Paul Walmsley <paul at pwsan.com>
>---
> arch/arm/mach-omap2/clockdomain.c | 30 ++++++++++++++++++++++++------
> 1 files changed, 24 insertions(+), 6 deletions(-)
>
>diff --git a/arch/arm/mach-omap2/clockdomain.c b/arch/arm/mach-
>omap2/clockdomain.c
>index a38a615..de4278c 100644
>--- a/arch/arm/mach-omap2/clockdomain.c
>+++ b/arch/arm/mach-omap2/clockdomain.c
>@@ -1,8 +1,8 @@
> /*
> * OMAP2/3/4 clockdomain framework functions
> *
>- * Copyright (C) 2008-2009 Texas Instruments, Inc.
>- * Copyright (C) 2008-2009 Nokia Corporation
>+ * Copyright (C) 2008-2010 Texas Instruments, Inc.
>+ * Copyright (C) 2008-2010 Nokia Corporation
> *
> * Written by Paul Walmsley and Jouni Högander
> * Added OMAP4 specific support by Abhijit Pagare <abhijitpagare at ti.com>
>@@ -891,8 +891,17 @@ 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);
>+ /*
>+ * XXX This should be removed once TI adds wakeup/sleep
>+ * dependency code and data for OMAP4.
>+ */
>+ if (cpu_is_omap44xx()) {
>+ WARN_ONCE(1, "clockdomain: OMAP4 wakeup/sleep dependency "
>+ "support is not yet implemented\n");
>+ } else {
>+ if (atomic_read(&clkdm->usecount) > 0)
>+ _clkdm_add_autodeps(clkdm);
>+ }
>
> _omap2_clkdm_set_hwsup(clkdm, 1);
>
>@@ -924,8 +933,17 @@ void omap2_clkdm_deny_idle(struct clockdomain *clkdm)
>
> _omap2_clkdm_set_hwsup(clkdm, 0);
>
>- if (atomic_read(&clkdm->usecount) > 0)
>- _clkdm_del_autodeps(clkdm);
>+ /*
>+ * XXX This should be removed once TI adds wakeup/sleep
>+ * dependency code and data for OMAP4.
>+ */
>+ if (cpu_is_omap44xx()) {
>+ WARN_ONCE(1, "clockdomain: OMAP4 wakeup/sleep dependency "
>+ "support is not yet implemented\n");
>+ } else {
>+ if (atomic_read(&clkdm->usecount) > 0)
>+ _clkdm_del_autodeps(clkdm);
>+ }
> }
>
>
>--
>1.6.6.GIT
More information about the linux-arm-kernel
mailing list