public inbox for linux-pm@vger.kernel.org
 help / color / mirror / Atom feed
* [-mm patch] kernel/power/main.c: make code static
       [not found] <20070628034321.38c9f12b.akpm@linux-foundation.org>
@ 2007-07-01 20:23 ` Adrian Bunk
  2007-07-01 23:12   ` Rafael J. Wysocki
  0 siblings, 1 reply; 2+ messages in thread
From: Adrian Bunk @ 2007-07-01 20:23 UTC (permalink / raw)
  To: Andrew Morton, Rafael J. Wysocki, Pavel Machek; +Cc: linux-pm, linux-kernel

On Thu, Jun 28, 2007 at 03:43:21AM -0700, Andrew Morton wrote:
>...
> Changes since 2.6.22-rc4-mm2:
>...
> +pm-reduce-code-duplication-between-mainc-and-userc.patch
>...
>  PM updates
>...


Some code can now become static.

Signed-off-by: Adrian Bunk <bunk@stusta.de>

---

 include/linux/pm.h   |    1 -
 kernel/power/main.c  |    4 ++--
 kernel/power/power.h |    1 -
 3 files changed, 2 insertions(+), 4 deletions(-)

--- linux-2.6.22-rc6-mm1/kernel/power/power.h.old	2007-07-01 00:21:34.000000000 +0200
+++ linux-2.6.22-rc6-mm1/kernel/power/power.h	2007-07-01 00:21:41.000000000 +0200
@@ -177,7 +177,6 @@
 				unsigned int, char *);
 
 /* kernel/power/main.c */
-extern int suspend_enter(suspend_state_t state);
 extern int suspend_devices_and_enter(suspend_state_t state);
 extern struct blocking_notifier_head pm_chain_head;
 
--- linux-2.6.22-rc6-mm1/include/linux/pm.h.old	2007-07-01 00:22:53.000000000 +0200
+++ linux-2.6.22-rc6-mm1/include/linux/pm.h	2007-07-01 00:23:04.000000000 +0200
@@ -138,7 +138,6 @@
  * @pm_ops: The new power management operations to set.
  */
 extern void pm_set_ops(struct pm_ops *pm_ops);
-extern struct pm_ops *pm_ops;
 extern int pm_suspend(suspend_state_t state);
 
 extern int pm_valid_only_mem(suspend_state_t state);
--- linux-2.6.22-rc6-mm1/kernel/power/main.c.old	2007-07-01 00:21:49.000000000 +0200
+++ linux-2.6.22-rc6-mm1/kernel/power/main.c	2007-07-01 00:23:11.000000000 +0200
@@ -31,7 +31,7 @@
 
 DEFINE_MUTEX(pm_mutex);
 
-struct pm_ops *pm_ops;
+static struct pm_ops *pm_ops;
 
 /**
  *	pm_set_ops - Set the global power method table. 
@@ -127,7 +127,7 @@
  *
  *	This function should be called after devices have been suspended.
  */
-int suspend_enter(suspend_state_t state)
+static int suspend_enter(suspend_state_t state)
 {
 	int error = 0;
 

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [-mm patch] kernel/power/main.c: make code static
  2007-07-01 20:23 ` [-mm patch] kernel/power/main.c: make code static Adrian Bunk
@ 2007-07-01 23:12   ` Rafael J. Wysocki
  0 siblings, 0 replies; 2+ messages in thread
From: Rafael J. Wysocki @ 2007-07-01 23:12 UTC (permalink / raw)
  To: Adrian Bunk; +Cc: Andrew Morton, linux-kernel, Pavel Machek, linux-pm

On Sunday, 1 July 2007 22:23, Adrian Bunk wrote:
> On Thu, Jun 28, 2007 at 03:43:21AM -0700, Andrew Morton wrote:
> >...
> > Changes since 2.6.22-rc4-mm2:
> >...
> > +pm-reduce-code-duplication-between-mainc-and-userc.patch
> >...
> >  PM updates
> >...
> 
> 
> Some code can now become static.
> 
> Signed-off-by: Adrian Bunk <bunk@stusta.de>

This clashes and partially overlaps with some patches that I have in the works.

Greetings,
Rafael


-- 
"Premature optimization is the root of all evil." - Donald Knuth

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2007-07-01 23:12 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <20070628034321.38c9f12b.akpm@linux-foundation.org>
2007-07-01 20:23 ` [-mm patch] kernel/power/main.c: make code static Adrian Bunk
2007-07-01 23:12   ` Rafael J. Wysocki

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox