Re: [PATCH] OMAP3 PM restore observability settings after off mode

2009-10-13 Thread Kevin Hilman 
Peter 'p2' De Schrijver peter.de-schrij...@nokia.com writes:

 This patch restores the observability settings after resuming from off mode.

 Signed-off-by: Peter 'p2' De Schrijver peter.de-schrij...@nokia.com

Thanks, pulling this into PM branch.

Kevin

 ---
  arch/arm/mach-omap2/debobs.c |9 +
  arch/arm/mach-omap2/pm34xx.c |4 
  arch/arm/plat-omap/include/mach/debobs.h |1 +
  3 files changed, 14 insertions(+), 0 deletions(-)

 diff --git a/arch/arm/mach-omap2/debobs.c b/arch/arm/mach-omap2/debobs.c
 index 4fbabef..d25b9a2 100644
 --- a/arch/arm/mach-omap2/debobs.c
 +++ b/arch/arm/mach-omap2/debobs.c
 @@ -190,6 +190,15 @@ static inline int __init _new_debobs_pad(struct 
 debobs_pad *pad, char *name,
  
  /* Public functions */
  
 +void debobs_restore(void)
 +{
 + struct debobs_pad *p = debobs_pads[0];
 + int i;
 +
 + for (i = 0; i  NUM_OF_DEBOBS_PADS; i++, p++)
 + debobs_set(p-core_obs, p-core_obs.value);
 +}
 +
  void debug_gpio_set(unsigned gpio, int value)
  {
   if (!debobs_initialized)
 diff --git a/arch/arm/mach-omap2/pm34xx.c b/arch/arm/mach-omap2/pm34xx.c
 index 553fe02..20c7ea2 100644
 --- a/arch/arm/mach-omap2/pm34xx.c
 +++ b/arch/arm/mach-omap2/pm34xx.c
 @@ -42,6 +42,7 @@
  #include mach/dma.h
  #include mach/gpmc.h
  #include mach/dma.h
 +#include mach/debobs.h
  #include asm/tlbflush.h
  
  #include cm.h
 @@ -124,6 +125,9 @@ static void omap3_core_restore_context(void)
   /* Restore the interrupt controller context */
   omap3_intc_restore_context();
   omap_dma_global_context_restore();
 + /* restore debobs context */
 + debobs_restore();
 +
  }
  
  static void omap3_save_secure_ram_context(u32 target_mpu_state)
 diff --git a/arch/arm/plat-omap/include/mach/debobs.h 
 b/arch/arm/plat-omap/include/mach/debobs.h
 index 67f765d..1e04bcd 100644
 --- a/arch/arm/plat-omap/include/mach/debobs.h
 +++ b/arch/arm/plat-omap/include/mach/debobs.h
 @@ -3,5 +3,6 @@
  
  void debug_gpio_set(unsigned gpio, int value);
  int debug_gpio_get(unsigned gpio);
 +void debobs_restore(void);
  
  #endif
 -- 
 1.6.2.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
--
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] OMAP3 PM restore observability settings after off mode

2009-10-13 Thread Kevin Hilman 
Peter 'p2' De Schrijver peter.de-schrij...@nokia.com writes:

 This patch restores the observability settings after resuming from off mode.

 Signed-off-by: Peter 'p2' De Schrijver peter.de-schrij...@nokia.com

I decided not to pull this one into PM branch, because...

 diff --git a/arch/arm/mach-omap2/pm34xx.c b/arch/arm/mach-omap2/pm34xx.c
 index 553fe02..20c7ea2 100644
 --- a/arch/arm/mach-omap2/pm34xx.c
 +++ b/arch/arm/mach-omap2/pm34xx.c
 @@ -42,6 +42,7 @@
  #include mach/dma.h
  #include mach/gpmc.h
  #include mach/dma.h
 +#include mach/debobs.h
  #include asm/tlbflush.h
  
  #include cm.h
 @@ -124,6 +125,9 @@ static void omap3_core_restore_context(void)
   /* Restore the interrupt controller context */
   omap3_intc_restore_context();
   omap_dma_global_context_restore();
 + /* restore debobs context */
 + debobs_restore();
 +

of compile failure when CONFIG_PM_DEBOBS is not enabled.

And, debobs should now go upstream indepenent of PM branch.

When I posted my PM debug series for the last merge window, I
originally included the debobs series.  It recieved some comments on
the list that need to be addressed before this can go upstream.

Now that the main PM debug stuff is in mainline, this series has no
more dependencies on the PM branch.  I think you should address the
the comments/questions raised on the list[1,2] and resubmit as a
series against mainline.

A quick test shows that the current set of 3 debobs patches currently
in the PM branch rebase easily against omap/master so should be easy
to rework for upstream acceptance.

Thanks,

Kevin

[1] http://marc.info/?l=linux-omapm=125049645823777w=2
[2] http://marc.info/?l=linux-omapm=125049651623911w=2
--
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] OMAP3 PM restore observability settings after off mode

2009-10-12 Thread Peter 'p2' De Schrijver 
This patch restores the observability settings after resuming from off mode.

Signed-off-by: Peter 'p2' De Schrijver peter.de-schrij...@nokia.com
---
 arch/arm/mach-omap2/debobs.c |9 +
 arch/arm/mach-omap2/pm34xx.c |4 
 arch/arm/plat-omap/include/mach/debobs.h |1 +
 3 files changed, 14 insertions(+), 0 deletions(-)

diff --git a/arch/arm/mach-omap2/debobs.c b/arch/arm/mach-omap2/debobs.c
index 4fbabef..d25b9a2 100644
--- a/arch/arm/mach-omap2/debobs.c
+++ b/arch/arm/mach-omap2/debobs.c
@@ -190,6 +190,15 @@ static inline int __init _new_debobs_pad(struct debobs_pad 
*pad, char *name,
 
 /* Public functions */
 
+void debobs_restore(void)
+{
+   struct debobs_pad *p = debobs_pads[0];
+   int i;
+
+   for (i = 0; i  NUM_OF_DEBOBS_PADS; i++, p++)
+   debobs_set(p-core_obs, p-core_obs.value);
+}
+
 void debug_gpio_set(unsigned gpio, int value)
 {
if (!debobs_initialized)
diff --git a/arch/arm/mach-omap2/pm34xx.c b/arch/arm/mach-omap2/pm34xx.c
index 553fe02..20c7ea2 100644
--- a/arch/arm/mach-omap2/pm34xx.c
+++ b/arch/arm/mach-omap2/pm34xx.c
@@ -42,6 +42,7 @@
 #include mach/dma.h
 #include mach/gpmc.h
 #include mach/dma.h
+#include mach/debobs.h
 #include asm/tlbflush.h
 
 #include cm.h
@@ -124,6 +125,9 @@ static void omap3_core_restore_context(void)
/* Restore the interrupt controller context */
omap3_intc_restore_context();
omap_dma_global_context_restore();
+   /* restore debobs context */
+   debobs_restore();
+
 }
 
 static void omap3_save_secure_ram_context(u32 target_mpu_state)
diff --git a/arch/arm/plat-omap/include/mach/debobs.h 
b/arch/arm/plat-omap/include/mach/debobs.h
index 67f765d..1e04bcd 100644
--- a/arch/arm/plat-omap/include/mach/debobs.h
+++ b/arch/arm/plat-omap/include/mach/debobs.h
@@ -3,5 +3,6 @@
 
 void debug_gpio_set(unsigned gpio, int value);
 int debug_gpio_get(unsigned gpio);
+void debobs_restore(void);
 
 #endif
-- 
1.6.2.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