[PATCH 2/2][resend] gpio_keys: Add suspend and resume

[Anti Sullin]

This patch adds suspend/resume support and enables wakeup from gpio_keys 
buttons.

Signed-off-by: Anti Sullin <anti.sullin@xxxxxxxxxxxxxx>

---
Compared to  previous patch sent, in this version the disable_irq_wake is
called only for those IRQs, that were enabled earlier.

---
diff -pur linux-2.6.23-rc5-old/drivers/input/keyboard/gpio_keys.c linux-2.6.23-rc5/drivers/input/keyboard/gpio_keys.c
--- linux-2.6.23-rc5-old/drivers/input/keyboard/gpio_keys.c	2007-09-18 17:43:54.000000000 +0300
+++ linux-2.6.23-rc5/drivers/input/keyboard/gpio_keys.c	2007-09-18 17:25:14.000000000 +0300
@@ -54,6 +54,7 @@ static int __devinit gpio_keys_probe(str
	struct gpio_keys_platform_data *pdata = pdev->dev.platform_data;
	struct input_dev *input;
	int i, error;
+	int wakeup = 0;

	input = input_allocate_device();
	if (!input)
@@ -89,6 +90,9 @@ static int __devinit gpio_keys_probe(str
			goto fail;
		}

+		if (button->wakeup) 
+			wakeup = 1;
+		
		input_set_capability(input, type, button->code);
	}

@@ -98,6 +102,8 @@ static int __devinit gpio_keys_probe(str
		goto fail;
	}

+	device_init_wakeup(&pdev->dev, wakeup);
+	
	return 0;

 fail:
@@ -120,14 +126,55 @@ static int __devexit gpio_keys_remove(st
		free_irq(irq, pdev);
	}

+	device_init_wakeup(&pdev->dev, 0);
	input_unregister_device(input);

	return 0;
}

+
+#ifdef CONFIG_PM
+static int gpio_keys_suspend(struct platform_device *pdev, pm_message_t state)
+{
+	struct gpio_keys_platform_data *pdata = pdev->dev.platform_data;
+	int i;
+
+	if (device_may_wakeup(&pdev->dev)) {
+		for (i = 0; i < pdata->nbuttons; i++) {
+			int irq = gpio_to_irq(pdata->buttons[i].gpio);
+			if (pdata->buttons[i].wakeup)
+				enable_irq_wake(irq);
+		}
+	}
+
+	return 0;
+}
+
+static int gpio_keys_resume(struct platform_device *pdev)
+{
+	struct gpio_keys_platform_data *pdata = pdev->dev.platform_data;
+	int i;
+
+	if (device_may_wakeup(&pdev->dev)) {
+		for (i = 0; i < pdata->nbuttons; i++) {
+			int irq = gpio_to_irq(pdata->buttons[i].gpio);
+			if (pdata->buttons[i].wakeup)
+				disable_irq_wake(irq);
+		}
+	}
+
+	return 0;
+}
+#else
+#define gpio_keys_suspend	NULL
+#define gpio_keys_resume	NULL
+#endif
+
struct platform_driver gpio_keys_device_driver = {
	.probe		= gpio_keys_probe,
	.remove		= __devexit_p(gpio_keys_remove),
+	.suspend	= gpio_keys_suspend,
+	.resume		= gpio_keys_resume,	
	.driver		= {
		.name	= "gpio-keys",
	}
diff -pur linux-2.6.23-rc5-old/include/linux/gpio_keys.h linux-2.6.23-rc5/include/linux/gpio_keys.h
--- linux-2.6.23-rc5-old/include/linux/gpio_keys.h	2007-09-01 09:08:24.000000000 +0300
+++ linux-2.6.23-rc5/include/linux/gpio_keys.h	2007-09-10 10:43:39.000000000 +0300
@@ -8,6 +8,7 @@ struct gpio_keys_button {
	int active_low;
	char *desc;
	int type;		/* input event type (EV_KEY, EV_SW) */
+	int wakeup;		/* configure the button as wake-up source from suspend */
};

struct gpio_keys_platform_data {


--
Anti Sullin
Embedded Software Engineer
Artec Design LLC
Türi 10C, 11313, Tallinn, Estonia
Phone: +3726718550; Direct: +3726718564
Fax: +3726718555; GSM: +3725583068
email: anti.sullin@xxxxxxxxxxxxxx
http://www.artecdesign.ee 

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/