linux-iio.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 0/5] staging:iio: New stuff (No rush!)
@ 2011-10-26 16:41 Jonathan Cameron
  2011-10-26 16:41 ` [PATCH 1/5] staging:iio: header reorganization Jonathan Cameron
                   ` (5 more replies)
  0 siblings, 6 replies; 7+ messages in thread
From: Jonathan Cameron @ 2011-10-26 16:41 UTC (permalink / raw)
  To: greg; +Cc: linux-iio, Jonathan Cameron

Hi Greg,

This lot are not even vaguely urgent. I am just sending them
to you to keep on top of my own patch queue.

The header reorganization was proposed by Lars-Peter and
adapted to fit in with the in kernel interface work.
Anyhow they'd become a mess so this cleanup makes life
somwhat nicer.

The userspace types fo iio_event data makes sense, but
obviously doesn't matter enough to push out in this cycle.

Not using bitmasks is a cleanup that should have been done
a long time ago.  The code evolved to the point where they
didn't make any sense at all (I'm not entirely sure they
did when we first introduced them either.) Dan Carpenter
raised this point as well as Lars-Peter.

The treewided only use shared to decide in interfaces patch
takes this futher.  Drivers very rarely (and arguably never
should) care whether the requested value is shared across
multiple channels or not.  Hence we push that logic up
the chain so that it controls only the userspace (sysfs)
interface naming (allowing in_accel_scale to apply to
in_accel_x_raw etc channels). It stops us having to
carry two versions of the enum entries and a lot
of silly workarounds for that.

Anyhow, all nice cleanups, mostly coming from Lars-Peter
poking around in stuff we had left alone for far too long!
Always good to get new eyes on the code.

Jonathan

Jonathan Cameron (2):
  staging:iio: header reorganization
  staging:iio:treewide only use shared to decide on interfaces

Lars-Peter Clausen (3):
  staging:iio: Use userspace types for iio_event_data
  iio:staging: Add documentation for IIO_EVENT_CODE
  staging:iio: Do not use bitmasks for channel info addresses

 drivers/staging/iio/accel/adis16201_core.c      |   35 ++--
 drivers/staging/iio/accel/adis16203_core.c      |   23 +--
 drivers/staging/iio/accel/adis16204_core.c      |   34 ++--
 drivers/staging/iio/accel/adis16209_core.c      |   31 ++--
 drivers/staging/iio/accel/adis16220_core.c      |   22 +-
 drivers/staging/iio/accel/adis16240_core.c      |   31 ++--
 drivers/staging/iio/accel/kxsd9.c               |    6 +-
 drivers/staging/iio/accel/lis3l02dq_core.c      |   19 +-
 drivers/staging/iio/accel/sca3000_core.c        |    7 +-
 drivers/staging/iio/accel/sca3000_ring.c        |    2 +-
 drivers/staging/iio/adc/ad7192.c                |   36 ++--
 drivers/staging/iio/adc/ad7280a.c               |    7 +-
 drivers/staging/iio/adc/ad7291.c                |   39 +++--
 drivers/staging/iio/adc/ad7298_core.c           |   43 +++--
 drivers/staging/iio/adc/ad7298_ring.c           |    2 +-
 drivers/staging/iio/adc/ad7476_core.c           |   20 +-
 drivers/staging/iio/adc/ad7476_ring.c           |    2 +-
 drivers/staging/iio/adc/ad7606_core.c           |    4 +-
 drivers/staging/iio/adc/ad7606_ring.c           |    2 +-
 drivers/staging/iio/adc/ad7780.c                |    6 +-
 drivers/staging/iio/adc/ad7793.c                |   52 +++---
 drivers/staging/iio/adc/ad7816.c                |    1 +
 drivers/staging/iio/adc/ad7887_core.c           |    8 +-
 drivers/staging/iio/adc/ad7887_ring.c           |    2 +-
 drivers/staging/iio/adc/ad799x_core.c           |    5 +-
 drivers/staging/iio/adc/ad799x_ring.c           |    2 +-
 drivers/staging/iio/adc/adt7310.c               |    2 +-
 drivers/staging/iio/adc/adt7410.c               |    1 +
 drivers/staging/iio/adc/max1363_core.c          |    7 +-
 drivers/staging/iio/adc/max1363_ring.c          |    2 +-
 drivers/staging/iio/addac/adt7316.c             |    1 +
 drivers/staging/iio/buffer.h                    |  227 ++++++++++++++++++++++
 drivers/staging/iio/buffer_generic.h            |  228 -----------------------
 drivers/staging/iio/cdc/ad7150.c                |    8 +-
 drivers/staging/iio/cdc/ad7152.c                |   38 ++--
 drivers/staging/iio/cdc/ad7746.c                |   50 +++---
 drivers/staging/iio/chrdev.h                    |   25 ---
 drivers/staging/iio/dac/ad5064.c                |    4 +-
 drivers/staging/iio/dac/ad5360.c                |   22 +-
 drivers/staging/iio/dac/ad5504.c                |    1 +
 drivers/staging/iio/dac/ad5686.c                |    4 +-
 drivers/staging/iio/dac/ad5791.c                |    8 +-
 drivers/staging/iio/events.h                    |  102 ++++++++++
 drivers/staging/iio/gyro/adis16060_core.c       |    8 +-
 drivers/staging/iio/gyro/adis16260_core.c       |   29 ++--
 drivers/staging/iio/gyro/adxrs450_core.c        |    8 +-
 drivers/staging/iio/iio.h                       |  102 +++++------
 drivers/staging/iio/iio_simple_dummy.c          |   46 +++--
 drivers/staging/iio/iio_simple_dummy_events.c   |    1 +
 drivers/staging/iio/impedance-analyzer/ad5933.c |    6 +-
 drivers/staging/iio/imu/adis16400_core.c        |  145 +++++++-------
 drivers/staging/iio/industrialio-buffer.c       |    2 +-
 drivers/staging/iio/industrialio-core.c         |   21 +-
 drivers/staging/iio/kfifo_buf.h                 |    2 +-
 drivers/staging/iio/light/isl29018.c            |    7 +-
 drivers/staging/iio/light/tsl2563.c             |    7 +-
 drivers/staging/iio/magnetometer/ak8975.c       |    4 +-
 drivers/staging/iio/magnetometer/hmc5843.c      |    4 +-
 drivers/staging/iio/meter/ade7758_core.c        |   32 ++--
 drivers/staging/iio/ring_sw.h                   |    2 +-
 drivers/staging/iio/sysfs.h                     |   43 -----
 drivers/staging/iio/types.h                     |   49 +++++
 62 files changed, 894 insertions(+), 795 deletions(-)
 create mode 100644 drivers/staging/iio/buffer.h
 delete mode 100644 drivers/staging/iio/buffer_generic.h
 delete mode 100644 drivers/staging/iio/chrdev.h
 create mode 100644 drivers/staging/iio/events.h
 create mode 100644 drivers/staging/iio/types.h

-- 
1.7.7

^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2011-11-27  0:36 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-10-26 16:41 [PATCH 0/5] staging:iio: New stuff (No rush!) Jonathan Cameron
2011-10-26 16:41 ` [PATCH 1/5] staging:iio: header reorganization Jonathan Cameron
2011-10-26 16:41 ` [PATCH 2/5] staging:iio: Use userspace types for iio_event_data Jonathan Cameron
2011-10-26 16:41 ` [PATCH 3/5] iio:staging: Add documentation for IIO_EVENT_CODE Jonathan Cameron
2011-10-26 16:41 ` [PATCH 4/5] staging:iio: Do not use bitmasks for channel info addresses Jonathan Cameron
2011-10-26 16:41 ` [PATCH 5/5] staging:iio:treewide only use shared to decide on interfaces Jonathan Cameron
2011-11-27  0:36 ` [PATCH 0/5] staging:iio: New stuff (No rush!) Greg KH

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).