* [RFC PATCH 0/4 V2] Add push based interface for non userspace iio users
@ 2012-04-10 20:38 Jonathan Cameron
2012-04-10 20:38 ` [PATCH 1/4] staging:iio: make all buffer access pass through the buffer_list Jonathan Cameron
` (2 more replies)
0 siblings, 3 replies; 15+ messages in thread
From: Jonathan Cameron @ 2012-04-10 20:38 UTC (permalink / raw)
To: linux-iio
Cc: device-drivers-devel, greg, dmitry.torokhov, broonie, alan, arnd,
linus.walleij, maxime.ripard, thomas.petazzoni, zdevai, w.sang,
marek.vasut, Jonathan Cameron
Changes since V1
Fixups as per Dmitry's email.
New element for the mapping to allow us to pass consumer related channel
specific data to the consumer driver. Here it's the fuzz etc bits for
input.
The input driver needs more testing, but I wanted this version out in
the hope that I would get some feedback on just how many drivers are
broken by the first patch!
That and I foolishly promissed an update earlier. Might be dependent
on something in the various trees I sent out earlier. I've been lazy
and not checked but I have it after them locally.
Jonathan
previous version text:
This set is large and relatively invasive. The first important thing
to verify is that the first patch doesn't break any of the existing
drivers.
Taking the patches in turn. Firstly this currenty should only effect
devices with software buffers so when I say buffers that is what I mean
in this description.
1) Re route all pushing to software buffers via iio_push_to_buffers.
We also need to modify drivers to make them aware of the fact that
they may well be pushing to multiple buffers. Where possible I
have done this by making use of iio_sw_buffer_preenable and adding
pre demux scan element size computation to the core (as this was
repeated quite a few times in drivers). Basically I've almost
certainly broken someone's driver so please test this one!
2) Now all drivers (that support buffers at all) should handle multiple
buffers we add a trivial call back buffer implementation. This simply
calls a callback function with new scans of data as and when they
are available.
3) A very rough and ready proof of concept for an input driver that
uses iio in a similar way to the hwmon driver that merged a few
days back. Clearly this needs more work!
One little question this 3rd patch raises is how we get board specific
details about channels into an iio consumer device. Here I thinking
things like 'fuzz' values for input.
To that end what do people think of adding a void *consumer_data
pointer to the struct iio_chan_map structure?
Other than that it's worth noting that this interface uses
exactly the same interface from the point of view of board files
as the pull interface does. (Hopefully avoiding what proved the most
controversial bit last time!).
One other practical issue post this patch set is that we can currently
only have either polled or interrupt driven clients of iio devices
but not both. I have some thoughts on how to deal with this as it
is obviously import to some SoC adc users.
Another future project will be making a cleaner break between those
buffering elements to do with the IIO direct userspace interface
and those used by other consumer drivers. If it looks like we
bolted on the in kernel users side of things that is because we
did!
Jonathan
Jonathan Cameron (4):
staging:iio: make all buffer access pass through the buffer_list
staging:iio: add a callback buffer for in kernel push interface
staging:iio:in kernel users: Add a data field for channel specific
info.
staging:iio: Proof of concept input driver.
drivers/staging/iio/Kconfig | 18 +
drivers/staging/iio/Makefile | 2 +
drivers/staging/iio/accel/adis16201_ring.c | 9 +-
drivers/staging/iio/accel/adis16203_ring.c | 11 +-
drivers/staging/iio/accel/adis16204_ring.c | 8 +-
drivers/staging/iio/accel/adis16209_ring.c | 9 +-
drivers/staging/iio/accel/adis16240_ring.c | 9 +-
drivers/staging/iio/accel/lis3l02dq_ring.c | 9 +-
drivers/staging/iio/adc/ad7192.c | 27 +-
drivers/staging/iio/adc/ad7298.h | 1 -
drivers/staging/iio/adc/ad7298_ring.c | 31 +-
drivers/staging/iio/adc/ad7476.h | 1 -
drivers/staging/iio/adc/ad7476_ring.c | 40 +-
drivers/staging/iio/adc/ad7606_ring.c | 11 +-
drivers/staging/iio/adc/ad7793.c | 26 +-
drivers/staging/iio/adc/ad7887.h | 1 -
drivers/staging/iio/adc/ad7887_ring.c | 28 +-
drivers/staging/iio/adc/ad799x.h | 1 -
drivers/staging/iio/adc/ad799x_ring.c | 27 +-
drivers/staging/iio/adc/max1363_ring.c | 6 +-
drivers/staging/iio/buffer.h | 24 +-
drivers/staging/iio/buffer_cb.c | 115 +++++
drivers/staging/iio/consumer.h | 48 ++
drivers/staging/iio/gyro/adis16260_ring.c | 8 +-
drivers/staging/iio/iio.h | 8 +
drivers/staging/iio/iio_input.c | 176 ++++++++
drivers/staging/iio/iio_input.h | 23 +
drivers/staging/iio/iio_simple_dummy_buffer.c | 16 +-
drivers/staging/iio/impedance-analyzer/ad5933.c | 14 +-
drivers/staging/iio/imu/adis16400_ring.c | 6 +-
drivers/staging/iio/industrialio-buffer.c | 553 ++++++++++++++---------
drivers/staging/iio/industrialio-core.c | 1 +
drivers/staging/iio/inkern.c | 1 +
drivers/staging/iio/machine.h | 2 +
drivers/staging/iio/meter/ade7758_ring.c | 25 +-
35 files changed, 840 insertions(+), 455 deletions(-)
create mode 100644 drivers/staging/iio/buffer_cb.c
create mode 100644 drivers/staging/iio/iio_input.c
create mode 100644 drivers/staging/iio/iio_input.h
--
1.7.9.4
^ permalink raw reply [flat|nested] 15+ messages in thread* [PATCH 1/4] staging:iio: make all buffer access pass through the buffer_list 2012-04-10 20:38 [RFC PATCH 0/4 V2] Add push based interface for non userspace iio users Jonathan Cameron @ 2012-04-10 20:38 ` Jonathan Cameron 2012-04-16 15:59 ` Lars-Peter Clausen 2012-04-10 20:38 ` [PATCH 3/4] staging:iio:in kernel users: Add a data field for channel specific info Jonathan Cameron 2012-04-10 20:38 ` [PATCH 4/4] staging:iio: Proof of concept input driver Jonathan Cameron 2 siblings, 1 reply; 15+ messages in thread From: Jonathan Cameron @ 2012-04-10 20:38 UTC (permalink / raw) To: linux-iio Cc: device-drivers-devel, greg, dmitry.torokhov, broonie, alan, arnd, linus.walleij, maxime.ripard, thomas.petazzoni, zdevai, w.sang, marek.vasut, Jonathan Cameron, Jonathan Cameron From: Jonathan Cameron <jic23@cam.ac.uk> Crucial step in allowing multiple interfaces. Main stages: 1) Ensure no trigger_handler touches the buffers directly. They should only care about active_scan_mask and scan_timestamp in indio_dev. 2) Ensure all setup ops for the buffers act on the general mode and the individual buffers in a consistent fashion. 3) Make use of iio_sw_buffer_preenable where possibe. It's never in a particular fast path so even if overcomplex it is worth using to cut down on code duplication. Signed-off-by: Jonathan Cameron <jic23@kernel.org> --- drivers/staging/iio/accel/adis16201_ring.c | 9 +- drivers/staging/iio/accel/adis16203_ring.c | 11 +- drivers/staging/iio/accel/adis16204_ring.c | 8 +- drivers/staging/iio/accel/adis16209_ring.c | 9 +- drivers/staging/iio/accel/adis16240_ring.c | 9 +- drivers/staging/iio/accel/lis3l02dq_ring.c | 9 +- drivers/staging/iio/adc/ad7192.c | 27 +- drivers/staging/iio/adc/ad7298.h | 1 - drivers/staging/iio/adc/ad7298_ring.c | 31 +- drivers/staging/iio/adc/ad7476.h | 1 - drivers/staging/iio/adc/ad7476_ring.c | 40 +- drivers/staging/iio/adc/ad7606_ring.c | 11 +- drivers/staging/iio/adc/ad7793.c | 26 +- drivers/staging/iio/adc/ad7887.h | 1 - drivers/staging/iio/adc/ad7887_ring.c | 28 +- drivers/staging/iio/adc/ad799x.h | 1 - drivers/staging/iio/adc/ad799x_ring.c | 27 +- drivers/staging/iio/adc/max1363_ring.c | 6 +- drivers/staging/iio/buffer.h | 24 +- drivers/staging/iio/gyro/adis16260_ring.c | 8 +- drivers/staging/iio/iio.h | 8 + drivers/staging/iio/iio_simple_dummy_buffer.c | 16 +- drivers/staging/iio/impedance-analyzer/ad5933.c | 14 +- drivers/staging/iio/imu/adis16400_ring.c | 6 +- drivers/staging/iio/industrialio-buffer.c | 553 ++++++++++++++--------- drivers/staging/iio/industrialio-core.c | 1 + drivers/staging/iio/meter/ade7758_ring.c | 25 +- 27 files changed, 455 insertions(+), 455 deletions(-) diff --git a/drivers/staging/iio/accel/adis16201_ring.c b/drivers/staging/iio/accel/adis16201_ring.c index 97f9e6b..f3673c2 100644 --- a/drivers/staging/iio/accel/adis16201_ring.c +++ b/drivers/staging/iio/accel/adis16201_ring.c @@ -62,13 +62,10 @@ static irqreturn_t adis16201_trigger_handler(int irq, void *p) struct iio_poll_func *pf = p; struct iio_dev *indio_dev = pf->indio_dev; struct adis16201_state *st = iio_priv(indio_dev); - struct iio_buffer *ring = indio_dev->buffer; - int i = 0; s16 *data; - size_t datasize = ring->access->get_bytes_per_datum(ring); - data = kmalloc(datasize, GFP_KERNEL); + data = kmalloc(indio_dev->scan_bytes, GFP_KERNEL); if (data == NULL) { dev_err(&st->us->dev, "memory alloc failed in ring bh"); return -ENOMEM; @@ -81,10 +78,10 @@ static irqreturn_t adis16201_trigger_handler(int irq, void *p) data[i] = be16_to_cpup((__be16 *)&(st->rx[i*2])); /* Guaranteed to be aligned with 8 byte boundary */ - if (ring->scan_timestamp) + if (indio_dev->scan_timestamp) *((s64 *)(data + ((i + 3)/4)*4)) = pf->timestamp; - ring->access->store_to(ring, (u8 *)data, pf->timestamp); + iio_push_to_buffers(indio_dev, (u8 *)data, pf->timestamp); iio_trigger_notify_done(indio_dev->trig); kfree(data); diff --git a/drivers/staging/iio/accel/adis16203_ring.c b/drivers/staging/iio/accel/adis16203_ring.c index 6a8963d..456bb22 100644 --- a/drivers/staging/iio/accel/adis16203_ring.c +++ b/drivers/staging/iio/accel/adis16203_ring.c @@ -62,13 +62,10 @@ static irqreturn_t adis16203_trigger_handler(int irq, void *p) struct iio_poll_func *pf = p; struct iio_dev *indio_dev = pf->indio_dev; struct adis16203_state *st = iio_priv(indio_dev); - struct iio_buffer *ring = indio_dev->buffer; - int i = 0; s16 *data; - size_t datasize = ring->access->get_bytes_per_datum(ring); - data = kmalloc(datasize, GFP_KERNEL); + data = kmalloc(indio_dev->scan_bytes, GFP_KERNEL); if (data == NULL) { dev_err(&st->us->dev, "memory alloc failed in ring bh"); return -ENOMEM; @@ -81,12 +78,10 @@ static irqreturn_t adis16203_trigger_handler(int irq, void *p) data[i] = be16_to_cpup((__be16 *)&(st->rx[i*2])); /* Guaranteed to be aligned with 8 byte boundary */ - if (ring->scan_timestamp) + if (indio_dev->scan_timestamp) *((s64 *)(data + ((i + 3)/4)*4)) = pf->timestamp; - ring->access->store_to(ring, - (u8 *)data, - pf->timestamp); + iio_push_to_buffers(indio_dev, (u8 *)data, pf->timestamp); iio_trigger_notify_done(indio_dev->trig); kfree(data); diff --git a/drivers/staging/iio/accel/adis16204_ring.c b/drivers/staging/iio/accel/adis16204_ring.c index 5c8ab73..cc0f699 100644 --- a/drivers/staging/iio/accel/adis16204_ring.c +++ b/drivers/staging/iio/accel/adis16204_ring.c @@ -60,12 +60,10 @@ static irqreturn_t adis16204_trigger_handler(int irq, void *p) struct iio_poll_func *pf = p; struct iio_dev *indio_dev = pf->indio_dev; struct adis16204_state *st = iio_priv(indio_dev); - struct iio_buffer *ring = indio_dev->buffer; int i = 0; s16 *data; - size_t datasize = ring->access->get_bytes_per_datum(ring); - data = kmalloc(datasize, GFP_KERNEL); + data = kmalloc(indio_dev->scan_bytes, GFP_KERNEL); if (data == NULL) { dev_err(&st->us->dev, "memory alloc failed in ring bh"); return -ENOMEM; @@ -78,10 +76,10 @@ static irqreturn_t adis16204_trigger_handler(int irq, void *p) data[i] = be16_to_cpup((__be16 *)&(st->rx[i*2])); /* Guaranteed to be aligned with 8 byte boundary */ - if (ring->scan_timestamp) + if (indio_dev->scan_timestamp) *((s64 *)(data + ((i + 3)/4)*4)) = pf->timestamp; - ring->access->store_to(ring, (u8 *)data, pf->timestamp); + iio_push_to_buffers(indio_dev, (u8 *)data, pf->timestamp); iio_trigger_notify_done(indio_dev->trig); kfree(data); diff --git a/drivers/staging/iio/accel/adis16209_ring.c b/drivers/staging/iio/accel/adis16209_ring.c index 57254b6..2c95f3d 100644 --- a/drivers/staging/iio/accel/adis16209_ring.c +++ b/drivers/staging/iio/accel/adis16209_ring.c @@ -60,13 +60,10 @@ static irqreturn_t adis16209_trigger_handler(int irq, void *p) struct iio_poll_func *pf = p; struct iio_dev *indio_dev = pf->indio_dev; struct adis16209_state *st = iio_priv(indio_dev); - struct iio_buffer *ring = indio_dev->buffer; - int i = 0; s16 *data; - size_t datasize = ring->access->get_bytes_per_datum(ring); - data = kmalloc(datasize , GFP_KERNEL); + data = kmalloc(indio_dev->scan_bytes, GFP_KERNEL); if (data == NULL) { dev_err(&st->us->dev, "memory alloc failed in ring bh"); return -ENOMEM; @@ -79,10 +76,10 @@ static irqreturn_t adis16209_trigger_handler(int irq, void *p) data[i] = be16_to_cpup((__be16 *)&(st->rx[i*2])); /* Guaranteed to be aligned with 8 byte boundary */ - if (ring->scan_timestamp) + if (indio_dev->scan_timestamp) *((s64 *)(data + ((i + 3)/4)*4)) = pf->timestamp; - ring->access->store_to(ring, (u8 *)data, pf->timestamp); + iio_push_to_buffers(indio_dev, (u8 *)data, pf->timestamp); iio_trigger_notify_done(indio_dev->trig); kfree(data); diff --git a/drivers/staging/iio/accel/adis16240_ring.c b/drivers/staging/iio/accel/adis16240_ring.c index 43ba84e..ee04154 100644 --- a/drivers/staging/iio/accel/adis16240_ring.c +++ b/drivers/staging/iio/accel/adis16240_ring.c @@ -57,13 +57,10 @@ static irqreturn_t adis16240_trigger_handler(int irq, void *p) struct iio_poll_func *pf = p; struct iio_dev *indio_dev = pf->indio_dev; struct adis16240_state *st = iio_priv(indio_dev); - struct iio_buffer *ring = indio_dev->buffer; - int i = 0; s16 *data; - size_t datasize = ring->access->get_bytes_per_datum(ring); - data = kmalloc(datasize, GFP_KERNEL); + data = kmalloc(indio_dev->scan_bytes, GFP_KERNEL); if (data == NULL) { dev_err(&st->us->dev, "memory alloc failed in ring bh"); return -ENOMEM; @@ -76,10 +73,10 @@ static irqreturn_t adis16240_trigger_handler(int irq, void *p) data[i] = be16_to_cpup((__be16 *)&(st->rx[i*2])); /* Guaranteed to be aligned with 8 byte boundary */ - if (ring->scan_timestamp) + if (indio_dev->scan_timestamp) *((s64 *)(data + ((i + 3)/4)*4)) = pf->timestamp; - ring->access->store_to(ring, (u8 *)data, pf->timestamp); + iio_push_to_buffers(indio_dev, (u8 *)data, pf->timestamp); iio_trigger_notify_done(indio_dev->trig); kfree(data); diff --git a/drivers/staging/iio/accel/lis3l02dq_ring.c b/drivers/staging/iio/accel/lis3l02dq_ring.c index 0fc3973..b493034 100644 --- a/drivers/staging/iio/accel/lis3l02dq_ring.c +++ b/drivers/staging/iio/accel/lis3l02dq_ring.c @@ -135,11 +135,10 @@ static irqreturn_t lis3l02dq_trigger_handler(int irq, void *p) { struct iio_poll_func *pf = p; struct iio_dev *indio_dev = pf->indio_dev; - struct iio_buffer *buffer = indio_dev->buffer; int len = 0; - size_t datasize = buffer->access->get_bytes_per_datum(buffer); - char *data = kmalloc(datasize, GFP_KERNEL); + char *data; + data = kmalloc(indio_dev->scan_bytes, GFP_KERNEL); if (data == NULL) { dev_err(indio_dev->dev.parent, "memory alloc failed in buffer bh"); @@ -150,11 +149,11 @@ static irqreturn_t lis3l02dq_trigger_handler(int irq, void *p) len = lis3l02dq_get_buffer_element(indio_dev, data); /* Guaranteed to be aligned with 8 byte boundary */ - if (buffer->scan_timestamp) + if (indio_dev->scan_timestamp) *(s64 *)(((phys_addr_t)data + len + sizeof(s64) - 1) & ~(sizeof(s64) - 1)) = pf->timestamp; - buffer->access->store_to(buffer, (u8 *)data, pf->timestamp); + iio_push_to_buffers(indio_dev, (u8 *)data, pf->timestamp); iio_trigger_notify_done(indio_dev->trig); kfree(data); diff --git a/drivers/staging/iio/adc/ad7192.c b/drivers/staging/iio/adc/ad7192.c index 9d3e037..fd920b0 100644 --- a/drivers/staging/iio/adc/ad7192.c +++ b/drivers/staging/iio/adc/ad7192.c @@ -456,31 +456,19 @@ out: static int ad7192_ring_preenable(struct iio_dev *indio_dev) { struct ad7192_state *st = iio_priv(indio_dev); - struct iio_buffer *ring = indio_dev->buffer; - size_t d_size; unsigned channel; + int ret; if (bitmap_empty(indio_dev->active_scan_mask, indio_dev->masklength)) return -EINVAL; + ret = iio_sw_buffer_preenable(indio_dev); + if (ret < 0) + return ret; + channel = find_first_bit(indio_dev->active_scan_mask, indio_dev->masklength); - d_size = bitmap_weight(indio_dev->active_scan_mask, - indio_dev->masklength) * - indio_dev->channels[0].scan_type.storagebits / 8; - - if (ring->scan_timestamp) { - d_size += sizeof(s64); - - if (d_size % sizeof(s64)) - d_size += sizeof(s64) - (d_size % sizeof(s64)); - } - - if (indio_dev->buffer->access->set_bytes_per_datum) - indio_dev->buffer->access-> - set_bytes_per_datum(indio_dev->buffer, d_size); - st->mode = (st->mode & ~AD7192_MODE_SEL(-1)) | AD7192_MODE_SEL(AD7192_MODE_CONT); st->conf = (st->conf & ~AD7192_CONF_CHAN(-1)) | @@ -522,7 +510,6 @@ static irqreturn_t ad7192_trigger_handler(int irq, void *p) { struct iio_poll_func *pf = p; struct iio_dev *indio_dev = pf->indio_dev; - struct iio_buffer *ring = indio_dev->buffer; struct ad7192_state *st = iio_priv(indio_dev); s64 dat64[2]; s32 *dat32 = (s32 *)dat64; @@ -533,10 +520,10 @@ static irqreturn_t ad7192_trigger_handler(int irq, void *p) indio_dev->channels[0].scan_type.realbits/8); /* Guaranteed to be aligned with 8 byte boundary */ - if (ring->scan_timestamp) + if (indio_dev->scan_timestamp) dat64[1] = pf->timestamp; - ring->access->store_to(ring, (u8 *)dat64, pf->timestamp); + iio_push_to_buffers(indio_dev, (u8 *)dat64, pf->timestamp); iio_trigger_notify_done(indio_dev->trig); st->irq_dis = false; diff --git a/drivers/staging/iio/adc/ad7298.h b/drivers/staging/iio/adc/ad7298.h index a0e5dea..5051a7e 100644 --- a/drivers/staging/iio/adc/ad7298.h +++ b/drivers/staging/iio/adc/ad7298.h @@ -38,7 +38,6 @@ struct ad7298_platform_data { struct ad7298_state { struct spi_device *spi; struct regulator *reg; - size_t d_size; u16 int_vref_mv; unsigned ext_ref; struct spi_transfer ring_xfer[10]; diff --git a/drivers/staging/iio/adc/ad7298_ring.c b/drivers/staging/iio/adc/ad7298_ring.c index feeb0ee..52ddfdf 100644 --- a/drivers/staging/iio/adc/ad7298_ring.c +++ b/drivers/staging/iio/adc/ad7298_ring.c @@ -28,25 +28,17 @@ static int ad7298_ring_preenable(struct iio_dev *indio_dev) { struct ad7298_state *st = iio_priv(indio_dev); - struct iio_buffer *ring = indio_dev->buffer; - size_t d_size; int i, m; unsigned short command; - int scan_count = bitmap_weight(indio_dev->active_scan_mask, - indio_dev->masklength); - d_size = scan_count * (AD7298_STORAGE_BITS / 8); + int scan_count, ret; - if (ring->scan_timestamp) { - d_size += sizeof(s64); + ret = iio_sw_buffer_preenable(indio_dev); + if (ret < 0) + return ret; - if (d_size % sizeof(s64)) - d_size += sizeof(s64) - (d_size % sizeof(s64)); - } - - if (ring->access->set_bytes_per_datum) - ring->access->set_bytes_per_datum(ring, d_size); - - st->d_size = d_size; + /* Now compute overall size */ + scan_count = bitmap_weight(indio_dev->active_scan_mask, + indio_dev->masklength); command = AD7298_WRITE | st->ext_ref; @@ -91,7 +83,6 @@ static irqreturn_t ad7298_trigger_handler(int irq, void *p) struct iio_poll_func *pf = p; struct iio_dev *indio_dev = pf->indio_dev; struct ad7298_state *st = iio_priv(indio_dev); - struct iio_buffer *ring = indio_dev->buffer; s64 time_ns; __u16 buf[16]; int b_sent, i; @@ -100,17 +91,17 @@ static irqreturn_t ad7298_trigger_handler(int irq, void *p) if (b_sent) return b_sent; - if (ring->scan_timestamp) { + if (indio_dev->scan_timestamp) { time_ns = iio_get_time_ns(); - memcpy((u8 *)buf + st->d_size - sizeof(s64), + memcpy((u8 *)buf + indio_dev->scan_bytes - sizeof(s64), &time_ns, sizeof(time_ns)); } for (i = 0; i < bitmap_weight(indio_dev->active_scan_mask, - indio_dev->masklength); i++) + indio_dev->masklength); i++) buf[i] = be16_to_cpu(st->rx_buf[i]); - indio_dev->buffer->access->store_to(ring, (u8 *)buf, time_ns); + iio_push_to_buffers(indio_dev, (u8 *)buf, time_ns); iio_trigger_notify_done(indio_dev->trig); return IRQ_HANDLED; diff --git a/drivers/staging/iio/adc/ad7476.h b/drivers/staging/iio/adc/ad7476.h index 27f696c..b1dd931 100644 --- a/drivers/staging/iio/adc/ad7476.h +++ b/drivers/staging/iio/adc/ad7476.h @@ -27,7 +27,6 @@ struct ad7476_state { struct spi_device *spi; const struct ad7476_chip_info *chip_info; struct regulator *reg; - size_t d_size; u16 int_vref_mv; struct spi_transfer xfer; struct spi_message msg; diff --git a/drivers/staging/iio/adc/ad7476_ring.c b/drivers/staging/iio/adc/ad7476_ring.c index d6af6c0..21bce39 100644 --- a/drivers/staging/iio/adc/ad7476_ring.c +++ b/drivers/staging/iio/adc/ad7476_ring.c @@ -20,36 +20,6 @@ #include "ad7476.h" -/** - * ad7476_ring_preenable() setup the parameters of the ring before enabling - * - * The complex nature of the setting of the number of bytes per datum is due - * to this driver currently ensuring that the timestamp is stored at an 8 - * byte boundary. - **/ -static int ad7476_ring_preenable(struct iio_dev *indio_dev) -{ - struct ad7476_state *st = iio_priv(indio_dev); - struct iio_buffer *ring = indio_dev->buffer; - - st->d_size = bitmap_weight(indio_dev->active_scan_mask, - indio_dev->masklength) * - st->chip_info->channel[0].scan_type.storagebits / 8; - - if (ring->scan_timestamp) { - st->d_size += sizeof(s64); - - if (st->d_size % sizeof(s64)) - st->d_size += sizeof(s64) - (st->d_size % sizeof(s64)); - } - - if (indio_dev->buffer->access->set_bytes_per_datum) - indio_dev->buffer->access-> - set_bytes_per_datum(indio_dev->buffer, st->d_size); - - return 0; -} - static irqreturn_t ad7476_trigger_handler(int irq, void *p) { struct iio_poll_func *pf = p; @@ -59,7 +29,7 @@ static irqreturn_t ad7476_trigger_handler(int irq, void *p) __u8 *rxbuf; int b_sent; - rxbuf = kzalloc(st->d_size, GFP_KERNEL); + rxbuf = kzalloc(indio_dev->scan_bytes, GFP_KERNEL); if (rxbuf == NULL) return -ENOMEM; @@ -70,11 +40,11 @@ static irqreturn_t ad7476_trigger_handler(int irq, void *p) time_ns = iio_get_time_ns(); - if (indio_dev->buffer->scan_timestamp) - memcpy(rxbuf + st->d_size - sizeof(s64), + if (indio_dev->scan_timestamp) + memcpy(rxbuf + indio_dev->scan_bytes - sizeof(s64), &time_ns, sizeof(time_ns)); - indio_dev->buffer->access->store_to(indio_dev->buffer, rxbuf, time_ns); + iio_push_to_buffers(indio_dev, rxbuf, time_ns); done: iio_trigger_notify_done(indio_dev->trig); kfree(rxbuf); @@ -83,7 +53,7 @@ done: } static const struct iio_buffer_setup_ops ad7476_ring_setup_ops = { - .preenable = &ad7476_ring_preenable, + .preenable = &iio_sw_buffer_preenable, .postenable = &iio_triggered_buffer_postenable, .predisable = &iio_triggered_buffer_predisable, }; diff --git a/drivers/staging/iio/adc/ad7606_ring.c b/drivers/staging/iio/adc/ad7606_ring.c index 1ef9fbc..2d36a7a 100644 --- a/drivers/staging/iio/adc/ad7606_ring.c +++ b/drivers/staging/iio/adc/ad7606_ring.c @@ -46,13 +46,11 @@ static void ad7606_poll_bh_to_ring(struct work_struct *work_s) struct ad7606_state *st = container_of(work_s, struct ad7606_state, poll_work); struct iio_dev *indio_dev = iio_priv_to_dev(st); - struct iio_buffer *ring = indio_dev->buffer; s64 time_ns; __u8 *buf; int ret; - buf = kzalloc(ring->access->get_bytes_per_datum(ring), - GFP_KERNEL); + buf = kzalloc(indio_dev->scan_bytes, GFP_KERNEL); if (buf == NULL) return; @@ -82,11 +80,10 @@ static void ad7606_poll_bh_to_ring(struct work_struct *work_s) time_ns = iio_get_time_ns(); - if (ring->scan_timestamp) - *((s64 *)(buf + ring->access->get_bytes_per_datum(ring) - - sizeof(s64))) = time_ns; + if (indio_dev->scan_timestamp) + *((s64 *)(buf + indio_dev->scan_bytes - sizeof(s64))) = time_ns; - ring->access->store_to(indio_dev->buffer, buf, time_ns); + iio_push_to_buffers(indio_dev, buf, time_ns); done: gpio_set_value(st->pdata->gpio_convst, 0); iio_trigger_notify_done(indio_dev->trig); diff --git a/drivers/staging/iio/adc/ad7793.c b/drivers/staging/iio/adc/ad7793.c index f7f7648d6..93f2391 100644 --- a/drivers/staging/iio/adc/ad7793.c +++ b/drivers/staging/iio/adc/ad7793.c @@ -319,31 +319,18 @@ out: static int ad7793_ring_preenable(struct iio_dev *indio_dev) { struct ad7793_state *st = iio_priv(indio_dev); - struct iio_buffer *ring = indio_dev->buffer; - size_t d_size; unsigned channel; + int ret; if (bitmap_empty(indio_dev->active_scan_mask, indio_dev->masklength)) return -EINVAL; + ret = iio_sw_buffer_preenable(indio_dev); + if (ret < 0) + return ret; channel = find_first_bit(indio_dev->active_scan_mask, indio_dev->masklength); - d_size = bitmap_weight(indio_dev->active_scan_mask, - indio_dev->masklength) * - indio_dev->channels[0].scan_type.storagebits / 8; - - if (ring->scan_timestamp) { - d_size += sizeof(s64); - - if (d_size % sizeof(s64)) - d_size += sizeof(s64) - (d_size % sizeof(s64)); - } - - if (indio_dev->buffer->access->set_bytes_per_datum) - indio_dev->buffer->access-> - set_bytes_per_datum(indio_dev->buffer, d_size); - st->mode = (st->mode & ~AD7793_MODE_SEL(-1)) | AD7793_MODE_SEL(AD7793_MODE_CONT); st->conf = (st->conf & ~AD7793_CONF_CHAN(-1)) | @@ -388,7 +375,6 @@ static irqreturn_t ad7793_trigger_handler(int irq, void *p) { struct iio_poll_func *pf = p; struct iio_dev *indio_dev = pf->indio_dev; - struct iio_buffer *ring = indio_dev->buffer; struct ad7793_state *st = iio_priv(indio_dev); s64 dat64[2]; s32 *dat32 = (s32 *)dat64; @@ -399,10 +385,10 @@ static irqreturn_t ad7793_trigger_handler(int irq, void *p) indio_dev->channels[0].scan_type.realbits/8); /* Guaranteed to be aligned with 8 byte boundary */ - if (ring->scan_timestamp) + if (indio_dev->scan_timestamp) dat64[1] = pf->timestamp; - ring->access->store_to(ring, (u8 *)dat64, pf->timestamp); + iio_push_to_buffers(indio_dev, (u8 *)dat64, pf->timestamp); iio_trigger_notify_done(indio_dev->trig); st->irq_dis = false; diff --git a/drivers/staging/iio/adc/ad7887.h b/drivers/staging/iio/adc/ad7887.h index bc53b65..2e09e54 100644 --- a/drivers/staging/iio/adc/ad7887.h +++ b/drivers/staging/iio/adc/ad7887.h @@ -63,7 +63,6 @@ struct ad7887_state { struct spi_device *spi; const struct ad7887_chip_info *chip_info; struct regulator *reg; - size_t d_size; u16 int_vref_mv; struct spi_transfer xfer[4]; struct spi_message msg[3]; diff --git a/drivers/staging/iio/adc/ad7887_ring.c b/drivers/staging/iio/adc/ad7887_ring.c index d180907..cdfcc91 100644 --- a/drivers/staging/iio/adc/ad7887_ring.c +++ b/drivers/staging/iio/adc/ad7887_ring.c @@ -29,22 +29,11 @@ static int ad7887_ring_preenable(struct iio_dev *indio_dev) { struct ad7887_state *st = iio_priv(indio_dev); - struct iio_buffer *ring = indio_dev->buffer; - - st->d_size = bitmap_weight(indio_dev->active_scan_mask, - indio_dev->masklength) * - st->chip_info->channel[0].scan_type.storagebits / 8; - - if (ring->scan_timestamp) { - st->d_size += sizeof(s64); - - if (st->d_size % sizeof(s64)) - st->d_size += sizeof(s64) - (st->d_size % sizeof(s64)); - } + int ret; - if (indio_dev->buffer->access->set_bytes_per_datum) - indio_dev->buffer->access-> - set_bytes_per_datum(indio_dev->buffer, st->d_size); + ret = iio_sw_buffer_preenable(indio_dev); + if (ret < 0) + return ret; /* We know this is a single long so can 'cheat' */ switch (*indio_dev->active_scan_mask) { @@ -83,7 +72,6 @@ static irqreturn_t ad7887_trigger_handler(int irq, void *p) struct iio_poll_func *pf = p; struct iio_dev *indio_dev = pf->indio_dev; struct ad7887_state *st = iio_priv(indio_dev); - struct iio_buffer *ring = indio_dev->buffer; s64 time_ns; __u8 *buf; int b_sent; @@ -92,7 +80,7 @@ static irqreturn_t ad7887_trigger_handler(int irq, void *p) indio_dev->masklength) * st->chip_info->channel[0].scan_type.storagebits / 8; - buf = kzalloc(st->d_size, GFP_KERNEL); + buf = kzalloc(indio_dev->scan_bytes, GFP_KERNEL); if (buf == NULL) return -ENOMEM; @@ -103,11 +91,11 @@ static irqreturn_t ad7887_trigger_handler(int irq, void *p) time_ns = iio_get_time_ns(); memcpy(buf, st->data, bytes); - if (ring->scan_timestamp) - memcpy(buf + st->d_size - sizeof(s64), + if (indio_dev->scan_timestamp) + memcpy(buf + indio_dev->scan_bytes - sizeof(s64), &time_ns, sizeof(time_ns)); - indio_dev->buffer->access->store_to(indio_dev->buffer, buf, time_ns); + iio_push_to_buffers(indio_dev, buf, time_ns); done: kfree(buf); iio_trigger_notify_done(indio_dev->trig); diff --git a/drivers/staging/iio/adc/ad799x.h b/drivers/staging/iio/adc/ad799x.h index 356f690..99f8abe 100644 --- a/drivers/staging/iio/adc/ad799x.h +++ b/drivers/staging/iio/adc/ad799x.h @@ -104,7 +104,6 @@ struct ad799x_chip_info { struct ad799x_state { struct i2c_client *client; const struct ad799x_chip_info *chip_info; - size_t d_size; struct iio_trigger *trig; struct regulator *reg; u16 int_vref_mv; diff --git a/drivers/staging/iio/adc/ad799x_ring.c b/drivers/staging/iio/adc/ad799x_ring.c index 069765c..f09b7ef 100644 --- a/drivers/staging/iio/adc/ad799x_ring.c +++ b/drivers/staging/iio/adc/ad799x_ring.c @@ -32,9 +32,7 @@ **/ static int ad799x_ring_preenable(struct iio_dev *indio_dev) { - struct iio_buffer *ring = indio_dev->buffer; struct ad799x_state *st = iio_priv(indio_dev); - /* * Need to figure out the current mode based upon the requested * scan mask in iio_dev @@ -43,21 +41,7 @@ static int ad799x_ring_preenable(struct iio_dev *indio_dev) if (st->id == ad7997 || st->id == ad7998) ad7997_8_set_scan_mode(st, *indio_dev->active_scan_mask); - st->d_size = bitmap_weight(indio_dev->active_scan_mask, - indio_dev->masklength) * 2; - - if (ring->scan_timestamp) { - st->d_size += sizeof(s64); - - if (st->d_size % sizeof(s64)) - st->d_size += sizeof(s64) - (st->d_size % sizeof(s64)); - } - - if (indio_dev->buffer->access->set_bytes_per_datum) - indio_dev->buffer->access-> - set_bytes_per_datum(indio_dev->buffer, st->d_size); - - return 0; + return iio_sw_buffer_preenable(indio_dev); } /** @@ -72,13 +56,12 @@ static irqreturn_t ad799x_trigger_handler(int irq, void *p) struct iio_poll_func *pf = p; struct iio_dev *indio_dev = pf->indio_dev; struct ad799x_state *st = iio_priv(indio_dev); - struct iio_buffer *ring = indio_dev->buffer; s64 time_ns; __u8 *rxbuf; int b_sent; u8 cmd; - rxbuf = kmalloc(st->d_size, GFP_KERNEL); + rxbuf = kmalloc(indio_dev->scan_bytes, GFP_KERNEL); if (rxbuf == NULL) goto out; @@ -111,11 +94,11 @@ static irqreturn_t ad799x_trigger_handler(int irq, void *p) time_ns = iio_get_time_ns(); - if (ring->scan_timestamp) - memcpy(rxbuf + st->d_size - sizeof(s64), + if (indio_dev->scan_timestamp) + memcpy(rxbuf + indio_dev->scan_bytes - sizeof(s64), &time_ns, sizeof(time_ns)); - ring->access->store_to(indio_dev->buffer, rxbuf, time_ns); + iio_push_to_buffers(indio_dev, rxbuf, time_ns); done: kfree(rxbuf); if (b_sent < 0) diff --git a/drivers/staging/iio/adc/max1363_ring.c b/drivers/staging/iio/adc/max1363_ring.c index d0a60a3..d20218d 100644 --- a/drivers/staging/iio/adc/max1363_ring.c +++ b/drivers/staging/iio/adc/max1363_ring.c @@ -54,7 +54,7 @@ static irqreturn_t max1363_trigger_handler(int irq, void *p) d_size = numvals*2; else d_size = numvals; - if (indio_dev->buffer->scan_timestamp) { + if (indio_dev->scan_timestamp) { d_size += sizeof(s64); if (d_size % sizeof(s64)) d_size += sizeof(s64) - (d_size % sizeof(s64)); @@ -78,9 +78,9 @@ static irqreturn_t max1363_trigger_handler(int irq, void *p) time_ns = iio_get_time_ns(); - if (indio_dev->buffer->scan_timestamp) + if (indio_dev->scan_timestamp) memcpy(rxbuf + d_size - sizeof(s64), &time_ns, sizeof(time_ns)); - iio_push_to_buffer(indio_dev->buffer, rxbuf, time_ns); + iio_push_to_buffers(indio_dev, rxbuf, time_ns); done: iio_trigger_notify_done(indio_dev->trig); diff --git a/drivers/staging/iio/buffer.h b/drivers/staging/iio/buffer.h index df2046d..df7ec46 100644 --- a/drivers/staging/iio/buffer.h +++ b/drivers/staging/iio/buffer.h @@ -67,7 +67,8 @@ struct iio_buffer_access_funcs { * @stufftoread: [INTERN] flag to indicate new data. * @demux_list: [INTERN] list of operations required to demux the scan. * @demux_bounce: [INTERN] buffer for doing gather from incoming scan. - **/ + * @buffer_list: [INTERN] entry in the devices list of current buffers. + */ struct iio_buffer { int length; int bytes_per_datum; @@ -83,9 +84,22 @@ struct iio_buffer { const struct attribute_group *attrs; struct list_head demux_list; unsigned char *demux_bounce; + struct list_head buffer_list; }; /** + * iio_update_buffers() - add or remove buffer from active list + * @indio_dev: device to add buffer to + * @insert_buffer: buffer to insert + * @remove_buffer: buffer_to_remove + * + * Note this will tear down the all buffering and build it up again + */ +int iio_update_buffers(struct iio_dev *indio_dev, + struct iio_buffer *insert_buffer, + struct iio_buffer *remove_buffer); + +/** * iio_buffer_init() - Initialize the buffer structure * @buffer: buffer to be initialized **/ @@ -116,12 +130,12 @@ int iio_scan_mask_set(struct iio_dev *indio_dev, struct iio_buffer *buffer, int bit); /** - * iio_push_to_buffer() - push to a registered buffer. - * @buffer: IIO buffer structure for device - * @scan: Full scan. + * iio_push_to_buffers() - push to a registered buffer. + * @indio_dev: iio_dev structure for device. + * @data: Full scan. * @timestamp: */ -int iio_push_to_buffer(struct iio_buffer *buffer, unsigned char *data, +int iio_push_to_buffers(struct iio_dev *indio_dev, unsigned char *data, s64 timestamp); int iio_update_demux(struct iio_dev *indio_dev); diff --git a/drivers/staging/iio/gyro/adis16260_ring.c b/drivers/staging/iio/gyro/adis16260_ring.c index 711f151..59cc200 100644 --- a/drivers/staging/iio/gyro/adis16260_ring.c +++ b/drivers/staging/iio/gyro/adis16260_ring.c @@ -63,12 +63,10 @@ static irqreturn_t adis16260_trigger_handler(int irq, void *p) struct iio_poll_func *pf = p; struct iio_dev *indio_dev = pf->indio_dev; struct adis16260_state *st = iio_priv(indio_dev); - struct iio_buffer *ring = indio_dev->buffer; int i = 0; s16 *data; - size_t datasize = ring->access->get_bytes_per_datum(ring); - data = kmalloc(datasize , GFP_KERNEL); + data = kmalloc(indio_dev->scan_bytes, GFP_KERNEL); if (data == NULL) { dev_err(&st->us->dev, "memory alloc failed in ring bh"); return -ENOMEM; @@ -81,10 +79,10 @@ static irqreturn_t adis16260_trigger_handler(int irq, void *p) data[i] = be16_to_cpup((__be16 *)&(st->rx[i*2])); /* Guaranteed to be aligned with 8 byte boundary */ - if (ring->scan_timestamp) + if (indio_dev->scan_timestamp) *((s64 *)(data + ((i + 3)/4)*4)) = pf->timestamp; - ring->access->store_to(ring, (u8 *)data, pf->timestamp); + iio_push_to_buffers(indio_dev, (u8 *)data, pf->timestamp); iio_trigger_notify_done(indio_dev->trig); kfree(data); diff --git a/drivers/staging/iio/iio.h b/drivers/staging/iio/iio.h index d96ee4b..8256eb9 100644 --- a/drivers/staging/iio/iio.h +++ b/drivers/staging/iio/iio.h @@ -300,12 +300,16 @@ struct iio_buffer_setup_ops { * and owner * @event_interface: [INTERN] event chrdevs associated with interrupt lines * @buffer: [DRIVER] any buffer present + * @buffer_list: [INTERN] list of all buffers currently attached + * @scan_bytes: [INTERN] num bytes captured to be fed to buffer demux * @mlock: [INTERN] lock used to prevent simultaneous device state * changes * @available_scan_masks: [DRIVER] optional array of allowed bitmasks * @masklength: [INTERN] the length of the mask established from * channels * @active_scan_mask: [INTERN] union of all scan masks requested by buffers + * @scan_timestamp: [INTERN] set if any buffers have requested timestamp + * @scan_index_timestamp:[INTERN] cache of the index to the timestamp * @trig: [INTERN] current device trigger (buffer modes) * @pollfunc: [DRIVER] function run on trigger being received * @channels: [DRIVER] channel specification structure table @@ -335,11 +339,15 @@ struct iio_dev { struct iio_event_interface *event_interface; struct iio_buffer *buffer; + struct list_head buffer_list; + int scan_bytes; struct mutex mlock; const unsigned long *available_scan_masks; unsigned masklength; const unsigned long *active_scan_mask; + bool scan_timestamp; + unsigned scan_index_timestamp; struct iio_trigger *trig; struct iio_poll_func *pollfunc; diff --git a/drivers/staging/iio/iio_simple_dummy_buffer.c b/drivers/staging/iio/iio_simple_dummy_buffer.c index bb4daf7..5be4749 100644 --- a/drivers/staging/iio/iio_simple_dummy_buffer.c +++ b/drivers/staging/iio/iio_simple_dummy_buffer.c @@ -46,14 +46,10 @@ static irqreturn_t iio_simple_dummy_trigger_h(int irq, void *p) { struct iio_poll_func *pf = p; struct iio_dev *indio_dev = pf->indio_dev; - struct iio_buffer *buffer = indio_dev->buffer; int len = 0; - /* - * The datasize is obtained from the buffer. It was stored when - * the preenable setup function was called. - */ - size_t datasize = buffer->access->get_bytes_per_datum(buffer); - u16 *data = kmalloc(datasize, GFP_KERNEL); + u16 *data; + + data = kmalloc(indio_dev->scan_bytes, GFP_KERNEL); if (data == NULL) return -ENOMEM; @@ -79,7 +75,7 @@ static irqreturn_t iio_simple_dummy_trigger_h(int irq, void *p) i < bitmap_weight(indio_dev->active_scan_mask, indio_dev->masklength); i++) { - j = find_next_bit(buffer->scan_mask, + j = find_next_bit(indio_dev->buffer->scan_mask, indio_dev->masklength, j + 1); /* random access read form the 'device' */ data[i] = fakedata[j]; @@ -87,11 +83,11 @@ static irqreturn_t iio_simple_dummy_trigger_h(int irq, void *p) } } /* Store a timestampe at an 8 byte boundary */ - if (buffer->scan_timestamp) + if (indio_dev->scan_timestamp) *(s64 *)(((phys_addr_t)data + len + sizeof(s64) - 1) & ~(sizeof(s64) - 1)) = iio_get_time_ns(); - buffer->access->store_to(buffer, (u8 *)data, pf->timestamp); + iio_push_to_buffers(indio_dev, (u8 *)data, pf->timestamp); kfree(data); diff --git a/drivers/staging/iio/impedance-analyzer/ad5933.c b/drivers/staging/iio/impedance-analyzer/ad5933.c index 06b9fe2..652f500 100644 --- a/drivers/staging/iio/impedance-analyzer/ad5933.c +++ b/drivers/staging/iio/impedance-analyzer/ad5933.c @@ -566,19 +566,14 @@ static const struct iio_info ad5933_info = { static int ad5933_ring_preenable(struct iio_dev *indio_dev) { struct ad5933_state *st = iio_priv(indio_dev); - size_t d_size; int ret; if (bitmap_empty(indio_dev->active_scan_mask, indio_dev->masklength)) return -EINVAL; - d_size = bitmap_weight(indio_dev->active_scan_mask, - indio_dev->masklength) * - ad5933_channels[1].scan_type.storagebits / 8; - - if (indio_dev->buffer->access->set_bytes_per_datum) - indio_dev->buffer->access-> - set_bytes_per_datum(indio_dev->buffer, d_size); + ret = iio_sw_buffer_preenable(indio_dev); + if (ret < 0) + return ret; ret = ad5933_reset(st); if (ret < 0) @@ -649,7 +644,6 @@ static void ad5933_work(struct work_struct *work) struct ad5933_state *st = container_of(work, struct ad5933_state, work.work); struct iio_dev *indio_dev = i2c_get_clientdata(st->client); - struct iio_buffer *ring = indio_dev->buffer; signed short buf[2]; unsigned char status; @@ -680,7 +674,7 @@ static void ad5933_work(struct work_struct *work) buf[0] = be16_to_cpu(buf[0]); } /* save datum to the ring */ - ring->access->store_to(ring, (u8 *)buf, iio_get_time_ns()); + iio_push_to_buffers(indio_dev, (u8 *)buf, iio_get_time_ns()); } else { /* no data available - try again later */ schedule_delayed_work(&st->work, st->poll_time_jiffies); diff --git a/drivers/staging/iio/imu/adis16400_ring.c b/drivers/staging/iio/imu/adis16400_ring.c index 8daa038..a0b227e 100644 --- a/drivers/staging/iio/imu/adis16400_ring.c +++ b/drivers/staging/iio/imu/adis16400_ring.c @@ -119,12 +119,12 @@ static irqreturn_t adis16400_trigger_handler(int irq, void *p) struct iio_buffer *ring = indio_dev->buffer; int i = 0, j, ret = 0; s16 *data; - size_t datasize = ring->access->get_bytes_per_datum(ring); + /* Asumption that long is enough for maximum channels */ unsigned long mask = *indio_dev->active_scan_mask; int scan_count = bitmap_weight(indio_dev->active_scan_mask, indio_dev->masklength); - data = kmalloc(datasize , GFP_KERNEL); + data = kmalloc(indio_dev->scan_bytes, GFP_KERNEL); if (data == NULL) { dev_err(&st->us->dev, "memory alloc failed in ring bh"); return -ENOMEM; @@ -152,7 +152,7 @@ static irqreturn_t adis16400_trigger_handler(int irq, void *p) /* Guaranteed to be aligned with 8 byte boundary */ if (ring->scan_timestamp) *((s64 *)(data + ((i + 3)/4)*4)) = pf->timestamp; - ring->access->store_to(indio_dev->buffer, (u8 *) data, pf->timestamp); + iio_push_to_buffers(indio_dev, (u8 *) data, pf->timestamp); iio_trigger_notify_done(indio_dev->trig); diff --git a/drivers/staging/iio/industrialio-buffer.c b/drivers/staging/iio/industrialio-buffer.c index 386ba76..db6e930 100644 --- a/drivers/staging/iio/industrialio-buffer.c +++ b/drivers/staging/iio/industrialio-buffer.c @@ -31,6 +31,16 @@ static const char * const iio_endian_prefix[] = { [IIO_LE] = "le", }; +static bool iio_buffer_is_primary_active(struct iio_dev *indio_dev) +{ + struct list_head *p; + + list_for_each(p, &indio_dev->buffer_list) + if (p == &indio_dev->buffer->buffer_list) + return true; + return false; +} + /** * iio_buffer_read_first_n_outer() - chrdev read for buffer access * @@ -124,15 +134,17 @@ static ssize_t iio_scan_el_store(struct device *dev, const char *buf, size_t len) { - int ret = 0; + int ret; bool state; struct iio_dev *indio_dev = dev_get_drvdata(dev); struct iio_buffer *buffer = indio_dev->buffer; struct iio_dev_attr *this_attr = to_iio_dev_attr(attr); - state = !(buf[0] == '0'); + ret = strtobool(buf, &state); + if (ret < 0) + return ret; mutex_lock(&indio_dev->mlock); - if (iio_buffer_enabled(indio_dev)) { + if (iio_buffer_is_primary_active(indio_dev)) { ret = -EBUSY; goto error_ret; } @@ -169,13 +181,16 @@ static ssize_t iio_scan_el_ts_store(struct device *dev, const char *buf, size_t len) { - int ret = 0; + int ret; struct iio_dev *indio_dev = dev_get_drvdata(dev); bool state; - state = !(buf[0] == '0'); + ret = strtobool(buf, &state); + if (ret < 0) + return ret; + mutex_lock(&indio_dev->mlock); - if (iio_buffer_enabled(indio_dev)) { + if (iio_buffer_is_primary_active(indio_dev)) { ret = -EBUSY; goto error_ret; } @@ -276,32 +291,30 @@ int iio_buffer_register(struct iio_dev *indio_dev, } attrcount = attrcount_orig; INIT_LIST_HEAD(&buffer->scan_el_dev_attr_list); - if (channels) { - /* new magic */ - for (i = 0; i < num_channels; i++) { - /* Establish necessary mask length */ - if (channels[i].scan_index > - (int)indio_dev->masklength - 1) - indio_dev->masklength - = indio_dev->channels[i].scan_index + 1; - - ret = iio_buffer_add_channel_sysfs(indio_dev, - &channels[i]); - if (ret < 0) - goto error_cleanup_dynamic; - attrcount += ret; - if (channels[i].type == IIO_TIMESTAMP) - buffer->scan_index_timestamp = - channels[i].scan_index; - } - if (indio_dev->masklength && buffer->scan_mask == NULL) { - buffer->scan_mask = kcalloc(BITS_TO_LONGS(indio_dev->masklength), - sizeof(*buffer->scan_mask), - GFP_KERNEL); - if (buffer->scan_mask == NULL) { - ret = -ENOMEM; - goto error_cleanup_dynamic; - } + for (i = 0; i < num_channels; i++) { + /* Establish necessary mask length */ + if (channels[i].scan_index > + (int)indio_dev->masklength - 1) + indio_dev->masklength + = indio_dev->channels[i].scan_index + 1; + + ret = iio_buffer_add_channel_sysfs(indio_dev, + &channels[i]); + if (ret < 0) + goto error_cleanup_dynamic; + attrcount += ret; + if (channels[i].type == IIO_TIMESTAMP) + indio_dev->scan_index_timestamp = + channels[i].scan_index; + } + if (indio_dev->masklength && buffer->scan_mask == NULL) { + buffer->scan_mask + = kcalloc(BITS_TO_LONGS(indio_dev->masklength), + sizeof(*buffer->scan_mask), + GFP_KERNEL); + if (buffer->scan_mask == NULL) { + ret = -ENOMEM; + goto error_cleanup_dynamic; } } @@ -389,106 +402,6 @@ ssize_t iio_buffer_write_length(struct device *dev, } EXPORT_SYMBOL(iio_buffer_write_length); -ssize_t iio_buffer_store_enable(struct device *dev, - struct device_attribute *attr, - const char *buf, - size_t len) -{ - int ret; - bool requested_state, current_state; - int previous_mode; - struct iio_dev *indio_dev = dev_get_drvdata(dev); - struct iio_buffer *buffer = indio_dev->buffer; - - mutex_lock(&indio_dev->mlock); - previous_mode = indio_dev->currentmode; - requested_state = !(buf[0] == '0'); - current_state = iio_buffer_enabled(indio_dev); - if (current_state == requested_state) { - printk(KERN_INFO "iio-buffer, current state requested again\n"); - goto done; - } - if (requested_state) { - if (indio_dev->setup_ops->preenable) { - ret = indio_dev->setup_ops->preenable(indio_dev); - if (ret) { - printk(KERN_ERR - "Buffer not started:" - "buffer preenable failed\n"); - goto error_ret; - } - } - if (buffer->access->request_update) { - ret = buffer->access->request_update(buffer); - if (ret) { - printk(KERN_INFO - "Buffer not started:" - "buffer parameter update failed\n"); - goto error_ret; - } - } - /* Definitely possible for devices to support both of these.*/ - if (indio_dev->modes & INDIO_BUFFER_TRIGGERED) { - if (!indio_dev->trig) { - printk(KERN_INFO - "Buffer not started: no trigger\n"); - ret = -EINVAL; - goto error_ret; - } - indio_dev->currentmode = INDIO_BUFFER_TRIGGERED; - } else if (indio_dev->modes & INDIO_BUFFER_HARDWARE) - indio_dev->currentmode = INDIO_BUFFER_HARDWARE; - else { /* should never be reached */ - ret = -EINVAL; - goto error_ret; - } - - if (indio_dev->setup_ops->postenable) { - ret = indio_dev->setup_ops->postenable(indio_dev); - if (ret) { - printk(KERN_INFO - "Buffer not started:" - "postenable failed\n"); - indio_dev->currentmode = previous_mode; - if (indio_dev->setup_ops->postdisable) - indio_dev->setup_ops-> - postdisable(indio_dev); - goto error_ret; - } - } - } else { - if (indio_dev->setup_ops->predisable) { - ret = indio_dev->setup_ops->predisable(indio_dev); - if (ret) - goto error_ret; - } - indio_dev->currentmode = INDIO_DIRECT_MODE; - if (indio_dev->setup_ops->postdisable) { - ret = indio_dev->setup_ops->postdisable(indio_dev); - if (ret) - goto error_ret; - } - } -done: - mutex_unlock(&indio_dev->mlock); - return len; - -error_ret: - mutex_unlock(&indio_dev->mlock); - return ret; -} -EXPORT_SYMBOL(iio_buffer_store_enable); - -ssize_t iio_buffer_show_enable(struct device *dev, - struct device_attribute *attr, - char *buf) -{ - struct iio_dev *indio_dev = dev_get_drvdata(dev); - return sprintf(buf, "%d\n", iio_buffer_enabled(indio_dev)); -} -EXPORT_SYMBOL(iio_buffer_show_enable); - -/* note NULL used as error indicator as it doesn't make sense. */ static const unsigned long *iio_scan_mask_match(const unsigned long *av_masks, unsigned int masklength, const unsigned long *mask) @@ -503,45 +416,220 @@ static const unsigned long *iio_scan_mask_match(const unsigned long *av_masks, return NULL; } -int iio_sw_buffer_preenable(struct iio_dev *indio_dev) +static int iio_compute_scan_bytes(struct iio_dev *indio_dev, const long *mask, + bool timestamp) { - struct iio_buffer *buffer = indio_dev->buffer; const struct iio_chan_spec *ch; unsigned bytes = 0; int length, i; - dev_dbg(&indio_dev->dev, "%s\n", __func__); /* How much space will the demuxed element take? */ - for_each_set_bit(i, buffer->scan_mask, + for_each_set_bit(i, mask, indio_dev->masklength) { ch = iio_find_channel_from_si(indio_dev, i); - length = ch->scan_type.storagebits/8; + length = ch->scan_type.storagebits / 8; bytes = ALIGN(bytes, length); bytes += length; } - if (buffer->scan_timestamp) { + if (timestamp) { ch = iio_find_channel_from_si(indio_dev, - buffer->scan_index_timestamp); - length = ch->scan_type.storagebits/8; + indio_dev->scan_index_timestamp); + length = ch->scan_type.storagebits / 8; bytes = ALIGN(bytes, length); bytes += length; } - buffer->access->set_bytes_per_datum(buffer, bytes); + return bytes; +} + +int iio_update_buffers(struct iio_dev *indio_dev, + struct iio_buffer *insert_buffer, + struct iio_buffer *remove_buffer) +{ + int ret; + struct iio_buffer *buffer; + unsigned long *compound_mask; + + /* Wind down existing buffers - iff there are any */ + if (!list_empty(&indio_dev->buffer_list)) { + if (indio_dev->setup_ops->predisable) { + ret = indio_dev->setup_ops->predisable(indio_dev); + if (ret) + goto error_ret; + } + indio_dev->currentmode = INDIO_DIRECT_MODE; + if (indio_dev->setup_ops->postdisable) { + ret = indio_dev->setup_ops->postdisable(indio_dev); + if (ret) + goto error_ret; + } + } + if (!indio_dev->available_scan_masks) + kfree(indio_dev->active_scan_mask); + + if (insert_buffer) + list_add(&insert_buffer->buffer_list, &indio_dev->buffer_list); + if (remove_buffer) + list_del(&remove_buffer->buffer_list); + + /* If no buffers in list, we are done */ + if (list_empty(&indio_dev->buffer_list)) + return 0; /* What scan mask do we actually have ?*/ + compound_mask = kzalloc(BITS_TO_LONGS(indio_dev->masklength) + *sizeof(long), GFP_KERNEL); + if (compound_mask == NULL) + return -ENOMEM; + list_for_each_entry(buffer, &indio_dev->buffer_list, buffer_list) { + bitmap_or(compound_mask, compound_mask, buffer->scan_mask, + indio_dev->masklength); + indio_dev->scan_timestamp |= buffer->scan_timestamp; + } if (indio_dev->available_scan_masks) indio_dev->active_scan_mask = iio_scan_mask_match(indio_dev->available_scan_masks, indio_dev->masklength, - buffer->scan_mask); + compound_mask); else - indio_dev->active_scan_mask = buffer->scan_mask; + indio_dev->active_scan_mask = compound_mask; + iio_update_demux(indio_dev); - if (indio_dev->info->update_scan_mode) - return indio_dev->info + /* Wind up again */ + if (indio_dev->setup_ops->preenable) { + ret = indio_dev->setup_ops->preenable(indio_dev); + if (ret) { + printk(KERN_ERR + "Buffer not started:" + "buffer preenable failed\n"); + goto error_free_compound_mask; + } + } + indio_dev->scan_bytes = + iio_compute_scan_bytes(indio_dev, + indio_dev->active_scan_mask, + indio_dev->scan_timestamp); + list_for_each_entry(buffer, &indio_dev->buffer_list, buffer_list) + if (buffer->access->request_update) { + ret = buffer->access->request_update(buffer); + if (ret) { + printk(KERN_INFO + "Buffer not started:" + "buffer parameter update failed\n"); + goto error_ret; + } + } + if (indio_dev->info->update_scan_mode) { + ret = indio_dev->info ->update_scan_mode(indio_dev, indio_dev->active_scan_mask); + if (ret < 0) + goto error_free_compound_mask; + } + /* Definitely possible for devices to support both of these.*/ + if (indio_dev->modes & INDIO_BUFFER_TRIGGERED) { + if (!indio_dev->trig) { + printk(KERN_INFO "Buffer not started: no trigger\n"); + ret = -EINVAL; + goto error_free_compound_mask; + } + indio_dev->currentmode = INDIO_BUFFER_TRIGGERED; + } else if (indio_dev->modes & INDIO_BUFFER_HARDWARE) { + indio_dev->currentmode = INDIO_BUFFER_HARDWARE; + } else { /* should never be reached */ + ret = -EINVAL; + goto error_free_compound_mask; + } + + if (indio_dev->setup_ops->postenable) { + ret = indio_dev->setup_ops->postenable(indio_dev); + if (ret) { + printk(KERN_INFO + "Buffer not started: postenable failed\n"); + indio_dev->currentmode = INDIO_DIRECT_MODE; + if (indio_dev->setup_ops->postdisable) + indio_dev->setup_ops->postdisable(indio_dev); + goto error_free_compound_mask; + } + } +error_free_compound_mask: + indio_dev->active_scan_mask = NULL; + kfree(compound_mask); +error_ret: + + return ret; +} +EXPORT_SYMBOL_GPL(iio_update_buffers); + +ssize_t iio_buffer_store_enable(struct device *dev, + struct device_attribute *attr, + const char *buf, + size_t len) +{ + int ret; + bool requested_state; + struct iio_dev *indio_dev = dev_get_drvdata(dev); + struct iio_buffer *pbuf = indio_dev->buffer; + struct list_head *p; + bool inlist = false; + + ret = strtobool(buf, &requested_state); + if (ret < 0) + return ret; + + mutex_lock(&indio_dev->mlock); + + /* Find out if it is in the list */ + list_for_each(p, &indio_dev->buffer_list) + if (p == &pbuf->buffer_list) { + inlist = true; + break; + } + /* Already enabled */ + if (inlist && requested_state) + goto done; + /* Already disabled */ + if (!inlist && !requested_state) + goto done; + + if (requested_state) + ret = iio_update_buffers(indio_dev, + indio_dev->buffer, NULL); + else + ret = iio_update_buffers(indio_dev, + NULL, indio_dev->buffer); + + if (ret < 0) + goto done; +done: + mutex_unlock(&indio_dev->mlock); + return (ret < 0) ? ret : len; +} +EXPORT_SYMBOL(iio_buffer_store_enable); + +ssize_t iio_buffer_show_enable(struct device *dev, + struct device_attribute *attr, + char *buf) +{ + struct iio_dev *indio_dev = dev_get_drvdata(dev); + return sprintf(buf, "%d\n", iio_buffer_enabled(indio_dev)); +} +EXPORT_SYMBOL(iio_buffer_show_enable); + +int iio_sw_buffer_preenable(struct iio_dev *indio_dev) +{ + struct iio_buffer *buffer; + unsigned bytes; + dev_dbg(&indio_dev->dev, "%s\n", __func__); + + list_for_each_entry(buffer, &indio_dev->buffer_list, buffer_list) + if (buffer->access->set_bytes_per_datum) { + bytes = iio_compute_scan_bytes(indio_dev, + buffer->scan_mask, + buffer->scan_timestamp); + + buffer->access->set_bytes_per_datum(buffer, bytes); + } return 0; } EXPORT_SYMBOL(iio_sw_buffer_preenable); @@ -550,7 +638,11 @@ EXPORT_SYMBOL(iio_sw_buffer_preenable); * iio_scan_mask_set() - set particular bit in the scan mask * @buffer: the buffer whose scan mask we are interested in * @bit: the bit to be set. - **/ + * + * Note that at this point we have no way of knowing what other + * buffers might request, hence this code only verifies that the + * individual buffers request is plausible. + */ int iio_scan_mask_set(struct iio_dev *indio_dev, struct iio_buffer *buffer, int bit) { @@ -580,7 +672,7 @@ int iio_scan_mask_set(struct iio_dev *indio_dev, return -EINVAL; } } - bitmap_copy(buffer->scan_mask, trialmask, indio_dev->masklength); + set_bit(bit, buffer->scan_mask); kfree(trialmask); @@ -629,105 +721,132 @@ static unsigned char *iio_demux(struct iio_buffer *buffer, return buffer->demux_bounce; } -int iio_push_to_buffer(struct iio_buffer *buffer, unsigned char *data, +static int iio_push_to_buffer(struct iio_buffer *buffer, unsigned char *data, s64 timestamp) { unsigned char *dataout = iio_demux(buffer, data); return buffer->access->store_to(buffer, dataout, timestamp); } -EXPORT_SYMBOL_GPL(iio_push_to_buffer); + +int iio_push_to_buffers(struct iio_dev *indio_dev, unsigned char *data, + s64 timestamp) +{ + int ret; + struct iio_buffer *buf; + + list_for_each_entry(buf, &indio_dev->buffer_list, buffer_list) { + ret = iio_push_to_buffer(buf, data, timestamp); + if (ret < 0) + return ret; + } + + return 0; +} +EXPORT_SYMBOL_GPL(iio_push_to_buffers); int iio_update_demux(struct iio_dev *indio_dev) { const struct iio_chan_spec *ch; - struct iio_buffer *buffer = indio_dev->buffer; - int ret, in_ind = -1, out_ind, length; - unsigned in_loc = 0, out_loc = 0; + struct iio_buffer *buffer; struct iio_demux_table *p, *q; + int ret; - /* Clear out any old demux */ - list_for_each_entry_safe(p, q, &buffer->demux_list, l) { - list_del(&p->l); - kfree(p); - } - kfree(buffer->demux_bounce); - buffer->demux_bounce = NULL; - - /* First work out which scan mode we will actually have */ - if (bitmap_equal(indio_dev->active_scan_mask, - buffer->scan_mask, - indio_dev->masklength)) - return 0; + list_for_each_entry(buffer, &indio_dev->buffer_list, buffer_list) { + unsigned in_loc = 0, out_loc = 0; + int in_ind = -1, out_ind, length; - /* Now we have the two masks, work from least sig and build up sizes */ - for_each_set_bit(out_ind, - indio_dev->active_scan_mask, - indio_dev->masklength) { - in_ind = find_next_bit(indio_dev->active_scan_mask, - indio_dev->masklength, - in_ind + 1); - while (in_ind != out_ind) { + /* Clear out any old demux */ + list_for_each_entry_safe(p, q, &buffer->demux_list, l) { + list_del(&p->l); + kfree(p); + } + kfree(buffer->demux_bounce); + buffer->demux_bounce = NULL; + + /* First work out which scan mode we will actually have */ + if (bitmap_equal(indio_dev->active_scan_mask, + buffer->scan_mask, + indio_dev->masklength)) + return 0; + + /* + * Now we have the two masks, work from least sig + * and build up sizes. + */ + for_each_set_bit(out_ind, + indio_dev->active_scan_mask, + indio_dev->masklength) { in_ind = find_next_bit(indio_dev->active_scan_mask, indio_dev->masklength, in_ind + 1); + while (in_ind != out_ind) { + in_ind = find_next_bit(indio_dev-> + active_scan_mask, + indio_dev->masklength, + in_ind + 1); + ch = iio_find_channel_from_si(indio_dev, + in_ind); + length = ch->scan_type.storagebits/8; + /* Make sure we are aligned */ + in_loc += length; + if (in_loc % length) + in_loc += length - in_loc % length; + } + p = kmalloc(sizeof(*p), GFP_KERNEL); + if (p == NULL) { + ret = -ENOMEM; + goto error_clear_mux_table; + } ch = iio_find_channel_from_si(indio_dev, in_ind); length = ch->scan_type.storagebits/8; - /* Make sure we are aligned */ - in_loc += length; + if (out_loc % length) + out_loc += length - out_loc % length; if (in_loc % length) in_loc += length - in_loc % length; + p->from = in_loc; + p->to = out_loc; + p->length = length; + list_add_tail(&p->l, &buffer->demux_list); + out_loc += length; + in_loc += length; } - p = kmalloc(sizeof(*p), GFP_KERNEL); - if (p == NULL) { - ret = -ENOMEM; - goto error_clear_mux_table; + /* Relies on scan_timestamp being last */ + if (buffer->scan_timestamp) { + p = kmalloc(sizeof(*p), GFP_KERNEL); + if (p == NULL) { + ret = -ENOMEM; + goto error_clear_mux_table; + } + ch = iio_find_channel_from_si(indio_dev, + indio_dev-> + scan_index_timestamp); + length = ch->scan_type.storagebits/8; + if (out_loc % length) + out_loc += length - out_loc % length; + if (in_loc % length) + in_loc += length - in_loc % length; + p->from = in_loc; + p->to = out_loc; + p->length = length; + list_add_tail(&p->l, &buffer->demux_list); + out_loc += length; + in_loc += length; } - ch = iio_find_channel_from_si(indio_dev, in_ind); - length = ch->scan_type.storagebits/8; - if (out_loc % length) - out_loc += length - out_loc % length; - if (in_loc % length) - in_loc += length - in_loc % length; - p->from = in_loc; - p->to = out_loc; - p->length = length; - list_add_tail(&p->l, &buffer->demux_list); - out_loc += length; - in_loc += length; - } - /* Relies on scan_timestamp being last */ - if (buffer->scan_timestamp) { - p = kmalloc(sizeof(*p), GFP_KERNEL); - if (p == NULL) { + buffer->demux_bounce = kzalloc(out_loc, GFP_KERNEL); + if (buffer->demux_bounce == NULL) { ret = -ENOMEM; goto error_clear_mux_table; } - ch = iio_find_channel_from_si(indio_dev, - buffer->scan_index_timestamp); - length = ch->scan_type.storagebits/8; - if (out_loc % length) - out_loc += length - out_loc % length; - if (in_loc % length) - in_loc += length - in_loc % length; - p->from = in_loc; - p->to = out_loc; - p->length = length; - list_add_tail(&p->l, &buffer->demux_list); - out_loc += length; - in_loc += length; - } - buffer->demux_bounce = kzalloc(out_loc, GFP_KERNEL); - if (buffer->demux_bounce == NULL) { - ret = -ENOMEM; - goto error_clear_mux_table; } return 0; error_clear_mux_table: - list_for_each_entry_safe(p, q, &buffer->demux_list, l) { - list_del(&p->l); - kfree(p); + list_for_each_entry(buffer, &indio_dev->buffer_list, buffer_list) { + list_for_each_entry_safe(p, q, &buffer->demux_list, l) { + list_del(&p->l); + kfree(p); + } } return ret; } diff --git a/drivers/staging/iio/industrialio-core.c b/drivers/staging/iio/industrialio-core.c index 2b51178..f6c6bd1 100644 --- a/drivers/staging/iio/industrialio-core.c +++ b/drivers/staging/iio/industrialio-core.c @@ -786,6 +786,7 @@ struct iio_dev *iio_allocate_device(int sizeof_priv) return NULL; } dev_set_name(&dev->dev, "iio:device%d", dev->id); + INIT_LIST_HEAD(&dev->buffer_list); } return dev; diff --git a/drivers/staging/iio/meter/ade7758_ring.c b/drivers/staging/iio/meter/ade7758_ring.c index c45b23b..69ec41a 100644 --- a/drivers/staging/iio/meter/ade7758_ring.c +++ b/drivers/staging/iio/meter/ade7758_ring.c @@ -62,7 +62,6 @@ static irqreturn_t ade7758_trigger_handler(int irq, void *p) { struct iio_poll_func *pf = p; struct iio_dev *indio_dev = pf->indio_dev; - struct iio_buffer *ring = indio_dev->buffer; struct ade7758_state *st = iio_priv(indio_dev); s64 dat64[2]; u32 *dat32 = (u32 *)dat64; @@ -72,10 +71,10 @@ static irqreturn_t ade7758_trigger_handler(int irq, void *p) *dat32 = get_unaligned_be32(&st->rx_buf[5]) & 0xFFFFFF; /* Guaranteed to be aligned with 8 byte boundary */ - if (ring->scan_timestamp) + if (indio_dev->scan_timestamp) dat64[1] = pf->timestamp; - ring->access->store_to(ring, (u8 *)dat64, pf->timestamp); + iio_push_to_buffers(indio_dev, (u8 *)dat64, pf->timestamp); iio_trigger_notify_done(indio_dev->trig); @@ -92,29 +91,19 @@ static irqreturn_t ade7758_trigger_handler(int irq, void *p) static int ade7758_ring_preenable(struct iio_dev *indio_dev) { struct ade7758_state *st = iio_priv(indio_dev); - struct iio_buffer *ring = indio_dev->buffer; - size_t d_size; unsigned channel; + int ret; if (!bitmap_empty(indio_dev->active_scan_mask, indio_dev->masklength)) return -EINVAL; + ret = iio_sw_buffer_preenable(indio_dev); + if (ret < 0) + return ret; + channel = find_first_bit(indio_dev->active_scan_mask, indio_dev->masklength); - d_size = st->ade7758_ring_channels[channel].scan_type.storagebits / 8; - - if (ring->scan_timestamp) { - d_size += sizeof(s64); - - if (d_size % sizeof(s64)) - d_size += sizeof(s64) - (d_size % sizeof(s64)); - } - - if (indio_dev->buffer->access->set_bytes_per_datum) - indio_dev->buffer->access-> - set_bytes_per_datum(indio_dev->buffer, d_size); - ade7758_write_waveform_type(&indio_dev->dev, st->ade7758_ring_channels[channel].address); -- 1.7.9.4 ^ permalink raw reply related [flat|nested] 15+ messages in thread
* Re: [PATCH 1/4] staging:iio: make all buffer access pass through the buffer_list 2012-04-10 20:38 ` [PATCH 1/4] staging:iio: make all buffer access pass through the buffer_list Jonathan Cameron @ 2012-04-16 15:59 ` Lars-Peter Clausen 2012-04-16 20:40 ` Jonathan Cameron 0 siblings, 1 reply; 15+ messages in thread From: Lars-Peter Clausen @ 2012-04-16 15:59 UTC (permalink / raw) To: Jonathan Cameron Cc: linux-iio, device-drivers-devel, greg, dmitry.torokhov, broonie, alan, arnd, linus.walleij, maxime.ripard, thomas.petazzoni, zdevai, w.sang, marek.vasut, Jonathan Cameron On 04/10/2012 10:38 PM, Jonathan Cameron wrote: > From: Jonathan Cameron <jic23@cam.ac.uk> > > Crucial step in allowing multiple interfaces. > > Main stages: > > 1) Ensure no trigger_handler touches the buffers directly. > They should only care about active_scan_mask and scan_timestamp > in indio_dev. > 2) Ensure all setup ops for the buffers act on the general mode > and the individual buffers in a consistent fashion. > 3) Make use of iio_sw_buffer_preenable where possibe. It's never > in a particular fast path so even if overcomplex it is worth > using to cut down on code duplication. In my opinion it would have been better to split 1, 2 and 3 into different patches. The driver specific bits look good, but I can't really say I do understand all that's changed in the core right now. > [...] > > int iio_update_demux(struct iio_dev *indio_dev) > { > const struct iio_chan_spec *ch; > - struct iio_buffer *buffer = indio_dev->buffer; > - int ret, in_ind = -1, out_ind, length; > - unsigned in_loc = 0, out_loc = 0; > + struct iio_buffer *buffer; > struct iio_demux_table *p, *q; > + int ret; > > - /* Clear out any old demux */ > - list_for_each_entry_safe(p, q, &buffer->demux_list, l) { > - list_del(&p->l); > - kfree(p); > - } > - kfree(buffer->demux_bounce); > - buffer->demux_bounce = NULL; > - > - /* First work out which scan mode we will actually have */ > - if (bitmap_equal(indio_dev->active_scan_mask, > - buffer->scan_mask, > - indio_dev->masklength)) > - return 0; > + list_for_each_entry(buffer, &indio_dev->buffer_list, buffer_list) { > + unsigned in_loc = 0, out_loc = 0; > + int in_ind = -1, out_ind, length; How about using a outer function here which loops over the buffers and a inner function, which handles a single buffer? This keeps the indention level low and also makes the diffstat a lot more readable. > > - /* Now we have the two masks, work from least sig and build up sizes */ > - for_each_set_bit(out_ind, > - indio_dev->active_scan_mask, > - indio_dev->masklength) { > - in_ind = find_next_bit(indio_dev->active_scan_mask, > - indio_dev->masklength, > - in_ind + 1); > - while (in_ind != out_ind) { > + /* Clear out any old demux */ > + list_for_each_entry_safe(p, q, &buffer->demux_list, l) { > + list_del(&p->l); > + kfree(p); > + } > + kfree(buffer->demux_bounce); > + buffer->demux_bounce = NULL; > + > + /* First work out which scan mode we will actually have */ > + if (bitmap_equal(indio_dev->active_scan_mask, > + buffer->scan_mask, > + indio_dev->masklength)) > + return 0; > + > + /* > + * Now we have the two masks, work from least sig > + * and build up sizes. > + */ > + for_each_set_bit(out_ind, > + indio_dev->active_scan_mask, > + indio_dev->masklength) { > in_ind = find_next_bit(indio_dev->active_scan_mask, > indio_dev->masklength, > in_ind + 1); > + while (in_ind != out_ind) { > + in_ind = find_next_bit(indio_dev-> > + active_scan_mask, > + indio_dev->masklength, > + in_ind + 1); > + ch = iio_find_channel_from_si(indio_dev, > + in_ind); > + length = ch->scan_type.storagebits/8; > + /* Make sure we are aligned */ > + in_loc += length; > + if (in_loc % length) > + in_loc += length - in_loc % length; > + } > + p = kmalloc(sizeof(*p), GFP_KERNEL); > + if (p == NULL) { > + ret = -ENOMEM; > + goto error_clear_mux_table; > + } > ch = iio_find_channel_from_si(indio_dev, in_ind); > length = ch->scan_type.storagebits/8; > - /* Make sure we are aligned */ > - in_loc += length; > + if (out_loc % length) > + out_loc += length - out_loc % length; > if (in_loc % length) > in_loc += length - in_loc % length; > + p->from = in_loc; > + p->to = out_loc; > + p->length = length; > + list_add_tail(&p->l, &buffer->demux_list); > + out_loc += length; > + in_loc += length; > } > - p = kmalloc(sizeof(*p), GFP_KERNEL); > - if (p == NULL) { > - ret = -ENOMEM; > - goto error_clear_mux_table; > + /* Relies on scan_timestamp being last */ > + if (buffer->scan_timestamp) { > + p = kmalloc(sizeof(*p), GFP_KERNEL); > + if (p == NULL) { > + ret = -ENOMEM; > + goto error_clear_mux_table; > + } > + ch = iio_find_channel_from_si(indio_dev, > + indio_dev-> > + scan_index_timestamp); > + length = ch->scan_type.storagebits/8; > + if (out_loc % length) > + out_loc += length - out_loc % length; > + if (in_loc % length) > + in_loc += length - in_loc % length; > + p->from = in_loc; > + p->to = out_loc; > + p->length = length; > + list_add_tail(&p->l, &buffer->demux_list); > + out_loc += length; > + in_loc += length; > } > - ch = iio_find_channel_from_si(indio_dev, in_ind); > - length = ch->scan_type.storagebits/8; > - if (out_loc % length) > - out_loc += length - out_loc % length; > - if (in_loc % length) > - in_loc += length - in_loc % length; > - p->from = in_loc; > - p->to = out_loc; > - p->length = length; > - list_add_tail(&p->l, &buffer->demux_list); > - out_loc += length; > - in_loc += length; > - } > - /* Relies on scan_timestamp being last */ > - if (buffer->scan_timestamp) { > - p = kmalloc(sizeof(*p), GFP_KERNEL); > - if (p == NULL) { > + buffer->demux_bounce = kzalloc(out_loc, GFP_KERNEL); > + if (buffer->demux_bounce == NULL) { > ret = -ENOMEM; > goto error_clear_mux_table; > } > - ch = iio_find_channel_from_si(indio_dev, > - buffer->scan_index_timestamp); > - length = ch->scan_type.storagebits/8; > - if (out_loc % length) > - out_loc += length - out_loc % length; > - if (in_loc % length) > - in_loc += length - in_loc % length; > - p->from = in_loc; > - p->to = out_loc; > - p->length = length; > - list_add_tail(&p->l, &buffer->demux_list); > - out_loc += length; > - in_loc += length; > - } > - buffer->demux_bounce = kzalloc(out_loc, GFP_KERNEL); > - if (buffer->demux_bounce == NULL) { > - ret = -ENOMEM; > - goto error_clear_mux_table; > } > return 0; > > error_clear_mux_table: > - list_for_each_entry_safe(p, q, &buffer->demux_list, l) { > - list_del(&p->l); > - kfree(p); > + list_for_each_entry(buffer, &indio_dev->buffer_list, buffer_list) { > + list_for_each_entry_safe(p, q, &buffer->demux_list, l) { > + list_del(&p->l); > + kfree(p); > + } > } > return ret; > } ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH 1/4] staging:iio: make all buffer access pass through the buffer_list 2012-04-16 15:59 ` Lars-Peter Clausen @ 2012-04-16 20:40 ` Jonathan Cameron 0 siblings, 0 replies; 15+ messages in thread From: Jonathan Cameron @ 2012-04-16 20:40 UTC (permalink / raw) To: Lars-Peter Clausen Cc: linux-iio, device-drivers-devel, greg, dmitry.torokhov, broonie, alan, arnd, linus.walleij, maxime.ripard, thomas.petazzoni, zdevai, w.sang, marek.vasut, Jonathan Cameron On 04/16/2012 04:59 PM, Lars-Peter Clausen wrote: > On 04/10/2012 10:38 PM, Jonathan Cameron wrote: >> From: Jonathan Cameron <jic23@cam.ac.uk> >> >> Crucial step in allowing multiple interfaces. >> >> Main stages: >> >> 1) Ensure no trigger_handler touches the buffers directly. >> They should only care about active_scan_mask and scan_timestamp >> in indio_dev. >> 2) Ensure all setup ops for the buffers act on the general mode >> and the individual buffers in a consistent fashion. >> 3) Make use of iio_sw_buffer_preenable where possibe. It's never >> in a particular fast path so even if overcomplex it is worth >> using to cut down on code duplication. > > In my opinion it would have been better to split 1, 2 and 3 into different > patches. I think you are being way too generous. This is an awful patch with all sorts of different things going on at the same time. Sorry for wasting your time. I'll put out a series with the various bits broken out... > > The driver specific bits look good, but I can't really say I do understand > all that's changed in the core right now. > >> [...] >> >> int iio_update_demux(struct iio_dev *indio_dev) >> { >> const struct iio_chan_spec *ch; >> - struct iio_buffer *buffer = indio_dev->buffer; >> - int ret, in_ind = -1, out_ind, length; >> - unsigned in_loc = 0, out_loc = 0; >> + struct iio_buffer *buffer; >> struct iio_demux_table *p, *q; >> + int ret; >> >> - /* Clear out any old demux */ >> - list_for_each_entry_safe(p, q, &buffer->demux_list, l) { >> - list_del(&p->l); >> - kfree(p); >> - } >> - kfree(buffer->demux_bounce); >> - buffer->demux_bounce = NULL; >> - >> - /* First work out which scan mode we will actually have */ >> - if (bitmap_equal(indio_dev->active_scan_mask, >> - buffer->scan_mask, >> - indio_dev->masklength)) >> - return 0; >> + list_for_each_entry(buffer, &indio_dev->buffer_list, buffer_list) { >> + unsigned in_loc = 0, out_loc = 0; >> + int in_ind = -1, out_ind, length; > > How about using a outer function here which loops over the buffers and a > inner function, which handles a single buffer? This keeps the indention > level low and also makes the diffstat a lot more readable. > >> >> - /* Now we have the two masks, work from least sig and build up sizes */ >> - for_each_set_bit(out_ind, >> - indio_dev->active_scan_mask, >> - indio_dev->masklength) { >> - in_ind = find_next_bit(indio_dev->active_scan_mask, >> - indio_dev->masklength, >> - in_ind + 1); >> - while (in_ind != out_ind) { >> + /* Clear out any old demux */ >> + list_for_each_entry_safe(p, q, &buffer->demux_list, l) { >> + list_del(&p->l); >> + kfree(p); >> + } >> + kfree(buffer->demux_bounce); >> + buffer->demux_bounce = NULL; >> + >> + /* First work out which scan mode we will actually have */ >> + if (bitmap_equal(indio_dev->active_scan_mask, >> + buffer->scan_mask, >> + indio_dev->masklength)) >> + return 0; >> + >> + /* >> + * Now we have the two masks, work from least sig >> + * and build up sizes. >> + */ >> + for_each_set_bit(out_ind, >> + indio_dev->active_scan_mask, >> + indio_dev->masklength) { >> in_ind = find_next_bit(indio_dev->active_scan_mask, >> indio_dev->masklength, >> in_ind + 1); >> + while (in_ind != out_ind) { >> + in_ind = find_next_bit(indio_dev-> >> + active_scan_mask, >> + indio_dev->masklength, >> + in_ind + 1); >> + ch = iio_find_channel_from_si(indio_dev, >> + in_ind); >> + length = ch->scan_type.storagebits/8; >> + /* Make sure we are aligned */ >> + in_loc += length; >> + if (in_loc % length) >> + in_loc += length - in_loc % length; >> + } >> + p = kmalloc(sizeof(*p), GFP_KERNEL); >> + if (p == NULL) { >> + ret = -ENOMEM; >> + goto error_clear_mux_table; >> + } >> ch = iio_find_channel_from_si(indio_dev, in_ind); >> length = ch->scan_type.storagebits/8; >> - /* Make sure we are aligned */ >> - in_loc += length; >> + if (out_loc % length) >> + out_loc += length - out_loc % length; >> if (in_loc % length) >> in_loc += length - in_loc % length; >> + p->from = in_loc; >> + p->to = out_loc; >> + p->length = length; >> + list_add_tail(&p->l, &buffer->demux_list); >> + out_loc += length; >> + in_loc += length; >> } >> - p = kmalloc(sizeof(*p), GFP_KERNEL); >> - if (p == NULL) { >> - ret = -ENOMEM; >> - goto error_clear_mux_table; >> + /* Relies on scan_timestamp being last */ >> + if (buffer->scan_timestamp) { >> + p = kmalloc(sizeof(*p), GFP_KERNEL); >> + if (p == NULL) { >> + ret = -ENOMEM; >> + goto error_clear_mux_table; >> + } >> + ch = iio_find_channel_from_si(indio_dev, >> + indio_dev-> >> + scan_index_timestamp); >> + length = ch->scan_type.storagebits/8; >> + if (out_loc % length) >> + out_loc += length - out_loc % length; >> + if (in_loc % length) >> + in_loc += length - in_loc % length; >> + p->from = in_loc; >> + p->to = out_loc; >> + p->length = length; >> + list_add_tail(&p->l, &buffer->demux_list); >> + out_loc += length; >> + in_loc += length; >> } >> - ch = iio_find_channel_from_si(indio_dev, in_ind); >> - length = ch->scan_type.storagebits/8; >> - if (out_loc % length) >> - out_loc += length - out_loc % length; >> - if (in_loc % length) >> - in_loc += length - in_loc % length; >> - p->from = in_loc; >> - p->to = out_loc; >> - p->length = length; >> - list_add_tail(&p->l, &buffer->demux_list); >> - out_loc += length; >> - in_loc += length; >> - } >> - /* Relies on scan_timestamp being last */ >> - if (buffer->scan_timestamp) { >> - p = kmalloc(sizeof(*p), GFP_KERNEL); >> - if (p == NULL) { >> + buffer->demux_bounce = kzalloc(out_loc, GFP_KERNEL); >> + if (buffer->demux_bounce == NULL) { >> ret = -ENOMEM; >> goto error_clear_mux_table; >> } >> - ch = iio_find_channel_from_si(indio_dev, >> - buffer->scan_index_timestamp); >> - length = ch->scan_type.storagebits/8; >> - if (out_loc % length) >> - out_loc += length - out_loc % length; >> - if (in_loc % length) >> - in_loc += length - in_loc % length; >> - p->from = in_loc; >> - p->to = out_loc; >> - p->length = length; >> - list_add_tail(&p->l, &buffer->demux_list); >> - out_loc += length; >> - in_loc += length; >> - } >> - buffer->demux_bounce = kzalloc(out_loc, GFP_KERNEL); >> - if (buffer->demux_bounce == NULL) { >> - ret = -ENOMEM; >> - goto error_clear_mux_table; >> } >> return 0; >> >> error_clear_mux_table: >> - list_for_each_entry_safe(p, q, &buffer->demux_list, l) { >> - list_del(&p->l); >> - kfree(p); >> + list_for_each_entry(buffer, &indio_dev->buffer_list, buffer_list) { >> + list_for_each_entry_safe(p, q, &buffer->demux_list, l) { >> + list_del(&p->l); >> + kfree(p); >> + } >> } >> return ret; >> } > > ^ permalink raw reply [flat|nested] 15+ messages in thread
* [PATCH 3/4] staging:iio:in kernel users: Add a data field for channel specific info. 2012-04-10 20:38 [RFC PATCH 0/4 V2] Add push based interface for non userspace iio users Jonathan Cameron 2012-04-10 20:38 ` [PATCH 1/4] staging:iio: make all buffer access pass through the buffer_list Jonathan Cameron @ 2012-04-10 20:38 ` Jonathan Cameron 2012-04-10 20:38 ` [PATCH 4/4] staging:iio: Proof of concept input driver Jonathan Cameron 2 siblings, 0 replies; 15+ messages in thread From: Jonathan Cameron @ 2012-04-10 20:38 UTC (permalink / raw) To: linux-iio Cc: device-drivers-devel, greg, dmitry.torokhov, broonie, alan, arnd, linus.walleij, maxime.ripard, thomas.petazzoni, zdevai, w.sang, marek.vasut, Jonathan Cameron Used to allow information about a given channel mapping to be passed through from board files to the consumer drivers. Signed-off-by: Jonathan Cameron <jic23@kernel.org> --- drivers/staging/iio/consumer.h | 2 ++ drivers/staging/iio/inkern.c | 1 + drivers/staging/iio/machine.h | 2 ++ 3 files changed, 5 insertions(+) diff --git a/drivers/staging/iio/consumer.h b/drivers/staging/iio/consumer.h index ef3c10c..d4863da 100644 --- a/drivers/staging/iio/consumer.h +++ b/drivers/staging/iio/consumer.h @@ -18,10 +18,12 @@ struct iio_chan_spec; * struct iio_channel - everything needed for a consumer to use a channel * @indio_dev: Device on which the channel exists. * @channel: Full description of the channel. + * @data: Data about the channel used by consumer. */ struct iio_channel { struct iio_dev *indio_dev; const struct iio_chan_spec *channel; + void *data; }; /** diff --git a/drivers/staging/iio/inkern.c b/drivers/staging/iio/inkern.c index de2c8ea..b895e7e 100644 --- a/drivers/staging/iio/inkern.c +++ b/drivers/staging/iio/inkern.c @@ -189,6 +189,7 @@ struct iio_channel *iio_st_channel_get_all(const char *name) if (name && strcmp(name, c->map->consumer_dev_name) != 0) continue; chans[mapind].indio_dev = c->indio_dev; + chans[mapind].data = c->map->consumer_data; chans[mapind].channel = iio_chan_spec_from_name(chans[mapind].indio_dev, c->map->adc_channel_label); diff --git a/drivers/staging/iio/machine.h b/drivers/staging/iio/machine.h index 0b1f19b..7409100 100644 --- a/drivers/staging/iio/machine.h +++ b/drivers/staging/iio/machine.h @@ -16,9 +16,11 @@ * @consumer_dev_name: Name to uniquely identify the consumer device. * @consumer_channel: Unique name used to idenitify the channel on the * consumer side. + * @consumer_data: Data about the channel for use by the consumer driver. */ struct iio_map { const char *adc_channel_label; const char *consumer_dev_name; const char *consumer_channel; + void *consumer_data; }; -- 1.7.9.4 ^ permalink raw reply related [flat|nested] 15+ messages in thread
* [PATCH 4/4] staging:iio: Proof of concept input driver. 2012-04-10 20:38 [RFC PATCH 0/4 V2] Add push based interface for non userspace iio users Jonathan Cameron 2012-04-10 20:38 ` [PATCH 1/4] staging:iio: make all buffer access pass through the buffer_list Jonathan Cameron 2012-04-10 20:38 ` [PATCH 3/4] staging:iio:in kernel users: Add a data field for channel specific info Jonathan Cameron @ 2012-04-10 20:38 ` Jonathan Cameron 2 siblings, 0 replies; 15+ messages in thread From: Jonathan Cameron @ 2012-04-10 20:38 UTC (permalink / raw) To: linux-iio Cc: device-drivers-devel, greg, dmitry.torokhov, broonie, alan, arnd, linus.walleij, maxime.ripard, thomas.petazzoni, zdevai, w.sang, marek.vasut, Jonathan Cameron, Jonathan Cameron From: Jonathan Cameron <jic23@cam.ac.uk> This is no where near ready to merge. Lots of stuff missing. Signed-off-by: Jonathan Cameron <jic23@kernel.org> --- drivers/staging/iio/Kconfig | 12 +++ drivers/staging/iio/Makefile | 1 + drivers/staging/iio/iio_input.c | 176 +++++++++++++++++++++++++++++++++++++++ drivers/staging/iio/iio_input.h | 23 +++++ 4 files changed, 212 insertions(+) diff --git a/drivers/staging/iio/Kconfig b/drivers/staging/iio/Kconfig index 4aff125..1850cd5 100644 --- a/drivers/staging/iio/Kconfig +++ b/drivers/staging/iio/Kconfig @@ -11,6 +11,18 @@ menuconfig IIO number of different physical interfaces (i2c, spi, etc). See drivers/staging/iio/Documentation for more information. if IIO + +config IIO_ST_INPUT + tristate "Input driver that uses channels specified via iio maps" + depends on INPUT + depends on IIO_BUFFER + select IIO_BUFFER_CB + help + Client driver for IIO via the push interfaces. Used to provide + and input interface for IIO devices that can feed a buffer on + a trigger interrupt. Note that all channels must come from a + single IIO supported device. + config IIO_ST_HWMON tristate "Hwmon driver that uses channels specified via iio maps" depends on HWMON diff --git a/drivers/staging/iio/Makefile b/drivers/staging/iio/Makefile index f55de94..f64c93b 100644 --- a/drivers/staging/iio/Makefile +++ b/drivers/staging/iio/Makefile @@ -19,6 +19,7 @@ iio_dummy-$(CONFIG_IIO_SIMPLE_DUMMY_BUFFER) += iio_simple_dummy_buffer.o obj-$(CONFIG_IIO_DUMMY_EVGEN) += iio_dummy_evgen.o obj-$(CONFIG_IIO_ST_HWMON) += iio_hwmon.o +obj-$(CONFIG_IIO_ST_INPUT) += iio_input.o obj-y += accel/ obj-y += adc/ diff --git a/drivers/staging/iio/iio_input.c b/drivers/staging/iio/iio_input.c new file mode 100644 index 0000000..4d21ee31 --- /dev/null +++ b/drivers/staging/iio/iio_input.c @@ -0,0 +1,176 @@ +/* + * The industrial I/O input client driver + * + * Copyright (c) 2011 Jonathan Cameron + * + * This program is free software; you can redistribute it and/or modify it + * under the terms of the GNU General Public License version 2 as published by + * the Free Software Foundation. + */ + +#include <linux/kernel.h> +#include <linux/slab.h> +#include <linux/module.h> +#include <linux/err.h> +#include <linux/platform_device.h> +#include <linux/input.h> +#include "buffer.h" +#include "consumer.h" +#include "iio_input.h" + +struct iio_input_state { + struct iio_cb_buffer *buff; + struct input_dev *idev; +}; + +static int iio_channel_value(u8 *data, + const struct iio_chan_spec *chan, + int *val) +{ + int value; + + if (chan->scan_type.sign == 's') { + switch (chan->scan_type.storagebits) { + case 8: + value = *(s8 *)(data); + break; + case 16: + value = *(s16 *)(data); + break; + case 32: + value = *(s32 *)(data); + break; + default: + return -EINVAL; + } + value >>= chan->scan_type.shift; + value &= (1 << chan->scan_type.realbits) - 1; + value = (value << (sizeof(value)*8 - chan->scan_type.realbits)) + >> (sizeof(value)*8 - chan->scan_type.realbits); + } else { + switch (chan->scan_type.storagebits) { + case 8: + value = *(u8 *)(data); + break; + case 16: + value = *(u16 *)(data); + break; + case 32: + value = *(u32 *)(data); + break; + default: + return -EINVAL; + } + value >>= chan->scan_type.shift; + value &= (1 << chan->scan_type.realbits) - 1; + } + *val = value; + + return 0; +} + +static int iio_input_store_to(u8 *data, void *private) +{ + struct iio_input_state *st = private; + struct iio_channel *channel; + struct iio_input_channel_data *input_data; + int offset = 0; + int value, ret; + + channel = iio_st_channel_cb_get_channels(st->buff); + while (channel->indio_dev) { + input_data = channel->data; + offset = ALIGN(offset, + channel->channel->scan_type.storagebits/8); + offset += channel->channel->scan_type.storagebits/8; + ret = iio_channel_value(&data[offset], + channel->channel, + &value); + if (ret < 0) + return ret; + input_report_abs(st->idev, input_data->code, value); + } + input_sync(st->idev); + + return 0; +} + +static int __devinit iio_input_probe(struct platform_device *pdev) +{ + struct iio_input_state *st; + int ret; + struct iio_channel *channel; + struct iio_input_channel_data *input_data; + + st = kzalloc(sizeof(*st), GFP_KERNEL); + if (st == NULL) + return -ENOMEM; + platform_set_drvdata(pdev, st); + st->buff = iio_st_channel_get_all_cb(dev_name(&pdev->dev), + &iio_input_store_to, + st); + if (IS_ERR(st->buff)) { + ret = PTR_ERR(st->buff); + goto error_free_state; + } + + st->idev = input_allocate_device(); + if (!st->idev) { + ret = -ENOMEM; + goto error_channels_release_all; + } + + __set_bit(EV_ABS, st->idev->evbit); + channel = iio_st_channel_cb_get_channels(st->buff); + while (channel->indio_dev) { + input_data = channel->data; + input_set_abs_params(st->idev, input_data->code, + input_data->min, + input_data->max, + input_data->fuzz, + input_data->flat); + } + + ret = input_register_device(st->idev); + if (ret < 0) + goto error_free_idev; + + /* NORMALLY IN THE OPEN */ + iio_st_channel_start_all_cb(st->buff); + + return 0; +error_free_idev: + input_free_device(st->idev); +error_channels_release_all: + iio_st_channel_release_all_cb(st->buff); +error_free_state: + kfree(st); + return ret; +} + +static int __devexit iio_input_remove(struct platform_device *pdev) +{ + struct iio_input_state *st = platform_get_drvdata(pdev); + /* NORMALLY IN THE CLOSE */ + iio_st_channel_stop_all_cb(st->buff); + input_unregister_device(st->idev); + iio_st_channel_release_all_cb(st->buff); + + kfree(st); + return 0; +} + +static struct platform_driver iio_input_driver = { + .driver = { + .name = "iio_snoop", + .owner = THIS_MODULE, + }, + .probe = iio_input_probe, + .remove = __devexit_p(iio_input_remove), +}; + +module_platform_driver(iio_input_driver); + +MODULE_AUTHOR("Jonathan Cameron <jic23@cam.ac.uk>"); +MODULE_DESCRIPTION("IIO input buffer driver"); +MODULE_LICENSE("GPL v2"); diff --git a/drivers/staging/iio/iio_input.h b/drivers/staging/iio/iio_input.h new file mode 100644 index 0000000..cfd1d34 --- /dev/null +++ b/drivers/staging/iio/iio_input.h @@ -0,0 +1,23 @@ +/* + * The industrial I/O input client driver + * + * Copyright (c) 2011 Jonathan Cameron + * + * This program is free software; you can redistribute it and/or modify it + * under the terms of the GNU General Public License version 2 as published by + * the Free Software Foundation. + */ + +/** + * iio_input_channel_data - description of the channel for input subsystem + * @code: Absolute axis. + * @min: Minimum value. + * @max: Maximum value. + * @fuzz: Used to filter noise from the event stream. + * @flat: Values within this value will be discarded by joydev + * and reported as 0 instead. + */ +struct iio_input_channel_data { + unsigned int code; + int min, max, fuzz, flat; +}; -- 1.7.9.4 ^ permalink raw reply related [flat|nested] 15+ messages in thread
* [PATCH 0/4 V2] staging:iio: Add support for multiple buffers
@ 2012-05-30 19:36 Jonathan Cameron
2012-05-30 19:36 ` [PATCH 4/4] staging:iio: Proof of concept input driver Jonathan Cameron
0 siblings, 1 reply; 15+ messages in thread
From: Jonathan Cameron @ 2012-05-30 19:36 UTC (permalink / raw)
To: linux-iio; +Cc: Jonathan Cameron
This is basically a rebase of the series from a few months back.
I held this one off as it's 'interesting and invasive'.
Basically this adds a level of indirection to the path between
the hardware and the software buffers. This allows us to have
multiple buffers each with their own demuxer. Clearly the
hardware has to be configured to provide all the channels
anyone is interested in, but then the flow looks something like.
Data ---> Demux1 ---> subset of data --> buffer1
|
-------> Demux2 ---> subset of data --> buffer2.
Note the demuxers should be very nearly transparent
if all the data is going on to the relevant buffer.
So you may ask ' Why?' In short - it allows us to
do clean data flow to multiple consumers such as an
IIO buffer, input, hwmon etc for devices running in
an trigger driven mode.
I also have a few thoughts on basic in kernel filtering
dependent on new buffer implementations.
Anyhow. Whilst the patch is relatively large it mostly
boils down to code for tearing down all current buffers,
reconfiguring the device and then building them all back
up again.
The input driver shoes the sort of client we might have.
(It probably needs more work) I'd like to get the
first three patches in the linux-next asap so they
get lots of testing prior to the next merge window.
Hence if anyone has time to take a look that would be
great!
Jonathan
Jonathan Cameron (4):
staging:iio: Add support for multiple buffers
staging:iio:in kernel users: Add a data field for channel specific
info.
staging:iio: add a callback buffer for in kernel push interface
staging:iio: Proof of concept input driver.
drivers/iio/Kconfig | 6 +
drivers/iio/Makefile | 1 +
drivers/iio/buffer_cb.c | 115 ++++++++
drivers/iio/industrialio-buffer.c | 335 +++++++++++++++--------
drivers/iio/industrialio-core.c | 1 +
drivers/iio/inkern.c | 1 +
drivers/staging/iio/Kconfig | 11 +
drivers/staging/iio/Makefile | 1 +
drivers/staging/iio/accel/adis16201_ring.c | 4 +-
drivers/staging/iio/accel/adis16203_ring.c | 6 +-
drivers/staging/iio/accel/adis16204_ring.c | 3 +-
drivers/staging/iio/accel/adis16209_ring.c | 3 +-
drivers/staging/iio/accel/adis16240_ring.c | 4 +-
drivers/staging/iio/accel/lis3l02dq_ring.c | 3 +-
drivers/staging/iio/adc/ad7192.c | 3 +-
drivers/staging/iio/adc/ad7298_ring.c | 5 +-
drivers/staging/iio/adc/ad7476_ring.c | 2 +-
drivers/staging/iio/adc/ad7606_ring.c | 3 +-
drivers/staging/iio/adc/ad7793.c | 3 +-
drivers/staging/iio/adc/ad7887_ring.c | 2 +-
drivers/staging/iio/adc/ad799x_ring.c | 3 +-
drivers/staging/iio/adc/max1363_ring.c | 2 +-
drivers/staging/iio/gyro/adis16260_ring.c | 3 +-
drivers/staging/iio/iio_input.c | 176 ++++++++++++
drivers/staging/iio/iio_input.h | 23 ++
drivers/staging/iio/iio_simple_dummy_buffer.c | 5 +-
drivers/staging/iio/impedance-analyzer/ad5933.c | 3 +-
drivers/staging/iio/imu/adis16400_ring.c | 2 +-
drivers/staging/iio/meter/ade7758_ring.c | 3 +-
include/linux/iio/buffer.h | 24 +-
include/linux/iio/consumer.h | 48 ++++
include/linux/iio/iio.h | 2 +
include/linux/iio/machine.h | 2 +
33 files changed, 651 insertions(+), 157 deletions(-)
create mode 100644 drivers/iio/buffer_cb.c
create mode 100644 drivers/staging/iio/iio_input.c
create mode 100644 drivers/staging/iio/iio_input.h
--
1.7.10.2
^ permalink raw reply [flat|nested] 15+ messages in thread* [PATCH 4/4] staging:iio: Proof of concept input driver. 2012-05-30 19:36 [PATCH 0/4 V2] staging:iio: Add support for multiple buffers Jonathan Cameron @ 2012-05-30 19:36 ` Jonathan Cameron 2012-06-08 15:27 ` Lars-Peter Clausen 0 siblings, 1 reply; 15+ messages in thread From: Jonathan Cameron @ 2012-05-30 19:36 UTC (permalink / raw) To: linux-iio; +Cc: Jonathan Cameron, Jonathan Cameron From: Jonathan Cameron <jic23@cam.ac.uk> This is no where near ready to merge. Lots of stuff missing. Signed-off-by: Jonathan Cameron <jic23@kernel.org> --- drivers/staging/iio/Kconfig | 11 +++ drivers/staging/iio/Makefile | 1 + drivers/staging/iio/iio_input.c | 176 +++++++++++++++++++++++++++++++++++++++ drivers/staging/iio/iio_input.h | 23 +++++ 4 files changed, 211 insertions(+) diff --git a/drivers/staging/iio/Kconfig b/drivers/staging/iio/Kconfig index 04cd6ec..022463e 100644 --- a/drivers/staging/iio/Kconfig +++ b/drivers/staging/iio/Kconfig @@ -4,6 +4,17 @@ menu "IIO staging drivers" depends on IIO +config IIO_ST_INPUT + tristate "Input driver that uses channels specified via iio maps" + depends on INPUT + depends on IIO_BUFFER + select IIO_BUFFER_CB + help + Client driver for IIO via the push interfaces. Used to provide + and input interface for IIO devices that can feed a buffer on + a trigger interrupt. Note that all channels must come from a + single IIO supported device. + config IIO_ST_HWMON tristate "Hwmon driver that uses channels specified via iio maps" depends on HWMON diff --git a/drivers/staging/iio/Makefile b/drivers/staging/iio/Makefile index fa6937d..9e71498 100644 --- a/drivers/staging/iio/Makefile +++ b/drivers/staging/iio/Makefile @@ -12,6 +12,7 @@ iio_dummy-$(CONFIG_IIO_SIMPLE_DUMMY_BUFFER) += iio_simple_dummy_buffer.o obj-$(CONFIG_IIO_DUMMY_EVGEN) += iio_dummy_evgen.o obj-$(CONFIG_IIO_ST_HWMON) += iio_hwmon.o +obj-$(CONFIG_IIO_ST_INPUT) += iio_input.o obj-y += accel/ obj-y += adc/ diff --git a/drivers/staging/iio/iio_input.c b/drivers/staging/iio/iio_input.c new file mode 100644 index 0000000..42f4cfb --- /dev/null +++ b/drivers/staging/iio/iio_input.c @@ -0,0 +1,176 @@ +/* + * The industrial I/O input client driver + * + * Copyright (c) 2011 Jonathan Cameron + * + * This program is free software; you can redistribute it and/or modify it + * under the terms of the GNU General Public License version 2 as published by + * the Free Software Foundation. + */ + +#include <linux/kernel.h> +#include <linux/slab.h> +#include <linux/module.h> +#include <linux/err.h> +#include <linux/platform_device.h> +#include <linux/input.h> +#include <linux/iio/buffer.h> +#include <linux/iio/consumer.h> +#include "iio_input.h" + +struct iio_input_state { + struct iio_cb_buffer *buff; + struct input_dev *idev; +}; + +static int iio_channel_value(u8 *data, + const struct iio_chan_spec *chan, + int *val) +{ + int value; + + if (chan->scan_type.sign == 's') { + switch (chan->scan_type.storagebits) { + case 8: + value = *(s8 *)(data); + break; + case 16: + value = *(s16 *)(data); + break; + case 32: + value = *(s32 *)(data); + break; + default: + return -EINVAL; + } + value >>= chan->scan_type.shift; + value &= (1 << chan->scan_type.realbits) - 1; + value = (value << (sizeof(value)*8 - chan->scan_type.realbits)) + >> (sizeof(value)*8 - chan->scan_type.realbits); + } else { + switch (chan->scan_type.storagebits) { + case 8: + value = *(u8 *)(data); + break; + case 16: + value = *(u16 *)(data); + break; + case 32: + value = *(u32 *)(data); + break; + default: + return -EINVAL; + } + value >>= chan->scan_type.shift; + value &= (1 << chan->scan_type.realbits) - 1; + } + *val = value; + + return 0; +} + +static int iio_input_store_to(u8 *data, void *private) +{ + struct iio_input_state *st = private; + struct iio_channel *channel; + struct iio_input_channel_data *input_data; + int offset = 0; + int value, ret; + + channel = iio_channel_cb_get_channels(st->buff); + while (channel->indio_dev) { + input_data = channel->data; + offset = ALIGN(offset, + channel->channel->scan_type.storagebits/8); + offset += channel->channel->scan_type.storagebits/8; + ret = iio_channel_value(&data[offset], + channel->channel, + &value); + if (ret < 0) + return ret; + input_report_abs(st->idev, input_data->code, value); + } + input_sync(st->idev); + + return 0; +} + +static int __devinit iio_input_probe(struct platform_device *pdev) +{ + struct iio_input_state *st; + int ret; + struct iio_channel *channel; + struct iio_input_channel_data *input_data; + + st = kzalloc(sizeof(*st), GFP_KERNEL); + if (st == NULL) + return -ENOMEM; + platform_set_drvdata(pdev, st); + st->buff = iio_channel_get_all_cb(dev_name(&pdev->dev), + &iio_input_store_to, + st); + if (IS_ERR(st->buff)) { + ret = PTR_ERR(st->buff); + goto error_free_state; + } + + st->idev = input_allocate_device(); + if (!st->idev) { + ret = -ENOMEM; + goto error_channels_release_all; + } + + __set_bit(EV_ABS, st->idev->evbit); + channel = iio_channel_cb_get_channels(st->buff); + while (channel->indio_dev) { + input_data = channel->data; + input_set_abs_params(st->idev, input_data->code, + input_data->min, + input_data->max, + input_data->fuzz, + input_data->flat); + } + + ret = input_register_device(st->idev); + if (ret < 0) + goto error_free_idev; + + /* NORMALLY IN THE OPEN */ + iio_channel_start_all_cb(st->buff); + + return 0; +error_free_idev: + input_free_device(st->idev); +error_channels_release_all: + iio_channel_release_all_cb(st->buff); +error_free_state: + kfree(st); + return ret; +} + +static int __devexit iio_input_remove(struct platform_device *pdev) +{ + struct iio_input_state *st = platform_get_drvdata(pdev); + /* NORMALLY IN THE CLOSE */ + iio_channel_stop_all_cb(st->buff); + input_unregister_device(st->idev); + iio_channel_release_all_cb(st->buff); + + kfree(st); + return 0; +} + +static struct platform_driver iio_input_driver = { + .driver = { + .name = "iio_snoop", + .owner = THIS_MODULE, + }, + .probe = iio_input_probe, + .remove = __devexit_p(iio_input_remove), +}; + +module_platform_driver(iio_input_driver); + +MODULE_AUTHOR("Jonathan Cameron <jic23@cam.ac.uk>"); +MODULE_DESCRIPTION("IIO input buffer driver"); +MODULE_LICENSE("GPL v2"); diff --git a/drivers/staging/iio/iio_input.h b/drivers/staging/iio/iio_input.h new file mode 100644 index 0000000..cfd1d34 --- /dev/null +++ b/drivers/staging/iio/iio_input.h @@ -0,0 +1,23 @@ +/* + * The industrial I/O input client driver + * + * Copyright (c) 2011 Jonathan Cameron + * + * This program is free software; you can redistribute it and/or modify it + * under the terms of the GNU General Public License version 2 as published by + * the Free Software Foundation. + */ + +/** + * iio_input_channel_data - description of the channel for input subsystem + * @code: Absolute axis. + * @min: Minimum value. + * @max: Maximum value. + * @fuzz: Used to filter noise from the event stream. + * @flat: Values within this value will be discarded by joydev + * and reported as 0 instead. + */ +struct iio_input_channel_data { + unsigned int code; + int min, max, fuzz, flat; +}; -- 1.7.10.2 ^ permalink raw reply related [flat|nested] 15+ messages in thread
* Re: [PATCH 4/4] staging:iio: Proof of concept input driver. 2012-05-30 19:36 ` [PATCH 4/4] staging:iio: Proof of concept input driver Jonathan Cameron @ 2012-06-08 15:27 ` Lars-Peter Clausen 2012-06-09 17:56 ` Jonathan Cameron 0 siblings, 1 reply; 15+ messages in thread From: Lars-Peter Clausen @ 2012-06-08 15:27 UTC (permalink / raw) To: Jonathan Cameron; +Cc: linux-iio, Jonathan Cameron On 05/30/2012 09:36 PM, Jonathan Cameron wrote: > From: Jonathan Cameron <jic23@cam.ac.uk> > > This is no where near ready to merge. Lots of stuff missing. > > Signed-off-by: Jonathan Cameron <jic23@kernel.org> > --- > drivers/staging/iio/Kconfig | 11 +++ > drivers/staging/iio/Makefile | 1 + > drivers/staging/iio/iio_input.c | 176 +++++++++++++++++++++++++++++++++++++++ > drivers/staging/iio/iio_input.h | 23 +++++ > 4 files changed, 211 insertions(+) > > diff --git a/drivers/staging/iio/Kconfig b/drivers/staging/iio/Kconfig > index 04cd6ec..022463e 100644 > --- a/drivers/staging/iio/Kconfig > +++ b/drivers/staging/iio/Kconfig > @@ -4,6 +4,17 @@ [...] > +}; > + > +static int iio_channel_value(u8 *data, > + const struct iio_chan_spec *chan, > + int *val) > +{ > + int value; > + > + if (chan->scan_type.sign == 's') { > + switch (chan->scan_type.storagebits) { > + case 8: > + value = *(s8 *)(data); > + break; > + case 16: > + value = *(s16 *)(data); > + break; > + case 32: > + value = *(s32 *)(data); > + break; > + default: > + return -EINVAL; > + } > + value >>= chan->scan_type.shift; > + value &= (1 << chan->scan_type.realbits) - 1; > + value = (value << (sizeof(value)*8 - chan->scan_type.realbits)) > + >> (sizeof(value)*8 - chan->scan_type.realbits); This looks scarey. There is a sign_extend32 function. It probably makes sense to add one which works on ints. Btw. I think until the sign extension the type should still be unsigned. This will also get rid of the duplicated code. > + } else { > + switch (chan->scan_type.storagebits) { > + case 8: > + value = *(u8 *)(data); > + break; > + case 16: > + value = *(u16 *)(data); > + break; > + case 32: > + value = *(u32 *)(data); > + break; > + default: > + return -EINVAL; > + } > + value >>= chan->scan_type.shift; > + value &= (1 << chan->scan_type.realbits) - 1; > + } > + *val = value; This function probably needs to be extended to deal with the endianness field of the scan type at some point. > + > + return 0; > +} > + > +static int iio_input_store_to(u8 *data, void *private) > +{ > + struct iio_input_state *st = private; > + struct iio_channel *channel; > + struct iio_input_channel_data *input_data; > + int offset = 0; > + int value, ret; > + > + channel = iio_channel_cb_get_channels(st->buff); > + while (channel->indio_dev) { > + input_data = channel->data; > + offset = ALIGN(offset, > + channel->channel->scan_type.storagebits/8); > + offset += channel->channel->scan_type.storagebits/8; Maybe I'm missing something, but shouldn't offset be increased after reading the data? > + ret = iio_channel_value(&data[offset], > + channel->channel, > + &value); > + if (ret < 0) > + return ret; > + input_report_abs(st->idev, input_data->code, value); > + } > + input_sync(st->idev); > + > + return 0; > +} > + [...] ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH 4/4] staging:iio: Proof of concept input driver. 2012-06-08 15:27 ` Lars-Peter Clausen @ 2012-06-09 17:56 ` Jonathan Cameron 0 siblings, 0 replies; 15+ messages in thread From: Jonathan Cameron @ 2012-06-09 17:56 UTC (permalink / raw) To: Lars-Peter Clausen; +Cc: linux-iio, Jonathan Cameron On 06/08/2012 04:27 PM, Lars-Peter Clausen wrote: > On 05/30/2012 09:36 PM, Jonathan Cameron wrote: >> From: Jonathan Cameron <jic23@cam.ac.uk> >> >> This is no where near ready to merge. Lots of stuff missing. >> >> Signed-off-by: Jonathan Cameron <jic23@kernel.org> >> --- >> drivers/staging/iio/Kconfig | 11 +++ >> drivers/staging/iio/Makefile | 1 + >> drivers/staging/iio/iio_input.c | 176 +++++++++++++++++++++++++++++++++++++++ >> drivers/staging/iio/iio_input.h | 23 +++++ >> 4 files changed, 211 insertions(+) >> >> diff --git a/drivers/staging/iio/Kconfig b/drivers/staging/iio/Kconfig >> index 04cd6ec..022463e 100644 >> --- a/drivers/staging/iio/Kconfig >> +++ b/drivers/staging/iio/Kconfig >> @@ -4,6 +4,17 @@ > [...] >> +}; >> + >> +static int iio_channel_value(u8 *data, >> + const struct iio_chan_spec *chan, >> + int *val) >> +{ >> + int value; >> + >> + if (chan->scan_type.sign == 's') { >> + switch (chan->scan_type.storagebits) { >> + case 8: >> + value = *(s8 *)(data); >> + break; >> + case 16: >> + value = *(s16 *)(data); >> + break; >> + case 32: >> + value = *(s32 *)(data); >> + break; >> + default: >> + return -EINVAL; >> + } >> + value >>= chan->scan_type.shift; >> + value &= (1 << chan->scan_type.realbits) - 1; >> + value = (value << (sizeof(value)*8 - chan->scan_type.realbits)) >> + >> (sizeof(value)*8 - chan->scan_type.realbits); > > This looks scarey. There is a sign_extend32 function. It probably makes > sense to add one which works on ints. Btw. I think until the sign extension > the type should still be unsigned. This will also get rid of the duplicated > code. Not sure there is any real reason not to use sign_extend32 as is. Can just convert much later at the input_report_abs call. > >> + } else { >> + switch (chan->scan_type.storagebits) { >> + case 8: >> + value = *(u8 *)(data); >> + break; >> + case 16: >> + value = *(u16 *)(data); >> + break; >> + case 32: >> + value = *(u32 *)(data); >> + break; >> + default: >> + return -EINVAL; >> + } >> + value >>= chan->scan_type.shift; >> + value &= (1 << chan->scan_type.realbits) - 1; >> + } >> + *val = value; > > This function probably needs to be extended to deal with the endianness > field of the scan type at some point. Yup. Lots of extensions needed to this driver.... >> + >> + return 0; >> +} >> + >> +static int iio_input_store_to(u8 *data, void *private) >> +{ >> + struct iio_input_state *st = private; >> + struct iio_channel *channel; >> + struct iio_input_channel_data *input_data; >> + int offset = 0; >> + int value, ret; >> + >> + channel = iio_channel_cb_get_channels(st->buff); >> + while (channel->indio_dev) { >> + input_data = channel->data; >> + offset = ALIGN(offset, >> + channel->channel->scan_type.storagebits/8); >> + offset += channel->channel->scan_type.storagebits/8; > > Maybe I'm missing something, but shouldn't offset be increased after reading > the data? Indeed. I'm clearly going mad ;) > >> + ret = iio_channel_value(&data[offset], >> + channel->channel, >> + &value); >> + if (ret < 0) >> + return ret; >> + input_report_abs(st->idev, input_data->code, value); >> + } >> + input_sync(st->idev); >> + >> + return 0; >> +} >> + > [...] > -- > 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 ^ permalink raw reply [flat|nested] 15+ messages in thread
* [PATCH 0/4 V3] staging:iio: Add support for multiple buffers
@ 2012-06-30 19:06 Jonathan Cameron
2012-06-30 19:06 ` [PATCH 4/4] staging:iio: Proof of concept input driver Jonathan Cameron
0 siblings, 1 reply; 15+ messages in thread
From: Jonathan Cameron @ 2012-06-30 19:06 UTC (permalink / raw)
To: linux-iio; +Cc: Jonathan Cameron
Changes since V2.
Improved handling of error conditions in the buffer insertion and
removal code. (suggested by Lars-Peter). I want to take another
look at these, but didn't want to keep people from testing this
in the meantime.
Also a number of minor fixes that came up during testing.
Thanks,
Jonathan
Jonathan Cameron (4):
staging:iio: Add support for multiple buffers
staging:iio:in kernel users: Add a data field for channel specific
info.
staging:iio: add a callback buffer for in kernel push interface
staging:iio: Proof of concept input driver.
drivers/iio/Kconfig | 6 +
drivers/iio/Makefile | 1 +
drivers/iio/adc/at91_adc.c | 4 +-
drivers/iio/buffer_cb.c | 115 +++++++
drivers/iio/industrialio-buffer.c | 378 ++++++++++++++++--------
drivers/iio/industrialio-core.c | 1 +
drivers/iio/inkern.c | 1 +
drivers/staging/iio/Kconfig | 11 +
drivers/staging/iio/Makefile | 1 +
drivers/staging/iio/accel/adis16201_ring.c | 4 +-
drivers/staging/iio/accel/adis16203_ring.c | 6 +-
drivers/staging/iio/accel/adis16204_ring.c | 3 +-
drivers/staging/iio/accel/adis16209_ring.c | 3 +-
drivers/staging/iio/accel/adis16240_ring.c | 4 +-
drivers/staging/iio/accel/lis3l02dq_ring.c | 3 +-
drivers/staging/iio/adc/ad7192.c | 3 +-
drivers/staging/iio/adc/ad7298_ring.c | 5 +-
drivers/staging/iio/adc/ad7476_ring.c | 2 +-
drivers/staging/iio/adc/ad7606_ring.c | 3 +-
drivers/staging/iio/adc/ad7793.c | 3 +-
drivers/staging/iio/adc/ad7887_ring.c | 2 +-
drivers/staging/iio/adc/ad799x_ring.c | 3 +-
drivers/staging/iio/adc/max1363_ring.c | 2 +-
drivers/staging/iio/gyro/adis16260_ring.c | 3 +-
drivers/staging/iio/iio_input.c | 221 ++++++++++++++
drivers/staging/iio/iio_input.h | 23 ++
drivers/staging/iio/iio_simple_dummy_buffer.c | 5 +-
drivers/staging/iio/impedance-analyzer/ad5933.c | 3 +-
drivers/staging/iio/imu/adis16400_ring.c | 2 +-
drivers/staging/iio/meter/ade7758_ring.c | 3 +-
include/linux/iio/buffer.h | 24 +-
include/linux/iio/consumer.h | 48 +++
include/linux/iio/iio.h | 2 +
include/linux/iio/machine.h | 2 +
34 files changed, 734 insertions(+), 166 deletions(-)
create mode 100644 drivers/iio/buffer_cb.c
create mode 100644 drivers/staging/iio/iio_input.c
create mode 100644 drivers/staging/iio/iio_input.h
--
1.7.11.1
^ permalink raw reply [flat|nested] 15+ messages in thread* [PATCH 4/4] staging:iio: Proof of concept input driver. 2012-06-30 19:06 [PATCH 0/4 V3] staging:iio: Add support for multiple buffers Jonathan Cameron @ 2012-06-30 19:06 ` Jonathan Cameron 0 siblings, 0 replies; 15+ messages in thread From: Jonathan Cameron @ 2012-06-30 19:06 UTC (permalink / raw) To: linux-iio; +Cc: Jonathan Cameron, Jonathan Cameron From: Jonathan Cameron <jic23@cam.ac.uk> This is no where near ready to merge. Lots of stuff missing. Signed-off-by: Jonathan Cameron <jic23@kernel.org> --- drivers/staging/iio/Kconfig | 11 ++ drivers/staging/iio/Makefile | 1 + drivers/staging/iio/iio_input.c | 221 ++++++++++++++++++++++++++++++++++++++++ drivers/staging/iio/iio_input.h | 23 +++++ 4 files changed, 256 insertions(+) diff --git a/drivers/staging/iio/Kconfig b/drivers/staging/iio/Kconfig index 04cd6ec..022463e 100644 --- a/drivers/staging/iio/Kconfig +++ b/drivers/staging/iio/Kconfig @@ -4,6 +4,17 @@ menu "IIO staging drivers" depends on IIO +config IIO_ST_INPUT + tristate "Input driver that uses channels specified via iio maps" + depends on INPUT + depends on IIO_BUFFER + select IIO_BUFFER_CB + help + Client driver for IIO via the push interfaces. Used to provide + and input interface for IIO devices that can feed a buffer on + a trigger interrupt. Note that all channels must come from a + single IIO supported device. + config IIO_ST_HWMON tristate "Hwmon driver that uses channels specified via iio maps" depends on HWMON diff --git a/drivers/staging/iio/Makefile b/drivers/staging/iio/Makefile index fa6937d..9e71498 100644 --- a/drivers/staging/iio/Makefile +++ b/drivers/staging/iio/Makefile @@ -12,6 +12,7 @@ iio_dummy-$(CONFIG_IIO_SIMPLE_DUMMY_BUFFER) += iio_simple_dummy_buffer.o obj-$(CONFIG_IIO_DUMMY_EVGEN) += iio_dummy_evgen.o obj-$(CONFIG_IIO_ST_HWMON) += iio_hwmon.o +obj-$(CONFIG_IIO_ST_INPUT) += iio_input.o obj-y += accel/ obj-y += adc/ diff --git a/drivers/staging/iio/iio_input.c b/drivers/staging/iio/iio_input.c new file mode 100644 index 0000000..207a392 --- /dev/null +++ b/drivers/staging/iio/iio_input.c @@ -0,0 +1,221 @@ +/* + * The industrial I/O input client driver + * + * Copyright (c) 2011 Jonathan Cameron + * + * This program is free software; you can redistribute it and/or modify it + * under the terms of the GNU General Public License version 2 as published by + * the Free Software Foundation. + */ + +#include <linux/kernel.h> +#include <linux/slab.h> +#include <linux/module.h> +#include <linux/err.h> +#include <linux/platform_device.h> +#include <linux/input.h> +#include <linux/iio/buffer.h> +#include <linux/iio/consumer.h> +#include "iio_input.h" + +struct iio_input_state { + struct iio_cb_buffer *buff; + struct input_dev *idev; +}; + +static int iio_channel_value(u8 *data, + const struct iio_chan_spec *chan, + s32 *val) +{ + u32 value = 0; + + if (chan->scan_type.sign == 's') { + switch (chan->scan_type.storagebits) { + case 8: + value = *(s8 *)(data); + break; + case 16: + switch (chan->scan_type.endianness) { + case IIO_CPU: + value = *(s16 *)(data); + break; + case IIO_BE: + value = be16_to_cpu(*(__be16 *)data); + break; + case IIO_LE: + value = le16_to_cpu(*(__le16 *)data); + break; + } + break; + case 32: + switch (chan->scan_type.endianness) { + case IIO_CPU: + value = *(s32 *)(data); + break; + case IIO_BE: + value = be32_to_cpu(*(__be32 *)data); + break; + case IIO_LE: + value = le32_to_cpu(*(__le32 *)data); + break; + } + break; + default: + return -EINVAL; + } + *val = sign_extend32(value, chan->scan_type.storagebits); + } else { + switch (chan->scan_type.storagebits) { + case 8: + value = *(u8 *)(data); + break; + case 16: + switch (chan->scan_type.endianness) { + case IIO_CPU: + value = *(u16 *)(data); + break; + case IIO_BE: + value = be16_to_cpu(*(__be16 *)data); + break; + case IIO_LE: + value = le16_to_cpu(*(__le16 *)data); + break; + } + break; + case 32: + switch (chan->scan_type.endianness) { + case IIO_CPU: + value = *(u32 *)(data); + break; + case IIO_BE: + value = be32_to_cpu(*(__be32 *)data); + break; + case IIO_LE: + value = le32_to_cpu(*(__le32 *)data); + break; + } + break; + default: + return -EINVAL; + } + value >>= chan->scan_type.shift; + value &= (1 << chan->scan_type.realbits) - 1; + *val = value; + } + + return 0; +} + +static int iio_input_store_to(u8 *data, void *private) +{ + struct iio_input_state *st = private; + struct iio_channel *channel; + struct iio_input_channel_data *input_data; + int offset = 0; + s32 value; + int ret; + + channel = iio_channel_cb_get_channels(st->buff); + while (channel->indio_dev) { + input_data = channel->data; + offset = ALIGN(offset, + channel->channel->scan_type.storagebits/8); + ret = iio_channel_value(&data[offset], + channel->channel, + &value); + if (ret < 0) + return ret; + offset += channel->channel->scan_type.storagebits/8; + + input_report_abs(st->idev, input_data->code, value); + channel++; + } + input_sync(st->idev); + + return 0; +} + +static int __devinit iio_input_probe(struct platform_device *pdev) +{ + struct iio_input_state *st; + int ret; + struct iio_channel *channel; + struct iio_input_channel_data *input_data; + + st = kzalloc(sizeof(*st), GFP_KERNEL); + if (st == NULL) + return -ENOMEM; + platform_set_drvdata(pdev, st); + st->buff = iio_channel_get_all_cb(dev_name(&pdev->dev), + &iio_input_store_to, + st); + if (IS_ERR(st->buff)) { + ret = PTR_ERR(st->buff); + goto error_free_state; + } + + st->idev = input_allocate_device(); + if (!st->idev) { + ret = -ENOMEM; + goto error_channels_release_all; + } + + __set_bit(EV_ABS, st->idev->evbit); + channel = iio_channel_cb_get_channels(st->buff); + while (channel->indio_dev) { + input_data = channel->data; + input_set_abs_params(st->idev, input_data->code, + input_data->min, + input_data->max, + input_data->fuzz, + input_data->flat); + channel++; + } + + ret = input_register_device(st->idev); + if (ret < 0) + goto error_free_idev; + + /* NORMALLY IN THE OPEN */ + ret = iio_channel_start_all_cb(st->buff); + if (ret < 0) + goto error_unregister_input; + + return 0; +error_unregister_input: + input_unregister_device(st->idev); +error_free_idev: + input_free_device(st->idev); +error_channels_release_all: + iio_channel_release_all_cb(st->buff); +error_free_state: + kfree(st); + return ret; +} + +static int __devexit iio_input_remove(struct platform_device *pdev) +{ + struct iio_input_state *st = platform_get_drvdata(pdev); + /* NORMALLY IN THE CLOSE */ + iio_channel_stop_all_cb(st->buff); + input_unregister_device(st->idev); + iio_channel_release_all_cb(st->buff); + + kfree(st); + return 0; +} + +static struct platform_driver iio_input_driver = { + .driver = { + .name = "iio_input_bridge", + .owner = THIS_MODULE, + }, + .probe = iio_input_probe, + .remove = __devexit_p(iio_input_remove), +}; + +module_platform_driver(iio_input_driver); + +MODULE_AUTHOR("Jonathan Cameron <jic23@cam.ac.uk>"); +MODULE_DESCRIPTION("IIO input buffer driver"); +MODULE_LICENSE("GPL v2"); diff --git a/drivers/staging/iio/iio_input.h b/drivers/staging/iio/iio_input.h new file mode 100644 index 0000000..cfd1d34 --- /dev/null +++ b/drivers/staging/iio/iio_input.h @@ -0,0 +1,23 @@ +/* + * The industrial I/O input client driver + * + * Copyright (c) 2011 Jonathan Cameron + * + * This program is free software; you can redistribute it and/or modify it + * under the terms of the GNU General Public License version 2 as published by + * the Free Software Foundation. + */ + +/** + * iio_input_channel_data - description of the channel for input subsystem + * @code: Absolute axis. + * @min: Minimum value. + * @max: Maximum value. + * @fuzz: Used to filter noise from the event stream. + * @flat: Values within this value will be discarded by joydev + * and reported as 0 instead. + */ +struct iio_input_channel_data { + unsigned int code; + int min, max, fuzz, flat; +}; -- 1.7.11.1 ^ permalink raw reply related [flat|nested] 15+ messages in thread
* [PATCH 0/4 V5] staging:iio: Add support for multiple buffers (testing required!)
@ 2012-10-13 9:24 Jonathan Cameron
2012-10-13 9:24 ` [PATCH 4/4] staging:iio: Proof of concept input driver Jonathan Cameron
0 siblings, 1 reply; 15+ messages in thread
From: Jonathan Cameron @ 2012-10-13 9:24 UTC (permalink / raw)
To: linux-iio; +Cc: Michael.Hennerich, lars, srinivas.pandruvada, Jonathan Cameron
In quick summary this is needed to do interrupt driven in kernel
clients as it adds the option to push to multiple 'buffer' interfaces.
Note that some of these may not actually do any buffering but rather
push directly into another subsytem (the example is input).
I would like to merge at least the first 3 patches early in this cycle.
What I need is some testing across a reasonable range of devices. Review
is welcome of course as well!
The complexity in this set is entirely in the changes to industrialio-buffer.c
concerning how we go about adding and removing client buffers. To do this
we need to work out which scan mask is appropriate given the requests of
all the given buffers (and whether we can satisfy it). I've been testing
with the max1363 driver which probably has the most complex set of available
scan masks. Thus when a new buffer is added or one is removed the whole
buffer set are torn down and built up again. This may be undesirable for
some applications but should occur only rarely.
The example driver is for bridging to the input subsystem and still needs
some work, but that can occur later in this merge cycle whereas the changes
to the core really want to go in asap.
Changes since v3
Rebased and added other drivers that have occured whilst this patch series
has been languishing in my local tree.
Changes since V2.
Improved handling of error conditions in the buffer insertion and
removal code. (suggested by Lars-Peter). I want to take another
look at these, but didn't want to keep people from testing this
in the meantime.
Also a number of minor fixes that came up during testing.
Thanks,
Jonathan
Jonathan Cameron (4):
staging:iio: Add support for multiple buffers
staging:iio:in kernel users: Add a data field for channel specific
info.
staging:iio: add a callback buffer for in kernel push interface
staging:iio: Proof of concept input driver.
drivers/iio/Kconfig | 6 +
drivers/iio/Makefile | 1 +
drivers/iio/accel/hid-sensor-accel-3d.c | 15 +-
drivers/iio/adc/ad7266.c | 3 +-
drivers/iio/adc/ad7476.c | 2 +-
drivers/iio/adc/ad_sigma_delta.c | 2 +-
drivers/iio/adc/at91_adc.c | 3 +-
drivers/iio/buffer_cb.c | 113 +++++++
drivers/iio/gyro/hid-sensor-gyro-3d.c | 15 +-
drivers/iio/industrialio-buffer.c | 377 ++++++++++++++++--------
drivers/iio/industrialio-core.c | 1 +
drivers/iio/inkern.c | 1 +
drivers/iio/light/adjd_s311.c | 3 +-
drivers/iio/light/hid-sensor-als.c | 15 +-
drivers/iio/magnetometer/hid-sensor-magn-3d.c | 15 +-
drivers/staging/iio/Kconfig | 11 +
drivers/staging/iio/Makefile | 1 +
drivers/staging/iio/accel/adis16201_ring.c | 2 +-
drivers/staging/iio/accel/adis16203_ring.c | 2 +-
drivers/staging/iio/accel/adis16204_ring.c | 2 +-
drivers/staging/iio/accel/adis16209_ring.c | 2 +-
drivers/staging/iio/accel/adis16240_ring.c | 2 +-
drivers/staging/iio/accel/lis3l02dq_ring.c | 2 +-
drivers/staging/iio/adc/ad7298_ring.c | 2 +-
drivers/staging/iio/adc/ad7606_ring.c | 2 +-
drivers/staging/iio/adc/ad7887_ring.c | 2 +-
drivers/staging/iio/adc/ad799x_ring.c | 2 +-
drivers/staging/iio/adc/max1363_ring.c | 2 +-
drivers/staging/iio/gyro/adis16260_ring.c | 2 +-
drivers/staging/iio/iio_input.c | 225 ++++++++++++++
drivers/staging/iio/iio_input.h | 23 ++
drivers/staging/iio/iio_simple_dummy_buffer.c | 5 +-
drivers/staging/iio/impedance-analyzer/ad5933.c | 4 +-
drivers/staging/iio/imu/adis16400_ring.c | 5 +-
drivers/staging/iio/meter/ade7758_ring.c | 2 +-
include/linux/iio/buffer.h | 24 +-
include/linux/iio/consumer.h | 48 +++
include/linux/iio/iio.h | 2 +
include/linux/iio/machine.h | 2 +
39 files changed, 740 insertions(+), 208 deletions(-)
create mode 100644 drivers/iio/buffer_cb.c
create mode 100644 drivers/staging/iio/iio_input.c
create mode 100644 drivers/staging/iio/iio_input.h
--
1.7.12.2
^ permalink raw reply [flat|nested] 15+ messages in thread* [PATCH 4/4] staging:iio: Proof of concept input driver. 2012-10-13 9:24 [PATCH 0/4 V5] staging:iio: Add support for multiple buffers (testing required!) Jonathan Cameron @ 2012-10-13 9:24 ` Jonathan Cameron 2012-10-13 9:57 ` Jonathan Cameron 0 siblings, 1 reply; 15+ messages in thread From: Jonathan Cameron @ 2012-10-13 9:24 UTC (permalink / raw) To: linux-iio Cc: Michael.Hennerich, lars, srinivas.pandruvada, Jonathan Cameron, Jonathan Cameron From: Jonathan Cameron <jic23@cam.ac.uk> This is no where near ready to merge. Lots of stuff missing. Signed-off-by: Jonathan Cameron <jic23@kernel.org> --- drivers/staging/iio/Kconfig | 11 ++ drivers/staging/iio/Makefile | 1 + drivers/staging/iio/iio_input.c | 225 ++++++++++++++++++++++++++++++++++++++++ drivers/staging/iio/iio_input.h | 23 ++++ 4 files changed, 260 insertions(+) diff --git a/drivers/staging/iio/Kconfig b/drivers/staging/iio/Kconfig index ca56c75..3d21f6d 100644 --- a/drivers/staging/iio/Kconfig +++ b/drivers/staging/iio/Kconfig @@ -4,6 +4,17 @@ menu "IIO staging drivers" depends on IIO +config IIO_ST_INPUT + tristate "Input driver that uses channels specified via iio maps" + depends on INPUT + depends on IIO_BUFFER + select IIO_BUFFER_CB + help + Client driver for IIO via the push interfaces. Used to provide + and input interface for IIO devices that can feed a buffer on + a trigger interrupt. Note that all channels must come from a + single IIO supported device. + config IIO_ST_HWMON tristate "Hwmon driver that uses channels specified via iio maps" depends on HWMON diff --git a/drivers/staging/iio/Makefile b/drivers/staging/iio/Makefile index fa6937d..9e71498 100644 --- a/drivers/staging/iio/Makefile +++ b/drivers/staging/iio/Makefile @@ -12,6 +12,7 @@ iio_dummy-$(CONFIG_IIO_SIMPLE_DUMMY_BUFFER) += iio_simple_dummy_buffer.o obj-$(CONFIG_IIO_DUMMY_EVGEN) += iio_dummy_evgen.o obj-$(CONFIG_IIO_ST_HWMON) += iio_hwmon.o +obj-$(CONFIG_IIO_ST_INPUT) += iio_input.o obj-y += accel/ obj-y += adc/ diff --git a/drivers/staging/iio/iio_input.c b/drivers/staging/iio/iio_input.c new file mode 100644 index 0000000..c34b325 --- /dev/null +++ b/drivers/staging/iio/iio_input.c @@ -0,0 +1,225 @@ +/* + * The industrial I/O input client driver + * + * Copyright (c) 2011 Jonathan Cameron + * + * This program is free software; you can redistribute it and/or modify it + * under the terms of the GNU General Public License version 2 as published by + * the Free Software Foundation. + */ + +#include <linux/kernel.h> +#include <linux/slab.h> +#include <linux/module.h> +#include <linux/err.h> +#include <linux/platform_device.h> +#include <linux/input.h> +#include <linux/iio/buffer.h> +#include <linux/iio/consumer.h> +#include "iio_input.h" + +struct iio_input_state { + struct iio_cb_buffer *buff; + struct input_dev *idev; +}; + +static int iio_channel_value(u8 *data, + const struct iio_chan_spec *chan, + s32 *val) +{ + u32 value = 0; + + if (chan->scan_type.sign == 's') { + switch (chan->scan_type.storagebits) { + case 8: + value = *(s8 *)(data); + break; + case 16: + switch (chan->scan_type.endianness) { + case IIO_CPU: + value = *(s16 *)(data); + break; + case IIO_BE: + value = be16_to_cpu(*(__be16 *)data); + break; + case IIO_LE: + value = le16_to_cpu(*(__le16 *)data); + break; + } + break; + case 32: + switch (chan->scan_type.endianness) { + case IIO_CPU: + value = *(s32 *)(data); + break; + case IIO_BE: + value = be32_to_cpu(*(__be32 *)data); + break; + case IIO_LE: + value = le32_to_cpu(*(__le32 *)data); + break; + } + break; + default: + return -EINVAL; + } + *val = sign_extend32(value, chan->scan_type.storagebits); + } else { + switch (chan->scan_type.storagebits) { + case 8: + value = *(u8 *)(data); + break; + case 16: + switch (chan->scan_type.endianness) { + case IIO_CPU: + value = *(u16 *)(data); + break; + case IIO_BE: + value = be16_to_cpu(*(__be16 *)data); + break; + case IIO_LE: + value = le16_to_cpu(*(__le16 *)data); + break; + } + break; + case 32: + switch (chan->scan_type.endianness) { + case IIO_CPU: + value = *(u32 *)(data); + break; + case IIO_BE: + value = be32_to_cpu(*(__be32 *)data); + break; + case IIO_LE: + value = le32_to_cpu(*(__le32 *)data); + break; + } + break; + default: + return -EINVAL; + } + value >>= chan->scan_type.shift; + value &= (1 << chan->scan_type.realbits) - 1; + *val = value; + } + + return 0; +} + +static int iio_input_store_to(u8 *data, void *private) +{ + struct iio_input_state *st = private; + struct iio_channel *channel; + struct iio_input_channel_data *input_data; + int offset = 0; + s32 value; + int ret; + + channel = iio_channel_cb_get_channels(st->buff); + while (channel->indio_dev) { + input_data = channel->data; + offset = ALIGN(offset, + channel->channel->scan_type.storagebits/8); + ret = iio_channel_value(&data[offset], + channel->channel, + &value); + if (ret < 0) + return ret; + offset += channel->channel->scan_type.storagebits/8; + + input_report_abs(st->idev, input_data->code, value); + channel++; + } + input_sync(st->idev); + + return 0; +} + +static int __devinit iio_input_probe(struct platform_device *pdev) +{ + struct iio_input_state *st; + int ret; + bool registered; + struct iio_channel *channel; + struct iio_input_channel_data *input_data; + + registered = false; + st = kzalloc(sizeof(*st), GFP_KERNEL); + if (st == NULL) + return -ENOMEM; + platform_set_drvdata(pdev, st); + st->buff = iio_channel_get_all_cb(dev_name(&pdev->dev), + &iio_input_store_to, + st); + if (IS_ERR(st->buff)) { + ret = PTR_ERR(st->buff); + goto error_free_state; + } + + st->idev = input_allocate_device(); + if (!st->idev) { + ret = -ENOMEM; + goto error_channels_release_all; + } + + __set_bit(EV_ABS, st->idev->evbit); + channel = iio_channel_cb_get_channels(st->buff); + while (channel->indio_dev) { + input_data = channel->data; + input_set_abs_params(st->idev, input_data->code, + input_data->min, + input_data->max, + input_data->fuzz, + input_data->flat); + channel++; + } + + ret = input_register_device(st->idev); + if (ret < 0) + goto error_free_idev; + registered = true; + /* NORMALLY IN THE OPEN */ + ret = iio_channel_start_all_cb(st->buff); + if (ret < 0) + goto error_unregister_input; + + return 0; +error_unregister_input: + if (registered) + input_unregister_device(st->idev); +error_free_idev: + if (!registered) + input_free_device(st->idev); +error_channels_release_all: + iio_channel_release_all_cb(st->buff); +error_free_state: + kfree(st); + return ret; +} + +static int __devexit iio_input_remove(struct platform_device *pdev) +{ + struct iio_input_state *st = platform_get_drvdata(pdev); + /* NORMALLY IN THE CLOSE */ + iio_channel_stop_all_cb(st->buff); + input_unregister_device(st->idev); + iio_channel_release_all_cb(st->buff); + + kfree(st); + return 0; +} + +static struct platform_driver iio_input_driver = { + .driver = { + .name = "iio_input_bridge", + .owner = THIS_MODULE, + }, + .probe = iio_input_probe, + .remove = __devexit_p(iio_input_remove), +}; + +module_platform_driver(iio_input_driver); + +MODULE_AUTHOR("Jonathan Cameron <jic23@cam.ac.uk>"); +MODULE_DESCRIPTION("IIO input buffer driver"); +MODULE_LICENSE("GPL v2"); diff --git a/drivers/staging/iio/iio_input.h b/drivers/staging/iio/iio_input.h new file mode 100644 index 0000000..cfd1d34 --- /dev/null +++ b/drivers/staging/iio/iio_input.h @@ -0,0 +1,23 @@ +/* + * The industrial I/O input client driver + * + * Copyright (c) 2011 Jonathan Cameron + * + * This program is free software; you can redistribute it and/or modify it + * under the terms of the GNU General Public License version 2 as published by + * the Free Software Foundation. + */ + +/** + * iio_input_channel_data - description of the channel for input subsystem + * @code: Absolute axis. + * @min: Minimum value. + * @max: Maximum value. + * @fuzz: Used to filter noise from the event stream. + * @flat: Values within this value will be discarded by joydev + * and reported as 0 instead. + */ +struct iio_input_channel_data { + unsigned int code; + int min, max, fuzz, flat; +}; -- 1.7.12.2 ^ permalink raw reply related [flat|nested] 15+ messages in thread
* Re: [PATCH 4/4] staging:iio: Proof of concept input driver. 2012-10-13 9:24 ` [PATCH 4/4] staging:iio: Proof of concept input driver Jonathan Cameron @ 2012-10-13 9:57 ` Jonathan Cameron 0 siblings, 0 replies; 15+ messages in thread From: Jonathan Cameron @ 2012-10-13 9:57 UTC (permalink / raw) To: Jonathan Cameron Cc: linux-iio, Michael.Hennerich, lars, srinivas.pandruvada, Jonathan Cameron On 10/13/2012 10:24 AM, Jonathan Cameron wrote: > From: Jonathan Cameron <jic23@cam.ac.uk> > > This is no where near ready to merge. Lots of stuff missing. It also contains an issue with using the input dev after freeing that has already been reported. Naughty me - I forgot about that one. Doesn't effect the usefulness of this as a test tool though! > > Signed-off-by: Jonathan Cameron <jic23@kernel.org> > --- > drivers/staging/iio/Kconfig | 11 ++ > drivers/staging/iio/Makefile | 1 + > drivers/staging/iio/iio_input.c | 225 ++++++++++++++++++++++++++++++++++++++++ > drivers/staging/iio/iio_input.h | 23 ++++ > 4 files changed, 260 insertions(+) > > diff --git a/drivers/staging/iio/Kconfig b/drivers/staging/iio/Kconfig > index ca56c75..3d21f6d 100644 > --- a/drivers/staging/iio/Kconfig > +++ b/drivers/staging/iio/Kconfig > @@ -4,6 +4,17 @@ > menu "IIO staging drivers" > depends on IIO > > +config IIO_ST_INPUT > + tristate "Input driver that uses channels specified via iio maps" > + depends on INPUT > + depends on IIO_BUFFER > + select IIO_BUFFER_CB > + help > + Client driver for IIO via the push interfaces. Used to provide > + and input interface for IIO devices that can feed a buffer on > + a trigger interrupt. Note that all channels must come from a > + single IIO supported device. > + > config IIO_ST_HWMON > tristate "Hwmon driver that uses channels specified via iio maps" > depends on HWMON > diff --git a/drivers/staging/iio/Makefile b/drivers/staging/iio/Makefile > index fa6937d..9e71498 100644 > --- a/drivers/staging/iio/Makefile > +++ b/drivers/staging/iio/Makefile > @@ -12,6 +12,7 @@ iio_dummy-$(CONFIG_IIO_SIMPLE_DUMMY_BUFFER) += iio_simple_dummy_buffer.o > obj-$(CONFIG_IIO_DUMMY_EVGEN) += iio_dummy_evgen.o > > obj-$(CONFIG_IIO_ST_HWMON) += iio_hwmon.o > +obj-$(CONFIG_IIO_ST_INPUT) += iio_input.o > > obj-y += accel/ > obj-y += adc/ > diff --git a/drivers/staging/iio/iio_input.c b/drivers/staging/iio/iio_input.c > new file mode 100644 > index 0000000..c34b325 > --- /dev/null > +++ b/drivers/staging/iio/iio_input.c > @@ -0,0 +1,225 @@ > +/* > + * The industrial I/O input client driver > + * > + * Copyright (c) 2011 Jonathan Cameron > + * > + * This program is free software; you can redistribute it and/or modify it > + * under the terms of the GNU General Public License version 2 as published by > + * the Free Software Foundation. > + */ > + > +#include <linux/kernel.h> > +#include <linux/slab.h> > +#include <linux/module.h> > +#include <linux/err.h> > +#include <linux/platform_device.h> > +#include <linux/input.h> > +#include <linux/iio/buffer.h> > +#include <linux/iio/consumer.h> > +#include "iio_input.h" > + > +struct iio_input_state { > + struct iio_cb_buffer *buff; > + struct input_dev *idev; > +}; > + > +static int iio_channel_value(u8 *data, > + const struct iio_chan_spec *chan, > + s32 *val) > +{ > + u32 value = 0; > + > + if (chan->scan_type.sign == 's') { > + switch (chan->scan_type.storagebits) { > + case 8: > + value = *(s8 *)(data); > + break; > + case 16: > + switch (chan->scan_type.endianness) { > + case IIO_CPU: > + value = *(s16 *)(data); > + break; > + case IIO_BE: > + value = be16_to_cpu(*(__be16 *)data); > + break; > + case IIO_LE: > + value = le16_to_cpu(*(__le16 *)data); > + break; > + } > + break; > + case 32: > + switch (chan->scan_type.endianness) { > + case IIO_CPU: > + value = *(s32 *)(data); > + break; > + case IIO_BE: > + value = be32_to_cpu(*(__be32 *)data); > + break; > + case IIO_LE: > + value = le32_to_cpu(*(__le32 *)data); > + break; > + } > + break; > + default: > + return -EINVAL; > + } > + *val = sign_extend32(value, chan->scan_type.storagebits); > + } else { > + switch (chan->scan_type.storagebits) { > + case 8: > + value = *(u8 *)(data); > + break; > + case 16: > + switch (chan->scan_type.endianness) { > + case IIO_CPU: > + value = *(u16 *)(data); > + break; > + case IIO_BE: > + value = be16_to_cpu(*(__be16 *)data); > + break; > + case IIO_LE: > + value = le16_to_cpu(*(__le16 *)data); > + break; > + } > + break; > + case 32: > + switch (chan->scan_type.endianness) { > + case IIO_CPU: > + value = *(u32 *)(data); > + break; > + case IIO_BE: > + value = be32_to_cpu(*(__be32 *)data); > + break; > + case IIO_LE: > + value = le32_to_cpu(*(__le32 *)data); > + break; > + } > + break; > + default: > + return -EINVAL; > + } > + value >>= chan->scan_type.shift; > + value &= (1 << chan->scan_type.realbits) - 1; > + *val = value; > + } > + > + return 0; > +} > + > +static int iio_input_store_to(u8 *data, void *private) > +{ > + struct iio_input_state *st = private; > + struct iio_channel *channel; > + struct iio_input_channel_data *input_data; > + int offset = 0; > + s32 value; > + int ret; > + > + channel = iio_channel_cb_get_channels(st->buff); > + while (channel->indio_dev) { > + input_data = channel->data; > + offset = ALIGN(offset, > + channel->channel->scan_type.storagebits/8); > + ret = iio_channel_value(&data[offset], > + channel->channel, > + &value); > + if (ret < 0) > + return ret; > + offset += channel->channel->scan_type.storagebits/8; > + > + input_report_abs(st->idev, input_data->code, value); > + channel++; > + } > + input_sync(st->idev); > + > + return 0; > +} > + > +static int __devinit iio_input_probe(struct platform_device *pdev) > +{ > + struct iio_input_state *st; > + int ret; > + bool registered; > + struct iio_channel *channel; > + struct iio_input_channel_data *input_data; > + > + registered = false; > + st = kzalloc(sizeof(*st), GFP_KERNEL); > + if (st == NULL) > + return -ENOMEM; > + platform_set_drvdata(pdev, st); > + st->buff = iio_channel_get_all_cb(dev_name(&pdev->dev), > + &iio_input_store_to, > + st); > + if (IS_ERR(st->buff)) { > + ret = PTR_ERR(st->buff); > + goto error_free_state; > + } > + > + st->idev = input_allocate_device(); > + if (!st->idev) { > + ret = -ENOMEM; > + goto error_channels_release_all; > + } > + > + __set_bit(EV_ABS, st->idev->evbit); > + channel = iio_channel_cb_get_channels(st->buff); > + while (channel->indio_dev) { > + input_data = channel->data; > + input_set_abs_params(st->idev, input_data->code, > + input_data->min, > + input_data->max, > + input_data->fuzz, > + input_data->flat); > + channel++; > + } > + > + ret = input_register_device(st->idev); > + if (ret < 0) > + goto error_free_idev; > + registered = true; > + /* NORMALLY IN THE OPEN */ > + ret = iio_channel_start_all_cb(st->buff); > + if (ret < 0) > + goto error_unregister_input; > + > + return 0; > +error_unregister_input: > + if (registered) > + input_unregister_device(st->idev); > +error_free_idev: > + if (!registered) > + input_free_device(st->idev); > +error_channels_release_all: > + iio_channel_release_all_cb(st->buff); > +error_free_state: > + kfree(st); > + return ret; > +} > + > +static int __devexit iio_input_remove(struct platform_device *pdev) > +{ > + struct iio_input_state *st = platform_get_drvdata(pdev); > + /* NORMALLY IN THE CLOSE */ > + iio_channel_stop_all_cb(st->buff); > + input_unregister_device(st->idev); > + iio_channel_release_all_cb(st->buff); > + > + kfree(st); > + return 0; > +} > + > +static struct platform_driver iio_input_driver = { > + .driver = { > + .name = "iio_input_bridge", > + .owner = THIS_MODULE, > + }, > + .probe = iio_input_probe, > + .remove = __devexit_p(iio_input_remove), > +}; > + > +module_platform_driver(iio_input_driver); > + > +MODULE_AUTHOR("Jonathan Cameron <jic23@cam.ac.uk>"); > +MODULE_DESCRIPTION("IIO input buffer driver"); > +MODULE_LICENSE("GPL v2"); > diff --git a/drivers/staging/iio/iio_input.h b/drivers/staging/iio/iio_input.h > new file mode 100644 > index 0000000..cfd1d34 > --- /dev/null > +++ b/drivers/staging/iio/iio_input.h > @@ -0,0 +1,23 @@ > +/* > + * The industrial I/O input client driver > + * > + * Copyright (c) 2011 Jonathan Cameron > + * > + * This program is free software; you can redistribute it and/or modify it > + * under the terms of the GNU General Public License version 2 as published by > + * the Free Software Foundation. > + */ > + > +/** > + * iio_input_channel_data - description of the channel for input subsystem > + * @code: Absolute axis. > + * @min: Minimum value. > + * @max: Maximum value. > + * @fuzz: Used to filter noise from the event stream. > + * @flat: Values within this value will be discarded by joydev > + * and reported as 0 instead. > + */ > +struct iio_input_channel_data { > + unsigned int code; > + int min, max, fuzz, flat; > +}; > ^ permalink raw reply [flat|nested] 15+ messages in thread
* [PATCH 0/4 V6] staging:iio: Add support for multiple buffers (testing required!)
@ 2012-10-31 10:30 Jonathan Cameron
2012-10-31 10:30 ` [PATCH 4/4] staging:iio: Proof of concept input driver Jonathan Cameron
0 siblings, 1 reply; 15+ messages in thread
From: Jonathan Cameron @ 2012-10-31 10:30 UTC (permalink / raw)
To: linux-iio; +Cc: Jonathan Cameron
Same as V5 but with the changes from Lars-Peter's review to close some
memory leaks and catch up with some changes in the subsystem that had
passed this by. It should also now be possible to disable the
timestamp! (oops).
Thanks to Lars for his work reviewing my patches and also for his
excellent work reviewing in general! Without him the whole subsystem
would be moving a lot slower.
Again, more testing would be good (thanks for those who already
have).
In quick summary this is needed to do interrupt driven in kernel
clients as it adds the option to push to multiple 'buffer' interfaces.
Note that some of these may not actually do any buffering but rather
push directly into another subsytem (the example is input).
I would like to merge at least the first 3 patches early in this cycle.
What I need is some testing across a reasonable range of devices. Review
is welcome of course as well!
The complexity in this set is entirely in the changes to industrialio-buffer.c
concerning how we go about adding and removing client buffers. To do this
we need to work out which scan mask is appropriate given the requests of
all the given buffers (and whether we can satisfy it). I've been testing
with the max1363 driver which probably has the most complex set of available
scan masks. Thus when a new buffer is added or one is removed the whole
buffer set are torn down and built up again. This may be undesirable for
some applications but should occur only rarely.
The example driver is for bridging to the input subsystem and still needs
some work, but that can occur later in this merge cycle whereas the changes
to the core really want to go in asap.
Changes since v3
Rebased and added other drivers that have occured whilst this patch series
has been languishing in my local tree.
Changes since V2.
Improved handling of error conditions in the buffer insertion and
removal code. (suggested by Lars-Peter). I want to take another
look at these, but didn't want to keep people from testing this
in the meantime.
Also a number of minor fixes that came up during testing.
Thanks,
Jonathan
Jonathan Cameron (4):
staging:iio: Add support for multiple buffers
staging:iio:in kernel users: Add a data field for channel specific
info.
staging:iio: add a callback buffer for in kernel push interface
staging:iio: Proof of concept input driver.
drivers/iio/Kconfig | 6 +
drivers/iio/Makefile | 1 +
drivers/iio/accel/hid-sensor-accel-3d.c | 15 +-
drivers/iio/adc/ad7266.c | 3 +-
drivers/iio/adc/ad7476.c | 2 +-
drivers/iio/adc/ad_sigma_delta.c | 2 +-
drivers/iio/adc/at91_adc.c | 3 +-
drivers/iio/buffer_cb.c | 113 +++++++
drivers/iio/gyro/hid-sensor-gyro-3d.c | 15 +-
drivers/iio/industrialio-buffer.c | 380 ++++++++++++++++--------
drivers/iio/industrialio-core.c | 1 +
drivers/iio/inkern.c | 1 +
drivers/iio/light/adjd_s311.c | 3 +-
drivers/iio/light/hid-sensor-als.c | 15 +-
drivers/iio/magnetometer/hid-sensor-magn-3d.c | 15 +-
drivers/staging/iio/Kconfig | 11 +
drivers/staging/iio/Makefile | 1 +
drivers/staging/iio/accel/adis16201_ring.c | 2 +-
drivers/staging/iio/accel/adis16203_ring.c | 2 +-
drivers/staging/iio/accel/adis16204_ring.c | 2 +-
drivers/staging/iio/accel/adis16209_ring.c | 2 +-
drivers/staging/iio/accel/adis16240_ring.c | 2 +-
drivers/staging/iio/accel/lis3l02dq_ring.c | 2 +-
drivers/staging/iio/adc/ad7298_ring.c | 2 +-
drivers/staging/iio/adc/ad7606_ring.c | 2 +-
drivers/staging/iio/adc/ad7887_ring.c | 2 +-
drivers/staging/iio/adc/ad799x_ring.c | 2 +-
drivers/staging/iio/adc/max1363_ring.c | 2 +-
drivers/staging/iio/gyro/adis16260_ring.c | 2 +-
drivers/staging/iio/iio_input.c | 225 ++++++++++++++
drivers/staging/iio/iio_input.h | 23 ++
drivers/staging/iio/iio_simple_dummy_buffer.c | 5 +-
drivers/staging/iio/impedance-analyzer/ad5933.c | 4 +-
drivers/staging/iio/imu/adis16400_ring.c | 5 +-
drivers/staging/iio/meter/ade7758_ring.c | 2 +-
include/linux/iio/buffer.h | 24 +-
include/linux/iio/consumer.h | 48 +++
include/linux/iio/iio.h | 2 +
include/linux/iio/machine.h | 2 +
39 files changed, 743 insertions(+), 208 deletions(-)
create mode 100644 drivers/iio/buffer_cb.c
create mode 100644 drivers/staging/iio/iio_input.c
create mode 100644 drivers/staging/iio/iio_input.h
--
1.7.12.4
^ permalink raw reply [flat|nested] 15+ messages in thread* [PATCH 4/4] staging:iio: Proof of concept input driver. 2012-10-31 10:30 [PATCH 0/4 V6] staging:iio: Add support for multiple buffers (testing required!) Jonathan Cameron @ 2012-10-31 10:30 ` Jonathan Cameron 2012-10-31 14:40 ` Peter Meerwald 0 siblings, 1 reply; 15+ messages in thread From: Jonathan Cameron @ 2012-10-31 10:30 UTC (permalink / raw) To: linux-iio; +Cc: Jonathan Cameron, Jonathan Cameron From: Jonathan Cameron <jic23@cam.ac.uk> This is no where near ready to merge. Lots of stuff missing. Signed-off-by: Jonathan Cameron <jic23@kernel.org> --- drivers/staging/iio/Kconfig | 11 ++ drivers/staging/iio/Makefile | 1 + drivers/staging/iio/iio_input.c | 225 ++++++++++++++++++++++++++++++++++++++++ drivers/staging/iio/iio_input.h | 23 ++++ 4 files changed, 260 insertions(+) diff --git a/drivers/staging/iio/Kconfig b/drivers/staging/iio/Kconfig index ca56c75..3d21f6d 100644 --- a/drivers/staging/iio/Kconfig +++ b/drivers/staging/iio/Kconfig @@ -4,6 +4,17 @@ menu "IIO staging drivers" depends on IIO +config IIO_ST_INPUT + tristate "Input driver that uses channels specified via iio maps" + depends on INPUT + depends on IIO_BUFFER + select IIO_BUFFER_CB + help + Client driver for IIO via the push interfaces. Used to provide + and input interface for IIO devices that can feed a buffer on + a trigger interrupt. Note that all channels must come from a + single IIO supported device. + config IIO_ST_HWMON tristate "Hwmon driver that uses channels specified via iio maps" depends on HWMON diff --git a/drivers/staging/iio/Makefile b/drivers/staging/iio/Makefile index fa6937d..9e71498 100644 --- a/drivers/staging/iio/Makefile +++ b/drivers/staging/iio/Makefile @@ -12,6 +12,7 @@ iio_dummy-$(CONFIG_IIO_SIMPLE_DUMMY_BUFFER) += iio_simple_dummy_buffer.o obj-$(CONFIG_IIO_DUMMY_EVGEN) += iio_dummy_evgen.o obj-$(CONFIG_IIO_ST_HWMON) += iio_hwmon.o +obj-$(CONFIG_IIO_ST_INPUT) += iio_input.o obj-y += accel/ obj-y += adc/ diff --git a/drivers/staging/iio/iio_input.c b/drivers/staging/iio/iio_input.c new file mode 100644 index 0000000..c34b325 --- /dev/null +++ b/drivers/staging/iio/iio_input.c @@ -0,0 +1,225 @@ +/* + * The industrial I/O input client driver + * + * Copyright (c) 2011 Jonathan Cameron + * + * This program is free software; you can redistribute it and/or modify it + * under the terms of the GNU General Public License version 2 as published by + * the Free Software Foundation. + */ + +#include <linux/kernel.h> +#include <linux/slab.h> +#include <linux/module.h> +#include <linux/err.h> +#include <linux/platform_device.h> +#include <linux/input.h> +#include <linux/iio/buffer.h> +#include <linux/iio/consumer.h> +#include "iio_input.h" + +struct iio_input_state { + struct iio_cb_buffer *buff; + struct input_dev *idev; +}; + +static int iio_channel_value(u8 *data, + const struct iio_chan_spec *chan, + s32 *val) +{ + u32 value = 0; + + if (chan->scan_type.sign == 's') { + switch (chan->scan_type.storagebits) { + case 8: + value = *(s8 *)(data); + break; + case 16: + switch (chan->scan_type.endianness) { + case IIO_CPU: + value = *(s16 *)(data); + break; + case IIO_BE: + value = be16_to_cpu(*(__be16 *)data); + break; + case IIO_LE: + value = le16_to_cpu(*(__le16 *)data); + break; + } + break; + case 32: + switch (chan->scan_type.endianness) { + case IIO_CPU: + value = *(s32 *)(data); + break; + case IIO_BE: + value = be32_to_cpu(*(__be32 *)data); + break; + case IIO_LE: + value = le32_to_cpu(*(__le32 *)data); + break; + } + break; + default: + return -EINVAL; + } + *val = sign_extend32(value, chan->scan_type.storagebits); + } else { + switch (chan->scan_type.storagebits) { + case 8: + value = *(u8 *)(data); + break; + case 16: + switch (chan->scan_type.endianness) { + case IIO_CPU: + value = *(u16 *)(data); + break; + case IIO_BE: + value = be16_to_cpu(*(__be16 *)data); + break; + case IIO_LE: + value = le16_to_cpu(*(__le16 *)data); + break; + } + break; + case 32: + switch (chan->scan_type.endianness) { + case IIO_CPU: + value = *(u32 *)(data); + break; + case IIO_BE: + value = be32_to_cpu(*(__be32 *)data); + break; + case IIO_LE: + value = le32_to_cpu(*(__le32 *)data); + break; + } + break; + default: + return -EINVAL; + } + value >>= chan->scan_type.shift; + value &= (1 << chan->scan_type.realbits) - 1; + *val = value; + } + + return 0; +} + +static int iio_input_store_to(u8 *data, void *private) +{ + struct iio_input_state *st = private; + struct iio_channel *channel; + struct iio_input_channel_data *input_data; + int offset = 0; + s32 value; + int ret; + + channel = iio_channel_cb_get_channels(st->buff); + while (channel->indio_dev) { + input_data = channel->data; + offset = ALIGN(offset, + channel->channel->scan_type.storagebits/8); + ret = iio_channel_value(&data[offset], + channel->channel, + &value); + if (ret < 0) + return ret; + offset += channel->channel->scan_type.storagebits/8; + + input_report_abs(st->idev, input_data->code, value); + channel++; + } + input_sync(st->idev); + + return 0; +} + +static int __devinit iio_input_probe(struct platform_device *pdev) +{ + struct iio_input_state *st; + int ret; + bool registered; + struct iio_channel *channel; + struct iio_input_channel_data *input_data; + + registered = false; + st = kzalloc(sizeof(*st), GFP_KERNEL); + if (st == NULL) + return -ENOMEM; + platform_set_drvdata(pdev, st); + st->buff = iio_channel_get_all_cb(dev_name(&pdev->dev), + &iio_input_store_to, + st); + if (IS_ERR(st->buff)) { + ret = PTR_ERR(st->buff); + goto error_free_state; + } + + st->idev = input_allocate_device(); + if (!st->idev) { + ret = -ENOMEM; + goto error_channels_release_all; + } + + __set_bit(EV_ABS, st->idev->evbit); + channel = iio_channel_cb_get_channels(st->buff); + while (channel->indio_dev) { + input_data = channel->data; + input_set_abs_params(st->idev, input_data->code, + input_data->min, + input_data->max, + input_data->fuzz, + input_data->flat); + channel++; + } + + ret = input_register_device(st->idev); + if (ret < 0) + goto error_free_idev; + registered = true; + /* NORMALLY IN THE OPEN */ + ret = iio_channel_start_all_cb(st->buff); + if (ret < 0) + goto error_unregister_input; + + return 0; +error_unregister_input: + if (registered) + input_unregister_device(st->idev); +error_free_idev: + if (!registered) + input_free_device(st->idev); +error_channels_release_all: + iio_channel_release_all_cb(st->buff); +error_free_state: + kfree(st); + return ret; +} + +static int __devexit iio_input_remove(struct platform_device *pdev) +{ + struct iio_input_state *st = platform_get_drvdata(pdev); + /* NORMALLY IN THE CLOSE */ + iio_channel_stop_all_cb(st->buff); + input_unregister_device(st->idev); + iio_channel_release_all_cb(st->buff); + + kfree(st); + return 0; +} + +static struct platform_driver iio_input_driver = { + .driver = { + .name = "iio_input_bridge", + .owner = THIS_MODULE, + }, + .probe = iio_input_probe, + .remove = __devexit_p(iio_input_remove), +}; + +module_platform_driver(iio_input_driver); + +MODULE_AUTHOR("Jonathan Cameron <jic23@cam.ac.uk>"); +MODULE_DESCRIPTION("IIO input buffer driver"); +MODULE_LICENSE("GPL v2"); diff --git a/drivers/staging/iio/iio_input.h b/drivers/staging/iio/iio_input.h new file mode 100644 index 0000000..cfd1d34 --- /dev/null +++ b/drivers/staging/iio/iio_input.h @@ -0,0 +1,23 @@ +/* + * The industrial I/O input client driver + * + * Copyright (c) 2011 Jonathan Cameron + * + * This program is free software; you can redistribute it and/or modify it + * under the terms of the GNU General Public License version 2 as published by + * the Free Software Foundation. + */ + +/** + * iio_input_channel_data - description of the channel for input subsystem + * @code: Absolute axis. + * @min: Minimum value. + * @max: Maximum value. + * @fuzz: Used to filter noise from the event stream. + * @flat: Values within this value will be discarded by joydev + * and reported as 0 instead. + */ +struct iio_input_channel_data { + unsigned int code; + int min, max, fuzz, flat; +}; -- 1.7.12.4 ^ permalink raw reply related [flat|nested] 15+ messages in thread
* Re: [PATCH 4/4] staging:iio: Proof of concept input driver. 2012-10-31 10:30 ` [PATCH 4/4] staging:iio: Proof of concept input driver Jonathan Cameron @ 2012-10-31 14:40 ` Peter Meerwald 2012-11-02 11:02 ` Jonathan Cameron 0 siblings, 1 reply; 15+ messages in thread From: Peter Meerwald @ 2012-10-31 14:40 UTC (permalink / raw) To: Jonathan Cameron; +Cc: linux-iio, Jonathan Cameron > +config IIO_ST_INPUT > + tristate "Input driver that uses channels specified via iio maps" > + depends on INPUT > + depends on IIO_BUFFER > + select IIO_BUFFER_CB > + help > + Client driver for IIO via the push interfaces. Used to provide > + and input interface for IIO devices that can feed a buffer on an input interface > + * Copyright (c) 2011 Jonathan Cameron 2012 is almost over > + case 32: > + switch (chan->scan_type.endianness) { > + case IIO_CPU: > + value = *(s32 *)(data); > + break; > + case IIO_BE: > + value = be32_to_cpu(*(__be32 *)data); > + break; > + case IIO_LE: > + value = le32_to_cpu(*(__le32 *)data); > + break; here's an extra space between le32 and * > +static struct platform_driver iio_input_driver = { > + .driver = { > + .name = "iio_input_bridge", > + .owner = THIS_MODULE, > + }, > + .probe = iio_input_probe, > + .remove = __devexit_p(iio_input_remove), > +}; > + > +module_platform_driver(iio_input_driver); > + > +MODULE_AUTHOR("Jonathan Cameron <jic23@cam.ac.uk>"); > +MODULE_DESCRIPTION("IIO input buffer driver"); > +MODULE_LICENSE("GPL v2"); > diff --git a/drivers/staging/iio/iio_input.h b/drivers/staging/iio/iio_input.h > new file mode 100644 > index 0000000..cfd1d34 > --- /dev/null > +++ b/drivers/staging/iio/iio_input.h > @@ -0,0 +1,23 @@ > +/* > + * The industrial I/O input client driver this thing is called iio_input_bridge, IIO input buffer_driver, instrustrial I/O input client driver within a few lines of code -- now is the time to christen the driver :) > + * > + * Copyright (c) 2011 Jonathan Cameron > + * > + * This program is free software; you can redistribute it and/or modify it > + * under the terms of the GNU General Public License version 2 as published by > + * the Free Software Foundation. > + */ > + > +/** > + * iio_input_channel_data - description of the channel for input subsystem > + * @code: Absolute axis. > + * @min: Minimum value. > + * @max: Maximum value. > + * @fuzz: Used to filter noise from the event stream. > + * @flat: Values within this value will be discarded by joydev > + * and reported as 0 instead. @flat: Values within the range [-flat, flat] will be discarded ... this is at least what I would guess it does; input.h is equally vague > + */ > +struct iio_input_channel_data { > + unsigned int code; > + int min, max, fuzz, flat; > +}; > -- Peter Meerwald +43-664-2444418 (mobile) ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH 4/4] staging:iio: Proof of concept input driver. 2012-10-31 14:40 ` Peter Meerwald @ 2012-11-02 11:02 ` Jonathan Cameron 0 siblings, 0 replies; 15+ messages in thread From: Jonathan Cameron @ 2012-11-02 11:02 UTC (permalink / raw) To: Peter Meerwald; +Cc: linux-iio, Jonathan Cameron I actually forgot to say in this posting - I'm not really intending to merge this driver just yet. It needs a fair bit of blugeoning then a bit of time to be hammered by the input list before it's ready. For now its just acting as an example user (and a means of testing the other patches). Still the review is obviously most welcome! Jonathan > >> +config IIO_ST_INPUT >> + tristate "Input driver that uses channels specified via iio maps" >> + depends on INPUT >> + depends on IIO_BUFFER >> + select IIO_BUFFER_CB >> + help >> + Client driver for IIO via the push interfaces. Used to provide >> + and input interface for IIO devices that can feed a buffer on > > an input interface good catch. > >> + * Copyright (c) 2011 Jonathan Cameron > > 2012 is almost over I've not changed much in here since 2011 though so will leave it be. Did this whilst still in accademia. Been doing a real job for a year and this patch is still kicking around... hohum. It's been a busy year. > >> + case 32: >> + switch (chan->scan_type.endianness) { >> + case IIO_CPU: >> + value = *(s32 *)(data); >> + break; >> + case IIO_BE: >> + value = be32_to_cpu(*(__be32 *)data); >> + break; >> + case IIO_LE: >> + value = le32_to_cpu(*(__le32 *)data); >> + break; > > here's an extra space between le32 and * eagle eyes :) > >> +static struct platform_driver iio_input_driver = { >> + .driver = { >> + .name = "iio_input_bridge", >> + .owner = THIS_MODULE, >> + }, >> + .probe = iio_input_probe, >> + .remove = __devexit_p(iio_input_remove), >> +}; >> + >> +module_platform_driver(iio_input_driver); >> + >> +MODULE_AUTHOR("Jonathan Cameron <jic23@cam.ac.uk>"); >> +MODULE_DESCRIPTION("IIO input buffer driver"); >> +MODULE_LICENSE("GPL v2"); >> diff --git a/drivers/staging/iio/iio_input.h b/drivers/staging/iio/iio_input.h >> new file mode 100644 >> index 0000000..cfd1d34 >> --- /dev/null >> +++ b/drivers/staging/iio/iio_input.h >> @@ -0,0 +1,23 @@ >> +/* >> + * The industrial I/O input client driver > > this thing is called > iio_input_bridge, > IIO input buffer_driver, > instrustrial I/O input client driver > within a few lines of code -- now is the time to christen the driver :) Fair point. iio client driver for the input subsystem (iio_to_input for driver name) > >> + * >> + * Copyright (c) 2011 Jonathan Cameron >> + * >> + * This program is free software; you can redistribute it and/or modify it >> + * under the terms of the GNU General Public License version 2 as published by >> + * the Free Software Foundation. >> + */ >> + >> +/** >> + * iio_input_channel_data - description of the channel for input subsystem >> + * @code: Absolute axis. >> + * @min: Minimum value. >> + * @max: Maximum value. >> + * @fuzz: Used to filter noise from the event stream. >> + * @flat: Values within this value will be discarded by joydev >> + * and reported as 0 instead. > > @flat: Values within the range [-flat, flat] will be discarded ... > > this is at least what I would guess it does; input.h is equally vague > I'll be honest - I'm not all that sure what it does either... Gone with your description. We'll see what Dmitry et al say when I post it to linux-input. >> + */ >> +struct iio_input_channel_data { >> + unsigned int code; >> + int min, max, fuzz, flat; >> +}; >> > ^ permalink raw reply [flat|nested] 15+ messages in thread
end of thread, other threads:[~2012-11-02 11:02 UTC | newest] Thread overview: 15+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2012-04-10 20:38 [RFC PATCH 0/4 V2] Add push based interface for non userspace iio users Jonathan Cameron 2012-04-10 20:38 ` [PATCH 1/4] staging:iio: make all buffer access pass through the buffer_list Jonathan Cameron 2012-04-16 15:59 ` Lars-Peter Clausen 2012-04-16 20:40 ` Jonathan Cameron 2012-04-10 20:38 ` [PATCH 3/4] staging:iio:in kernel users: Add a data field for channel specific info Jonathan Cameron 2012-04-10 20:38 ` [PATCH 4/4] staging:iio: Proof of concept input driver Jonathan Cameron -- strict thread matches above, loose matches on Subject: below -- 2012-05-30 19:36 [PATCH 0/4 V2] staging:iio: Add support for multiple buffers Jonathan Cameron 2012-05-30 19:36 ` [PATCH 4/4] staging:iio: Proof of concept input driver Jonathan Cameron 2012-06-08 15:27 ` Lars-Peter Clausen 2012-06-09 17:56 ` Jonathan Cameron 2012-06-30 19:06 [PATCH 0/4 V3] staging:iio: Add support for multiple buffers Jonathan Cameron 2012-06-30 19:06 ` [PATCH 4/4] staging:iio: Proof of concept input driver Jonathan Cameron 2012-10-13 9:24 [PATCH 0/4 V5] staging:iio: Add support for multiple buffers (testing required!) Jonathan Cameron 2012-10-13 9:24 ` [PATCH 4/4] staging:iio: Proof of concept input driver Jonathan Cameron 2012-10-13 9:57 ` Jonathan Cameron 2012-10-31 10:30 [PATCH 0/4 V6] staging:iio: Add support for multiple buffers (testing required!) Jonathan Cameron 2012-10-31 10:30 ` [PATCH 4/4] staging:iio: Proof of concept input driver Jonathan Cameron 2012-10-31 14:40 ` Peter Meerwald 2012-11-02 11:02 ` Jonathan Cameron
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).