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,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH, MAILING_LIST_MULTI,MENTIONS_GIT_HOSTING,SPF_HELO_NONE,SPF_PASS, 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 6D84AC48BCF for ; Wed, 9 Jun 2021 14:36:45 +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 375D460FF2 for ; Wed, 9 Jun 2021 14:36:45 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 375D460FF2 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=Huawei.com 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=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:References:In-Reply-To: Message-ID:Subject:CC: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=kAD3APIM0NIHDRMahaYebBPHgt1D7h1BPobiLGJmrkk=; b=4Wew16K8m08le4 NBOgoadV3mHmdmIXqU//bAwUR8EfnsSwHTvF4t2iFlifumb0/0VaH1lNrXsohAfcwyVke4epXwrb0 0oVOrH0CajmLOmdrsXyBmiQPm7a2f2dB5TqUBL6bjJyLoI+5NAtMju8HWQuBLtpHtnLWA03LVk7Fe ldFKpfe3sXCUW0l7N4gzV4DzJbz+aiXe2Mh6scDjsfeN5Rb3OxsUCcsBxBrfQPlGfhGrHRVmjDAF+ pZnJxcLcCP0TasTkOwJB7Qd9LZch6hRQaxqV0mFa5E27hyST4wIiNs8SksO1D9EoXOUsy/KfwItPD 4zoMz8hxKEPob76RCEjA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1lqzHw-00EKy4-LD; Wed, 09 Jun 2021 14:34:37 +0000 Received: from frasgout.his.huawei.com ([185.176.79.56]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1lqykY-00EANz-1I for linux-arm-kernel@lists.infradead.org; Wed, 09 Jun 2021 14:00:08 +0000 Received: from fraeml705-chm.china.huawei.com (unknown [172.18.147.206]) by frasgout.his.huawei.com (SkyGuard) with ESMTP id 4G0T5R0L0fz6L69t; Wed, 9 Jun 2021 21:50:35 +0800 (CST) Received: from lhreml710-chm.china.huawei.com (10.201.108.61) by fraeml705-chm.china.huawei.com (10.206.15.54) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.2176.2; Wed, 9 Jun 2021 15:59:54 +0200 Received: from localhost (10.52.124.126) by lhreml710-chm.china.huawei.com (10.201.108.61) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2176.2; Wed, 9 Jun 2021 14:59:54 +0100 Date: Wed, 9 Jun 2021 14:59:51 +0100 From: Jonathan Cameron To: William Breathitt Gray CC: , , , , , , , , , , , , , , , , , Subject: Re: [PATCH v11 00/33] Introduce the Counter character device interface Message-ID: <20210609145951.000005dd@Huawei.com> In-Reply-To: References: Organization: Huawei Technologies Research and Development (UK) Ltd. X-Mailer: Claws Mail 3.17.8 (GTK+ 2.24.33; i686-w64-mingw32) MIME-Version: 1.0 X-Originating-IP: [10.52.124.126] X-ClientProxiedBy: lhreml716-chm.china.huawei.com (10.201.108.67) To lhreml710-chm.china.huawei.com (10.201.108.61) X-CFilter-Loop: Reflected X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20210609_070006_438341_3D5B5D0D X-CRM114-Status: GOOD ( 24.32 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , 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 Wed, 9 Jun 2021 10:31:03 +0900 William Breathitt Gray wrote: > Changes in v11: > - Migrated Intel QEP to new interface > - Context adjustments made for documentation > > I pulled out a lot of bits and pieces to their own patches; hopefully > that makes reviewing this patchset much simpler than before. This > patchset is also available on my personal public git repo for anyone who > wants a quick way to clone: > https://gitlab.com/vilhelmgray/iio/-/tree/counter_chrdev_v11 > > The patches preceding "counter: Internalize sysfs interface code" are > primarily cleanup and fixes that can be picked up and applied now to the > IIO tree if so desired. The "counter: Internalize sysfs interface code" > patch as well may be considered for pickup because it is relatively safe > and makes no changes to the userspace interface. Let us aim to do that in the interests of cutting down on the volume for future versions! However, it is made tricky by the fact patch 22 hasn't made it to me or lore.kernel.org. Could you send that one again. Hopefully the in-reply-to will ensure it's correctly threaded. Jonathan > > To summarize the main points of this patchset: there are no changes to > the existing Counter sysfs userspace interface; a Counter character > device interface is introduced that allows Counter events and associated > data to be read() by userspace; the events_configure() and > watch_validate() driver callbacks are introduced to support Counter > events; and IRQ support is added to the 104-QUAD-8 driver, serving as an > example of how to support the new Counter events functionality. > > Something that should still be discussed: should the struct > counter_event "status" member be 8 bits or 32 bits wide? This member > will provide the return status (system error number) of an event > operation. > > William Breathitt Gray (33): > docs: counter: Consolidate Counter sysfs attributes documentation > docs: counter: Fix spelling > counter: 104-quad-8: Remove pointless comment > counter: 104-quad-8: Return error when invalid mode during > ceiling_write > counter: 104-quad-8: Annotate hardware config module parameter > counter: 104-quad-8: Add const qualifiers for > quad8_preset_register_set > counter: 104-quad-8: Add const qualifier for functions_list array > counter: interrupt-cnt: Add const qualifier for functions_list array > counter: microchip-tcb-capture: Add const qualifier for functions_list > array > counter: stm32-lptimer-cnt: Add const qualifier for functions_list > array > counter: stm32-timer-cnt: Add const qualifier for functions_list array > counter: 104-quad-8: Add const qualifier for actions_list array > counter: ftm-quaddec: Add const qualifier for actions_list array > counter: interrupt-cnt: Add const qualifier for actions_list array > counter: microchip-tcb-capture: Add const qualifier for actions_list > array > counter: stm32-lptimer-cnt: Add const qualifier for actions_list array > counter: stm32-timer-cnt: Add const qualifier for actions_list array > counter: Return error code on invalid modes > counter: Standardize to ERANGE for limit exceeded errors > counter: Rename counter_signal_value to counter_signal_level > counter: Rename counter_count_function to counter_function > counter: Internalize sysfs interface code > counter: Update counter.h comments to reflect sysfs internalization > docs: counter: Update to reflect sysfs internalization > counter: Move counter enums to uapi header > counter: Add character device interface > docs: counter: Document character device interface > tools/counter: Create Counter tools > counter: Implement signalZ_action_component_id sysfs attribute > counter: Implement *_component_id sysfs attributes > counter: Implement events_queue_size sysfs attribute > counter: 104-quad-8: Replace mutex with spinlock > counter: 104-quad-8: Add IRQ support for the ACCES 104-QUAD-8 > > Documentation/ABI/testing/sysfs-bus-counter | 112 +- > .../ABI/testing/sysfs-bus-counter-104-quad-8 | 61 - > .../ABI/testing/sysfs-bus-counter-ftm-quaddec | 16 - > Documentation/driver-api/generic-counter.rst | 368 +++- > .../userspace-api/ioctl/ioctl-number.rst | 1 + > MAINTAINERS | 7 +- > drivers/counter/104-quad-8.c | 739 ++++---- > drivers/counter/Kconfig | 6 +- > drivers/counter/Makefile | 1 + > drivers/counter/counter-chrdev.c | 486 ++++++ > drivers/counter/counter-chrdev.h | 14 + > drivers/counter/counter-core.c | 192 +++ > drivers/counter/counter-sysfs.c | 953 +++++++++++ > drivers/counter/counter-sysfs.h | 13 + > drivers/counter/counter.c | 1496 ----------------- > drivers/counter/ftm-quaddec.c | 61 +- > drivers/counter/intel-qep.c | 150 +- > drivers/counter/interrupt-cnt.c | 75 +- > drivers/counter/microchip-tcb-capture.c | 105 +- > drivers/counter/stm32-lptimer-cnt.c | 176 +- > drivers/counter/stm32-timer-cnt.c | 149 +- > drivers/counter/ti-eqep.c | 221 +-- > include/linux/counter.h | 716 ++++---- > include/linux/counter_enum.h | 45 - > include/uapi/linux/counter.h | 133 ++ > tools/Makefile | 13 +- > tools/counter/Build | 1 + > tools/counter/Makefile | 53 + > tools/counter/counter_example.c | 95 ++ > 29 files changed, 3586 insertions(+), 2872 deletions(-) > delete mode 100644 Documentation/ABI/testing/sysfs-bus-counter-104-quad-8 > delete mode 100644 Documentation/ABI/testing/sysfs-bus-counter-ftm-quaddec > create mode 100644 drivers/counter/counter-chrdev.c > create mode 100644 drivers/counter/counter-chrdev.h > create mode 100644 drivers/counter/counter-core.c > create mode 100644 drivers/counter/counter-sysfs.c > create mode 100644 drivers/counter/counter-sysfs.h > delete mode 100644 drivers/counter/counter.c > delete mode 100644 include/linux/counter_enum.h > create mode 100644 include/uapi/linux/counter.h > create mode 100644 tools/counter/Build > create mode 100644 tools/counter/Makefile > create mode 100644 tools/counter/counter_example.c > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel