Re: [PATCH v6 09/12] Input: Add power key driver for Rockchip RK805 PMIC

Dmitry Torokhov
2017-06-20T03:06:50Z

On Thu, Jun 08, 2017 at 03:30:27PM +0800, Jianhong Chen wrote:
> From: chenjh <chenjh@rock-chips.com>
>
> This driver provides a input driver for the power
> key on the Rockchip RK805 PMIC.
>
> Signed-off-by: chenjh <chenjh@rock-chips.com>
> ---
> drivers/input/misc/Kconfig | 6 +++
> drivers/input/misc/Makefile | 1 +
> drivers/input/misc/rk805-pwrkey.c | 111 ++++++++++++++++++++++++++++++++++++++
> 3 files changed, 118 insertions(+)
> create mode 100644 drivers/input/misc/rk805-pwrkey.c
>
> diff --git a/drivers/input/misc/Kconfig b/drivers/input/misc/Kconfig
> index 5b6c522..72dc4f5 100644
> --- a/drivers/input/misc/Kconfig
> +++ b/drivers/input/misc/Kconfig
> @@ -571,6 +571,12 @@ config INPUT_PWM_BEEPER
> To compile this driver as a module, choose M here: the module will be
> called pwm-beeper.
>
> +config INPUT_RK805_PWRKEY
> + tristate "Rockchip RK805 PMIC power key support"
> + depends on MFD_RK808
> + help
> + Select this option to enable power key driver for RK805.
> +
> config INPUT_GPIO_ROTARY_ENCODER
> tristate "Rotary encoders connected to GPIO pins"
> depends on GPIOLIB || COMPILE_TEST
> diff --git a/drivers/input/misc/Makefile b/drivers/input/misc/Makefile
> index b10523f..c27052a 100644
> --- a/drivers/input/misc/Makefile
> +++ b/drivers/input/misc/Makefile
> @@ -63,6 +63,7 @@ obj-$(CONFIG_INPUT_REGULATOR_HAPTIC) += regulator-haptic.o
> obj-$(CONFIG_INPUT_RETU_PWRBUTTON) += retu-pwrbutton.o
> obj-$(CONFIG_INPUT_AXP20X_PEK) += axp20x-pek.o
> obj-$(CONFIG_INPUT_GPIO_ROTARY_ENCODER) += rotary_encoder.o
> +obj-$(CONFIG_INPUT_RK805_PWRKEY) += rk805-pwrkey.o
> obj-$(CONFIG_INPUT_SGI_BTNS) += sgi_btns.o
> obj-$(CONFIG_INPUT_SIRFSOC_ONKEY) += sirfsoc-onkey.o
> obj-$(CONFIG_INPUT_SOC_BUTTON_ARRAY) += soc_button_array.o
> diff --git a/drivers/input/misc/rk805-pwrkey.c b/drivers/input/misc/rk805-pwrkey.c
> new file mode 100644
> index 0000000..188eed5
> --- /dev/null
> +++ b/drivers/input/misc/rk805-pwrkey.c
> @@ -0,0 +1,111 @@
> +/*
> + * Rockchip RK805 PMIC Power Key driver
> + *
> + * Copyright (c) 2017, Fuzhou Rockchip Electronics Co., Ltd
> + *
> + * Author: Chen Jianhong <chenjh@rock-chips.com>
> + *
> + * 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; either version 2 of the License, or (at your
> + * option) any later version.
> + *
> + */
> +#include <linux/errno.h>
> +#include <linux/init.h>
> +#include <linux/input.h>
> +#include <linux/interrupt.h>
> +#include <linux/kernel.h>
> +#include <linux/module.h>
> +#include <linux/platform_device.h>
> +
> +static irqreturn_t pwrkey_fall_irq(int irq, void *_pwr)
> +{
> + struct input_dev *pwr = _pwr;
> +
> + input_report_key(pwr, KEY_POWER, 1);
> + input_sync(pwr);
> +
> + return IRQ_HANDLED;
> +}
> +
> +static irqreturn_t pwrkey_rise_irq(int irq, void *_pwr)
> +{
> + struct input_dev *pwr = _pwr;
> +
> + input_report_key(pwr, KEY_POWER, 0);
> + input_sync(pwr);
> +
> + return IRQ_HANDLED;
> +}
> +
> +static int rk805_pwrkey_probe(struct platform_device *pdev)
> +{
> + struct input_dev *pwr;
> + int fall_irq, rise_irq;
> + int err;
> +
> + pwr = devm_input_allocate_device(&pdev->dev);
> + if (!pwr) {
> + dev_err(&pdev->dev, "Can't allocate power button\n");
> + return -ENOMEM;
> + }
> +
> + pwr->name = "rk805 pwrkey";
> + pwr->phys = "rk805_pwrkey/input0";
> + pwr->dev.parent = &pdev->dev;
> + input_set_capability(pwr, EV_KEY, KEY_POWER);
> +
> + fall_irq = platform_get_irq(pdev, 0);
> + if (fall_irq < 0) {
> + dev_err(&pdev->dev, "Can't get fall irq: %d\n", fall_irq);
> + return fall_irq;
> + }
> +
> + rise_irq = platform_get_irq(pdev, 1);
> + if (rise_irq < 0) {
> + dev_err(&pdev->dev, "Can't get rise irq: %d\n", rise_irq);
> + return rise_irq;
> + }
> +
> + err = devm_request_threaded_irq(&pwr->dev, fall_irq, NULL,
> + pwrkey_fall_irq,
> + IRQF_TRIGGER_FALLING | IRQF_ONESHOT,
> + "rk805_pwrkey_fall", pwr);

I think you want devm_request_any_context_irq() here as your interrupt
handler does not need to sleep.

> + if (err < 0) {
> + dev_err(&pdev->dev, "Can't register fall irq: %d\n", err);
> + return err;
> + }
> +
> + err = devm_request_threaded_irq(&pwr->dev, rise_irq, NULL,
> + pwrkey_rise_irq,
> + IRQF_TRIGGER_RISING | IRQF_ONESHOT,
> + "rk805_pwrkey_rise", pwr);
> + if (err < 0) {
> + dev_err(&pdev->dev, "Can't register rise irq: %d\n", err);
> + return err;
> + }
> +
> + err = input_register_device(pwr);
> + if (err) {
> + dev_err(&pdev->dev, "Can't register power button: %d\n", err);
> + return err;
> + }
> +
> + platform_set_drvdata(pdev, pwr);
> + device_init_wakeup(&pdev->dev, true);
> +
> + return 0;
> +}
> +
> +static struct platform_driver rk805_pwrkey_driver = {
> + .probe = rk805_pwrkey_probe,
> + .driver = {
> + .name = "rk805-pwrkey",
> + },
> +};
> +module_platform_driver(rk805_pwrkey_driver);
> +
> +MODULE_AUTHOR("Chen Jianghong <chenjh@rock-chips.com>");
> +MODULE_DESCRIPTION("RK805 PMIC Power Key driver");
> +MODULE_LICENSE("GPL v2");
> --
> 1.9.1
>
>

--
Dmitry