* [PATCH] ARM: AM33xx: hwmod: Add INIT_NO_IDLE flag for debugss hwmod
@ 2014-04-09 5:47 Lokesh Vutla
2014-04-11 18:00 ` Paul Walmsley
0 siblings, 1 reply; 5+ messages in thread
From: Lokesh Vutla @ 2014-04-09 5:47 UTC (permalink / raw)
To: linux-arm-kernel
During boot, when hwmod tries to cut clocks for debugss it always
gets stuck in transition state and throws the following warning:
[ 0.139581] omap_hwmod: debugss: _wait_target_disable failed
As per the information provided by folks, clocks to debugss cannot be cut.
So adding HWMOD_INIT_NO_IDLE flag to debugss hwmod.
Signed-off-by: Lokesh Vutla <lokeshvutla@ti•com>
---
Tested on BeagleBone Black.
arch/arm/mach-omap2/omap_hwmod_33xx_data.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm/mach-omap2/omap_hwmod_33xx_data.c b/arch/arm/mach-omap2/omap_hwmod_33xx_data.c
index 6b406ca..ec18ce0 100644
--- a/arch/arm/mach-omap2/omap_hwmod_33xx_data.c
+++ b/arch/arm/mach-omap2/omap_hwmod_33xx_data.c
@@ -222,6 +222,7 @@ static struct omap_hwmod am33xx_debugss_hwmod = {
.name = "debugss",
.class = &am33xx_debugss_hwmod_class,
.clkdm_name = "l3_aon_clkdm",
+ .flags = HWMOD_INIT_NO_IDLE,
.main_clk = "trace_clk_div_ck",
.prcm = {
.omap4 = {
--
1.7.9.5
^ permalink raw reply related [flat|nested] 5+ messages in thread* [PATCH] ARM: AM33xx: hwmod: Add INIT_NO_IDLE flag for debugss hwmod
2014-04-09 5:47 [PATCH] ARM: AM33xx: hwmod: Add INIT_NO_IDLE flag for debugss hwmod Lokesh Vutla
@ 2014-04-11 18:00 ` Paul Walmsley
2014-04-11 18:09 ` Paul Walmsley
0 siblings, 1 reply; 5+ messages in thread
From: Paul Walmsley @ 2014-04-11 18:00 UTC (permalink / raw)
To: linux-arm-kernel
On Wed, 9 Apr 2014, Lokesh Vutla wrote:
> During boot, when hwmod tries to cut clocks for debugss it always
> gets stuck in transition state and throws the following warning:
>
> [ 0.139581] omap_hwmod: debugss: _wait_target_disable failed
>
> As per the information provided by folks, clocks to debugss cannot be cut.
> So adding HWMOD_INIT_NO_IDLE flag to debugss hwmod.
>
> Signed-off-by: Lokesh Vutla <lokeshvutla@ti•com>
Thanks, queued for v3.15-rc.
- Paul
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH] ARM: AM33xx: hwmod: Add INIT_NO_IDLE flag for debugss hwmod
2014-04-11 18:00 ` Paul Walmsley
@ 2014-04-11 18:09 ` Paul Walmsley
2014-04-14 8:29 ` Lokesh Vutla
0 siblings, 1 reply; 5+ messages in thread
From: Paul Walmsley @ 2014-04-11 18:09 UTC (permalink / raw)
To: linux-arm-kernel
On Fri, 11 Apr 2014, Paul Walmsley wrote:
> On Wed, 9 Apr 2014, Lokesh Vutla wrote:
>
> > During boot, when hwmod tries to cut clocks for debugss it always
> > gets stuck in transition state and throws the following warning:
> >
> > [ 0.139581] omap_hwmod: debugss: _wait_target_disable failed
> >
> > As per the information provided by folks, clocks to debugss cannot be cut.
> > So adding HWMOD_INIT_NO_IDLE flag to debugss hwmod.
> >
> > Signed-off-by: Lokesh Vutla <lokeshvutla@ti•com>
>
> Thanks, queued for v3.15-rc.
Hmmm. On second thought, this doesn't look like the right fix. Could
you please comment on the issues raised here:
https://patchwork.kernel.org/patch/2212111/
Looks to me like there should be at least one device driver, either for
the entire DEBUGSS, or for the individual IP blocks inside the DEBUGSS.
Dropping this patch for now.
- Paul
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH] ARM: AM33xx: hwmod: Add INIT_NO_IDLE flag for debugss hwmod
2014-04-11 18:09 ` Paul Walmsley
@ 2014-04-14 8:29 ` Lokesh Vutla
2014-05-07 23:55 ` Paul Walmsley
0 siblings, 1 reply; 5+ messages in thread
From: Lokesh Vutla @ 2014-04-14 8:29 UTC (permalink / raw)
To: linux-arm-kernel
Hi Paul,
On Friday 11 April 2014 11:39 PM, Paul Walmsley wrote:
> On Fri, 11 Apr 2014, Paul Walmsley wrote:
>
>> On Wed, 9 Apr 2014, Lokesh Vutla wrote:
>>
>>> During boot, when hwmod tries to cut clocks for debugss it always
>>> gets stuck in transition state and throws the following warning:
>>>
>>> [ 0.139581] omap_hwmod: debugss: _wait_target_disable failed
>>>
>>> As per the information provided by folks, clocks to debugss cannot be cut.
>>> So adding HWMOD_INIT_NO_IDLE flag to debugss hwmod.
>>>
>>> Signed-off-by: Lokesh Vutla <lokeshvutla@ti•com>
>>
>> Thanks, queued for v3.15-rc.
>
> Hmmm. On second thought, this doesn't look like the right fix. Could
> you please comment on the issues raised here:
>
> https://patchwork.kernel.org/patch/2212111/
Yes, I initially created a driver for enabling and disabling clocks for DEBUGSS.
But I always see that DEBUGSS is always stuck in transition whenever clocks are cut
to DEBUGSS.
During boot also when hwmod tries to cut clocks the following warning comes:
[ 0.139581] omap_hwmod: debugss: _wait_target_disable failed.
As confirmed by the hardware team that this is a bug in silicon that clocks cannot
be cut to DEBUGSS.
So I am just adding HWMOD_INIT_NO_IDLE flag to debugss hwmod.
I have also tested suspend-resume with $subject patch with TI internal tree.
Please let me know if I am not clear.
Thanks and regards,
Lokesh
>
> Looks to me like there should be at least one device driver, either for
> the entire DEBUGSS, or for the individual IP blocks inside the DEBUGSS.
>
> Dropping this patch for now.
>
>
> - Paul
>
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH] ARM: AM33xx: hwmod: Add INIT_NO_IDLE flag for debugss hwmod
2014-04-14 8:29 ` Lokesh Vutla
@ 2014-05-07 23:55 ` Paul Walmsley
0 siblings, 0 replies; 5+ messages in thread
From: Paul Walmsley @ 2014-05-07 23:55 UTC (permalink / raw)
To: linux-arm-kernel
Hi Lokesh
On Mon, 14 Apr 2014, Lokesh Vutla wrote:
> On Friday 11 April 2014 11:39 PM, Paul Walmsley wrote:
> > On Fri, 11 Apr 2014, Paul Walmsley wrote:
> >
> >> On Wed, 9 Apr 2014, Lokesh Vutla wrote:
> >>
> >>> During boot, when hwmod tries to cut clocks for debugss it always
> >>> gets stuck in transition state and throws the following warning:
> >>>
> >>> [ 0.139581] omap_hwmod: debugss: _wait_target_disable failed
> >>>
> >>> As per the information provided by folks, clocks to debugss cannot be cut.
> >>> So adding HWMOD_INIT_NO_IDLE flag to debugss hwmod.
> >>>
> >>> Signed-off-by: Lokesh Vutla <lokeshvutla@ti•com>
> >>
> >> Thanks, queued for v3.15-rc.
> >
> > Hmmm. On second thought, this doesn't look like the right fix. Could
> > you please comment on the issues raised here:
> >
> > https://patchwork.kernel.org/patch/2212111/
> Yes, I initially created a driver for enabling and disabling clocks for DEBUGSS.
> But I always see that DEBUGSS is always stuck in transition whenever clocks are cut
> to DEBUGSS.
> During boot also when hwmod tries to cut clocks the following warning comes:
> [ 0.139581] omap_hwmod: debugss: _wait_target_disable failed.
>
> As confirmed by the hardware team that this is a bug in silicon that clocks cannot
> be cut to DEBUGSS.
> So I am just adding HWMOD_INIT_NO_IDLE flag to debugss hwmod.
>
> I have also tested suspend-resume with $subject patch with TI internal tree.
>
> Please let me know if I am not clear.
OK, here's my question. Does the DEBUGSS module correctly enter a
low-power state when going to suspend, even when the DEBUGSS is still
enabled? Or is it the case that we should still disable it in the hwmod
code and simply skip the call to _wait_target_disable()?
If the former, then please repost your patch with some comments in the
hwmod data to describe this case, and note that the module will still
enter a low-power state appropriately when the MPUSS goes into standby (if
that is indeed the behavior).
If the latter, then it seems like the right approach is to add a hwmod
flag to indicate that we should still disable it, but not wait for the
disable to complete.
I guess the most direct way to test this would be to pull out a ammeter
and verify that the idle current is the same in both cases.
How does that sound?
- Paul
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2014-05-07 23:55 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-04-09 5:47 [PATCH] ARM: AM33xx: hwmod: Add INIT_NO_IDLE flag for debugss hwmod Lokesh Vutla
2014-04-11 18:00 ` Paul Walmsley
2014-04-11 18:09 ` Paul Walmsley
2014-04-14 8:29 ` Lokesh Vutla
2014-05-07 23:55 ` Paul Walmsley
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox