On Thu, Jun 08, 2017 at 03:30:27PM +0800, Jianhong Chen wrote:
> From: chenjh <che...@rock-chips.com>
> 
> This driver provides a input driver for the power
> key on the Rockchip RK805 PMIC.
> 
> Signed-off-by: chenjh <che...@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 <che...@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 <che...@rock-chips.com>");
> +MODULE_DESCRIPTION("RK805 PMIC Power Key driver");
> +MODULE_LICENSE("GPL v2");
> -- 
> 1.9.1
> 
> 

-- 
Dmitry

Reply via email to