Re: [PATCH 5/6] ARM: OMAP4: suspend: Do not get dsp/tesla out of reset

2012-02-27 Thread Kevin Hilman
Tero Kristo t-kri...@ti.com writes:

 From: Rajendra Nayak rna...@ti.com

 With no driver handling DSP/TESLA, if brought out of reset, it stays
 active and does not assert standby.

 Signed-off-by: Rajendra Nayak rna...@ti.com
 Signed-off-by: Tero Kristo t-kri...@ti.com

Subject says 'OMAP4: suspend'.  How does HWMOD_INIT_NO_RESET affect
suspend?

Kevin

 ---
  arch/arm/mach-omap2/omap_hwmod_44xx_data.c |1 +
  1 files changed, 1 insertions(+), 0 deletions(-)

 diff --git a/arch/arm/mach-omap2/omap_hwmod_44xx_data.c 
 b/arch/arm/mach-omap2/omap_hwmod_44xx_data.c
 index ef0524c..81d90da 100644
 --- a/arch/arm/mach-omap2/omap_hwmod_44xx_data.c
 +++ b/arch/arm/mach-omap2/omap_hwmod_44xx_data.c
 @@ -1160,6 +1160,7 @@ static struct omap_hwmod omap44xx_dsp_hwmod = {
   .name   = dsp,
   .class  = omap44xx_dsp_hwmod_class,
   .clkdm_name = tesla_clkdm,
 + .flags  = HWMOD_INIT_NO_RESET,
   .mpu_irqs   = omap44xx_dsp_irqs,
   .rst_lines  = omap44xx_dsp_resets,
   .rst_lines_cnt  = ARRAY_SIZE(omap44xx_dsp_resets),
--
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


Re: [PATCH 5/6] ARM: OMAP4: suspend: Do not get dsp/tesla out of reset

2012-02-27 Thread Rajendra Nayak

Hi Kevin,

On Tuesday 28 February 2012 05:58 AM, Kevin Hilman wrote:

Tero Kristot-kri...@ti.com  writes:


From: Rajendra Nayakrna...@ti.com

With no driver handling DSP/TESLA, if brought out of reset, it stays
active and does not assert standby.

Signed-off-by: Rajendra Nayakrna...@ti.com
Signed-off-by: Tero Kristot-kri...@ti.com


Subject says 'OMAP4: suspend'.  How does HWMOD_INIT_NO_RESET affect
suspend?


The v2 of this patch posted by Tero has a better $subject as suggested
by Benoit which says 'ARM: OMAP4: hwmod_data: Do not get DSP out of
reset at boot time'

The intention of this patch is to *not* get DSP out of reset at boot,
because once out of reset DSP needs some code that executes a 'wfi'
equivalent for it to assert a standby and allow suspend to sleep.

regards,
Rajendra



Kevin


---
  arch/arm/mach-omap2/omap_hwmod_44xx_data.c |1 +
  1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/arch/arm/mach-omap2/omap_hwmod_44xx_data.c 
b/arch/arm/mach-omap2/omap_hwmod_44xx_data.c
index ef0524c..81d90da 100644
--- a/arch/arm/mach-omap2/omap_hwmod_44xx_data.c
+++ b/arch/arm/mach-omap2/omap_hwmod_44xx_data.c
@@ -1160,6 +1160,7 @@ static struct omap_hwmod omap44xx_dsp_hwmod = {
.name   = dsp,
.class  =omap44xx_dsp_hwmod_class,
.clkdm_name = tesla_clkdm,
+   .flags  = HWMOD_INIT_NO_RESET,
.mpu_irqs   = omap44xx_dsp_irqs,
.rst_lines  = omap44xx_dsp_resets,
.rst_lines_cnt  = ARRAY_SIZE(omap44xx_dsp_resets),


--
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


Re: [PATCH 5/6] ARM: OMAP4: suspend: Do not get dsp/tesla out of reset

2012-02-23 Thread Cousson, Benoit

On 2/23/2012 2:56 PM, Shilimkar, Santosh wrote:

$Subject
s/dsp/DSP
s/tesla/TESLA


Even better... remove tesla :-)

This is an internal codename that at some point should be removed from 
the clkdm name. I just did not have the time to do that for OMAP4, but 
it will not exist anymore on OMAP5.


Nit: The subject should be:
ARM: OMAP4: hwmod data: Do not get DSP out of reset at boot time

Regards,
Benoit



On Thu, Feb 23, 2012 at 6:51 PM, Tero Kristot-kri...@ti.com  wrote:

From: Rajendra Nayakrna...@ti.com

With no driver handling DSP/TESLA, if brought out of reset, it stays
active and does not assert standby.


Which lead to IVHA PD not transitiong and hence gating CHIP retention.
You may want to add these bits as well.


Signed-off-by: Rajendra Nayakrna...@ti.com
Signed-off-by: Tero Kristot-kri...@ti.com
---


Patch is fine though script needs to be updated I guess

Regards
Santosh
--
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


--
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


[PATCH 5/6] ARM: OMAP4: suspend: Do not get dsp/tesla out of reset

2012-02-23 Thread Tero Kristo
From: Rajendra Nayak rna...@ti.com

With no driver handling DSP/TESLA, if brought out of reset, it stays
active and does not assert standby.

Signed-off-by: Rajendra Nayak rna...@ti.com
Signed-off-by: Tero Kristo t-kri...@ti.com
---
 arch/arm/mach-omap2/omap_hwmod_44xx_data.c |1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/arch/arm/mach-omap2/omap_hwmod_44xx_data.c 
b/arch/arm/mach-omap2/omap_hwmod_44xx_data.c
index ef0524c..81d90da 100644
--- a/arch/arm/mach-omap2/omap_hwmod_44xx_data.c
+++ b/arch/arm/mach-omap2/omap_hwmod_44xx_data.c
@@ -1160,6 +1160,7 @@ static struct omap_hwmod omap44xx_dsp_hwmod = {
.name   = dsp,
.class  = omap44xx_dsp_hwmod_class,
.clkdm_name = tesla_clkdm,
+   .flags  = HWMOD_INIT_NO_RESET,
.mpu_irqs   = omap44xx_dsp_irqs,
.rst_lines  = omap44xx_dsp_resets,
.rst_lines_cnt  = ARRAY_SIZE(omap44xx_dsp_resets),
-- 
1.7.4.1

--
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