Hi Joseph, On Fri, Jul 21, 2017 at 07:18:23PM +0800, Joseph Chen wrote: > This driver provides a input driver for the power > key on the Rockchip RK805 PMIC. > > Signed-off-by: Joseph Chen <chenjh@xxxxxxxxxxxxxx> > --- > 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..e0fc66e > --- /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: Joseph Chen <chenjh@xxxxxxxxxxxxxx> > + * > + * 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; This assignment is not needed because it is done for you by devm_input_allocate_device(). I added pwr->id.bustype = BUS_HOST; > + 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_any_context_irq(&pwr->dev, fall_irq, > + pwrkey_fall_irq, > + IRQF_TRIGGER_FALLING | IRQF_ONESHOT, > + "rk805_pwrkey_fall", pwr); > + if (err < 0) { > + dev_err(&pdev->dev, "Can't register fall irq: %d\n", err); > + return err; > + } > + > + err = devm_request_any_context_irq(&pwr->dev, rise_irq, > + 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("Joseph Chen <chenjh@xxxxxxxxxxxxxx>"); > +MODULE_DESCRIPTION("RK805 PMIC Power Key driver"); > +MODULE_LICENSE("GPL v2"); Given that the license notice says "GPL v2 or later" I changed module license to read just "GPL" (see include/linux/module.h for various strings accepted by MODULE_LICENSE and their meaning). Otherwise applied. Thanks. -- Dmitry -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html