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 Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id DCF0CC43334 for ; Sat, 16 Jul 2022 14:46:43 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229558AbiGPOqm (ORCPT ); Sat, 16 Jul 2022 10:46:42 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:50856 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229499AbiGPOqm (ORCPT ); Sat, 16 Jul 2022 10:46:42 -0400 Received: from dfw.source.kernel.org (dfw.source.kernel.org [139.178.84.217]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 18E48193E1; Sat, 16 Jul 2022 07:46:41 -0700 (PDT) Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id A57026114B; Sat, 16 Jul 2022 14:46:40 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 07D40C34114; Sat, 16 Jul 2022 14:46:38 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1657982800; bh=4QvxNZ2FXer/MBSeMYgiz4DE9IC/aVS4+NEw/i4U0Ko=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=rQ+/yDpHD//TaP3shmZ7TDgyJPkg7FsWk6bTFHqWFhOJgdT2hRbYVbB4Snk03n+ye Mj11rhvoPsjsYwpoEAf0uy2WYHGFWMBsLJO4UXrwSqIqDKopN4DgW983sW1TwUVpGf d4tziTHjvSDKCxFvzkx3CS31hCrNxoWEJVBfON1xDTUpteEQes6dZb2gaa/w6wj06d +Lm/bJPyP3/fDYYcOh8gMivy2qhRh7p9xtwjSkdHkTuRcJu+XCNRMd1Bya5WRUVlUN 0lAxaKV6uRW7Zxrbd9mDm7oAMVik8/wQsCSpdAEFfmJalwSk+UEKDyTaIALbm8x4UA Nv60f5440OTlQ== Date: Sat, 16 Jul 2022 15:56:33 +0100 From: Jonathan Cameron To: Jagath Jog J Cc: Andy Shevchenko , linux-iio , Linux Kernel Mailing List Subject: Re: [PATCH v1 1/2] iio: Add new event type gesture and use direction for single and double tap Message-ID: <20220716155633.244b9243@jic23-huawei> In-Reply-To: References: <20220613191706.31239-1-jagathjog1996@gmail.com> <20220613191706.31239-2-jagathjog1996@gmail.com> <20220619135427.4ffc2d3d@jic23-huawei> X-Mailer: Claws Mail 4.1.0 (GTK 3.24.34; x86_64-pc-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-iio@vger.kernel.org On Sun, 26 Jun 2022 18:51:54 +0530 Jagath Jog J wrote: > Hi Jonathan, > > Sorry for the delay in replying. > > On Sun, Jun 19, 2022 at 6:15 PM Jonathan Cameron wrote: > > > > On Tue, 14 Jun 2022 00:47:05 +0530 > > Jagath Jog J wrote: > > > > > Add new event type for tap called gesture and the direction can be used > > > to differentiate single and double tap. This may be used by accelerometer > > > sensors to express single and double tap events. For directional tap, > > > modifiers like IIO_MOD_(X/Y/Z) can be used along with singletap and > > > doubletap direction. > > > > > > Signed-off-by: Jagath Jog J > > > > Hi, > > > > With fresh eyes I think we need to rethink the use of _period to make sure > > we have 'space' for another very like ABI element which is the maximum > > time between events for them to be considered a double tap. > > > > Jonathan > > > > > --- > > > Documentation/ABI/testing/sysfs-bus-iio | 24 ++++++++++++++++++++++++ > > > drivers/iio/industrialio-event.c | 5 ++++- > > > include/uapi/linux/iio/types.h | 3 +++ > > > tools/iio/iio_event_monitor.c | 8 +++++++- > > > 4 files changed, 38 insertions(+), 2 deletions(-) > > > > > > diff --git a/Documentation/ABI/testing/sysfs-bus-iio b/Documentation/ABI/testing/sysfs-bus-iio > > > index 3e00d7f7ee22..4eaf85e01911 100644 > > > --- a/Documentation/ABI/testing/sysfs-bus-iio > > > +++ b/Documentation/ABI/testing/sysfs-bus-iio > > > @@ -2035,3 +2035,27 @@ Description: > > > Available range for the forced calibration value, expressed as: > > > > > > - a range specified as "[min step max]" > > > + > > > +What: /sys/.../events/in_accel_gesture_singletap_en > > > +What: /sys/.../events/in_accel_gesture_doubletap_en > > > +KernelVersion: 5.19 > > > +Contact: linux-iio@vger.kernel.org > > > +Description: > > > + Device generates an event on a single or double tap. > > > + > > > +What: /sys/.../events/in_accel_gesture_singletap_value > > > +What: /sys/.../events/in_accel_gesture_doubletap_value > > > +KernelVersion: 5.19 > > > +Contact: linux-iio@vger.kernel.org > > > +Description: > > > + Specifies the threshold value that the device is comparing > > > + against to generate the tap gesture event. Units and exact > > > + meaning of value are device specific. > > > > I'm fine with this one being device specific, as likely a complex alg > > involved. > > > > > + > > > +What: /sys/.../events/in_accel_gesture_doubletap_period > > > +KernelVersion: 5.19 > > > +Contact: linux-iio@vger.kernel.org > > > +Description: > > > + Minimum time period between before and after the double tap > > > + event. Units and exact meaning of period value are device > > > + specific. > > > > I think the units need to be standard. Also would this work better > > as a description? > > > > Minimum time in seconds between the two taps making up a double > > tap event. > > The values for the time between the two taps are not in terms of seconds, > here period value is in terms of data samples which depends on > the data rate. These time based controls in IIO are always defined in seconds. You'll have to correct for the data rate in the driver. Either just make the available options change with data rate, or have the driver do a 'nearest possible' choice based on what was requested and what is possible at the current data rate. It's more complex but it gives a consistent ABI across devices that work in many different ways. Jonathan > > > > > Raises a question though. How would we specify the maximum time? I.e. > > if taps are further apart than Xseconds, they are reported as two single > > taps. > > > > Maybe reusing period here isn't a good idea and we need to have new ABI for > > this? > > > > > > > >