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=-15.5 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,INCLUDES_CR_TRAILER,INCLUDES_PATCH,MAILING_LIST_MULTI, SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED,USER_AGENT_SANE_2 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 EE386C433DB for ; Sun, 14 Feb 2021 16:59:37 +0000 (UTC) Received: from merlin.infradead.org (merlin.infradead.org [205.233.59.134]) (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 7A1A3614A7 for ; Sun, 14 Feb 2021 16:59:37 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 7A1A3614A7 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=kernel.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+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=merlin.20170209; h=Sender:Content-Transfer-Encoding: Content-Type:Cc:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:MIME-Version:References:In-Reply-To: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=4BigwQuiD6S3BqbYPR4s7trioMJEVSgmxYWabt18FKw=; b=P3Z2FvZY2FNnpwVljZgqrhfA0 3JRB1EBeb3KJxITGub+rlwyXxWp9i6RN4XxwDrk9SE8EcXDKuSqdSWSidwr5qa180lLwPvcboVBre LxA9PE10oNho4YXlmUu6912zkU9+Bs31zkZ+4c6DKRQ1mixID99xakyol8/VtIphGecsunk0Qf3ph 5u612S/0OJIbqL2NFrusJtQLLt/lLvUhKFga/6JQgNXrLVQ5c6/xTKfrkUPenIf4YXWR9tdKc4+F1 w3CxhK0zUL9al9QH4H1mQ7lD7AS3oLx+csPCFjdiGC6WUKRDbVBJQL68asL9oI+CYYgzDU+BDGtt5 hAQtSdyHA==; Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1lBKin-000253-9I; Sun, 14 Feb 2021 16:58:09 +0000 Received: from mail.kernel.org ([198.145.29.99]) by merlin.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1lBKik-00024f-0i for linux-arm-kernel@lists.infradead.org; Sun, 14 Feb 2021 16:58:07 +0000 Received: from archlinux (cpc108967-cmbg20-2-0-cust86.5-4.cable.virginm.net [81.101.6.87]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id 43E00614A7; Sun, 14 Feb 2021 16:57:59 +0000 (UTC) Date: Sun, 14 Feb 2021 16:57:52 +0000 From: Jonathan Cameron To: William Breathitt Gray Subject: Re: [PATCH v8 01/22] docs: counter: Consolidate Counter sysfs attributes documentation Message-ID: <20210214165752.59d41f98@archlinux> In-Reply-To: <362a71a1eba98108d009db8c929170a6ed8cf110.1613131238.git.vilhelm.gray@gmail.com> References: <362a71a1eba98108d009db8c929170a6ed8cf110.1613131238.git.vilhelm.gray@gmail.com> X-Mailer: Claws Mail 3.17.8 (GTK+ 2.24.33; x86_64-pc-linux-gnu) MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20210214_115806_213123_E4B7283C X-CRM114-Status: GOOD ( 23.54 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: kamel.bouhara@bootlin.com, gwendal@chromium.org, a.fatoum@pengutronix.de, david@lechnology.com, linux-iio@vger.kernel.org, patrick.havelange@essensium.com, alexandre.belloni@bootlin.com, mcoquelin.stm32@gmail.com, linux-kernel@vger.kernel.org, o.rempel@pengutronix.de, kernel@pengutronix.de, fabrice.gasnier@st.com, syednwaris@gmail.com, linux-stm32@st-md-mailman.stormreply.com, linux-arm-kernel@lists.infradead.org, alexandre.torgue@st.com Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Fri, 12 Feb 2021 21:13:25 +0900 William Breathitt Gray wrote: > Duplicate ABIs are not valid, so let's consolidate these sysfs > attributes into the main sysfs-bus-counter documentation file. This is sensible to do, but is there any duplicate ABI? I went looking for any resulting build problems and couldn't see what was actually duplicated. Jonathan > > Cc: Patrick Havelange > Signed-off-by: William Breathitt Gray > --- > Documentation/ABI/testing/sysfs-bus-counter | 76 ++++++++++++++++++- > .../ABI/testing/sysfs-bus-counter-104-quad-8 | 61 --------------- > .../ABI/testing/sysfs-bus-counter-ftm-quaddec | 16 ---- > MAINTAINERS | 4 +- > 4 files changed, 75 insertions(+), 82 deletions(-) > delete mode 100644 Documentation/ABI/testing/sysfs-bus-counter-104-quad-8 > delete mode 100644 Documentation/ABI/testing/sysfs-bus-counter-ftm-quaddec > > diff --git a/Documentation/ABI/testing/sysfs-bus-counter b/Documentation/ABI/testing/sysfs-bus-counter > index 566bd99fe0a5..29f54484183f 100644 > --- a/Documentation/ABI/testing/sysfs-bus-counter > +++ b/Documentation/ABI/testing/sysfs-bus-counter > @@ -57,6 +57,7 @@ Description: > What: /sys/bus/counter/devices/counterX/countY/count_mode_available > What: /sys/bus/counter/devices/counterX/countY/error_noise_available > What: /sys/bus/counter/devices/counterX/countY/function_available > +What: /sys/bus/counter/devices/counterX/countY/prescaler_available > What: /sys/bus/counter/devices/counterX/countY/signalZ_action_available > KernelVersion: 5.2 > Contact: linux-iio@vger.kernel.org > @@ -154,6 +155,15 @@ Description: > Count Y. If possible, this should match the name of the > respective channel as it appears in the device datasheet. > > +What: /sys/bus/counter/devices/counterX/countY/prescaler > +KernelVersion: 5.2 > +Contact: linux-iio@vger.kernel.org > +Description: > + Configure the prescaler value associated with Count Y. > + On the FlexTimer, the counter clock source passes through a > + prescaler (i.e. a counter). This acts like a clock > + divider. > + > What: /sys/bus/counter/devices/counterX/countY/preset > KernelVersion: 5.2 > Contact: linux-iio@vger.kernel.org > @@ -215,11 +225,45 @@ Description: > Read-only attribute that indicates the total number of Signals > belonging to the Counter. > > -What: /sys/bus/counter/devices/counterX/signalY/signal > +What: /sys/bus/counter/devices/counterX/signalY/cable_fault > +KernelVersion: 5.7 > +Contact: linux-iio@vger.kernel.org > +Description: > + Read-only attribute that indicates whether a differential > + encoder cable fault (not connected or loose wires) is detected > + for the respective channel of Signal Y. Valid attribute values > + are boolean. Detection must first be enabled via the > + corresponding cable_fault_enable attribute. > + > +What: /sys/bus/counter/devices/counterX/signalY/cable_fault_enable > +KernelVersion: 5.7 > +Contact: linux-iio@vger.kernel.org > +Description: > + Whether detection of differential encoder cable faults for the > + respective channel of Signal Y is enabled. Valid attribute > + values are boolean. > + > +What: /sys/bus/counter/devices/counterX/signalY/filter_clock_prescaler > +KernelVersion: 5.7 > +Contact: linux-iio@vger.kernel.org > +Description: > + Filter clock factor for input Signal Y. This prescaler value > + affects the inputs of both quadrature pair signals. > + > +What: /sys/bus/counter/devices/counterX/signalY/index_polarity > KernelVersion: 5.2 > Contact: linux-iio@vger.kernel.org > Description: > - Signal data of Signal Y represented as a string. > + Active level of index input Signal Y; irrelevant in > + non-synchronous load mode. > + > +What: /sys/bus/counter/devices/counterX/signalY/index_polarity_available > +What: /sys/bus/counter/devices/counterX/signalY/synchronous_mode_available > +KernelVersion: 5.2 > +Contact: linux-iio@vger.kernel.org > +Description: > + Discrete set of available values for the respective Signal Y > + configuration are listed in this file. > > What: /sys/bus/counter/devices/counterX/signalY/name > KernelVersion: 5.2 > @@ -228,3 +272,31 @@ Description: > Read-only attribute that indicates the device-specific name of > Signal Y. If possible, this should match the name of the > respective signal as it appears in the device datasheet. > + > +What: /sys/bus/counter/devices/counterX/signalY/signal > +KernelVersion: 5.2 > +Contact: linux-iio@vger.kernel.org > +Description: > + Signal data of Signal Y represented as a string. > + > +What: /sys/bus/counter/devices/counterX/signalY/synchronous_mode > +KernelVersion: 5.2 > +Contact: linux-iio@vger.kernel.org > +Description: > + Configure the counter associated with Signal Y for > + non-synchronous or synchronous load mode. Synchronous load mode > + cannot be selected in non-quadrature (Pulse-Direction) clock > + mode. > + > + non-synchronous: > + A logic low level is the active level at this index > + input. The index function (as enabled via preset_enable) > + is performed directly on the active level of the index > + input. > + > + synchronous: > + Intended for interfacing with encoder Index output in > + quadrature clock mode. The active level is configured > + via index_polarity. The index function (as enabled via > + preset_enable) is performed synchronously with the > + quadrature clock on the active level of the index input. > diff --git a/Documentation/ABI/testing/sysfs-bus-counter-104-quad-8 b/Documentation/ABI/testing/sysfs-bus-counter-104-quad-8 > deleted file mode 100644 > index eac32180c40d..000000000000 > --- a/Documentation/ABI/testing/sysfs-bus-counter-104-quad-8 > +++ /dev/null > @@ -1,61 +0,0 @@ > -What: /sys/bus/counter/devices/counterX/signalY/cable_fault > -KernelVersion: 5.7 > -Contact: linux-iio@vger.kernel.org > -Description: > - Read-only attribute that indicates whether a differential > - encoder cable fault (not connected or loose wires) is detected > - for the respective channel of Signal Y. Valid attribute values > - are boolean. Detection must first be enabled via the > - corresponding cable_fault_enable attribute. > - > -What: /sys/bus/counter/devices/counterX/signalY/cable_fault_enable > -KernelVersion: 5.7 > -Contact: linux-iio@vger.kernel.org > -Description: > - Whether detection of differential encoder cable faults for the > - respective channel of Signal Y is enabled. Valid attribute > - values are boolean. > - > -What: /sys/bus/counter/devices/counterX/signalY/filter_clock_prescaler > -KernelVersion: 5.7 > -Contact: linux-iio@vger.kernel.org > -Description: > - Filter clock factor for input Signal Y. This prescaler value > - affects the inputs of both quadrature pair signals. > - > -What: /sys/bus/counter/devices/counterX/signalY/index_polarity > -KernelVersion: 5.2 > -Contact: linux-iio@vger.kernel.org > -Description: > - Active level of index input Signal Y; irrelevant in > - non-synchronous load mode. > - > -What: /sys/bus/counter/devices/counterX/signalY/index_polarity_available > -What: /sys/bus/counter/devices/counterX/signalY/synchronous_mode_available > -KernelVersion: 5.2 > -Contact: linux-iio@vger.kernel.org > -Description: > - Discrete set of available values for the respective Signal Y > - configuration are listed in this file. > - > -What: /sys/bus/counter/devices/counterX/signalY/synchronous_mode > -KernelVersion: 5.2 > -Contact: linux-iio@vger.kernel.org > -Description: > - Configure the counter associated with Signal Y for > - non-synchronous or synchronous load mode. Synchronous load mode > - cannot be selected in non-quadrature (Pulse-Direction) clock > - mode. > - > - non-synchronous: > - A logic low level is the active level at this index > - input. The index function (as enabled via preset_enable) > - is performed directly on the active level of the index > - input. > - > - synchronous: > - Intended for interfacing with encoder Index output in > - quadrature clock mode. The active level is configured > - via index_polarity. The index function (as enabled via > - preset_enable) is performed synchronously with the > - quadrature clock on the active level of the index input. > diff --git a/Documentation/ABI/testing/sysfs-bus-counter-ftm-quaddec b/Documentation/ABI/testing/sysfs-bus-counter-ftm-quaddec > deleted file mode 100644 > index 7d2e7b363467..000000000000 > --- a/Documentation/ABI/testing/sysfs-bus-counter-ftm-quaddec > +++ /dev/null > @@ -1,16 +0,0 @@ > -What: /sys/bus/counter/devices/counterX/countY/prescaler_available > -KernelVersion: 5.2 > -Contact: linux-iio@vger.kernel.org > -Description: > - Discrete set of available values for the respective Count Y > - configuration are listed in this file. Values are delimited by > - newline characters. > - > -What: /sys/bus/counter/devices/counterX/countY/prescaler > -KernelVersion: 5.2 > -Contact: linux-iio@vger.kernel.org > -Description: > - Configure the prescaler value associated with Count Y. > - On the FlexTimer, the counter clock source passes through a > - prescaler (i.e. a counter). This acts like a clock > - divider. > diff --git a/MAINTAINERS b/MAINTAINERS > index e697044d34d6..d858582c917b 100644 > --- a/MAINTAINERS > +++ b/MAINTAINERS > @@ -297,7 +297,6 @@ M: William Breathitt Gray > M: Syed Nayyar Waris > L: linux-iio@vger.kernel.org > S: Maintained > -F: Documentation/ABI/testing/sysfs-bus-counter-104-quad-8 > F: drivers/counter/104-quad-8.c > > ACCES PCI-IDIO-16 GPIO DRIVER > @@ -4540,7 +4539,7 @@ COUNTER SUBSYSTEM > M: William Breathitt Gray > L: linux-iio@vger.kernel.org > S: Maintained > -F: Documentation/ABI/testing/sysfs-bus-counter* > +F: Documentation/ABI/testing/sysfs-bus-counter > F: Documentation/driver-api/generic-counter.rst > F: drivers/counter/ > F: include/linux/counter.h > @@ -6925,7 +6924,6 @@ FLEXTIMER FTM-QUADDEC DRIVER > M: Patrick Havelange > L: linux-iio@vger.kernel.org > S: Maintained > -F: Documentation/ABI/testing/sysfs-bus-counter-ftm-quaddec > F: Documentation/devicetree/bindings/counter/ftm-quaddec.txt > F: drivers/counter/ftm-quaddec.c > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel