da9030_battery: Fix race between event handler and monitor

[pandora-kernel.git] / arch / arm / mach-sa1100 / collie_pm.c

diff --git a/arch/arm/mach-sa1100/collie_pm.c b/arch/arm/mach-sa1100/collie_pm.c
index b39307f..444f266 100644 (file)
--- a/arch/arm/mach-sa1100/collie_pm.c
+++ b/arch/arm/mach-sa1100/collie_pm.c
@@ -22,6 +22,7 @@
 #include <linux/interrupt.h>
 #include <linux/device.h>
 #include <linux/platform_device.h>
+#include <linux/gpio.h>
 
 #include <asm/irq.h>
 #include <mach/hardware.h>
@@ -58,6 +59,9 @@ static void collie_charger_init(void)
                return;
        }
 
+       gpio_request(COLLIE_GPIO_CHARGE_ON, "charge on");
+       gpio_direction_output(COLLIE_GPIO_CHARGE_ON, 1);
+
        ucb1x00_io_set_dir(ucb, 0, COLLIE_TC35143_GPIO_MBAT_ON | COLLIE_TC35143_GPIO_TMP_ON |
                                   COLLIE_TC35143_GPIO_BBAT_ON);
        return;
@@ -73,17 +77,11 @@ static void collie_measure_temp(int on)
 
 static void collie_charge(int on)
 {
-       extern struct platform_device colliescoop_device;
-
        /* Zaurus seems to contain LTC1731; it should know when to
         * stop charging itself, so setting charge on should be
         * relatively harmless (as long as it is not done too often).
         */
-       if (on) {
-               set_scoop_gpio(&colliescoop_device.dev, COLLIE_SCP_CHARGE_ON);
-       } else {
-               reset_scoop_gpio(&colliescoop_device.dev, COLLIE_SCP_CHARGE_ON);
-       }
+       gpio_set_value(COLLIE_GPIO_CHARGE_ON, on);
 }
 
 static void collie_discharge(int on)