From: Jonathan Cameron <jic23@cam.ac.uk>
To: Jonathan Cameron <jic23@cam.ac.uk>
Cc: linux-iio@vger.kernel.org
Subject: Re: [PATCH 00/12] staging:iio: New stuff and reworks.
Date: Fri, 02 Sep 2011 17:24:53 +0100 [thread overview]
Message-ID: <4E610355.4080400@cam.ac.uk> (raw)
In-Reply-To: <1314353795-7437-1-git-send-email-jic23@cam.ac.uk>
Sent on to Greg now.
On 08/26/11 11:16, Jonathan Cameron wrote:
> Hi All,
>
> This is the interesting series of the day. I could have broken
> it down futher, but then it would mostly be single patches.
>
> A fair bit of this you will have seen before, but this is the form
> I intend to send to greg if no one shouts.
>
> So what's here:
>
> 1) Add in_/out_ prefixes to all channels generate via iio_chan_spec.
> We will need a cleanup series adding them to all the stuff provided
> directly by the drivers. (what fun).
> 2) Docs for the above
>
> 3) Endian description for scan elements. Here we have a cpu version
> internally. Note that will not be leaving staging. All drivers
> should get it right themselves.
>
> 4) Reorganize the default scan mask settings so that they won't cause
> trouble (will insert a separate patch here to handle the issue Michael
> has found with adis16400.
>
> 5) Bitmaps for channel masks. This is fiddly, but does allow for devices
> with a lot more channels so we need to do this or something similar.
>
> 6) 64bit event codes. Gives us more space and avoids the reuse issues
> with the previous 32bit ones. Very few disadvantages and a lot of
> advantages!
>
> 7) rework the attribute registration. This is invasive and nasty but
> necessary if we want udev to know attributes have been created. Upshot
> of that discussion about allowing empty groups in sysfs (which was
> the wrong way to solve this). Side effect is we can get rid of that
> hideous regdone trick that was in lots of drivers.
>
> 8) Make the address element of iio_device_attr 64 bit to allow for new
> long event codes. Makes life a lot easier in some corner cases.
>
> 9) the EV_BIT macro had some odd naming. Trivial fix.
>
> 10) Get ready for CDC (capacitance to digital convertors) being handled
> properly.
>
> 11) Differential channel handling rework. the IIO_VOLTAGE_DIFF channel
> type was a bad idea. Allow all channel types to be differential by
> adding an explicit flag. This means that the differential CDCs
> are much cleaner. Requires updating all previous differential users.
> Whilst there cleared out some IIO_CHAN macro usage as that's slowly
> going away.
>
> What fun,
>
> All comments welcome.
>
> Jonathan
>
> Jonathan Cameron (12):
> staging:iio: ABI rework - add in_ or out_ prefix to channnels
> staging:iio:Documentation sysfs-bus-iio add in and out prefixes
> staging:iio:scan element types: introduce endian description to the
> data format.
> staging:iio:various move default scan mask setting after ring
> register or remove
> staging:iio: Switch the channel masks to bitmaps so as to allow for
> more channels.
> staging:iio:events - new 64 bit code structure and push out drivers.
> staging:iio: rework of attribute registration.
> staging:iio:attrs - make address a u64 to allow event codes to be
> used.
> staging:iio:naming in the EV_BIT macro fix.
> staging:iio: Add capacitance type and average_raw chan info.
> staging:iio:add adaptive event types and missing extract_type macro.
> staging:iio: Differential channel handling - use explicit flag rather
> than types.
>
> drivers/staging/iio/Documentation/sysfs-bus-iio | 536 +++++++++++-----------
> drivers/staging/iio/accel/adis16201_core.c | 16 +-
> drivers/staging/iio/accel/adis16201_ring.c | 9 -
> drivers/staging/iio/accel/adis16203_core.c | 17 +-
> drivers/staging/iio/accel/adis16203_ring.c | 7 -
> drivers/staging/iio/accel/adis16204_core.c | 18 +-
> drivers/staging/iio/accel/adis16204_ring.c | 7 -
> drivers/staging/iio/accel/adis16209_core.c | 16 +-
> drivers/staging/iio/accel/adis16209_ring.c | 10 -
> drivers/staging/iio/accel/adis16240_core.c | 15 +-
> drivers/staging/iio/accel/adis16240_ring.c | 8 -
> drivers/staging/iio/accel/lis3l02dq_core.c | 26 +-
> drivers/staging/iio/accel/lis3l02dq_ring.c | 10 +-
> drivers/staging/iio/accel/sca3000_core.c | 16 +-
> drivers/staging/iio/accel/sca3000_ring.c | 4 -
> drivers/staging/iio/adc/ad7150.c | 20 +-
> drivers/staging/iio/adc/ad7192.c | 14 +-
> drivers/staging/iio/adc/ad7280a.c | 26 +-
> drivers/staging/iio/adc/ad7291.c | 12 +-
> drivers/staging/iio/adc/ad7298_core.c | 18 +-
> drivers/staging/iio/adc/ad7298_ring.c | 4 +-
> drivers/staging/iio/adc/ad7476_core.c | 16 +-
> drivers/staging/iio/adc/ad7606_core.c | 17 +-
> drivers/staging/iio/adc/ad7745.c | 18 +-
> drivers/staging/iio/adc/ad7793.c | 210 ++++++---
> drivers/staging/iio/adc/ad7816.c | 15 +-
> drivers/staging/iio/adc/ad7887.h | 4 +-
> drivers/staging/iio/adc/ad7887_core.c | 20 +-
> drivers/staging/iio/adc/ad7887_ring.c | 10 +-
> drivers/staging/iio/adc/ad799x.h | 4 +-
> drivers/staging/iio/adc/ad799x_core.c | 19 +-
> drivers/staging/iio/adc/ad799x_ring.c | 18 +-
> drivers/staging/iio/adc/adt7310.c | 12 +-
> drivers/staging/iio/adc/adt7410.c | 11 +-
> drivers/staging/iio/adc/adt75.c | 12 +-
> drivers/staging/iio/adc/max1363.h | 8 +-
> drivers/staging/iio/adc/max1363_core.c | 263 +++++------
> drivers/staging/iio/adc/max1363_ring.c | 22 +-
> drivers/staging/iio/addac/adt7316.c | 13 +-
> drivers/staging/iio/chrdev.h | 2 +-
> drivers/staging/iio/dac/ad5504.c | 40 +-
> drivers/staging/iio/dac/ad5624r_spi.c | 1 -
> drivers/staging/iio/dac/ad5686.c | 83 ++---
> drivers/staging/iio/dac/ad5791.c | 2 +
> drivers/staging/iio/gyro/adis16060_core.c | 8 +-
> drivers/staging/iio/gyro/adis16080_core.c | 9 +-
> drivers/staging/iio/gyro/adis16260_core.c | 24 +-
> drivers/staging/iio/gyro/adis16260_ring.c | 7 -
> drivers/staging/iio/iio.h | 77 +++-
> drivers/staging/iio/iio_core.h | 3 +-
> drivers/staging/iio/impedance-analyzer/ad5933.c | 18 +-
> drivers/staging/iio/imu/adis16400_core.c | 16 +-
> drivers/staging/iio/imu/adis16400_ring.c | 9 +-
> drivers/staging/iio/industrialio-core.c | 448 +++++++++----------
> drivers/staging/iio/industrialio-ring.c | 223 +++++++---
> drivers/staging/iio/industrialio-trigger.c | 8 +-
> drivers/staging/iio/light/tsl2563.c | 21 +-
> drivers/staging/iio/meter/ade7753.c | 15 +-
> drivers/staging/iio/meter/ade7754.c | 16 +-
> drivers/staging/iio/meter/ade7758.h | 2 +-
> drivers/staging/iio/meter/ade7758_core.c | 20 +-
> drivers/staging/iio/meter/ade7758_ring.c | 2 +-
> drivers/staging/iio/meter/ade7759.c | 12 +-
> drivers/staging/iio/resolver/ad2s1210.c | 1 +
> drivers/staging/iio/ring_generic.h | 67 +---
> drivers/staging/iio/sysfs.h | 46 ++-
> 66 files changed, 1334 insertions(+), 1347 deletions(-)
>
prev parent reply other threads:[~2011-09-02 16:24 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-08-26 10:16 [PATCH 00/12] staging:iio: New stuff and reworks Jonathan Cameron
2011-08-26 10:16 ` [PATCH 01/12] staging:iio: ABI rework - add in_ or out_ prefix to channnels Jonathan Cameron
2011-08-26 10:16 ` [PATCH 02/12] staging:iio:Documentation sysfs-bus-iio add in and out prefixes Jonathan Cameron
2011-08-26 10:16 ` [PATCH 03/12] staging:iio:scan element types: introduce endian description to the data format Jonathan Cameron
2011-08-26 10:16 ` [PATCH 04/12] staging:iio:various move default scan mask setting after ring register or remove Jonathan Cameron
2011-08-26 10:16 ` [PATCH 05/12] staging:iio: Switch the channel masks to bitmaps so as to allow for more channels Jonathan Cameron
2011-08-26 10:16 ` [PATCH 06/12] staging:iio:events - new 64 bit code structure and push out drivers Jonathan Cameron
2011-08-26 10:16 ` [PATCH 07/12] staging:iio: rework of attribute registration Jonathan Cameron
2011-08-26 10:16 ` [PATCH 08/12] staging:iio:attrs - make address a u64 to allow event codes to be used Jonathan Cameron
2011-08-26 10:16 ` [PATCH 09/12] staging:iio:naming in the EV_BIT macro fix Jonathan Cameron
2011-08-26 10:16 ` [PATCH 10/12] staging:iio: Add capacitance type and average_raw chan info Jonathan Cameron
2011-08-26 10:16 ` [PATCH 11/12] staging:iio:add adaptive event types and missing extract_type macro Jonathan Cameron
2011-08-26 10:16 ` [PATCH 12/12] staging:iio: Differential channel handling - use explicit flag rather than types Jonathan Cameron
2011-09-02 16:24 ` Jonathan Cameron [this message]
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=4E610355.4080400@cam.ac.uk \
--to=jic23@cam.ac.uk \
--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.