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 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).