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=-7.2 required=3.0 tests=DKIM_ADSP_CUSTOM_MED, DKIM_SIGNED,DKIM_VALID,FREEMAIL_FORGED_FROMDOMAIN,FREEMAIL_FROM, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,MENTIONS_GIT_HOSTING, SPF_PASS,USER_AGENT_MUTT 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 F007DC43381 for ; Thu, 7 Mar 2019 12:02:36 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id BE35020835 for ; Thu, 7 Mar 2019 12:02:36 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="U9XN81pW"; dkim=fail reason="signature verification failed" (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="t8m9aVN1" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org BE35020835 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=gmail.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20170209; h=Sender: Content-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=JYyMH9XVJJiL3e4DqSu1qnZVvLpXA+XGZDQ7VXqPVHg=; b=U9XN81pWaGZBVs Dmx9j9anOmKFTxShav8OIo7xZgUGxi8M7X/0+iztBtn5opmAu5EHeFRTMbtdj96dtHjPgEgaux44n RUQLLQbIxrwU1cTUb4iitDWu/o88cd10pK5haeYUZAjDJM0AJcY3QK9AyNbYXYSiN+HOWYAP0jFSc ykNLG0e35Dg5lC9e3GcvTqy4NRDkzQjcLJMWfN4/+ZybYxvswSaLdIYhVPkIMX6sxy/dmvUejjp0n DfIxdEMeMkvCZ4EejaLJf7Rf+ICUSaoUz/3UfPjUOZkMSZ3KU5vaeMlpflFMY8+Le5xsO/ercRc6s owV7LQEnkjx5ji4nMmKw==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.90_1 #2 (Red Hat Linux)) id 1h1rjM-00059n-0g; Thu, 07 Mar 2019 12:02:32 +0000 Received: from mail-pg1-x542.google.com ([2607:f8b0:4864:20::542]) by bombadil.infradead.org with esmtps (Exim 4.90_1 #2 (Red Hat Linux)) id 1h1rjC-00052f-6i for linux-arm-kernel@lists.infradead.org; Thu, 07 Mar 2019 12:02:23 +0000 Received: by mail-pg1-x542.google.com with SMTP id 196so11088991pgf.13 for ; Thu, 07 Mar 2019 04:02:22 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to:user-agent; bh=cCcMg5pc+2dNLPmNs9eB2iLJmfA5xUcfuYlcrGmXBtI=; b=t8m9aVN1NyLjx3zvRRZW/3NykwiPUA78aCbsbuDmMg6BeqZP3fQukxDil6iSqzp+R1 VRJ/qnURD8zjBcIyT69Dp0Mx4S944f0JkP01+J2Xx6VAwtkqbKD6qAvtXJXhcr5tMwQW xBMNXZqsvI717a1lUa3mLvRl5eRL3dOImbNicA6T8lKqEoXpUIYdekx+r18WEfehXGQI L+VQu35YuFAx+cuAp7OOALV89YItWnm49WZwajN4fZhPPkrzUrBJ5+WQOPhG2RvOgULZ GrNR3Bl3xDnlOd5k3ILm3W0jdoR+3vSph4aS7SDa1diIi0acPY+fQZis2rmNX7DOwEok b6Zw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to:user-agent; bh=cCcMg5pc+2dNLPmNs9eB2iLJmfA5xUcfuYlcrGmXBtI=; b=aO7/Tl6XkrmyPZff5Q04SMNb1lkPD/z2vXns5FEPhD9pjMSuTAQBuUbQpw+cYmCrWi TpnE9Y7eKFZUeQ4cQPFnj/+OScBV/cCTb/14c6QK15OXJdBTIMtNMWeJl1uDWF3yHwIC x0dyKi3Nw/Rd9uqJACzpvipyy1dEfSrI+vVpQGhipdJC3x6ugJ0OIEap2VSBivGUMFiv 5n7qkOUkCMotP4wf9Fy3HTKecVQPBNfPWJ0dVkKH/R9adCeNypUSLmjnkzMMz8Jq+n/4 lzBc42rImXGQKNv0vhhWCPVKlD7LIITUZuOpbtWD9UlOm5aodmWctDXy8D9LhgU8gz5s M73A== X-Gm-Message-State: APjAAAXfyzonJBQ9Q6Dx+LZGOocAO1ypjDALxcbaK9cw1Glz557kX3hG MtWKqEBUvzUbD77FwLtujD0= X-Google-Smtp-Source: APXvYqx5MhCcSKXMxnG5wwoct3amjub16sCaozTvfpqlDSxmpqAhXnq4a3i6NMkiQbsgY16Cg59N1A== X-Received: by 2002:a65:64d9:: with SMTP id t25mr11165549pgv.244.1551960141488; Thu, 07 Mar 2019 04:02:21 -0800 (PST) Received: from icarus ([2001:268:c0a4:120d:c70:4af9:86e2:2]) by smtp.gmail.com with ESMTPSA id q62sm14179288pfi.183.2019.03.07.04.02.14 (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Thu, 07 Mar 2019 04:02:20 -0800 (PST) Date: Thu, 7 Mar 2019 21:03:13 +0900 From: William Breathitt Gray To: Patrick Havelange , jic23@kernel.org Subject: Re: [PATCH v2 0/7] FlexTimer Module Quadrature decoder counter Message-ID: <20190307120224.GC7851@icarus> References: <20190306111208.7454-1-patrick.havelange@essensium.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20190306111208.7454-1-patrick.havelange@essensium.com> User-Agent: Mutt/1.11.3 (2019-02-01) X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20190307_040222_240455_BDD0EA19 X-CRM114-Status: GOOD ( 18.40 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Mark Rutland , devicetree@vger.kernel.org, linux-pwm@vger.kernel.org, linux-iio@vger.kernel.org, linuxppc-dev@lists.ozlabs.org, Daniel Lezcano , linux-kernel@vger.kernel.org, Li Yang , Rob Herring , Thierry Reding , linux-arm-kernel@lists.infradead.org, Thomas Gleixner , Shawn Guo , Esben Haabendal Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Wed, Mar 06, 2019 at 12:12:01PM +0100, Patrick Havelange wrote: > This patch serie is to be applied on top of > https://patchwork.kernel.org/project/linux-iio/list/?series=147 > (a more recent version of the serie is available here : > https://gitlab.com/vilhelmgray/iio/tree/generic_counter_v10 ) > > Main changes in v2: > The code is a bit simpler, thanks to more use of devm_* functions. > The polling/32bit signed version has been dropped, as not needed and > no other driver is doing that. > > > Patrick Havelange (7): > include/fsl: add common FlexTimer #defines in a separate header. > drivers/pwm: pwm-fsl-ftm: use common header for FlexTimer #defines > drivers/clocksource: timer-fsl-ftm: use common header for FlexTimer > #defines > dt-bindings: counter: ftm-quaddec > counter: add FlexTimer Module Quadrature decoder counter driver > counter: ftm-quaddec: Documentation: Add specific counter sysfs > documentation > LS1021A: dtsi: add ftm quad decoder entries > > .../ABI/testing/sysfs-bus-counter-ftm-quaddec | 16 + > .../bindings/counter/ftm-quaddec.txt | 18 + > arch/arm/boot/dts/ls1021a.dtsi | 28 ++ > drivers/clocksource/timer-fsl-ftm.c | 15 +- > drivers/counter/Kconfig | 9 + > drivers/counter/Makefile | 1 + > drivers/counter/ftm-quaddec.c | 356 ++++++++++++++++++ > drivers/pwm/pwm-fsl-ftm.c | 44 +-- > include/linux/fsl/ftm.h | 88 +++++ > 9 files changed, 519 insertions(+), 56 deletions(-) > create mode 100644 Documentation/ABI/testing/sysfs-bus-counter-ftm-quaddec > create mode 100644 Documentation/devicetree/bindings/counter/ftm-quaddec.txt > create mode 100644 drivers/counter/ftm-quaddec.c > create mode 100644 include/linux/fsl/ftm.h > > -- > 2.19.1 Patrick, I see you dropped the polling support in this version. If the need arises in the future, we can discuss a possible ways of resolving your latency issues; I imagine interrupts during overflow/underflow events to be a common behavior among counter devices so those too may result in latency issues as you discovered in your case. Jonathan, If you are satisfied with the changes in this patchset, let me know which patches you like and I'll add respective Reviewed-by tags for you for the next Counter subsystem introduction patchset submission. William Breathitt Gray _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel