All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] IIO: Why maintain separate and shared knowledge internally?
@ 2011-10-21 15:39 Jonathan Cameron
  2011-10-21 15:39 ` [PATCH] staging:iio:treewide only use shared to decide on interfaces Jonathan Cameron
  0 siblings, 1 reply; 5+ messages in thread
From: Jonathan Cameron @ 2011-10-21 15:39 UTC (permalink / raw)
  To: linux-iio; +Cc: lars, michael.hennerich, device-drivers-devel, Jonathan Cameron

We don't use it anywhere.  Post Lars-Peter's cleanup to
use the enum values directly, it seems sensible to get rid
of the constraint that each should be listed in two variants,
shared and separate.  Lars-Peter suggested some macros to
simplify the whole (1 << INFO ELEMENT) | ( 1 <<...
mess we have in drivers.  If we are going to have these,
might as well use this oportunity to make the pairs in
the mask explicit by having only one enum entry and adding
the logic to these macros.

Will break all out of tree drivers (more or less).
Sorry, but the fix is simple.

Note - dependent on patch 1 but not 2 of Lars-Peters series.
2 is replaced as part of this.

Jonathan

Jonathan Cameron (1):
  staging:iio:treewide only use shared to decide on interfaces

 drivers/staging/iio/accel/adis16201_core.c      |   33 +++---
 drivers/staging/iio/accel/adis16203_core.c      |   21 ++--
 drivers/staging/iio/accel/adis16204_core.c      |   32 +++---
 drivers/staging/iio/accel/adis16209_core.c      |   29 +++---
 drivers/staging/iio/accel/adis16220_core.c      |   22 ++--
 drivers/staging/iio/accel/adis16240_core.c      |   29 +++---
 drivers/staging/iio/accel/kxsd9.c               |    6 +-
 drivers/staging/iio/accel/lis3l02dq_core.c      |   16 ++--
 drivers/staging/iio/accel/sca3000_core.c        |    4 +-
 drivers/staging/iio/adc/ad7192.c                |   34 +++---
 drivers/staging/iio/adc/ad7280a.c               |    6 +-
 drivers/staging/iio/adc/ad7291.c                |   38 ++++---
 drivers/staging/iio/adc/ad7298_core.c           |   41 ++++---
 drivers/staging/iio/adc/ad7476_core.c           |   18 ++--
 drivers/staging/iio/adc/ad7606_core.c           |    2 +-
 drivers/staging/iio/adc/ad7780.c                |    6 +-
 drivers/staging/iio/adc/ad7793.c                |   50 ++++----
 drivers/staging/iio/adc/ad7887_core.c           |    6 +-
 drivers/staging/iio/adc/ad799x_core.c           |    2 +-
 drivers/staging/iio/adc/max1363_core.c          |    4 +-
 drivers/staging/iio/cdc/ad7150.c                |    6 +-
 drivers/staging/iio/cdc/ad7152.c                |   38 +++---
 drivers/staging/iio/cdc/ad7746.c                |   50 ++++----
 drivers/staging/iio/dac/ad5064.c                |    4 +-
 drivers/staging/iio/dac/ad5360.c                |   22 ++--
 drivers/staging/iio/dac/ad5686.c                |    4 +-
 drivers/staging/iio/dac/ad5791.c                |    8 +-
 drivers/staging/iio/gyro/adis16060_core.c       |    8 +-
 drivers/staging/iio/gyro/adis16260_core.c       |   27 ++---
 drivers/staging/iio/gyro/adxrs450_core.c        |    8 +-
 drivers/staging/iio/iio.h                       |   62 +++++++---
 drivers/staging/iio/iio_simple_dummy.c          |   43 ++++---
 drivers/staging/iio/impedance-analyzer/ad5933.c |    4 +-
 drivers/staging/iio/imu/adis16400_core.c        |  143 +++++++++++------------
 drivers/staging/iio/industrialio-core.c         |   19 ++--
 drivers/staging/iio/light/isl29018.c            |    7 +-
 drivers/staging/iio/light/tsl2563.c             |    6 +-
 drivers/staging/iio/magnetometer/ak8975.c       |    4 +-
 drivers/staging/iio/magnetometer/hmc5843.c      |    4 +-
 drivers/staging/iio/meter/ade7758_core.c        |   30 +++---
 40 files changed, 468 insertions(+), 428 deletions(-)

-- 
1.7.7

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

end of thread, other threads:[~2011-10-24  8:58 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-10-21 15:39 [PATCH] IIO: Why maintain separate and shared knowledge internally? Jonathan Cameron
2011-10-21 15:39 ` [PATCH] staging:iio:treewide only use shared to decide on interfaces Jonathan Cameron
2011-10-24  8:30   ` Lars-Peter Clausen
2011-10-24  8:33     ` Jonathan Cameron
2011-10-24  8:58       ` Lars-Peter Clausen

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.