Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753732AbcKIARI (ORCPT ); Tue, 8 Nov 2016 19:17:08 -0500 Received: from mail-pf0-f196.google.com ([209.85.192.196]:35696 "EHLO mail-pf0-f196.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932094AbcKIARE (ORCPT ); Tue, 8 Nov 2016 19:17:04 -0500 Date: Tue, 8 Nov 2016 16:17:00 -0800 From: Dmitry Torokhov To: Steve Twiss Cc: DEVICETREE , LINUX-INPUT , LINUX-KERNEL , Mark Rutland , Rob Herring , Eduardo Valentin , Guenter Roeck , LINUX-PM , LINUX-WATCHDOG , Lee Jones , Liam Girdwood , Mark Brown , Support Opensource , Wim Van Sebroeck , Zhang Rui Subject: Re: [PATCH V3 1/9] Documentation: devicetree: input: additions for da9061 onkey driver Message-ID: <20161109001700.GC8719@dtor-ws> References: <48abf6ce51f8ef02421ea2dd4f4e309b440efc85.1477929725.git.stwiss.opensource@diasemi.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <48abf6ce51f8ef02421ea2dd4f4e309b440efc85.1477929725.git.stwiss.opensource@diasemi.com> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 5071 Lines: 141 On Mon, Oct 31, 2016 at 04:02:02PM +0000, Steve Twiss wrote: > From: Steve Twiss > > Add binding information for DA9061 onkey. > > This patch updates the compatible string "dlg,da9061-onkey" to support > DA9061, removes the reference to KEY_SLEEP (which the driver no longer > supports) and fixes a typo in the example for DA9063. > > Supporting KEY_SLEEP was not the general convention and the typical > solution should have been for KEY_POWER to support both cases of suspend > and S/W power off. This change was sent to the DA9063 ONKEY device > driver in a separate patch, but the documentation was not updated at > that time. > - f889bea Report KEY_POWER instead of KEY_SLEEP during power key-press > > This patch also adds two new examples, one for DA9062 and one for DA9061. > The DA9061 examples uses a fall-back compatible string for the DA9062 > onkey driver. > > Signed-off-by: Steve Twiss Applied, thank you. > > --- > This patch applies against linux-next and v4.8 > > v2 -> v3 > - Patch renamed from [PATCH V2 01/10] to [PATCH V3 1/9] > - Each compatible line should be a valid combination of compatible > strings, alter DA9061 line to include the fall back compatible string > - Update the commit message to include KEY_SLEEP removal explanation and > a link back to the original device driver commit. This will allow full > traceability back to the original patch change > - Link in information about associated patches from this set without > describing them as being explicitly dependent on this binding > > v1 -> v2 > - Patch renamed from [PATCH V1 06/10] to [PATCH V2 01/10] -- these > changes were made to fix checkpatch warnings caused by the patch > set dependency order > - Typo s/ther/the/ in commit message > - Explanation about why KEY_SLEEP was removed (see below) > - Addition of DA9062 example > - Addition of a DA9061 example to follow the driver fall-back compatible > convention being applied for this device driver > > Hi, > > There is also new binding examples for DA9062 and DA9061. Importantly, > the Linux device driver changes for DA9061 were rejected because the > DA9062 device driver can be reused. For this reason, the DA9061 example > uses a fall-back compatible string. > > Other information: > The device driver from this patch set (associated with this binding) was > applied by Dmitry Torokhov on 26-Oct-2016. See: > - [PATCH V2 07/10] Input: da9061: onkey driver. > - https://lkml.org/lkml/2016/10/26/1169 > > Regards, > Steve Twiss, Dialog Semiconductor Ltd. > > > .../devicetree/bindings/input/da9062-onkey.txt | 45 ++++++++++++++-------- > 1 file changed, 30 insertions(+), 15 deletions(-) > > diff --git a/Documentation/devicetree/bindings/input/da9062-onkey.txt b/Documentation/devicetree/bindings/input/da9062-onkey.txt > index ab0e048..5f9fbc6 100644 > --- a/Documentation/devicetree/bindings/input/da9062-onkey.txt > +++ b/Documentation/devicetree/bindings/input/da9062-onkey.txt > @@ -1,32 +1,47 @@ > -* Dialog DA9062/63 OnKey Module > +* Dialog DA9061/62/63 OnKey Module > > -This module is part of the DA9062/DA9063. For more details about entire > -chips see Documentation/devicetree/bindings/mfd/da9062.txt and > -Documentation/devicetree/bindings/mfd/da9063.txt > +This module is part of the DA9061/DA9062/DA9063. For more details about entire > +DA9062 and DA9061 chips see Documentation/devicetree/bindings/mfd/da9062.txt > +For DA9063 see Documentation/devicetree/bindings/mfd/da9063.txt > > -This module provides KEY_POWER, KEY_SLEEP and events. > +This module provides the KEY_POWER event. > > Required properties: > > -- compatible: should be one of: > - dlg,da9062-onkey > - dlg,da9063-onkey > +- compatible: should be one of the following valid compatible string lines: > + "dlg,da9061-onkey", "dlg,da9062-onkey" > + "dlg,da9062-onkey" > + "dlg,da9063-onkey" > > Optional properties: > > - - dlg,disable-key-power : Disable power-down using a long key-press. If this > +- dlg,disable-key-power : Disable power-down using a long key-press. If this > entry exists the OnKey driver will remove support for the KEY_POWER key > - press. If this entry does not exist then by default the key-press > - triggered power down is enabled and the OnKey will support both KEY_POWER > - and KEY_SLEEP. > + press when triggered using a long press of the OnKey. > > -Example: > - > - pmic0: da9062@58 { > +Example: DA9063 > > + pmic0: da9063@58 { > onkey { > compatible = "dlg,da9063-onkey"; > dlg,disable-key-power; > }; > + }; > + > +Example: DA9062 > + > + pmic0: da9062@58 { > + onkey { > + compatible = "dlg,da9062-onkey"; > + dlg,disable-key-power; > + }; > + }; > + > +Example: DA9061 using a fall-back compatible for the DA9062 onkey driver > > + pmic0: da9061@58 { > + onkey { > + compatible = "dlg,da9061-onkey", "dlg,da9062-onkey"; > + dlg,disable-key-power; > + }; > }; > -- > end-of-patch for PATCH V3 > -- Dmitry