mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Mike Frysinger <vapier.adi@gmail.com>
To: Greg Kroah-Hartman <gregkh@suse.de>, Kay Sievers <kay.sievers@vrfy.org>
Cc: Jonathan Cameron <jic23@cam.ac.uk>,
	LKML <linux-kernel@vger.kernel.org>,
	Dmitry Torokhov <dtor@mail.ru>,
	"Hennerich, Michael" <Michael.Hennerich@analog.com>,
	Manuel Stahl <manuel.stahl@iis.fraunhofer.de>,
	Robin Getz <rgetz@blackfin.uclinux.org>,
	"Trisal, Kalhan" <kalhan.trisal@intel.com>,
	"Zhang, Xing Z" <xing.z.zhang@intel.com>,
	Ira Snyder <iws@ovro.caltech.edu>,
	Jean Delvare <khali@linux-fr.org>,
	Samu Onkalo <samu.p.onkalo@nokia.com>,
	Stefani Seibold <stefani@seibold.net>
Subject: Re: [RFC] Staging: IIO: New ABI V2
Date: Mon, 15 Feb 2010 19:58:12 -0500	[thread overview]
Message-ID: <8bd0f97a1002151658l742444cg18b2a38846cbfe37@mail.gmail.com> (raw)
In-Reply-To: <201002151526.53539.rgetz@blackfin.uclinux.org>

On Mon, Feb 15, 2010 at 15:26, Robin Getz wrote:
> On Fri 5 Feb 2010 13:21, Jonathan Cameron pondered:
>> Here is another iteration of the ABI spec for IIO with changes made
>> in response to http://lkml.org/lkml/2010/1/20/195 and
>> http://marc.info/?l=lm-sensors&m=126496271320649&w=2 along with a few
>> other general tidy ups.
>>
>> If there are no major issues raised, we may begin working on the move
>> to this ABI shortly on the basis any minor changes can always get
>> cleaned up before those patches merge.  I'll also be doing a formal
>> version of this file for in kernel documentation once things are
>> fairly stable with all of the information not relevant to this
>> discussion.
>>
>> Changes since v1:
>>
>> * iio is now a bus with directory changes resulting in this document.
>> * moved to in0_raw etc for voltage sensors to avoid confusion with
>>   a completely different ABI from hwmon.  Jean made the point that
>>   we shouldn't take this to far, but as things currently stand there
>>   is no disadvantage in this name change.
>> * dropped freefall event for now. More discussions need to be had on this
>>   and in a straight IIO world we normally won't care about this one anyway.
>> * 'device' naming changed for the various subsidiary devices so as make
>>   the interconnections more obvious.  I haven't tried implementing this
>>   yet, but I think the small amount of pain involved is worth it for
>>   increased clarity. The only exception is triggers where the connections
>>   are not specified as a given trigger may not have and IIO device
>>   associated with it.  Anyone suggest a scheme for this? (see about 10
>>   lines below to clarify what I mean here!)
>> * As conversion of the max1363 driver over to a consistent scan_element
>>   interface will mean that these will only apply to the ring buffer
>>   (rather than direct capture), scan_elements is moved into the ring
>>   buffer device directory.
>> * Switch ring for simply buffer as it might be a fifo or other buffer
>>   form instead.
>>
>> At times I may have suppressed links that would be created by the tree of
>> devices. In the flat base directory a given driver can now create the
>> following:
>>
>> device[n]
>> device[n]:ring
>> device[n]:ring:access
>> device[n]:ring:event
>> device[n]:event[m]
>> trigger[o]
>>
>
> What exists today still requires a copy_[to|from]_user when using the ring
> buffer (and then another cache_flush if you are dma'ing things). These seems
> pretty expensive and will consume extra cycles that will limit throughput.
>
> Any thoughts to a mmaped interface directly to the IIO ring buffer, so the
> system could avoid some of the above overhead? (This is what we had to do for
> some other drivers - which were able to handle a 40 MSample/second data
> processed by userspace for a soft radio).

does sysfs currently support mmap-ing of files in there ?
-mike

  reply	other threads:[~2010-02-16  0:58 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-02-05 18:21 Jonathan Cameron
2010-02-05 22:32 ` Ira W. Snyder
2010-02-06 13:59   ` Jonathan Cameron
2010-02-06 14:49     ` Jean Delvare
2010-02-06 17:05     ` Ira W. Snyder
2010-02-15 20:26 ` Robin Getz
2010-02-16  0:58   ` Mike Frysinger [this message]
2010-02-16  2:49     ` Greg KH
2010-02-16 11:03       ` Jonathan Cameron
2010-02-16 19:18         ` Robin Getz
2010-02-17 10:50           ` Jonathan Cameron

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=8bd0f97a1002151658l742444cg18b2a38846cbfe37@mail.gmail.com \
    --to=vapier.adi@gmail.com \
    --cc=Michael.Hennerich@analog.com \
    --cc=dtor@mail.ru \
    --cc=gregkh@suse.de \
    --cc=iws@ovro.caltech.edu \
    --cc=jic23@cam.ac.uk \
    --cc=kalhan.trisal@intel.com \
    --cc=kay.sievers@vrfy.org \
    --cc=khali@linux-fr.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=manuel.stahl@iis.fraunhofer.de \
    --cc=rgetz@blackfin.uclinux.org \
    --cc=samu.p.onkalo@nokia.com \
    --cc=stefani@seibold.net \
    --cc=xing.z.zhang@intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®