From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-6.8 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 85DA2C2BBC7 for ; Thu, 9 Apr 2020 22:39:15 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 687D52083E for ; Thu, 9 Apr 2020 22:39:15 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726795AbgDIWjL (ORCPT ); Thu, 9 Apr 2020 18:39:11 -0400 Received: from relay9-d.mail.gandi.net ([217.70.183.199]:58675 "EHLO relay9-d.mail.gandi.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726689AbgDIWjL (ORCPT ); Thu, 9 Apr 2020 18:39:11 -0400 X-Originating-IP: 86.202.105.35 Received: from localhost (lfbn-lyo-1-9-35.w86-202.abo.wanadoo.fr [86.202.105.35]) (Authenticated sender: alexandre.belloni@bootlin.com) by relay9-d.mail.gandi.net (Postfix) with ESMTPSA id 78876FF803; Thu, 9 Apr 2020 22:39:07 +0000 (UTC) Date: Fri, 10 Apr 2020 00:39:07 +0200 From: Alexandre Belloni To: Dmitry Torokhov Cc: Kamel Bouhara , William Breathitt Gray , Rob Herring , Mark Rutland , Nicolas Ferre , Ludovic Desroches , linux-arm-kernel@lists.infradead.org, Thomas Petazzoni , linux-input@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-iio@vger.kernel.org Subject: Re: [PATCH 2/3] Input: rotary-encoder-counter: add DT bindings Message-ID: <20200409223907.GW3628@piout.net> References: <20200406155806.1295169-1-kamel.bouhara@bootlin.com> <20200406155806.1295169-3-kamel.bouhara@bootlin.com> <20200409222115.GT75430@dtor-ws> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20200409222115.GT75430@dtor-ws> Sender: linux-input-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-input@vger.kernel.org Hi Dmitry, On 09/04/2020 15:21:15-0700, Dmitry Torokhov wrote: > On Mon, Apr 06, 2020 at 05:58:05PM +0200, Kamel Bouhara wrote: > > Add dt binding for the counter variant of the rotary encoder driver. > > > > Signed-off-by: Kamel Bouhara > > --- > > .../input/rotary-encoder-counter.yaml | 67 +++++++++++++++++++ > > 1 file changed, 67 insertions(+) > > create mode 100644 Documentation/devicetree/bindings/input/rotary-encoder-counter.yaml > > > > diff --git a/Documentation/devicetree/bindings/input/rotary-encoder-counter.yaml b/Documentation/devicetree/bindings/input/rotary-encoder-counter.yaml > > new file mode 100644 > > index 000000000000..a59f7c1faf0c > > --- /dev/null > > +++ b/Documentation/devicetree/bindings/input/rotary-encoder-counter.yaml > > @@ -0,0 +1,67 @@ > > +# SPDX-License-Identifier: GPL-2.0 > > +%YAML 1.2 > > +--- > > +$id: http://devicetree.org/schemas/input/rotary-encoder-counter.yaml# > > +$schema: http://devicetree.org/meta-schemas/core.yaml# > > + > > +title: Rotary Encoder Counter > > + > > +maintainers: > > + - Kamel Bouhara > > + > > +description: > > + Registers a Rotary encoder connected through a counter device. > > + > > +properties: > > + compatible: > > + const: rotary-encoder-counter > > I wonder if a separate driver is really needed. The original driver be > taught to use counter device when available? > By the original driver, do you mean drivers/input/misc/rotary_encoder.c that is using gpios ? > > + > > + counter: > > + description: Phandle for the counter device providing rotary position. > > + > > + linux-axis: > > + description: The input subsystem axis to map to this rotary encoder. > > + type: boolean > > + > > + qdec-mode: > > + description: | > > + Quadrature decoder function to set in the counter device. > > + 3: x1-PHA > > + 4: x1-PHB > > + 5: x2-PHA > > + 6: x2-PHB > > + 7: x4-PHA and PHB > > Is it really property of the rotary encoder itself or property of the > counter device? > The mode the quadrature decoder has to be put in depends on both the rotary encoder and the qdec. > > + > > + steps: > > + description: Number of steps in a full turnaround of the encoder. > > + Only relevant for absolute axis. Defaults to 24 which is a typical > > + value for such devices. > > + > > + relative-axis: > > + description: Register a relative axis rather than an absolute one. > > + type: boolean > > + > > + rollover: > > + description: Automatic rollover when the rotary value becomes greater > > + than the specified steps or smaller than 0. For absolute axis only. > > + type: boolean > > + > > + poll-interval: > > + description: Poll interval at which the position is read from the counter > > + device (default 500ms). > > Is there a way found counters to signal an interrupt? > For some counters, there are interrupts available, this is not trivial with the counter that is the target of this work but this is on the TODO list. Of course, this will also require adding a bit more to the in-kernel counter API to allow registering a callback that would be called when an interrupt happens. -- Alexandre Belloni, Bootlin Embedded Linux and Kernel engineering https://bootlin.com