Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752814AbbEKHbt (ORCPT ); Mon, 11 May 2015 03:31:49 -0400 Received: from mail-wi0-f174.google.com ([209.85.212.174]:37275 "EHLO mail-wi0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752264AbbEKHbp (ORCPT ); Mon, 11 May 2015 03:31:45 -0400 MIME-Version: 1.0 In-Reply-To: <554D0347.9000903@kernel.org> References: <1431092025-14355-1-git-send-email-daniel.baluta@intel.com> <554D0347.9000903@kernel.org> Date: Mon, 11 May 2015 10:31:44 +0300 X-Google-Sender-Auth: 7kfJcmx_elai9WnijGimVEhNfjs Message-ID: Subject: Re: [PATCH v6 0/4] Add initial configfs support for IIO From: Daniel Baluta To: Jonathan Cameron , Lars-Peter Clausen Cc: Daniel Baluta , Joel Becker , Hartmut Knaack , Linux Kernel Mailing List , "linux-iio@vger.kernel.org" , "octavian.purdila@intel.com" , Paul Bolle , patrick.porlan@intel.com, adriana.reus@intel.com, constantin.musca@intel.com, marten@intuitiveaerial.com Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 4069 Lines: 89 On Fri, May 8, 2015 at 9:41 PM, Jonathan Cameron wrote: > On 08/05/15 09:33, Daniel Baluta wrote: >> This patchset introduces IIO software triggers, offers a way of configuring >> them via configfs and adds the IIO hrtimer based interrupt source to be used >> with software triggers. >> >> The arhitecture is now split in 3 parts, to remove all IIO trigger specific >> parts from IIO configfs core: >> >> (1) IIO software triggers - are independent of configfs. >> (2) IIO configfs - offers a generic way of creating IIO objects. So far we can >> create software triggers. >> (3) IIO hrtimer trigger - is the first interrupt source for software triggers >> (with syfs to follow). Each trigger type can implement its own set of >> attributes. > I'm happy with the whole series, but given Lars' involvement would like to > give him a bit of time for another look / reviewed-by / acked-by or similar. > > Obviously comments from anyone else welcome as well. > > Now all we need to do is all the other uses for configfs that we've > discussed in earlier threads :) Easy really ;) Agree. :) The important thing now is to get Acked-by/Reviewed-by on the interface because once accepted this cannot be changed. For the rest of the code we can send patches later. >> >> Changes since v5: (after Lars comments) >> * the most important change is that we moved sampling_frequency attribute >> from configfs to trigger's directory in /sys. >> * couple of const added to strings >> * documentation to public API in sw_trigger.h >> * replace pr_err with WARN_ONCE in trigger_make_group to avoid spamming >> kernel log, but without leaving user clueless in case of errors. >> * we still need to decide if we get a real gain by adding min/max limits >> for sampling frequency in /config dir. Anyhow, this can be done in a later >> patch. >> * fix race in hrtimer_remove >> >> Changes since v4: >> * patch 1/4 >> - fixed "new line" nit in industrialio-sw-trigger.c >> - added license header in sw_trigger.h o >> * patch 2/4 >> - none >> * patch 3/4 >> - none >> * patch 4/4 >> - removed "Further work" chapter in iio_configfs.txt >> - added configfs-iio file in Documentation/ABI/testing >> >> Daniel Baluta (4): >> iio: core: Introduce IIO software triggers >> iio: core: Introduce IIO configfs support >> iio: trigger: Introduce IIO hrtimer based trigger >> iio: Documentation: Add IIO configfs documentation >> >> Documentation/ABI/testing/configfs-iio | 20 ++++ >> Documentation/iio/iio_configfs.txt | 58 ++++++++++ >> drivers/iio/Kconfig | 16 +++ >> drivers/iio/Makefile | 2 + >> drivers/iio/industrialio-configfs.c | 117 ++++++++++++++++++++ >> drivers/iio/industrialio-sw-trigger.c | 115 +++++++++++++++++++ >> drivers/iio/trigger/Kconfig | 10 ++ >> drivers/iio/trigger/Makefile | 2 + >> drivers/iio/trigger/iio-trig-hrtimer.c | 194 +++++++++++++++++++++++++++++++++ >> include/linux/iio/sw_trigger.h | 85 +++++++++++++++ >> 10 files changed, 619 insertions(+) >> create mode 100644 Documentation/ABI/testing/configfs-iio >> create mode 100644 Documentation/iio/iio_configfs.txt >> create mode 100644 drivers/iio/industrialio-configfs.c >> create mode 100644 drivers/iio/industrialio-sw-trigger.c >> create mode 100644 drivers/iio/trigger/iio-trig-hrtimer.c >> create mode 100644 include/linux/iio/sw_trigger.h >> > > -- > To unsubscribe from this list: send the line "unsubscribe linux-iio" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/