[PATCH v2 0/3] NI Ettus Research USRP E3x0 Button driver

[PATCH v2 0/3] NI Ettus Research USRP E3x0 Button driver

[Moritz Fischer]

This patchset adds NI Ettus Research USRP E3x0 power button support and
corresponding devicetree bindings and MAINTAINER entries.

It was developed and tested on our NI Ettus Research USRP E310 embedded
SDR on top of 3.19-rc3.

Moritz Fischer (3):
  input: misc: Support NI Ettus Research USRP E3x0 Button
  doc: dt: Add documentation for e3x0-button bindings.
  Update MAINTAINERS file with e3x0-button info.

 .../devicetree/bindings/input/e3x0-button.txt      |  25 ++++
 MAINTAINERS                                        |   8 ++
 drivers/input/misc/Kconfig                         |  10 ++
 drivers/input/misc/Makefile                        |   1 +
 drivers/input/misc/e3x0-button.c                   | 132 +++++++++++++++++++++
 5 files changed, 176 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/input/e3x0-button.txt
 create mode 100644 drivers/input/misc/e3x0-button.c

-- 
1.9.3

[PATCH v2 1/3] input: misc: Support NI Ettus Research USRP E3x0 Button

[Moritz Fischer]

Signed-off-by: Moritz Fischer <moritz.fischer@ettus.com>
---
 drivers/input/misc/Kconfig       |  10 +++
 drivers/input/misc/Makefile      |   1 +
 drivers/input/misc/e3x0-button.c | 132 +++++++++++++++++++++++++++++++++++++++
 3 files changed, 143 insertions(+)

diff --git a/drivers/input/misc/Kconfig b/drivers/input/misc/Kconfig
index 23297ab..84a56b4 100644
--- a/drivers/input/misc/Kconfig
+++ b/drivers/input/misc/Kconfig
@@ -93,6 +93,16 @@ config INPUT_BMA150
 	  To compile this driver as a module, choose M here: the
 	  module will be called bma150.
 
+config INPUT_E3X0_BUTTON
+	tristate "NI Ettus Research USRP E3x0 Button support."
+	default n
+	help
+	  Say Y here to enable support for the NI Ettus Research
+	  USRP E3x0 Button.
+
+	  To compile this driver as a module, choose M here: the
+	  module will be called e3x0_button.
+
 config INPUT_PCSPKR
 	tristate "PC Speaker support"
 	depends on PCSPKR_PLATFORM
diff --git a/drivers/input/misc/Makefile b/drivers/input/misc/Makefile
index 19c7603..b1775a5 100644
--- a/drivers/input/misc/Makefile
+++ b/drivers/input/misc/Makefile
@@ -26,6 +26,7 @@ obj-$(CONFIG_INPUT_COBALT_BTNS)		+= cobalt_btns.o
 obj-$(CONFIG_INPUT_DA9052_ONKEY)	+= da9052_onkey.o
 obj-$(CONFIG_INPUT_DA9055_ONKEY)	+= da9055_onkey.o
 obj-$(CONFIG_INPUT_DM355EVM)		+= dm355evm_keys.o
+obj-$(CONFIG_INPUT_E3X0_BUTTON)		+= e3x0-button.o
 obj-$(CONFIG_INPUT_DRV260X_HAPTICS)	+= drv260x.o
 obj-$(CONFIG_INPUT_DRV2667_HAPTICS)	+= drv2667.o
 obj-$(CONFIG_INPUT_GP2A)		+= gp2ap002a00f.o
diff --git a/drivers/input/misc/e3x0-button.c b/drivers/input/misc/e3x0-button.c
new file mode 100644
index 0000000..0f4e7f3
--- /dev/null
+++ b/drivers/input/misc/e3x0-button.c
@@ -0,0 +1,132 @@
+/*
+ * Copyright (c) 2014, National Instruments Corp. All rights reserved.
+ *
+ * Driver for NI Ettus Research USRP E3x0 Button Driver
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; version 2 of the License.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU General Public License for more details.
+ */
+
+#include <linux/device.h>
+#include <linux/kernel.h>
+#include <linux/module.h>
+#include <linux/platform_device.h>
+#include <linux/input.h>
+#include <linux/interrupt.h>
+#include <linux/of.h>
+#include <linux/slab.h>
+
+static irqreturn_t e3x0_button_release_handler(int irq, void *data)
+{
+	struct input_dev *idev = data;
+
+	input_report_key(idev, KEY_POWER, 0);
+	input_sync(idev);
+
+	return IRQ_HANDLED;
+}
+
+static irqreturn_t e3x0_button_press_handler(int irq, void *data)
+{
+	struct input_dev *idev = data;
+
+	input_report_key(idev, KEY_POWER, 1);
+	input_sync(idev);
+
+	return IRQ_HANDLED;
+}
+
+static int e3x0_button_probe(struct platform_device *pdev)
+{
+	struct input_dev *input;
+	int irq_press, irq_release;
+	int error;
+
+	irq_press = platform_get_irq_byname(pdev, "press");
+	if (irq_press < 0) {
+		dev_err(&pdev->dev, "No IRQ for 'press', error=%d\n",
+			irq_press);
+		return irq_press;
+	}
+
+	irq_release = platform_get_irq_byname(pdev, "release");
+	if (irq_release < 0) {
+		dev_err(&pdev->dev, "No IRQ for 'release', error=%d\n",
+			irq_release);
+		return irq_release;
+	}
+
+	input = devm_input_allocate_device(&pdev->dev);
+	if (!input)
+		return -ENOMEM;
+
+	input->name = "NI Ettus Research USRP E3x0 Button Driver";
+	input->phys = "e3x0_button/input0";
+	input->dev.parent = &pdev->dev;
+
+	input_set_capability(input, EV_KEY, KEY_POWER);
+
+	error = devm_request_irq(&pdev->dev, irq_press,
+					     e3x0_button_press_handler, 0,
+					     "e3x0-button", input);
+	if (error < 0) {
+		dev_err(&pdev->dev, "Failed to request 'press' IRQ#%d: %d\n",
+			irq_press, error);
+		return error;
+	}
+
+	error = devm_request_irq(&pdev->dev, irq_release,
+					     e3x0_button_release_handler, 0,
+					     "e3x0-button", input);
+	if (error < 0) {
+		dev_err(&pdev->dev, "Failed to request 'release' IRQ#%d: %d\n",
+			irq_release, error);
+		return error;
+	}
+
+	error = input_register_device(input);
+	if (error) {
+		dev_err(&pdev->dev, "Can't register input device: %d\n", error);
+		return error;
+	}
+
+	platform_set_drvdata(pdev, input);
+	device_init_wakeup(&pdev->dev, 1);
+	return 0;
+}
+
+static int e3x0_button_remove(struct platform_device *pdev)
+{
+	device_init_wakeup(&pdev->dev, 0);
+	return 0;
+}
+
+#ifdef CONFIG_OF
+static const struct of_device_id e3x0_button_match[] = {
+	{ .compatible = "ettus,e3x0-button", },
+	{}
+};
+MODULE_DEVICE_TABLE(of, e3x0_button_match);
+#endif
+
+static struct platform_driver e3x0_button_driver = {
+	.driver		= {
+		.name	= "e3x0-button",
+		.of_match_table = of_match_ptr(e3x0_button_match),
+	},
+	.probe		= e3x0_button_probe,
+	.probe		= e3x0_button_remove,
+};
+
+module_platform_driver(e3x0_button_driver);
+
+MODULE_LICENSE("GPL v2");
+MODULE_AUTHOR("Moritz Fischer <moritz.fischer@ettus.com>");
+MODULE_DESCRIPTION("NI Ettus Research USRP E3x0 Button driver");
+MODULE_ALIAS("platform:e3x0-button");
-- 
1.9.3

[PATCH v2 2/3] doc: dt: Add documentation for e3x0-button bindings.

[Moritz Fischer]

Signed-off-by: Moritz Fischer <moritz.fischer@ettus.com>
---
 .../devicetree/bindings/input/e3x0-button.txt      | 25 ++++++++++++++++++++++
 1 file changed, 25 insertions(+)

diff --git a/Documentation/devicetree/bindings/input/e3x0-button.txt b/Documentation/devicetree/bindings/input/e3x0-button.txt
new file mode 100644
index 0000000..751665e
--- /dev/null
+++ b/Documentation/devicetree/bindings/input/e3x0-button.txt
@@ -0,0 +1,25 @@
+National Instruments Ettus Research USRP E3x0 button driver
+
+This module is part of the NI Ettus Research USRP E3x0 SDR.
+
+This module provides a simple power button event via two interrupts.
+
+Required properties:
+- compatible: should be one of the following
+  - "ettus,e3x0-button": For devices such as the NI Ettus Research USRP E3x0
+- interrupt-parent:
+  - a phandle to the interrupt controller that it is attached to.
+- interrupts: should be one of the following
+  - <0 30 1>, <0 31 1>: For devices such as the NI Ettus Research USRP E3x0
+- interrupt-names: should be one of the following
+  - "press", "release": For devices such as the NI Ettus Research USRP E3x0
+
+Note: Interrupt numbers might vary depending on the FPGA configuration.
+
+Example:
+	button {
+		compatible = "ettus,e3x0-button";
+		interrupt-parent = <&intc>;
+		interrupts = <0 30 1>, <0 31 1>;
+		interrupt-names = "press", "release";
+	}
-- 
1.9.3

[PATCH v2 3/3] Update MAINTAINERS file with e3x0-button info.

[Moritz Fischer]

Signed-off-by: Moritz Fischer <moritz.fischer@ettus.com>
---
 MAINTAINERS | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index ddb9ac8..65358d1 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -3466,6 +3466,14 @@ M:	"Maciej W. Rozycki" <macro@linux-mips.org>
 S:	Maintained
 F:	drivers/tty/serial/dz.*
 
+E3X0 POWER BUTTON DRIVER
+M:	Moritz Fischer <moritz.fischer@ettus.com>
+L:	usrp-users@lists.ettus.com
+W:	http://www.ettus.com
+S:	Supported
+F:	drivers/input/misc/e3x0-button.c
+F:	Documentation/devicetree/bindings/input/e3x0-button.txt
+
 E4000 MEDIA DRIVER
 M:	Antti Palosaari <crope@iki.fi>
 L:	linux-media@vger.kernel.org
-- 
1.9.3

Re: [PATCH v2 1/3] input: misc: Support NI Ettus Research USRP E3x0 Button

[Dmitry Torokhov]

On Thu, Jan 08, 2015 at 11:43:09PM +0100, Moritz Fischer wrote:
> Signed-off-by: Moritz Fischer <moritz.fischer@ettus.com>
> ---
>  drivers/input/misc/Kconfig       |  10 +++
>  drivers/input/misc/Makefile      |   1 +
>  drivers/input/misc/e3x0-button.c | 132 +++++++++++++++++++++++++++++++++++++++
>  3 files changed, 143 insertions(+)
> 
> diff --git a/drivers/input/misc/Kconfig b/drivers/input/misc/Kconfig
> index 23297ab..84a56b4 100644
> --- a/drivers/input/misc/Kconfig
> +++ b/drivers/input/misc/Kconfig
> @@ -93,6 +93,16 @@ config INPUT_BMA150
>  	  To compile this driver as a module, choose M here: the
>  	  module will be called bma150.
>  
> +config INPUT_E3X0_BUTTON
> +	tristate "NI Ettus Research USRP E3x0 Button support."
> +	default n
> +	help
> +	  Say Y here to enable support for the NI Ettus Research
> +	  USRP E3x0 Button.
> +
> +	  To compile this driver as a module, choose M here: the
> +	  module will be called e3x0_button.
> +
>  config INPUT_PCSPKR
>  	tristate "PC Speaker support"
>  	depends on PCSPKR_PLATFORM
> diff --git a/drivers/input/misc/Makefile b/drivers/input/misc/Makefile
> index 19c7603..b1775a5 100644
> --- a/drivers/input/misc/Makefile
> +++ b/drivers/input/misc/Makefile
> @@ -26,6 +26,7 @@ obj-$(CONFIG_INPUT_COBALT_BTNS)		+= cobalt_btns.o
>  obj-$(CONFIG_INPUT_DA9052_ONKEY)	+= da9052_onkey.o
>  obj-$(CONFIG_INPUT_DA9055_ONKEY)	+= da9055_onkey.o
>  obj-$(CONFIG_INPUT_DM355EVM)		+= dm355evm_keys.o
> +obj-$(CONFIG_INPUT_E3X0_BUTTON)		+= e3x0-button.o
>  obj-$(CONFIG_INPUT_DRV260X_HAPTICS)	+= drv260x.o
>  obj-$(CONFIG_INPUT_DRV2667_HAPTICS)	+= drv2667.o
>  obj-$(CONFIG_INPUT_GP2A)		+= gp2ap002a00f.o
> diff --git a/drivers/input/misc/e3x0-button.c b/drivers/input/misc/e3x0-button.c
> new file mode 100644
> index 0000000..0f4e7f3
> --- /dev/null
> +++ b/drivers/input/misc/e3x0-button.c
> @@ -0,0 +1,132 @@
> +/*
> + * Copyright (c) 2014, National Instruments Corp. All rights reserved.
> + *
> + * Driver for NI Ettus Research USRP E3x0 Button Driver
> + *
> + * This program is free software; you can redistribute it and/or modify
> + * it under the terms of the GNU General Public License as published by
> + * the Free Software Foundation; version 2 of the License.
> + *
> + * This program is distributed in the hope that it will be useful,
> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
> + * GNU General Public License for more details.
> + */
> +
> +#include <linux/device.h>
> +#include <linux/kernel.h>
> +#include <linux/module.h>
> +#include <linux/platform_device.h>
> +#include <linux/input.h>
> +#include <linux/interrupt.h>
> +#include <linux/of.h>
> +#include <linux/slab.h>
> +
> +static irqreturn_t e3x0_button_release_handler(int irq, void *data)
> +{
> +	struct input_dev *idev = data;
> +
> +	input_report_key(idev, KEY_POWER, 0);
> +	input_sync(idev);
> +
> +	return IRQ_HANDLED;
> +}
> +
> +static irqreturn_t e3x0_button_press_handler(int irq, void *data)
> +{
> +	struct input_dev *idev = data;
> +
> +	input_report_key(idev, KEY_POWER, 1);
> +	input_sync(idev);
> +
> +	return IRQ_HANDLED;
> +}
> +
> +static int e3x0_button_probe(struct platform_device *pdev)
> +{
> +	struct input_dev *input;
> +	int irq_press, irq_release;
> +	int error;
> +
> +	irq_press = platform_get_irq_byname(pdev, "press");
> +	if (irq_press < 0) {
> +		dev_err(&pdev->dev, "No IRQ for 'press', error=%d\n",
> +			irq_press);
> +		return irq_press;
> +	}
> +
> +	irq_release = platform_get_irq_byname(pdev, "release");
> +	if (irq_release < 0) {
> +		dev_err(&pdev->dev, "No IRQ for 'release', error=%d\n",
> +			irq_release);
> +		return irq_release;
> +	}
> +
> +	input = devm_input_allocate_device(&pdev->dev);
> +	if (!input)
> +		return -ENOMEM;
> +
> +	input->name = "NI Ettus Research USRP E3x0 Button Driver";
> +	input->phys = "e3x0_button/input0";
> +	input->dev.parent = &pdev->dev;
> +
> +	input_set_capability(input, EV_KEY, KEY_POWER);
> +
> +	error = devm_request_irq(&pdev->dev, irq_press,
> +					     e3x0_button_press_handler, 0,
> +					     "e3x0-button", input);
> +	if (error < 0) {
> +		dev_err(&pdev->dev, "Failed to request 'press' IRQ#%d: %d\n",
> +			irq_press, error);
> +		return error;
> +	}
> +
> +	error = devm_request_irq(&pdev->dev, irq_release,
> +					     e3x0_button_release_handler, 0,
> +					     "e3x0-button", input);
> +	if (error < 0) {
> +		dev_err(&pdev->dev, "Failed to request 'release' IRQ#%d: %d\n",
> +			irq_release, error);
> +		return error;
> +	}
> +
> +	error = input_register_device(input);
> +	if (error) {
> +		dev_err(&pdev->dev, "Can't register input device: %d\n", error);
> +		return error;
> +	}
> +
> +	platform_set_drvdata(pdev, input);
> +	device_init_wakeup(&pdev->dev, 1);
> +	return 0;
> +}
> +
> +static int e3x0_button_remove(struct platform_device *pdev)
> +{
> +	device_init_wakeup(&pdev->dev, 0);
> +	return 0;
> +}
> +
> +#ifdef CONFIG_OF
> +static const struct of_device_id e3x0_button_match[] = {
> +	{ .compatible = "ettus,e3x0-button", },
> +	{}
> +};
> +MODULE_DEVICE_TABLE(of, e3x0_button_match);
> +#endif
> +
> +static struct platform_driver e3x0_button_driver = {
> +	.driver		= {
> +		.name	= "e3x0-button",
> +		.of_match_table = of_match_ptr(e3x0_button_match),
> +	},
> +	.probe		= e3x0_button_probe,
> +	.probe		= e3x0_button_remove,

Nope ;)

BTW, I think you need to implement PM ops in the driver and do

|-------if (device_may_wakeup(dev))
|-------|-------enable_irq_wake(platform_get_irq_byname(pdev, "press"));

in suspend and undo it in resume.

Thanks.

-- 
Dmitry

Re: [PATCH v2 1/3] input: misc: Support NI Ettus Research USRP E3x0 Button

[Moritz Fischer]

On Thu, Jan 8, 2015 at 11:50 PM, Dmitry Torokhov
<dmitry.torokhov@gmail.com> wrote:
>
> On Thu, Jan 08, 2015 at 11:43:09PM +0100, Moritz Fischer wrote:
> > Signed-off-by: Moritz Fischer <moritz.fischer@ettus.com>
> > ---
> >  drivers/input/misc/Kconfig       |  10 +++
> >  drivers/input/misc/Makefile      |   1 +
> >  drivers/input/misc/e3x0-button.c | 132 +++++++++++++++++++++++++++++++++++++++
> >  3 files changed, 143 insertions(+)
> >
> > diff --git a/drivers/input/misc/Kconfig b/drivers/input/misc/Kconfig
> > index 23297ab..84a56b4 100644
> > --- a/drivers/input/misc/Kconfig
> > +++ b/drivers/input/misc/Kconfig
> > @@ -93,6 +93,16 @@ config INPUT_BMA150
> >         To compile this driver as a module, choose M here: the
> >         module will be called bma150.
> >
> > +config INPUT_E3X0_BUTTON
> > +     tristate "NI Ettus Research USRP E3x0 Button support."
> > +     default n
> > +     help
> > +       Say Y here to enable support for the NI Ettus Research
> > +       USRP E3x0 Button.
> > +
> > +       To compile this driver as a module, choose M here: the
> > +       module will be called e3x0_button.
> > +
> >  config INPUT_PCSPKR
> >       tristate "PC Speaker support"
> >       depends on PCSPKR_PLATFORM
> > diff --git a/drivers/input/misc/Makefile b/drivers/input/misc/Makefile
> > index 19c7603..b1775a5 100644
> > --- a/drivers/input/misc/Makefile
> > +++ b/drivers/input/misc/Makefile
> > @@ -26,6 +26,7 @@ obj-$(CONFIG_INPUT_COBALT_BTNS)             += cobalt_btns.o
> >  obj-$(CONFIG_INPUT_DA9052_ONKEY)     += da9052_onkey.o
> >  obj-$(CONFIG_INPUT_DA9055_ONKEY)     += da9055_onkey.o
> >  obj-$(CONFIG_INPUT_DM355EVM)         += dm355evm_keys.o
> > +obj-$(CONFIG_INPUT_E3X0_BUTTON)              += e3x0-button.o
> >  obj-$(CONFIG_INPUT_DRV260X_HAPTICS)  += drv260x.o
> >  obj-$(CONFIG_INPUT_DRV2667_HAPTICS)  += drv2667.o
> >  obj-$(CONFIG_INPUT_GP2A)             += gp2ap002a00f.o
> > diff --git a/drivers/input/misc/e3x0-button.c b/drivers/input/misc/e3x0-button.c
> > new file mode 100644
> > index 0000000..0f4e7f3
> > --- /dev/null
> > +++ b/drivers/input/misc/e3x0-button.c
> > @@ -0,0 +1,132 @@
> > +/*
> > + * Copyright (c) 2014, National Instruments Corp. All rights reserved.
> > + *
> > + * Driver for NI Ettus Research USRP E3x0 Button Driver
> > + *
> > + * This program is free software; you can redistribute it and/or modify
> > + * it under the terms of the GNU General Public License as published by
> > + * the Free Software Foundation; version 2 of the License.
> > + *
> > + * This program is distributed in the hope that it will be useful,
> > + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
> > + * GNU General Public License for more details.
> > + */
> > +
> > +#include <linux/device.h>
> > +#include <linux/kernel.h>
> > +#include <linux/module.h>
> > +#include <linux/platform_device.h>
> > +#include <linux/input.h>
> > +#include <linux/interrupt.h>
> > +#include <linux/of.h>
> > +#include <linux/slab.h>
> > +
> > +static irqreturn_t e3x0_button_release_handler(int irq, void *data)
> > +{
> > +     struct input_dev *idev = data;
> > +
> > +     input_report_key(idev, KEY_POWER, 0);
> > +     input_sync(idev);
> > +
> > +     return IRQ_HANDLED;
> > +}
> > +
> > +static irqreturn_t e3x0_button_press_handler(int irq, void *data)
> > +{
> > +     struct input_dev *idev = data;
> > +
> > +     input_report_key(idev, KEY_POWER, 1);
> > +     input_sync(idev);
> > +
> > +     return IRQ_HANDLED;
> > +}
> > +
> > +static int e3x0_button_probe(struct platform_device *pdev)
> > +{
> > +     struct input_dev *input;
> > +     int irq_press, irq_release;
> > +     int error;
> > +
> > +     irq_press = platform_get_irq_byname(pdev, "press");
> > +     if (irq_press < 0) {
> > +             dev_err(&pdev->dev, "No IRQ for 'press', error=%d\n",
> > +                     irq_press);
> > +             return irq_press;
> > +     }
> > +
> > +     irq_release = platform_get_irq_byname(pdev, "release");
> > +     if (irq_release < 0) {
> > +             dev_err(&pdev->dev, "No IRQ for 'release', error=%d\n",
> > +                     irq_release);
> > +             return irq_release;
> > +     }
> > +
> > +     input = devm_input_allocate_device(&pdev->dev);
> > +     if (!input)
> > +             return -ENOMEM;
> > +
> > +     input->name = "NI Ettus Research USRP E3x0 Button Driver";
> > +     input->phys = "e3x0_button/input0";
> > +     input->dev.parent = &pdev->dev;
> > +
> > +     input_set_capability(input, EV_KEY, KEY_POWER);
> > +
> > +     error = devm_request_irq(&pdev->dev, irq_press,
> > +                                          e3x0_button_press_handler, 0,
> > +                                          "e3x0-button", input);
> > +     if (error < 0) {
> > +             dev_err(&pdev->dev, "Failed to request 'press' IRQ#%d: %d\n",
> > +                     irq_press, error);
> > +             return error;
> > +     }
> > +
> > +     error = devm_request_irq(&pdev->dev, irq_release,
> > +                                          e3x0_button_release_handler, 0,
> > +                                          "e3x0-button", input);
> > +     if (error < 0) {
> > +             dev_err(&pdev->dev, "Failed to request 'release' IRQ#%d: %d\n",
> > +                     irq_release, error);
> > +             return error;
> > +     }
> > +
> > +     error = input_register_device(input);
> > +     if (error) {
> > +             dev_err(&pdev->dev, "Can't register input device: %d\n", error);
> > +             return error;
> > +     }
> > +
> > +     platform_set_drvdata(pdev, input);
> > +     device_init_wakeup(&pdev->dev, 1);
> > +     return 0;
> > +}
> > +
> > +static int e3x0_button_remove(struct platform_device *pdev)
> > +{
> > +     device_init_wakeup(&pdev->dev, 0);
> > +     return 0;
> > +}
> > +
> > +#ifdef CONFIG_OF
> > +static const struct of_device_id e3x0_button_match[] = {
> > +     { .compatible = "ettus,e3x0-button", },
> > +     {}
> > +};
> > +MODULE_DEVICE_TABLE(of, e3x0_button_match);
> > +#endif
> > +
> > +static struct platform_driver e3x0_button_driver = {
> > +     .driver         = {
> > +             .name   = "e3x0-button",
> > +             .of_match_table = of_match_ptr(e3x0_button_match),
> > +     },
> > +     .probe          = e3x0_button_probe,
> > +     .probe          = e3x0_button_remove,
>
> Nope ;)

D'Oh ... sorry about that.

> BTW, I think you need to implement PM ops in the driver and do
>
> |-------if (device_may_wakeup(dev))
> |-------|-------enable_irq_wake(platform_get_irq_byname(pdev, "press"));
>
> in suspend and undo it in resume.

I submitted a v3 including the changes. Please let me know if there's
other stuff that needs fixing.

Thanks for taking the time!

Cheers,

Moritz