From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <SRS0=l8JL=JI=gmail.com=dmitry.torokhov@kernel.org>
Date: Fri, 22 Jun 2018 11:53:31 -0700
From: Dmitry Torokhov <dmitry.torokhov@gmail.com>
Subject: Re: [PATCH 0/5]: input: pm8941-pwrkey: Add support for reboot reason
Message-ID: <20180622185331.GE92912@dtor-ws>
References: <20180618143548.29900-1-vkoul@kernel.org>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20180618143548.29900-1-vkoul@kernel.org>
To: Vinod Koul <vinod.koul@linaro.org>
Cc: linux-input@vger.kernel.org, linux-pm@vger.kernel.org, Rob Herring <robh+dt@kernel.org>, Sebastian Reichel <sre@kernel.org>, devicetree@vger.kernel.org, linux-arm-msm@vger.kernel.org, Vinod Koul <vkoul@kernel.org>, Bjorn Andersson <bjorn.andersson@linaro.org>
List-ID: <devicetree@vger.kernel.org>

On Mon, Jun 18, 2018 at 08:05:43PM +0530, Vinod Koul wrote:
> To add support for reboot reason there have been some attempts [1], [2]
> in past. Based on these discussions we added a new pon driver and made
> pwrkey and resin as child nodes to pon.
> 
> Since the pwrkey and resin are similar, abstract pwrkey driver
> and then add support for resin in the same driver.
> 
> Since the patches touch reset and input subsystems, it would be nice if
> folks can ACK and we carry thru one subsystem, I would prefer that to be
> input, other way would work fine too.

This looks reasonable to me, let's add Bjorn.

> 
> [1]: https://patchwork.kernel.org/patch/9751627/
> [2]: https://patchwork.kernel.org/patch/10381801/
> 
> Vinod Koul (5):
>   dt-bindings: power: reset: Add qcom pon binding
>   power: reset: qcom-pon: Add Qcom PON driver
>   dt-bindings: Input: Add additional property to qcom pwrkey
>   input: pm8941-pwrkey: Abstract register offsets and event code
>   input: pm8941-pwrkey: Add resin entry
> 
>  .../bindings/input/qcom,pm8941-pwrkey.txt          |  9 +++
>  .../devicetree/bindings/power/reset/qcom,pon.txt   | 47 +++++++++++
>  drivers/input/misc/pm8941-pwrkey.c                 | 64 ++++++++++++---
>  drivers/power/reset/Kconfig                        |  8 ++
>  drivers/power/reset/Makefile                       |  1 +
>  drivers/power/reset/qcom-pon.c                     | 91 ++++++++++++++++++++++
>  6 files changed, 210 insertions(+), 10 deletions(-)
>  create mode 100644 Documentation/devicetree/bindings/power/reset/qcom,pon.txt
>  create mode 100644 drivers/power/reset/qcom-pon.c
> 
> -- 
> 2.14.4
> 

-- 
Dmitry