| Message ID | 48abf6ce51f8ef02421ea2dd4f4e309b440efc85.1477929725.git.stwiss.opensource@diasemi.com (mailing list archive) |
|---|---|
| State | Accepted |
| Headers | show |
On Mon, Oct 31, 2016 at 04:02:02PM +0000, Steve Twiss wrote: > From: Steve Twiss <stwiss.opensource@diasemi.com> > > 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 <stwiss.opensource@diasemi.com> 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 >
On Mon, Oct 31, 2016 at 04:02:02PM +0000, Steve Twiss wrote: > From: Steve Twiss <stwiss.opensource@diasemi.com> > > 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 <stwiss.opensource@diasemi.com> > > --- > 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(-) Acked-by: Rob Herring <robh@kernel.org> -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
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; + }; };