From: Lars-Peter Clausen <lars@metafoo.de>
To: Jonathan Cameron <jic23@kernel.org>
Cc: linux-iio@vger.kernel.org
Subject: Re: [PATCH 00/13 V2] IIO: Make the value attribute optional for channels
Date: Fri, 13 Apr 2012 16:31:36 +0200 [thread overview]
Message-ID: <4F8838C8.5010900@metafoo.de> (raw)
In-Reply-To: <1334311471-23054-1-git-send-email-jic23@kernel.org>
On 04/13/2012 12:04 PM, Jonathan Cameron wrote:
> This is as per Lars-Peter's suggestion + actually covers 'all' (I hope)
> of the drivers rather than missing cdc and impedance directories.
Ok, this one is not so easy to verify. I tried to and couldn't find anything
expect two minor formatting issues. So feel free to add
Acked-by: Lars-Peter Clausen <lars@metafoo.de>
to all of the patches.
But you missed the meter subdirectory and the dummy driver, I'll send the
patches as a follow up.
>
> So now we have IIO_CHAN_INFO_RAW and IIO_CHAN_INFO_PROCESSED.
>
> This lets us drop a chunk of special case logic and generally cleans
> things up. On top of this allows channels with no reading or writing
> abilities which was the original point.
>
> Jonathan
>
> original patch message was:
>
> Dear All,
>
> This came out of a discussion about Michael's variable gain amplifier
> driver the other day. We have devices that have a channel that we
> can't actually read, but know other things about. (there are other
> cases that we have previously fudged around).
>
> Anyhow, here is the promised series making the value optional.
> Has recieved limited testing and I may well have fried someones
> driver so please do take a look and test test test!
>
> Dependent on the IIO_CHAN removal series.
>
> Note I'll probably handle this set a little differently from
> usual and send the first 9 to Greg asap, but hold the 10th
> that actually makes them active until I'm sure there aren't
> any drivers going to cross with it and end up broken.
>
> Jonathan
>
> Jonathan Cameron (13):
> staging:iio: add a raw and processed elements to info_mask
> staging:iio:accel Add IIO_CHAN_INFO_RAW entries to all drivers.
> staging:iio:adc Add IIO_CHAN_INFO_RAW entries to all drivers.
> staging:iio:dac Add IIO_CHAN_INFO_RAW entries to all drivers.
> staging:iio:gyro Add IIO_CHAN_INFO_RAW entries to all drivers.
> staging:iio:imu Add IIO_CHAN_INFO_RAW entries to all drivers.
> staging:iio:light Add IIO_CHAN_INFO_RAW/PROCESSED entries to all
> drivers.
> staging:iio:magnetometer Add IIO_CHAN_INFO_RAW entries to all
> drivers.
> staging:iio:resolver Add IIO_CHAN_INFO_RAW entries to all drivers.
> staging:iio:impedance Add IIO_CHAN_INFO_RAW/PROCESSED entries to all
> drivers.
> staging:iio:cdc Add IIO_CHAN_INFO_RAW/PROCESSED entries to all
> drivers.
> staging:iio: Make read / write attributes for channel values
> optional.
> staging:iio: drop procesed_val element of chan_spec.
>
> drivers/staging/iio/accel/adis16201_core.c | 25 +++--
> drivers/staging/iio/accel/adis16203_core.c | 17 ++-
> drivers/staging/iio/accel/adis16204_core.c | 17 ++-
> drivers/staging/iio/accel/adis16209_core.c | 24 +++--
> drivers/staging/iio/accel/adis16220_core.c | 15 ++-
> drivers/staging/iio/accel/adis16240_core.c | 18 +++-
> drivers/staging/iio/accel/kxsd9.c | 6 +-
> drivers/staging/iio/accel/lis3l02dq_core.c | 5 +-
> drivers/staging/iio/accel/sca3000_core.c | 4 +-
> drivers/staging/iio/adc/ad7192.c | 11 ++-
> drivers/staging/iio/adc/ad7280a.c | 7 +-
> drivers/staging/iio/adc/ad7291.c | 8 +-
> drivers/staging/iio/adc/ad7298_core.c | 8 +-
> drivers/staging/iio/adc/ad7476_core.c | 5 +-
> drivers/staging/iio/adc/ad7606_core.c | 19 ++--
> drivers/staging/iio/adc/ad7780.c | 8 +-
> drivers/staging/iio/adc/ad7793.c | 38 +++++---
> drivers/staging/iio/adc/ad7887_core.c | 8 +-
> drivers/staging/iio/adc/ad799x_core.c | 40 +++++++-
> drivers/staging/iio/adc/lpc32xx_adc.c | 15 ++--
> drivers/staging/iio/adc/max1363_core.c | 5 +-
> drivers/staging/iio/adc/spear_adc.c | 5 +-
> drivers/staging/iio/cdc/ad7150.c | 8 +-
> drivers/staging/iio/cdc/ad7152.c | 14 ++-
> drivers/staging/iio/cdc/ad7746.c | 25 +++--
> drivers/staging/iio/dac/ad5064.c | 7 +-
> drivers/staging/iio/dac/ad5360.c | 7 +-
> drivers/staging/iio/dac/ad5380.c | 7 +-
> drivers/staging/iio/dac/ad5421.c | 7 +-
> drivers/staging/iio/dac/ad5446.c | 5 +-
> drivers/staging/iio/dac/ad5504.c | 7 +-
> drivers/staging/iio/dac/ad5624r_spi.c | 5 +-
> drivers/staging/iio/dac/ad5686.c | 7 +-
> drivers/staging/iio/dac/ad5764.c | 7 +-
> drivers/staging/iio/dac/ad5791.c | 7 +-
> drivers/staging/iio/gyro/adis16060_core.c | 8 +-
> drivers/staging/iio/gyro/adis16080_core.c | 6 +-
> drivers/staging/iio/gyro/adis16130_core.c | 2 +
> drivers/staging/iio/gyro/adis16260_core.c | 15 ++-
> drivers/staging/iio/gyro/adxrs450_core.c | 14 ++-
> drivers/staging/iio/iio.h | 22 +++--
> drivers/staging/iio/impedance-analyzer/ad5933.c | 11 ++-
> drivers/staging/iio/imu/adis16400_core.c | 121 +++++++++++++++--------
> drivers/staging/iio/industrialio-core.c | 24 +----
> drivers/staging/iio/light/isl29018.c | 9 +-
> drivers/staging/iio/light/tsl2563.c | 11 ++-
> drivers/staging/iio/magnetometer/ak8975.c | 5 +-
> drivers/staging/iio/magnetometer/hmc5843.c | 5 +-
> drivers/staging/iio/resolver/ad2s1200.c | 2 +
> drivers/staging/iio/resolver/ad2s1210.c | 2 +
> drivers/staging/iio/resolver/ad2s90.c | 1 +
> 51 files changed, 436 insertions(+), 243 deletions(-)
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-iio" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
next prev parent reply other threads:[~2012-04-13 14:29 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-04-13 10:04 [PATCH 00/13 V2] IIO: Make the value attribute optional for channels Jonathan Cameron
2012-04-13 10:04 ` [PATCH 01/13] staging:iio: add a raw and processed elements to info_mask Jonathan Cameron
2012-04-13 10:04 ` [PATCH 02/13] staging:iio:accel Add IIO_CHAN_INFO_RAW entries to all drivers Jonathan Cameron
2012-04-13 14:33 ` Lars-Peter Clausen
2012-04-13 10:04 ` [PATCH 03/13] staging:iio:adc " Jonathan Cameron
2012-04-13 14:35 ` Lars-Peter Clausen
2012-04-13 10:04 ` [PATCH 04/13] staging:iio:dac " Jonathan Cameron
2012-04-13 10:04 ` [PATCH 05/13] staging:iio:gyro " Jonathan Cameron
2012-04-13 10:04 ` [PATCH 06/13] staging:iio:imu " Jonathan Cameron
2012-04-13 10:04 ` [PATCH 07/13] staging:iio:light Add IIO_CHAN_INFO_RAW/PROCESSED " Jonathan Cameron
2012-04-13 10:04 ` [PATCH 08/13] staging:iio:magnetometer Add IIO_CHAN_INFO_RAW " Jonathan Cameron
2012-04-13 10:04 ` [PATCH 09/13] staging:iio:resolver " Jonathan Cameron
2012-04-13 10:04 ` [PATCH 10/13] staging:iio:impedance Add IIO_CHAN_INFO_RAW/PROCESSED " Jonathan Cameron
2012-04-13 10:04 ` [PATCH 11/13] staging:iio:cdc " Jonathan Cameron
2012-04-13 10:04 ` [PATCH 12/13] staging:iio: Make read / write attributes for channel values optional Jonathan Cameron
2012-04-13 10:04 ` [PATCH 13/13] staging:iio: drop procesed_val element of chan_spec Jonathan Cameron
2012-04-13 10:37 ` Lars-Peter Clausen
2012-04-13 10:39 ` Jonathan Cameron
2012-04-13 14:31 ` Lars-Peter Clausen [this message]
2012-04-13 15:06 ` [PATCH 00/13 V2] IIO: Make the value attribute optional for channels 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=4F8838C8.5010900@metafoo.de \
--to=lars@metafoo.de \
--cc=jic23@kernel.org \
--cc=linux-iio@vger.kernel.org \
/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 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.