On 12/05/2017 09:56 PM, mfornero@xxxxxxxxx wrote:
> From: Matt Fornero <matt.fornero@xxxxxxxxxxxxx>
>
> Add a sysfs attribute that exposes buffer data available to userspace.
> This attribute can be checked at runtime to determine the overall buffer
> fill level (across all allocated buffers).
>
> Signed-off-by: Matt Fornero <matt.fornero@xxxxxxxxxxxxx>
Looks sensible to me. I think we need an update to the ABI documentation as
well for this in Documentation/ABI/testing/sysfs-bus-iio
> ---
> v1 -> v2:
> - Use iio_buffer_data_available() for all buffers instead
> of only DMA buffers
>
> drivers/iio/industrialio-buffer.c | 15 +++++++++++++++
> 1 file changed, 15 insertions(+)
>
> diff --git a/drivers/iio/industrialio-buffer.c b/drivers/iio/industrialio-buffer.c
> index d80f830..e7bdfa3 100644
> --- a/drivers/iio/industrialio-buffer.c
> +++ b/drivers/iio/industrialio-buffer.c
> @@ -1121,6 +1121,18 @@ static ssize_t iio_buffer_store_watermark(struct device *dev,
> return ret ? ret : len;
> }
>
> +static ssize_t iio_dma_show_data_available(struct device *dev,
> + struct device_attribute *attr,
> + char *buf)
> +{
> + struct iio_dev *indio_dev = dev_to_iio_dev(dev);
> + size_t bytes;
> +
> + bytes = iio_buffer_data_available(indio_dev->buffer);
> +
> + return sprintf(buf, "%llu\n", (unsigned long long)bytes);
Since this is the kernel we shouldn't need those MSVC workarounds ;)
"%zu" should work
> +}
[..]
--
To unsubscribe from this list: send the line "unsubscribe linux-iio" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
