From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751292AbeCHXeY (ORCPT ); Thu, 8 Mar 2018 18:34:24 -0500 Received: from mail-pl0-f67.google.com ([209.85.160.67]:36713 "EHLO mail-pl0-f67.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750783AbeCHXeW (ORCPT ); Thu, 8 Mar 2018 18:34:22 -0500 X-Google-Smtp-Source: AG47ELtrnubJcdsCt/jWS3n6Jif1Qt+mGsT3RwzmEGD6HmHRv6o7u9IZdUDLr4JWHgWLTRqESTj2sw== Date: Thu, 8 Mar 2018 15:34:19 -0800 From: Dmitry Torokhov To: Andrey Smirnov Cc: linux-input@vger.kernel.org, linux-kernel@vger.kernel.org, Rob Herring , Mark Rutland , devicetree@vger.kernel.org, Guenter Roeck , Chris Healy , Lucas Stach Subject: Re: [PATCH v3 1/2] input: Add RAVE SP Powerbutton driver Message-ID: <20180308233419.GB198135@dtor-ws> References: <20180307020134.26293-1-andrew.smirnov@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20180307020134.26293-1-andrew.smirnov@gmail.com> User-Agent: Mutt/1.9.2 (2017-12-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Mar 06, 2018 at 06:01:33PM -0800, Andrey Smirnov wrote: > Add driver that properly handles input event emitted by RAVE SP > devices. > > Cc: Dmitry Torokhov > Cc: linux-input@vger.kernel.org > Cc: linux-kernel@vger.kernel.org > Cc: Rob Herring > Cc: Mark Rutland > Cc: devicetree@vger.kernel.org > Cc: Guenter Roeck > Cc: Chris Healy > Cc: Lucas Stach > Reviewed-by: Lucas Stach > Signed-off-by: Andrey Smirnov Applied, thank you. > --- > > Changes since [v2]: > > - Collected Reviewed-by from Lucas > > Changes since [v1]: > > - Removed redundant dev.parent assignment > > - Various cosmetic changes > > [v2] lkml.kernel.org/r/20180301165527.22274-1-andrew.smirnov@gmail.com > [v1] lkml.kernel.org/r/20180226154130.25774-1-andrew.smirnov@gmail.com > > drivers/input/misc/Kconfig | 9 ++++ > drivers/input/misc/Makefile | 1 + > drivers/input/misc/rave-sp-pwrbutton.c | 94 ++++++++++++++++++++++++++++++++++ > 3 files changed, 104 insertions(+) > create mode 100644 drivers/input/misc/rave-sp-pwrbutton.c > > diff --git a/drivers/input/misc/Kconfig b/drivers/input/misc/Kconfig > index 62a1312a7387..6a3c753b093b 100644 > --- a/drivers/input/misc/Kconfig > +++ b/drivers/input/misc/Kconfig > @@ -841,4 +841,13 @@ config INPUT_HISI_POWERKEY > To compile this driver as a module, choose M here: the > module will be called hisi_powerkey. > > +config INPUT_RAVE_SP_PWRBUTTON > + tristate "RAVE SP Power button Driver" > + depends on RAVE_SP_CORE > + help > + Say Y here if you want to enable power key reporting from RAVE SP > + > + To compile this driver as a module, choose M here: the > + module will be called rave-sp-pwrbutton. > + > endif > diff --git a/drivers/input/misc/Makefile b/drivers/input/misc/Makefile > index a8f61af865aa..8cc58f362bb8 100644 > --- a/drivers/input/misc/Makefile > +++ b/drivers/input/misc/Makefile > @@ -60,6 +60,7 @@ obj-$(CONFIG_INPUT_PMIC8XXX_PWRKEY) += pmic8xxx-pwrkey.o > obj-$(CONFIG_INPUT_POWERMATE) += powermate.o > obj-$(CONFIG_INPUT_PWM_BEEPER) += pwm-beeper.o > obj-$(CONFIG_INPUT_PWM_VIBRA) += pwm-vibra.o > +obj-$(CONFIG_INPUT_RAVE_SP_PWRBUTTON) += rave-sp-pwrbutton.o > obj-$(CONFIG_INPUT_RB532_BUTTON) += rb532_button.o > obj-$(CONFIG_INPUT_REGULATOR_HAPTIC) += regulator-haptic.o > obj-$(CONFIG_INPUT_RETU_PWRBUTTON) += retu-pwrbutton.o > diff --git a/drivers/input/misc/rave-sp-pwrbutton.c b/drivers/input/misc/rave-sp-pwrbutton.c > new file mode 100644 > index 000000000000..bcab3cdb7ebd > --- /dev/null > +++ b/drivers/input/misc/rave-sp-pwrbutton.c > @@ -0,0 +1,94 @@ > +// SPDX-License-Identifier: GPL-2.0+ > +// > +// Power Button driver for RAVE SP > +// > +// Copyright (C) 2017 Zodiac Inflight Innovations > +// > +// > + > +#include > +#include > +#include > +#include > +#include > + > +#define RAVE_SP_EVNT_BUTTON_PRESS (RAVE_SP_EVNT_BASE + 0x00) > + > +struct rave_sp_power_button { > + struct input_dev *idev; > + struct notifier_block nb; > +}; > + > +static int rave_sp_power_button_event(struct notifier_block *nb, > + unsigned long action, void *data) > +{ > + struct rave_sp_power_button *pb = > + container_of(nb, struct rave_sp_power_button, nb); > + const u8 event = rave_sp_action_unpack_event(action); > + const u8 value = rave_sp_action_unpack_value(action); > + struct input_dev *idev = pb->idev; > + > + if (event == RAVE_SP_EVNT_BUTTON_PRESS) { > + input_report_key(idev, KEY_POWER, value); > + input_sync(idev); > + > + return NOTIFY_STOP; > + } > + > + return NOTIFY_DONE; > +} > + > +static int rave_sp_pwrbutton_probe(struct platform_device *pdev) > +{ > + struct device *dev = &pdev->dev; > + struct rave_sp_power_button *pb; > + struct input_dev *idev; > + int error; > + > + pb = devm_kzalloc(dev, sizeof(*pb), GFP_KERNEL); > + if (!pb) > + return -ENOMEM; > + > + idev = devm_input_allocate_device(dev); > + if (!idev) > + return -ENOMEM; > + > + idev->name = pdev->name; > + > + input_set_capability(idev, EV_KEY, KEY_POWER); > + > + error = input_register_device(idev); > + if (error) > + return error; > + > + pb->idev = idev; > + pb->nb.notifier_call = rave_sp_power_button_event; > + pb->nb.priority = 128; > + > + error = devm_rave_sp_register_event_notifier(dev, &pb->nb); > + if (error) > + return error; > + > + return 0; > +} > + > +static const struct of_device_id rave_sp_pwrbutton_of_match[] = { > + { .compatible = "zii,rave-sp-pwrbutton" }, > + {} > +}; > + > +static struct platform_driver rave_sp_pwrbutton_driver = { > + .probe = rave_sp_pwrbutton_probe, > + .driver = { > + .name = KBUILD_MODNAME, > + .of_match_table = rave_sp_pwrbutton_of_match, > + }, > +}; > +module_platform_driver(rave_sp_pwrbutton_driver); > + > +MODULE_DEVICE_TABLE(of, rave_sp_pwrbutton_of_match); > +MODULE_LICENSE("GPL"); > +MODULE_AUTHOR("Andrey Vostrikov "); > +MODULE_AUTHOR("Nikita Yushchenko "); > +MODULE_AUTHOR("Andrey Smirnov "); > +MODULE_DESCRIPTION("RAVE SP Power Button driver"); > -- > 2.14.3 > -- Dmitry