Re: [PATCH] ARM: OMAP2+: Fix: Don't wait for the idle status if modulemode is not supported
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-cous...@ti.com Cc: Paul Walmsley p...@pwsan.com Cc: Rajendra Nayak rna...@ti.com Thanks. I've modified this patch somewhat - please let me know if you have any comments. - Paul From: Benoit Cousson b-cous...@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-cous...@ti.com Cc: Paul Walmsley p...@pwsan.com Cc: Rajendra Nayak rna...@ti.com [p...@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 p...@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) +
Re: [PATCH] ARM: OMAP2+: Fix: Don't wait for the idle status if modulemode is not supported
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 Coussonb-cous...@ti.com Cc: Paul Walmsleyp...@pwsan.com Cc: Rajendra Nayakrna...@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 Coussonb-cous...@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 Coussonb-cous...@ti.com Cc: Paul Walmsleyp...@pwsan.com Cc: Rajendra Nayakrna...@ti.com [p...@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 Walmsleyp...@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
[PATCH] ARM: OMAP2+: Fix: 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-cous...@ti.com Cc: Paul Walmsley p...@pwsan.com Cc: Rajendra Nayak rna...@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 -- To unsubscribe from this list: send the line unsubscribe linux-omap in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html