[PATCH 00/19] staging:iio:buffer clean ups.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]


Hi Greg,

Dependent on previous two series:

Add scan demux unit..
iio:staging:inkern pull interface for staging tree.

This lot are the result of a thorough read of the buffering code.
Nothing terribly interesting. Some are things allowed by example
due to the new version of the sw_ring_preenable function in the
earlier series.

Lots of them are removal of the ability to query into a running
buffer to get the most recent value.  There was no real known
usecase for this. Only one buffer supported it. It made for
some really fiddly code.  Hence after a discussion on linux-iio
we decided to stop drivers doing this and remove the support for
it.

Note the discussion involved the possibility that an equivalent
more general option may come back as part of the work to
support simultaneous in kernel pull and push users of an IIO
device.  Still this code was not needed for that!

The last 3 patches originally fell in a separate series and
so have probably slipped by everyone.  They are trivial
though so I am reasonably happy to send them on to you now.

Thanks,

Jonathan

Previous text:

A fairly straight forward set of cleanups of the buffer code.
Recent discussion concluded that having access to channels being
captured to a buffer by sysfs was an unnecessary complexity so that
ist dropped.  The other patches get rid of some infrastructure and
move the setup ops from a buffer instance to the device (as they will
be relevant to all buffers when we allow more than one)

Anyhow, if people could verify I haven't messed any of this up, that
would be great!

(following is for the last 3).

These were after the inkern pull interface stuff in my tree, but
with a closer look they clearly aren't dependent on it so I'll tack
them on to the end of the previous series if everyone is happy.


Jonathan Cameron (19):
  staging:iio:buffer drop bpe field.
  staging:iio: remove userspace access to bytes per datum.
  staging:iio:buffer move setup ops from buffer instance to iio_dev
  staging:iio: scrap scan_count and ensure all drivers use
    active_scan_mask
  staging:iio:buffer remove unused owner field from struct iio_buffer
  staging:iio:accel:lis3l02dq scrap reading from buffer for sysfs
    access.
  staging:iio:adc:ad7192 scrap read from buffer for sysfs access.
  staging:iio:adc:ad7476 scrap read from buffer for sysfs access.
  staging:iio:adc:ad7606 remove buffer access for sysfs interfaces.
  staging:iio:adc:ad7606 remove buffer access to data from sysfs read.
  staging:iio:adc:ad7993 stop reading from buffer for sysfs raw read.
  staging:iio:adc:ad7887 stop reading from buffer for sysfs access.
  staging:iio:adc:ad799x stop reading from buffer for sysfs accesses
  staging:iio:adc:max1363 stop reading from buffer for sysfs access
  staging:iio:ring_sw don't provide read last function.
  staging:iio:buffer stop allowing for read_last callback.
  staging:iio:buffer scrap to_iio_buffer as it no longer has meaning.
  staging:iio:buffer struct iio_buffer doesn't need an indio_dev
    pointer.
  staging:iio:buffer.h update struct buffer docs.

 drivers/staging/iio/accel/adis16201_ring.c      |   14 ++---
 drivers/staging/iio/accel/adis16203_ring.c      |   14 ++---
 drivers/staging/iio/accel/adis16204_ring.c      |   14 ++---
 drivers/staging/iio/accel/adis16209_ring.c      |    9 ++--
 drivers/staging/iio/accel/adis16240_ring.c      |    9 ++--
 drivers/staging/iio/accel/lis3l02dq.h           |   12 -----
 drivers/staging/iio/accel/lis3l02dq_core.c      |    9 +--
 drivers/staging/iio/accel/lis3l02dq_ring.c      |   60 ++++++-----------------
 drivers/staging/iio/accel/sca3000_core.c        |    6 +-
 drivers/staging/iio/accel/sca3000_ring.c        |   15 ++----
 drivers/staging/iio/adc/ad7192.c                |   34 +++----------
 drivers/staging/iio/adc/ad7298.h                |    5 --
 drivers/staging/iio/adc/ad7298_core.c           |    8 +--
 drivers/staging/iio/adc/ad7298_ring.c           |   43 +++-------------
 drivers/staging/iio/adc/ad7476.h                |    5 --
 drivers/staging/iio/adc/ad7476_core.c           |    2 +-
 drivers/staging/iio/adc/ad7476_ring.c           |   29 +----------
 drivers/staging/iio/adc/ad7606.h                |    1 -
 drivers/staging/iio/adc/ad7606_core.c           |    2 +-
 drivers/staging/iio/adc/ad7606_ring.c           |   28 +----------
 drivers/staging/iio/adc/ad7793.c                |   35 +++----------
 drivers/staging/iio/adc/ad7887.h                |    5 --
 drivers/staging/iio/adc/ad7887_core.c           |    2 +-
 drivers/staging/iio/adc/ad7887_ring.c           |   44 ++--------------
 drivers/staging/iio/adc/ad799x.h                |    6 --
 drivers/staging/iio/adc/ad799x_core.c           |    3 +-
 drivers/staging/iio/adc/ad799x_ring.c           |   45 +++--------------
 drivers/staging/iio/adc/max1363.h               |    7 ---
 drivers/staging/iio/adc/max1363_core.c          |   51 ++++++++-----------
 drivers/staging/iio/adc/max1363_ring.c          |   38 +--------------
 drivers/staging/iio/buffer.h                    |   54 ++++----------------
 drivers/staging/iio/gyro/adis16260_core.c       |   15 ++++--
 drivers/staging/iio/gyro/adis16260_ring.c       |    9 ++--
 drivers/staging/iio/iio.h                       |   16 ++++++
 drivers/staging/iio/iio_simple_dummy_buffer.c   |   12 ++--
 drivers/staging/iio/impedance-analyzer/ad5933.c |   20 ++++---
 drivers/staging/iio/imu/adis16400_ring.c        |   23 +++++----
 drivers/staging/iio/industrialio-buffer.c       |   52 ++++++-------------
 drivers/staging/iio/kfifo_buf.c                 |    4 +-
 drivers/staging/iio/meter/ade7758_ring.c        |   10 ++--
 drivers/staging/iio/ring_sw.c                   |   37 +--------------
 41 files changed, 217 insertions(+), 590 deletions(-)

-- 
1.7.7.4

--
To unsubscribe from this list: send the line "unsubscribe linux-iio" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Photo]     [Yosemite News]    [Yosemite Photos]    [Free Online Dating]     [Linux Kernel]     [Linux SCSI]     [XFree86]

Add to Google Powered by Linux