On 30/12/16 09:23, Song Hongyan wrote:
> Add support for max_sampling_frequency, this will help user to
> set sampling frequency to an acceptable range.
>
> Signed-off-by: Song Hongyan <hongyan.song@xxxxxxxxx>
Hi,
We've recently added an extended version of _available which includes
the option for
[start_value step end_value] which will handle this well without having
to added additional ABI.
It's not well (read as 'not') documented as yet though except for
a few specific drivers that use it. It's one of the tasks on my todo
list once I've caught up with reviews.
See the read_avail callback in struct iio_info and the associated
info_mask_separate_available etc.
Hopefully this new approach will finally solve the whole 'range'
of an attribute question.
Jonathan
> ---
> Documentation/ABI/testing/sysfs-bus-iio | 7 +++++++
> drivers/iio/industrialio-core.c | 1 +
> include/linux/iio/iio.h | 1 +
> 3 files changed, 9 insertions(+)
>
> diff --git a/Documentation/ABI/testing/sysfs-bus-iio b/Documentation/ABI/testing/sysfs-bus-iio
> index 60b7406..cbcd2b8 100644
> --- a/Documentation/ABI/testing/sysfs-bus-iio
> +++ b/Documentation/ABI/testing/sysfs-bus-iio
> @@ -54,6 +54,13 @@ Description:
> relevant directories. If it effects all of the above
> then it is to be found in the base device directory.
>
> +What: /sys/bus/iio/devices/iio:deviceX/max_sampling_frequency
> +KernelVersion: 4.11
> +Contact: linux-iio@xxxxxxxxxxxxxxx
> +Description:
> + This parameter shows the max sampling frequency, which can
> + help user to set sampling frequency to an acceptable range.
> +
> What: /sys/bus/iio/devices/iio:deviceX/sampling_frequency_available
> What: /sys/.../iio:deviceX/buffer/sampling_frequency_available
> What: /sys/bus/iio/devices/triggerX/sampling_frequency_available
> diff --git a/drivers/iio/industrialio-core.c b/drivers/iio/industrialio-core.c
> index 72fc96a..5203e75 100644
> --- a/drivers/iio/industrialio-core.c
> +++ b/drivers/iio/industrialio-core.c
> @@ -140,6 +140,7 @@ struct bus_type iio_bus_type = {
> [IIO_CHAN_INFO_HIGH_PASS_FILTER_3DB_FREQUENCY]
> = "filter_high_pass_3db_frequency",
> [IIO_CHAN_INFO_SAMP_FREQ] = "sampling_frequency",
> + [IIO_CHAN_INFO_MAX_SAMP_FREQ] = "max_sampling_frequency",
> [IIO_CHAN_INFO_FREQUENCY] = "frequency",
> [IIO_CHAN_INFO_PHASE] = "phase",
> [IIO_CHAN_INFO_HARDWAREGAIN] = "hardwaregain",
> diff --git a/include/linux/iio/iio.h b/include/linux/iio/iio.h
> index b4a0679..f9eba2e 100644
> --- a/include/linux/iio/iio.h
> +++ b/include/linux/iio/iio.h
> @@ -34,6 +34,7 @@ enum iio_chan_info_enum {
> IIO_CHAN_INFO_LOW_PASS_FILTER_3DB_FREQUENCY,
> IIO_CHAN_INFO_HIGH_PASS_FILTER_3DB_FREQUENCY,
> IIO_CHAN_INFO_SAMP_FREQ,
> + IIO_CHAN_INFO_MAX_SAMP_FREQ,
> IIO_CHAN_INFO_FREQUENCY,
> IIO_CHAN_INFO_PHASE,
> IIO_CHAN_INFO_HARDWAREGAIN,
>
--
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
