linux-kernel - Re: [PATCH v6 0/4] Add initial configfs support for IIO

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CAEnQRZCMkXYv6XO2dG7MRP0XEQm4A5t2zPHWiSdeR=X-9C=CaQ@mail.gmail.com>
Date:	Mon, 11 May 2015 10:31:44 +0300
From:	Daniel Baluta <daniel.baluta@...el.com>
To:	Jonathan Cameron <jic23@...nel.org>,
	Lars-Peter Clausen <lars@...afoo.de>
Cc:	Daniel Baluta <daniel.baluta@...el.com>,
	Joel Becker <jlbec@...lplan.org>,
	Hartmut Knaack <knaack.h@....de>,
	Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
	"linux-iio@...r.kernel.org" <linux-iio@...r.kernel.org>,
	"octavian.purdila@...el.com" <octavian.purdila@...el.com>,
	Paul Bolle <pebolle@...cali.nl>, patrick.porlan@...el.com,
	adriana.reus@...el.com, constantin.musca@...el.com,
	marten@...uitiveaerial.com
Subject: Re: [PATCH v6 0/4] Add initial configfs support for IIO

On Fri, May 8, 2015 at 9:41 PM, Jonathan Cameron <jic23@...nel.org> 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@...r.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@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/