diff options
author | Johannes Berg <johannes@sipsolutions.net> | 2007-05-06 17:50:49 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@woody.linux-foundation.org> | 2007-05-07 15:12:59 -0400 |
commit | ab3bfca7abf3fd0fe41d26d839610a787aa7e587 (patch) | |
tree | e62e58beaf9d22d4156b27d4523c3728bf5769bc /include/linux | |
parent | b1296cc48b39355241470ef934a5e2270e3f23bd (diff) |
remove software_suspend()
Remove software_suspend() and all its users since
pm_suspend(PM_SUSPEND_DISK) should be equivalent and there's no point in
having two interfaces for the same thing.
The patch also changes the valid_state function to return 0 (false) for
PM_SUSPEND_DISK when SOFTWARE_SUSPEND is not configured instead of
accepting it and having the whole thing fail later.
Signed-off-by: Johannes Berg <johannes@sipsolutions.net>
Acked-by: "Rafael J. Wysocki" <rjw@sisk.pl>
Cc: Pavel Machek <pavel@ucw.cz>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'include/linux')
-rw-r--r-- | include/linux/init.h | 2 | ||||
-rw-r--r-- | include/linux/suspend.h | 8 |
2 files changed, 1 insertions, 9 deletions
diff --git a/include/linux/init.h b/include/linux/init.h index 9abf120ec9f8..dbbdbd1bec77 100644 --- a/include/linux/init.h +++ b/include/linux/init.h | |||
@@ -233,7 +233,7 @@ void __init parse_early_param(void); | |||
233 | #define __obsolete_setup(str) /* nothing */ | 233 | #define __obsolete_setup(str) /* nothing */ |
234 | #endif | 234 | #endif |
235 | 235 | ||
236 | /* Data marked not to be saved by software_suspend() */ | 236 | /* Data marked not to be saved by software suspend */ |
237 | #define __nosavedata __attribute__ ((__section__ (".data.nosave"))) | 237 | #define __nosavedata __attribute__ ((__section__ (".data.nosave"))) |
238 | 238 | ||
239 | /* This means "can be init if no module support, otherwise module load | 239 | /* This means "can be init if no module support, otherwise module load |
diff --git a/include/linux/suspend.h b/include/linux/suspend.h index 3cc4d6394c07..3aecc96acc76 100644 --- a/include/linux/suspend.h +++ b/include/linux/suspend.h | |||
@@ -33,8 +33,6 @@ static inline void pm_restore_console(void) {} | |||
33 | #endif | 33 | #endif |
34 | 34 | ||
35 | #if defined(CONFIG_PM) && defined(CONFIG_SOFTWARE_SUSPEND) | 35 | #if defined(CONFIG_PM) && defined(CONFIG_SOFTWARE_SUSPEND) |
36 | /* kernel/power/swsusp.c */ | ||
37 | extern int software_suspend(void); | ||
38 | /* kernel/power/snapshot.c */ | 36 | /* kernel/power/snapshot.c */ |
39 | extern void __init register_nosave_region(unsigned long, unsigned long); | 37 | extern void __init register_nosave_region(unsigned long, unsigned long); |
40 | extern int swsusp_page_is_forbidden(struct page *); | 38 | extern int swsusp_page_is_forbidden(struct page *); |
@@ -42,12 +40,6 @@ extern void swsusp_set_page_free(struct page *); | |||
42 | extern void swsusp_unset_page_free(struct page *); | 40 | extern void swsusp_unset_page_free(struct page *); |
43 | extern unsigned long get_safe_page(gfp_t gfp_mask); | 41 | extern unsigned long get_safe_page(gfp_t gfp_mask); |
44 | #else | 42 | #else |
45 | static inline int software_suspend(void) | ||
46 | { | ||
47 | printk("Warning: fake suspend called\n"); | ||
48 | return -ENOSYS; | ||
49 | } | ||
50 | |||
51 | static inline void register_nosave_region(unsigned long b, unsigned long e) {} | 43 | static inline void register_nosave_region(unsigned long b, unsigned long e) {} |
52 | static inline int swsusp_page_is_forbidden(struct page *p) { return 0; } | 44 | static inline int swsusp_page_is_forbidden(struct page *p) { return 0; } |
53 | static inline void swsusp_set_page_free(struct page *p) {} | 45 | static inline void swsusp_set_page_free(struct page *p) {} |