diff options
author | Russell King <rmk@dyn-67.arm.linux.org.uk> | 2007-05-15 06:22:48 -0400 |
---|---|---|
committer | Russell King <rmk+kernel@arm.linux.org.uk> | 2007-07-12 09:28:06 -0400 |
commit | 88dfe98c688e1700a4a9f73f8b7d570f4f52170d (patch) | |
tree | b6810766664731716c24138ac8a1c247602b77d8 /arch/arm/mach-pxa/pxa27x.c | |
parent | e176bb05fec4c00450302a75e81f8da3dc9e309e (diff) |
[ARM] pxa: Fix PXA27x suspend type validation, remove pxa_pm_prepare()
pxa_pm_prepare() tried to validate the suspend method type. As
noted in previous commits:
eb9289eb20df6b54214c45ac7c6bf5179a149026
9c372d06ce9ddf65e1393f9ea22a6d6bd5f96b42
e8c9c502690efd24b7055bf608e7a3c34216848b
the checking of the suspend type in the 'prepare' method is the
wrong place to do this; use the 'valid' method instead. This
means that pxa_pm_prepare() can be entirely removed.
Signed-off-by: Russell King <rmk+kernel@arm.linux.org.uk>
Diffstat (limited to 'arch/arm/mach-pxa/pxa27x.c')
-rw-r--r-- | arch/arm/mach-pxa/pxa27x.c | 19 |
1 files changed, 6 insertions, 13 deletions
diff --git a/arch/arm/mach-pxa/pxa27x.c b/arch/arm/mach-pxa/pxa27x.c index be6ca7d3afa2..ac40fa5f7709 100644 --- a/arch/arm/mach-pxa/pxa27x.c +++ b/arch/arm/mach-pxa/pxa27x.c | |||
@@ -123,17 +123,6 @@ EXPORT_SYMBOL(get_lcdclk_frequency_10khz); | |||
123 | 123 | ||
124 | #ifdef CONFIG_PM | 124 | #ifdef CONFIG_PM |
125 | 125 | ||
126 | int pxa_pm_prepare(suspend_state_t state) | ||
127 | { | ||
128 | switch (state) { | ||
129 | case PM_SUSPEND_MEM: | ||
130 | case PM_SUSPEND_STANDBY: | ||
131 | return 0; | ||
132 | default: | ||
133 | return -EINVAL; | ||
134 | } | ||
135 | } | ||
136 | |||
137 | void pxa_cpu_pm_enter(suspend_state_t state) | 126 | void pxa_cpu_pm_enter(suspend_state_t state) |
138 | { | 127 | { |
139 | extern void pxa_cpu_standby(void); | 128 | extern void pxa_cpu_standby(void); |
@@ -163,10 +152,14 @@ void pxa_cpu_pm_enter(suspend_state_t state) | |||
163 | } | 152 | } |
164 | } | 153 | } |
165 | 154 | ||
155 | static int pxa27x_pm_valid(suspend_state_t state) | ||
156 | { | ||
157 | return state == PM_SUSPEND_MEM || state == PM_SUSPEND_STANDBY; | ||
158 | } | ||
159 | |||
166 | static struct pm_ops pxa27x_pm_ops = { | 160 | static struct pm_ops pxa27x_pm_ops = { |
167 | .prepare = pxa_pm_prepare, | ||
168 | .enter = pxa_pm_enter, | 161 | .enter = pxa_pm_enter, |
169 | .valid = pm_valid_only_mem, | 162 | .valid = pxa27x_pm_valid, |
170 | }; | 163 | }; |
171 | #endif | 164 | #endif |
172 | 165 | ||