* [PATCH] ARM: OMAP2+: Fix: Don't wait for the idle status if modulemode is not supported
@ 2011-12-12 14:39 Benoit Cousson
2011-12-16 10:21 ` Paul Walmsley
0 siblings, 1 reply; 3+ messages in thread
From: Benoit Cousson @ 2011-12-12 14:39 UTC (permalink / raw)
To: linux-arm-kernel
If the module does not have any modulemode, the _disable_module function
will do nothing. There is then no point waiting for a idle status change.
It will remove the following warnings.
[ 0.331848] omap_hwmod: dmm: _wait_target_disable failed
[ 0.339935] omap_hwmod: emif_fw: _wait_target_disable failed
[ 0.348358] omap_hwmod: l3_main_1: _wait_target_disable failed
[ 0.356964] omap_hwmod: l3_main_2: _wait_target_disable failed
[ 0.365600] omap_hwmod: l4_abe: _wait_target_disable failed
[ 0.373931] omap_hwmod: l4_cfg: _wait_target_disable failed
[ 0.382263] omap_hwmod: l4_per: _wait_target_disable failed
[ 0.391113] omap_hwmod: l4_wkup: _wait_target_disable failed
[ 0.399536] omap_hwmod: dma_system: _wait_target_disable failed
[ 0.408325] omap_hwmod: dss_core: _wait_target_disable failed
[ 0.416839] omap_hwmod: dss_dispc: _wait_target_disable failed
[ 0.425445] omap_hwmod: dss_dsi1: _wait_target_disable failed
[ 0.433990] omap_hwmod: dss_dsi2: _wait_target_disable failed
[ 0.442504] omap_hwmod: dss_hdmi: _wait_target_disable failed
[ 0.451019] omap_hwmod: dss_rfbi: _wait_target_disable failed
[ 0.459564] omap_hwmod: dss_venc: _wait_target_disable failed
[ 0.489471] omap_hwmod: mailbox: _wait_target_disable failed
[ 0.505920] omap_hwmod: spinlock: _wait_target_disable failed
Note: For such module, the state is managed automatically by HW according
to clock domain transition. It is then not possible to wait for idle even
later in the _idle function since the status will change at clock domain
boundary.
Signed-off-by: Benoit Cousson <b-cousson@ti.com>
Cc: Paul Walmsley <paul@pwsan.com>
Cc: Rajendra Nayak <rnayak@ti.com>
---
arch/arm/mach-omap2/omap_hwmod.c | 29 ++++++++++++++++++++---------
1 files changed, 20 insertions(+), 9 deletions(-)
diff --git a/arch/arm/mach-omap2/omap_hwmod.c b/arch/arm/mach-omap2/omap_hwmod.c
index 207a2ff..dc9acac 100644
--- a/arch/arm/mach-omap2/omap_hwmod.c
+++ b/arch/arm/mach-omap2/omap_hwmod.c
@@ -710,16 +710,16 @@ static void _enable_module(struct omap_hwmod *oh)
* @oh: struct omap_hwmod *
*
* Disable the PRCM module mode related to the hwmod @oh.
- * No return value.
+ * Return EINVAL if the modulemode is not supported and 0 in case of success.
*/
-static void _disable_module(struct omap_hwmod *oh)
+int _disable_module(struct omap_hwmod *oh)
{
/* The module mode does not exist prior OMAP4 */
if (cpu_is_omap24xx() || cpu_is_omap34xx())
- return;
+ return -EINVAL;
if (!oh->clkdm || !oh->prcm.omap4.modulemode)
- return;
+ return -EINVAL;
pr_debug("omap_hwmod: %s: _disable_module\n", oh->name);
@@ -727,6 +727,8 @@ static void _disable_module(struct omap_hwmod *oh)
oh->clkdm->cm_inst,
oh->clkdm->clkdm_offs,
oh->prcm.omap4.clkctrl_offs);
+
+ return 0;
}
/**
@@ -1537,11 +1539,20 @@ static int _idle(struct omap_hwmod *oh)
if (oh->class->sysc)
_idle_sysc(oh);
_del_initiator_dep(oh, mpu_oh);
- _disable_module(oh);
- ret = _wait_target_disable(oh);
- if (ret)
- pr_warn("omap_hwmod: %s: _wait_target_disable failed\n",
- oh->name);
+ ret = _disable_module(oh);
+
+ /*
+ * If the module does not have any modulemode, the previous
+ * function will do nothing. There is then no point waiting
+ * for a idle status change.
+ */
+ if (!ret) {
+ ret = _wait_target_disable(oh);
+ if (ret)
+ pr_warn("omap_hwmod: %s: _wait_target_disable failed\n",
+ oh->name);
+ }
+
/*
* The module must be in idle mode before disabling any parents
* clocks. Otherwise, the parent clock might be disabled before
--
1.7.0.4
^ permalink raw reply related [flat|nested] 3+ messages in thread
* [PATCH] ARM: OMAP2+: Fix: Don't wait for the idle status if modulemode is not supported
2011-12-12 14:39 [PATCH] ARM: OMAP2+: Fix: Don't wait for the idle status if modulemode is not supported Benoit Cousson
@ 2011-12-16 10:21 ` Paul Walmsley
2011-12-16 10:28 ` Cousson, Benoit
0 siblings, 1 reply; 3+ messages in thread
From: Paul Walmsley @ 2011-12-16 10:21 UTC (permalink / raw)
To: linux-arm-kernel
Hi
On Mon, 12 Dec 2011, Benoit Cousson wrote:
> If the module does not have any modulemode, the _disable_module function
> will do nothing. There is then no point waiting for a idle status change.
>
> It will remove the following warnings.
>
> [ 0.331848] omap_hwmod: dmm: _wait_target_disable failed
> [ 0.339935] omap_hwmod: emif_fw: _wait_target_disable failed
> [ 0.348358] omap_hwmod: l3_main_1: _wait_target_disable failed
> [ 0.356964] omap_hwmod: l3_main_2: _wait_target_disable failed
> [ 0.365600] omap_hwmod: l4_abe: _wait_target_disable failed
> [ 0.373931] omap_hwmod: l4_cfg: _wait_target_disable failed
> [ 0.382263] omap_hwmod: l4_per: _wait_target_disable failed
> [ 0.391113] omap_hwmod: l4_wkup: _wait_target_disable failed
> [ 0.399536] omap_hwmod: dma_system: _wait_target_disable failed
> [ 0.408325] omap_hwmod: dss_core: _wait_target_disable failed
> [ 0.416839] omap_hwmod: dss_dispc: _wait_target_disable failed
> [ 0.425445] omap_hwmod: dss_dsi1: _wait_target_disable failed
> [ 0.433990] omap_hwmod: dss_dsi2: _wait_target_disable failed
> [ 0.442504] omap_hwmod: dss_hdmi: _wait_target_disable failed
> [ 0.451019] omap_hwmod: dss_rfbi: _wait_target_disable failed
> [ 0.459564] omap_hwmod: dss_venc: _wait_target_disable failed
> [ 0.489471] omap_hwmod: mailbox: _wait_target_disable failed
> [ 0.505920] omap_hwmod: spinlock: _wait_target_disable failed
>
> Note: For such module, the state is managed automatically by HW according
> to clock domain transition. It is then not possible to wait for idle even
> later in the _idle function since the status will change at clock domain
> boundary.
>
> Signed-off-by: Benoit Cousson <b-cousson@ti.com>
> Cc: Paul Walmsley <paul@pwsan.com>
> Cc: Rajendra Nayak <rnayak@ti.com>
Thanks. I've modified this patch somewhat - please let me know if you
have any comments.
- Paul
From: Benoit Cousson <b-cousson@ti.com>
Date: Mon, 12 Dec 2011 15:39:47 +0100
Subject: [PATCH] ARM: OMAP4: hwmod: Don't wait for the idle status if
modulemode is not supported
If the module does not have any modulemode, the _disable_module function
will do nothing. There is then no point waiting for a idle status change.
It will remove the following warnings.
[ 0.331848] omap_hwmod: dmm: _wait_target_disable failed
[ 0.339935] omap_hwmod: emif_fw: _wait_target_disable failed
[ 0.348358] omap_hwmod: l3_main_1: _wait_target_disable failed
[ 0.356964] omap_hwmod: l3_main_2: _wait_target_disable failed
[ 0.365600] omap_hwmod: l4_abe: _wait_target_disable failed
[ 0.373931] omap_hwmod: l4_cfg: _wait_target_disable failed
[ 0.382263] omap_hwmod: l4_per: _wait_target_disable failed
[ 0.391113] omap_hwmod: l4_wkup: _wait_target_disable failed
[ 0.399536] omap_hwmod: dma_system: _wait_target_disable failed
[ 0.408325] omap_hwmod: dss_core: _wait_target_disable failed
[ 0.416839] omap_hwmod: dss_dispc: _wait_target_disable failed
[ 0.425445] omap_hwmod: dss_dsi1: _wait_target_disable failed
[ 0.433990] omap_hwmod: dss_dsi2: _wait_target_disable failed
[ 0.442504] omap_hwmod: dss_hdmi: _wait_target_disable failed
[ 0.451019] omap_hwmod: dss_rfbi: _wait_target_disable failed
[ 0.459564] omap_hwmod: dss_venc: _wait_target_disable failed
[ 0.489471] omap_hwmod: mailbox: _wait_target_disable failed
[ 0.505920] omap_hwmod: spinlock: _wait_target_disable failed
Note: For such module, the state is managed automatically by HW according
to clock domain transition. It is then not possible to wait for idle even
later in the _idle function since the status will change at clock domain
boundary.
Signed-off-by: Benoit Cousson <b-cousson@ti.com>
Cc: Paul Walmsley <paul@pwsan.com>
Cc: Rajendra Nayak <rnayak@ti.com>
[paul at pwsan.com: renamed fns to indicate that they are OMAP4-only; moved
_wait_target_disable() into _disable_module(), removing duplicate code]
Signed-off-by: Paul Walmsley <paul@pwsan.com>
---
arch/arm/mach-omap2/omap_hwmod.c | 98 +++++++++++++++++++-------------------
1 files changed, 49 insertions(+), 49 deletions(-)
diff --git a/arch/arm/mach-omap2/omap_hwmod.c b/arch/arm/mach-omap2/omap_hwmod.c
index 207a2ff..ebace0f 100644
--- a/arch/arm/mach-omap2/omap_hwmod.c
+++ b/arch/arm/mach-omap2/omap_hwmod.c
@@ -706,27 +706,65 @@ static void _enable_module(struct omap_hwmod *oh)
}
/**
- * _disable_module - enable CLKCTRL modulemode on OMAP4
+ * _omap4_wait_target_disable - wait for a module to be disabled on OMAP4
+ * @oh: struct omap_hwmod *
+ *
+ * Wait for a module @oh to enter slave idle. Returns 0 if the module
+ * does not have an IDLEST bit or if the module successfully enters
+ * slave idle; otherwise, pass along the return value of the
+ * appropriate *_cm*_wait_module_idle() function.
+ */
+static int _omap4_wait_target_disable(struct omap_hwmod *oh)
+{
+ if (!cpu_is_omap44xx())
+ return 0;
+
+ if (!oh)
+ return -EINVAL;
+
+ if (oh->_int_flags & _HWMOD_NO_MPU_PORT)
+ return 0;
+
+ if (oh->flags & HWMOD_NO_IDLEST)
+ return 0;
+
+ return omap4_cminst_wait_module_idle(oh->clkdm->prcm_partition,
+ oh->clkdm->cm_inst,
+ oh->clkdm->clkdm_offs,
+ oh->prcm.omap4.clkctrl_offs);
+}
+
+/**
+ * _omap4_disable_module - enable CLKCTRL modulemode on OMAP4
* @oh: struct omap_hwmod *
*
* Disable the PRCM module mode related to the hwmod @oh.
- * No return value.
+ * Return EINVAL if the modulemode is not supported and 0 in case of success.
*/
-static void _disable_module(struct omap_hwmod *oh)
+static int _omap4_disable_module(struct omap_hwmod *oh)
{
+ int v;
+
/* The module mode does not exist prior OMAP4 */
- if (cpu_is_omap24xx() || cpu_is_omap34xx())
- return;
+ if (!cpu_is_omap44xx())
+ return -EINVAL;
if (!oh->clkdm || !oh->prcm.omap4.modulemode)
- return;
+ return -EINVAL;
- pr_debug("omap_hwmod: %s: _disable_module\n", oh->name);
+ pr_debug("omap_hwmod: %s: %s\n", oh->name, __func__);
omap4_cminst_module_disable(oh->clkdm->prcm_partition,
oh->clkdm->cm_inst,
oh->clkdm->clkdm_offs,
oh->prcm.omap4.clkctrl_offs);
+
+ v = _omap4_wait_target_disable(oh);
+ if (v)
+ pr_warn("omap_hwmod: %s: _wait_target_disable failed\n",
+ oh->name);
+
+ return 0;
}
/**
@@ -1153,36 +1191,6 @@ static int _wait_target_ready(struct omap_hwmod *oh)
}
/**
- * _wait_target_disable - wait for a module to be disabled
- * @oh: struct omap_hwmod *
- *
- * Wait for a module @oh to enter slave idle. Returns 0 if the module
- * does not have an IDLEST bit or if the module successfully enters
- * slave idle; otherwise, pass along the return value of the
- * appropriate *_cm*_wait_module_idle() function.
- */
-static int _wait_target_disable(struct omap_hwmod *oh)
-{
- /* TODO: For now just handle OMAP4+ */
- if (cpu_is_omap24xx() || cpu_is_omap34xx())
- return 0;
-
- if (!oh)
- return -EINVAL;
-
- if (oh->_int_flags & _HWMOD_NO_MPU_PORT)
- return 0;
-
- if (oh->flags & HWMOD_NO_IDLEST)
- return 0;
-
- return omap4_cminst_wait_module_idle(oh->clkdm->prcm_partition,
- oh->clkdm->cm_inst,
- oh->clkdm->clkdm_offs,
- oh->prcm.omap4.clkctrl_offs);
-}
-
-/**
* _lookup_hardreset - fill register bit info for this hwmod/reset line
* @oh: struct omap_hwmod *
* @name: name of the reset line in the context of this hwmod
@@ -1524,8 +1532,6 @@ static int _enable(struct omap_hwmod *oh)
*/
static int _idle(struct omap_hwmod *oh)
{
- int ret;
-
pr_debug("omap_hwmod: %s: idling\n", oh->name);
if (oh->_state != _HWMOD_STATE_ENABLED) {
@@ -1537,11 +1543,9 @@ static int _idle(struct omap_hwmod *oh)
if (oh->class->sysc)
_idle_sysc(oh);
_del_initiator_dep(oh, mpu_oh);
- _disable_module(oh);
- ret = _wait_target_disable(oh);
- if (ret)
- pr_warn("omap_hwmod: %s: _wait_target_disable failed\n",
- oh->name);
+
+ _omap4_disable_module(oh);
+
/*
* The module must be in idle mode before disabling any parents
* clocks. Otherwise, the parent clock might be disabled before
@@ -1642,11 +1646,7 @@ static int _shutdown(struct omap_hwmod *oh)
if (oh->_state == _HWMOD_STATE_ENABLED) {
_del_initiator_dep(oh, mpu_oh);
/* XXX what about the other system initiators here? dma, dsp */
- _disable_module(oh);
- ret = _wait_target_disable(oh);
- if (ret)
- pr_warn("omap_hwmod: %s: _wait_target_disable failed\n",
- oh->name);
+ _omap4_disable_module(oh);
_disable_clocks(oh);
if (oh->clkdm)
clkdm_hwmod_disable(oh->clkdm, oh);
--
1.7.7.3
^ permalink raw reply related [flat|nested] 3+ messages in thread
* [PATCH] ARM: OMAP2+: Fix: Don't wait for the idle status if modulemode is not supported
2011-12-16 10:21 ` Paul Walmsley
@ 2011-12-16 10:28 ` Cousson, Benoit
0 siblings, 0 replies; 3+ messages in thread
From: Cousson, Benoit @ 2011-12-16 10:28 UTC (permalink / raw)
To: linux-arm-kernel
Hi Paul,
On 12/16/2011 11:21 AM, Paul Walmsley wrote:
> Hi
>
> On Mon, 12 Dec 2011, Benoit Cousson wrote:
>
>> If the module does not have any modulemode, the _disable_module function
>> will do nothing. There is then no point waiting for a idle status change.
>>
>> It will remove the following warnings.
>>
>> [ 0.331848] omap_hwmod: dmm: _wait_target_disable failed
>> [ 0.339935] omap_hwmod: emif_fw: _wait_target_disable failed
>> [ 0.348358] omap_hwmod: l3_main_1: _wait_target_disable failed
>> [ 0.356964] omap_hwmod: l3_main_2: _wait_target_disable failed
>> [ 0.365600] omap_hwmod: l4_abe: _wait_target_disable failed
>> [ 0.373931] omap_hwmod: l4_cfg: _wait_target_disable failed
>> [ 0.382263] omap_hwmod: l4_per: _wait_target_disable failed
>> [ 0.391113] omap_hwmod: l4_wkup: _wait_target_disable failed
>> [ 0.399536] omap_hwmod: dma_system: _wait_target_disable failed
>> [ 0.408325] omap_hwmod: dss_core: _wait_target_disable failed
>> [ 0.416839] omap_hwmod: dss_dispc: _wait_target_disable failed
>> [ 0.425445] omap_hwmod: dss_dsi1: _wait_target_disable failed
>> [ 0.433990] omap_hwmod: dss_dsi2: _wait_target_disable failed
>> [ 0.442504] omap_hwmod: dss_hdmi: _wait_target_disable failed
>> [ 0.451019] omap_hwmod: dss_rfbi: _wait_target_disable failed
>> [ 0.459564] omap_hwmod: dss_venc: _wait_target_disable failed
>> [ 0.489471] omap_hwmod: mailbox: _wait_target_disable failed
>> [ 0.505920] omap_hwmod: spinlock: _wait_target_disable failed
>>
>> Note: For such module, the state is managed automatically by HW according
>> to clock domain transition. It is then not possible to wait for idle even
>> later in the _idle function since the status will change at clock domain
>> boundary.
>>
>> Signed-off-by: Benoit Cousson<b-cousson@ti.com>
>> Cc: Paul Walmsley<paul@pwsan.com>
>> Cc: Rajendra Nayak<rnayak@ti.com>
>
> Thanks. I've modified this patch somewhat - please let me know if you
> have any comments.
Cool, it is indeed much better like that since that code was OMAP4 only.
Thanks,
Benoit
> From: Benoit Cousson<b-cousson@ti.com>
> Date: Mon, 12 Dec 2011 15:39:47 +0100
> Subject: [PATCH] ARM: OMAP4: hwmod: Don't wait for the idle status if
> modulemode is not supported
>
> If the module does not have any modulemode, the _disable_module function
> will do nothing. There is then no point waiting for a idle status change.
>
> It will remove the following warnings.
>
> [ 0.331848] omap_hwmod: dmm: _wait_target_disable failed
> [ 0.339935] omap_hwmod: emif_fw: _wait_target_disable failed
> [ 0.348358] omap_hwmod: l3_main_1: _wait_target_disable failed
> [ 0.356964] omap_hwmod: l3_main_2: _wait_target_disable failed
> [ 0.365600] omap_hwmod: l4_abe: _wait_target_disable failed
> [ 0.373931] omap_hwmod: l4_cfg: _wait_target_disable failed
> [ 0.382263] omap_hwmod: l4_per: _wait_target_disable failed
> [ 0.391113] omap_hwmod: l4_wkup: _wait_target_disable failed
> [ 0.399536] omap_hwmod: dma_system: _wait_target_disable failed
> [ 0.408325] omap_hwmod: dss_core: _wait_target_disable failed
> [ 0.416839] omap_hwmod: dss_dispc: _wait_target_disable failed
> [ 0.425445] omap_hwmod: dss_dsi1: _wait_target_disable failed
> [ 0.433990] omap_hwmod: dss_dsi2: _wait_target_disable failed
> [ 0.442504] omap_hwmod: dss_hdmi: _wait_target_disable failed
> [ 0.451019] omap_hwmod: dss_rfbi: _wait_target_disable failed
> [ 0.459564] omap_hwmod: dss_venc: _wait_target_disable failed
> [ 0.489471] omap_hwmod: mailbox: _wait_target_disable failed
> [ 0.505920] omap_hwmod: spinlock: _wait_target_disable failed
>
> Note: For such module, the state is managed automatically by HW according
> to clock domain transition. It is then not possible to wait for idle even
> later in the _idle function since the status will change at clock domain
> boundary.
>
> Signed-off-by: Benoit Cousson<b-cousson@ti.com>
> Cc: Paul Walmsley<paul@pwsan.com>
> Cc: Rajendra Nayak<rnayak@ti.com>
> [paul at pwsan.com: renamed fns to indicate that they are OMAP4-only; moved
> _wait_target_disable() into _disable_module(), removing duplicate code]
> Signed-off-by: Paul Walmsley<paul@pwsan.com>
> ---
> arch/arm/mach-omap2/omap_hwmod.c | 98 +++++++++++++++++++-------------------
> 1 files changed, 49 insertions(+), 49 deletions(-)
>
> diff --git a/arch/arm/mach-omap2/omap_hwmod.c b/arch/arm/mach-omap2/omap_hwmod.c
> index 207a2ff..ebace0f 100644
> --- a/arch/arm/mach-omap2/omap_hwmod.c
> +++ b/arch/arm/mach-omap2/omap_hwmod.c
> @@ -706,27 +706,65 @@ static void _enable_module(struct omap_hwmod *oh)
> }
>
> /**
> - * _disable_module - enable CLKCTRL modulemode on OMAP4
> + * _omap4_wait_target_disable - wait for a module to be disabled on OMAP4
> + * @oh: struct omap_hwmod *
> + *
> + * Wait for a module @oh to enter slave idle. Returns 0 if the module
> + * does not have an IDLEST bit or if the module successfully enters
> + * slave idle; otherwise, pass along the return value of the
> + * appropriate *_cm*_wait_module_idle() function.
> + */
> +static int _omap4_wait_target_disable(struct omap_hwmod *oh)
> +{
> + if (!cpu_is_omap44xx())
> + return 0;
> +
> + if (!oh)
> + return -EINVAL;
> +
> + if (oh->_int_flags& _HWMOD_NO_MPU_PORT)
> + return 0;
> +
> + if (oh->flags& HWMOD_NO_IDLEST)
> + return 0;
> +
> + return omap4_cminst_wait_module_idle(oh->clkdm->prcm_partition,
> + oh->clkdm->cm_inst,
> + oh->clkdm->clkdm_offs,
> + oh->prcm.omap4.clkctrl_offs);
> +}
> +
> +/**
> + * _omap4_disable_module - enable CLKCTRL modulemode on OMAP4
> * @oh: struct omap_hwmod *
> *
> * Disable the PRCM module mode related to the hwmod @oh.
> - * No return value.
> + * Return EINVAL if the modulemode is not supported and 0 in case of success.
> */
> -static void _disable_module(struct omap_hwmod *oh)
> +static int _omap4_disable_module(struct omap_hwmod *oh)
> {
> + int v;
> +
> /* The module mode does not exist prior OMAP4 */
> - if (cpu_is_omap24xx() || cpu_is_omap34xx())
> - return;
> + if (!cpu_is_omap44xx())
> + return -EINVAL;
>
> if (!oh->clkdm || !oh->prcm.omap4.modulemode)
> - return;
> + return -EINVAL;
>
> - pr_debug("omap_hwmod: %s: _disable_module\n", oh->name);
> + pr_debug("omap_hwmod: %s: %s\n", oh->name, __func__);
>
> omap4_cminst_module_disable(oh->clkdm->prcm_partition,
> oh->clkdm->cm_inst,
> oh->clkdm->clkdm_offs,
> oh->prcm.omap4.clkctrl_offs);
> +
> + v = _omap4_wait_target_disable(oh);
> + if (v)
> + pr_warn("omap_hwmod: %s: _wait_target_disable failed\n",
> + oh->name);
> +
> + return 0;
> }
>
> /**
> @@ -1153,36 +1191,6 @@ static int _wait_target_ready(struct omap_hwmod *oh)
> }
>
> /**
> - * _wait_target_disable - wait for a module to be disabled
> - * @oh: struct omap_hwmod *
> - *
> - * Wait for a module @oh to enter slave idle. Returns 0 if the module
> - * does not have an IDLEST bit or if the module successfully enters
> - * slave idle; otherwise, pass along the return value of the
> - * appropriate *_cm*_wait_module_idle() function.
> - */
> -static int _wait_target_disable(struct omap_hwmod *oh)
> -{
> - /* TODO: For now just handle OMAP4+ */
> - if (cpu_is_omap24xx() || cpu_is_omap34xx())
> - return 0;
> -
> - if (!oh)
> - return -EINVAL;
> -
> - if (oh->_int_flags& _HWMOD_NO_MPU_PORT)
> - return 0;
> -
> - if (oh->flags& HWMOD_NO_IDLEST)
> - return 0;
> -
> - return omap4_cminst_wait_module_idle(oh->clkdm->prcm_partition,
> - oh->clkdm->cm_inst,
> - oh->clkdm->clkdm_offs,
> - oh->prcm.omap4.clkctrl_offs);
> -}
> -
> -/**
> * _lookup_hardreset - fill register bit info for this hwmod/reset line
> * @oh: struct omap_hwmod *
> * @name: name of the reset line in the context of this hwmod
> @@ -1524,8 +1532,6 @@ static int _enable(struct omap_hwmod *oh)
> */
> static int _idle(struct omap_hwmod *oh)
> {
> - int ret;
> -
> pr_debug("omap_hwmod: %s: idling\n", oh->name);
>
> if (oh->_state != _HWMOD_STATE_ENABLED) {
> @@ -1537,11 +1543,9 @@ static int _idle(struct omap_hwmod *oh)
> if (oh->class->sysc)
> _idle_sysc(oh);
> _del_initiator_dep(oh, mpu_oh);
> - _disable_module(oh);
> - ret = _wait_target_disable(oh);
> - if (ret)
> - pr_warn("omap_hwmod: %s: _wait_target_disable failed\n",
> - oh->name);
> +
> + _omap4_disable_module(oh);
> +
> /*
> * The module must be in idle mode before disabling any parents
> * clocks. Otherwise, the parent clock might be disabled before
> @@ -1642,11 +1646,7 @@ static int _shutdown(struct omap_hwmod *oh)
> if (oh->_state == _HWMOD_STATE_ENABLED) {
> _del_initiator_dep(oh, mpu_oh);
> /* XXX what about the other system initiators here? dma, dsp */
> - _disable_module(oh);
> - ret = _wait_target_disable(oh);
> - if (ret)
> - pr_warn("omap_hwmod: %s: _wait_target_disable failed\n",
> - oh->name);
> + _omap4_disable_module(oh);
> _disable_clocks(oh);
> if (oh->clkdm)
> clkdm_hwmod_disable(oh->clkdm, oh);
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2011-12-16 10:28 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-12-12 14:39 [PATCH] ARM: OMAP2+: Fix: Don't wait for the idle status if modulemode is not supported Benoit Cousson
2011-12-16 10:21 ` Paul Walmsley
2011-12-16 10:28 ` Cousson, Benoit
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).