linux-stable.git/drivers/iio/buffer, branch linux-5.17.y

iio: buffer-dma: Use round_down() instead of rounddown()

2021-11-27T15:12:44+00:00

We know that the buffer's alignment will always be a power of two;
therefore, we can use the faster round_down() macro.

Signed-off-by: Paul Cercueil 
Reviewed-by: Alexandru Ardelean 
Link: https://lore.kernel.org/r/20211115141925.60164-4-paul@crapouillou.net
Signed-off-by: Jonathan Cameron

iio: triggered-buffer: extend support to configure output buffers

2021-10-19T07:30:45+00:00

Now that output (kfifo) buffers are supported, we need to extend the
{devm_}iio_triggered_buffer_setup_ext() parameter list to take a direction
parameter.

This allows us to attach an output triggered buffer to a DAC device.
Unfortunately it's a bit difficult to add another macro to avoid changing 5
drivers where {devm_}iio_triggered_buffer_setup_ext() is used.
Well, it's doable, but may not be worth the trouble vs just updating all
these 5 drivers.

Signed-off-by: Alexandru Ardelean 
Signed-off-by: Mihail Chindris 
Link: https://lore.kernel.org/r/20211007080035.2531-4-mihail.chindris@analog.com
Signed-off-by: Jonathan Cameron

iio: kfifo-buffer: Add output buffer support

2021-10-19T07:30:45+00:00

Add output buffer support to the kfifo buffer implementation.

The implementation is straight forward and mostly just wraps the kfifo
API to provide the required operations.

Signed-off-by: Lars-Peter Clausen 
Signed-off-by: Alexandru Ardelean 
Signed-off-by: Mihail Chindris 
Link: https://lore.kernel.org/r/20211007080035.2531-3-mihail.chindris@analog.com
Signed-off-by: Jonathan Cameron

iio: core: move @id from struct iio_dev to struct iio_dev_opaque

2021-05-17T12:49:13+00:00

Continuing from Alexandru Ardelean's introduction of the split between
driver modifiable fields and those that should only be set by the core.

This could have been done in two steps to make the actual move after
introducing iio_device_id() but there seemed limited point to that
given how mechanical the majority of the patch is.

Includes fixup from Alex for missing mxs-lradc-adc conversion.

Signed-off-by: Jonathan Cameron 
Reviewed-by: Alexandru Ardelean 
Link: https://lore.kernel.org/r/20210426174911.397061-2-jic23@kernel.org

iio: triggered-buffer: simplify devm_iio_triggered_buffer_setup_ext()

2021-05-17T12:49:07+00:00

Use devm_add_action_or_reset() instead of devres_alloc() and
devres_add(), which works the same. This will simplify the
code. There is no functional changes.

Signed-off-by: Yicong Yang 
Reviewed-by: Lars-Peter Clausen 
Reviewed-by: Nuno Sa 
Reviewed-by: Andy Shevchenko 
Link: https://lore.kernel.org/r/1617881896-3164-5-git-send-email-yangyicong@hisilicon.com
Signed-off-by: Jonathan Cameron

iio: hw_consumer: simplify devm_iio_hw_consumer_alloc()

2021-05-17T12:49:06+00:00

Use devm_add_action_or_reset() instead of devres_alloc() and
devres_add(), which works the same. This will simplify the
code. There is no functional changes.

Signed-off-by: Yicong Yang 
Reviewed-by: Lars-Peter Clausen 
Reviewed-by: Nuno Sa 
Reviewed-by: Andy Shevchenko 
Link: https://lore.kernel.org/r/1617881896-3164-4-git-send-email-yangyicong@hisilicon.com
Signed-off-by: Jonathan Cameron

iio: buffer-dmaengine: simplify __devm_iio_dmaengine_buffer_free()

2021-05-17T12:49:06+00:00

Use devm_add_action_or_reset() instead of devres_alloc() and
devres_add(), which works the same. This will simplify the
code. There is no functional changes.

Signed-off-by: Yicong Yang 
Reviewed-by: Lars-Peter Clausen 
Reviewed-by: Nuno Sa 
Reviewed-by: Andy Shevchenko 
Link: https://lore.kernel.org/r/1617881896-3164-3-git-send-email-yangyicong@hisilicon.com
Signed-off-by: Jonathan Cameron

iio: kfifo: add devm_iio_triggered_buffer_setup_ext variant

2021-03-25T19:13:52+00:00

This is similar to the {devm_}iio_triggered_buffer_setup_ext variants added
via commit 5164c7889857 ("iio: triggered-buffer: add
{devm_}iio_triggered_buffer_setup_ext variants").

These can be used to pass extra buffer attributes to the buffer object.
This is a bit of temporary mechanism (hopefully) so that drivers that want
to allocate a kfifo buffer with extra buffer attributes, don't need to
include 'buffer_impl.h' directly. This can also become an API function (in
it's own right, unfortunately), but it may be a little less bad vs drivers
having to include 'buffer_impl.h'.

So, far the drivers that want to pass buffer attributes, all have to do
with some HW FIFO attributes, so there may be a chance of unifying them
into IIO core somehow (as some standard API). But, until that happens, we
just need to let them register their HW FIFO attributes directly (without
having to let them include 'buffer_impl.h' directly).

Signed-off-by: Alexandru Ardelean 
Link: https://lore.kernel.org/r/20210311091042.22417-1-aardelean@deviqon.com
Signed-off-by: Jonathan Cameron

iio: buffer: kfifo_buf: kernel-doc, typo in function name.

2021-03-25T19:13:51+00:00

Should have been _kfifo_ and was _fifo_

Signed-off-by: Jonathan Cameron 
Acked-by: Alexandru Ardelean 
Reviewed-by: Lars-Peter Clausen 
Link: https://lore.kernel.org/r/20210314164655.408461-9-jic23@kernel.org

iio: kfifo: mask flags without zero-check in devm_iio_kfifo_buffer_setup()

2021-03-25T19:13:49+00:00

As pointed by Lars, this doesn't require a zero-check. Also, while looking
at this a little closer at it (again), the masking can be done later, as
there is a zero-check for 'mode_flags' anyway, which returns -EINVAL. And
we only need the 'mode_flags' later in the logic.

This change is more of a tweak.

Fixes: e36db6a06937 ("iio: kfifo: add devm_iio_kfifo_buffer_setup() helper")
Cc: Lars-Peter Clausen 
Signed-off-by: Alexandru Ardelean 
Link: https://lore.kernel.org/r/20210306162834.7339-1-ardeleanalex@gmail.com
Signed-off-by: Jonathan Cameron