diff options
| author | Lars-Peter Clausen <lars@metafoo.de> | 2013-09-15 18:50:00 +0200 |
|---|---|---|
| committer | Jonathan Cameron <jic23@kernel.org> | 2013-09-15 20:02:07 +0200 |
| commit | 5d65d92045cb7d3b2c45020c0e62d6d1c1d34f37 (patch) | |
| tree | a996bac655ff390d6331d82f6121c6b7aab7bbdf /drivers/iio/buffer_cb.c | |
| parent | iio:buffer_cb: Constify iio_cb_access (diff) | |
| download | linux-5d65d92045cb7d3b2c45020c0e62d6d1c1d34f37.tar.xz linux-5d65d92045cb7d3b2c45020c0e62d6d1c1d34f37.zip | |
iio: iio_push_to_buffers(): Change type of 'data' to const void *
Change the type of the 'data' parameter for iio_push_to_buffers() from 'u8 *' to
'const void *'. Drivers typically use the correct type (e.g. __be16 *) for their
data buffer. When passing the buffer to iio_push_to_buffers() it needs to be
cast to 'u8 *' for the compiler to not complain (and also having to add __force
if we want to keep sparse happy as well). Since the buffer implementation should
not care about the data layout (except the size of one sample) using a void
pointer is the correct thing to do. Also make it const as the buffer
implementations are not supposed to modify it.
Signed-off-by: Lars-Peter Clausen <lars@metafoo.de>
Signed-off-by: Jonathan Cameron <jic23@kernel.org>
Diffstat (limited to 'drivers/iio/buffer_cb.c')
| -rw-r--r-- | drivers/iio/buffer_cb.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/drivers/iio/buffer_cb.c b/drivers/iio/buffer_cb.c index f406889248c8..578f7199ed51 100644 --- a/drivers/iio/buffer_cb.c +++ b/drivers/iio/buffer_cb.c @@ -7,12 +7,12 @@ struct iio_cb_buffer { struct iio_buffer buffer; - int (*cb)(u8 *data, void *private); + int (*cb)(const void *data, void *private); void *private; struct iio_channel *channels; }; -static int iio_buffer_cb_store_to(struct iio_buffer *buffer, u8 *data) +static int iio_buffer_cb_store_to(struct iio_buffer *buffer, const void *data) { struct iio_cb_buffer *cb_buff = container_of(buffer, struct iio_cb_buffer, @@ -26,7 +26,7 @@ static const struct iio_buffer_access_funcs iio_cb_access = { }; struct iio_cb_buffer *iio_channel_get_all_cb(struct device *dev, - int (*cb)(u8 *data, + int (*cb)(const void *data, void *private), void *private) { |