2 files changed, 15 insertions, 36 deletions

diff --git a/drivers/acpi/ec.c b/drivers/acpi/ec.c
index 8e665f2e3138..fca4140a50a9 100644
--- a/drivers/acpi/ec.c
+++ b/drivers/acpi/ec.c
@@ -170,22 +170,19 @@ acpi_ec_enter_burst_mode (
         status = acpi_ec_read_status(ec);
         if (status != -EINVAL &&
                 !(status & ACPI_EC_FLAG_BURST)){
-                ACPI_DEBUG_PRINT((ACPI_DB_INFO,"entering burst mode \n"));
                 acpi_hw_low_level_write(8, ACPI_EC_BURST_ENABLE, &ec->command_addr);
                 status = acpi_ec_wait(ec, ACPI_EC_EVENT_OBF);
                 if (status){
                         acpi_enable_gpe(NULL, ec->gpe_bit, ACPI_NOT_ISR);
-                        ACPI_DEBUG_PRINT((ACPI_DB_ERROR," status = %d\n", status));
                         return_VALUE(-EINVAL);
                 }
                 acpi_hw_low_level_read(8, &tmp, &ec->data_addr);
                 acpi_enable_gpe(NULL, ec->gpe_bit, ACPI_NOT_ISR);
                 if(tmp != 0x90 ) {/* Burst ACK byte*/
-                        ACPI_DEBUG_PRINT((ACPI_DB_ERROR,"Ack failed \n"));
                         return_VALUE(-EINVAL);
                 }
-        } else
-                ACPI_DEBUG_PRINT((ACPI_DB_INFO,"already be in burst mode \n"));
+        }
+
         atomic_set(&ec->leaving_burst , 0);
         return_VALUE(0);
 }
@@ -202,7 +199,6 @@ acpi_ec_leave_burst_mode (
         status = acpi_ec_read_status(ec);
         if (status != -EINVAL &&
                 (status & ACPI_EC_FLAG_BURST)){
-                ACPI_DEBUG_PRINT((ACPI_DB_INFO,"leaving burst mode\n"));
                 acpi_hw_low_level_write(8, ACPI_EC_BURST_DISABLE, &ec->command_addr);
                 status = acpi_ec_wait(ec, ACPI_EC_FLAG_IBF);
                 if (status){
@@ -212,14 +208,7 @@ acpi_ec_leave_burst_mode (
                 }
                 acpi_enable_gpe(NULL, ec->gpe_bit, ACPI_NOT_ISR);
                 status = acpi_ec_read_status(ec);
-                if (status != -EINVAL &&
-                        (status & ACPI_EC_FLAG_BURST)) {
-                        ACPI_DEBUG_PRINT((ACPI_DB_ERROR,"------->status fail\n"));
-                        return_VALUE(-EINVAL);
-                }
-        }else
-                ACPI_DEBUG_PRINT((ACPI_DB_INFO,"already be in Non-burst mode \n"));
-        ACPI_DEBUG_PRINT((ACPI_DB_INFO,"leaving burst mode\n"));
+        }
 
         return_VALUE(0);
 }
diff --git a/drivers/acpi/sleep/poweroff.c b/drivers/acpi/sleep/poweroff.c
index 1fc86e6b5ab9..186b182c5825 100644
--- a/drivers/acpi/sleep/poweroff.c
+++ b/drivers/acpi/sleep/poweroff.c
@@ -19,8 +19,6 @@
 
 int acpi_sleep_prepare(u32 acpi_state)
 {
-        /* Flag to do not allow second time invocation for S5 state */
-        static int shutdown_prepared = 0;
 #ifdef CONFIG_ACPI_SLEEP
         /* do we have a wakeup address for S2 and S3? */
         /* Here, we support only S4BIOS, those we set the wakeup address */
@@ -38,28 +36,23 @@ int acpi_sleep_prepare(u32 acpi_state)
         acpi_enable_wakeup_device_prep(acpi_state);
 #endif
         if (acpi_state == ACPI_STATE_S5) {
-                /* Check if we were already called */
-                if (shutdown_prepared)
-                        return 0;
                 acpi_wakeup_gpe_poweroff_prepare();
-                shutdown_prepared = 1;
         }
         acpi_enter_sleep_state_prep(acpi_state);
         return 0;
 }
 
+#ifdef CONFIG_PM
+
 void acpi_power_off(void)
 {
+        /* acpi_sleep_prepare(ACPI_STATE_S5) should have already been called */
         printk("%s called\n", __FUNCTION__);
-        acpi_sleep_prepare(ACPI_STATE_S5);
         local_irq_disable();
         /* Some SMP machines only can poweroff in boot CPU */
-        set_cpus_allowed(current, cpumask_of_cpu(0));
         acpi_enter_sleep_state(ACPI_STATE_S5);
 }
 
-#ifdef CONFIG_PM
-
 static int acpi_shutdown(struct sys_device *x)
 {
         return acpi_sleep_prepare(ACPI_STATE_S5);
@@ -75,8 +68,6 @@ static struct sys_device device_acpi = {
         .cls = &acpi_sysclass,
 };
 
-#endif
-
 static int acpi_poweroff_init(void)
 {
         if (!acpi_disabled) {
@@ -86,19 +77,18 @@ static int acpi_poweroff_init(void)
                 status =
                     acpi_get_sleep_type_data(ACPI_STATE_S5, &type_a, &type_b);
                 if (ACPI_SUCCESS(status)) {
-                        pm_power_off = acpi_power_off;
-#ifdef CONFIG_PM
-                        {
-                                int error;
-                                error = sysdev_class_register(&acpi_sysclass);
-                                if (!error)
-                                        error = sysdev_register(&device_acpi);
-                                return error;
-                        }
-#endif
+                        int error;
+                        error = sysdev_class_register(&acpi_sysclass);
+                        if (!error)
+                                error = sysdev_register(&device_acpi);
+                        if (!error)
+                                pm_power_off = acpi_power_off;
+                        return error;
                 }
         }
         return 0;
 }
 
 late_initcall(acpi_poweroff_init);
+
+#endif /* CONFIG_PM */