Thread (23 messages) 23 messages, 6 authors, 2016-11-11

Re: [PATCH V3 1/9] Documentation: devicetree: input: additions for da9061 onkey driver

From: Rob Herring <robh@kernel.org>
Date: 2016-11-09 18:24:51
Also in: linux-input, linux-pm, linux-watchdog, lkml

On Mon, Oct 31, 2016 at 04:02:02PM +0000, Steve Twiss wrote:
From: Steve Twiss <redacted>

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 <redacted>

---
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>
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help