]> pilppa.com Git - linux-2.6-omap-h63xx.git/commitdiff
[ARM] pxa: Fix PXA27x suspend type validation, remove pxa_pm_prepare()
authorRussell King <rmk@dyn-67.arm.linux.org.uk>
Tue, 15 May 2007 10:22:48 +0000 (11:22 +0100)
committerRussell King <rmk+kernel@arm.linux.org.uk>
Thu, 12 Jul 2007 13:28:06 +0000 (14:28 +0100)
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>
arch/arm/common/sharpsl_pm.c
arch/arm/mach-pxa/pxa25x.c
arch/arm/mach-pxa/pxa27x.c

index 62cb50816cc674655be18324ac1c83a005845081..111a7fa5debee4ace5a39f1aa5c50b2ab3927683 100644 (file)
@@ -766,7 +766,6 @@ static void sharpsl_apm_get_power_status(struct apm_power_info *info)
 }
 
 static struct pm_ops sharpsl_pm_ops = {
-       .prepare        = pxa_pm_prepare,
        .enter          = corgi_pxa_pm_enter,
        .valid          = pm_valid_only_mem,
 };
index 8560439ba0b909cc7c31e8c3cc8f31a728ee4c32..fe29c6e03a4f1658fec0a764ed0782dc54ab3cc4 100644 (file)
@@ -106,18 +106,6 @@ EXPORT_SYMBOL(get_lcdclk_frequency_10khz);
 
 #ifdef CONFIG_PM
 
-int pxa_pm_prepare(suspend_state_t state)
-{
-       switch (state) {
-       case PM_SUSPEND_MEM:
-               break;
-       default:
-               return -EINVAL;
-       }
-
-       return 0;
-}
-
 void pxa_cpu_pm_enter(suspend_state_t state)
 {
        extern void pxa_cpu_suspend(unsigned int);
@@ -135,7 +123,6 @@ void pxa_cpu_pm_enter(suspend_state_t state)
 }
 
 static struct pm_ops pxa25x_pm_ops = {
-       .prepare        = pxa_pm_prepare,
        .enter          = pxa_pm_enter,
        .valid          = pm_valid_only_mem,
 };
index be6ca7d3afa28f1bd50fa72c57b76859f3ec2e34..ac40fa5f7709e7a16d1200ed7d1afc96e026541b 100644 (file)
@@ -123,17 +123,6 @@ EXPORT_SYMBOL(get_lcdclk_frequency_10khz);
 
 #ifdef CONFIG_PM
 
-int pxa_pm_prepare(suspend_state_t state)
-{
-       switch (state) {
-       case PM_SUSPEND_MEM:
-       case PM_SUSPEND_STANDBY:
-               return 0;
-       default:
-               return -EINVAL;
-       }
-}
-
 void pxa_cpu_pm_enter(suspend_state_t state)
 {
        extern void pxa_cpu_standby(void);
@@ -163,10 +152,14 @@ void pxa_cpu_pm_enter(suspend_state_t state)
        }
 }
 
+static int pxa27x_pm_valid(suspend_state_t state)
+{
+       return state == PM_SUSPEND_MEM || state == PM_SUSPEND_STANDBY;
+}
+
 static struct pm_ops pxa27x_pm_ops = {
-       .prepare        = pxa_pm_prepare,
        .enter          = pxa_pm_enter,
-       .valid          = pm_valid_only_mem,
+       .valid          = pxa27x_pm_valid,
 };
 #endif