Re: [PATCH v2 6/7] counter: ftm-quaddec: Documentation: Add specific counter sysfs documentation

From: William Breathitt Gray
Date: Thu Mar 07 2019 - 06:41:28 EST


On Wed, Mar 06, 2019 at 12:12:07PM +0100, Patrick Havelange wrote:
> This adds documentation for the specific prescaler entry.
>
> Signed-off-by: Patrick Havelange <patrick.havelange@xxxxxxxxxxxxx>
> ---
> Changes v2
> - Add doc for prescaler entry
> ---
> .../ABI/testing/sysfs-bus-counter-ftm-quaddec | 16 ++++++++++++++++
> 1 file changed, 16 insertions(+)
> create mode 100644 Documentation/ABI/testing/sysfs-bus-counter-ftm-quaddec
>
> diff --git a/Documentation/ABI/testing/sysfs-bus-counter-ftm-quaddec b/Documentation/ABI/testing/sysfs-bus-counter-ftm-quaddec
> new file mode 100644
> index 000000000000..2da629d6d485
> --- /dev/null
> +++ b/Documentation/ABI/testing/sysfs-bus-counter-ftm-quaddec
> @@ -0,0 +1,16 @@
> +What: /sys/bus/counter/devices/counterX/countY/prescaler_available
> +KernelVersion: 5.1
> +Contact: linux-iio@xxxxxxxxxxxxxxx
> +Description:
> + Discrete set of available values for the respective Count Y
> + configuration are listed in this file. Values are delimited by
> + newline characters.
> +
> +What: /sys/bus/counter/devices/counterX/countY/prescaler
> +KernelVersion: 5.1
> +Contact: linux-iio@xxxxxxxxxxxxxxx
> +Description:
> + Configure the prescaler value associated with Count Y.
> + On the FlexTimer, the counter clock source passes through a
> + prescaler that is a 7-bit counter. This acts like a clock
> + divider.
> --
> 2.19.1

Hmm, prescalers seem common enough among counter devices to permit these
attributes to be listed in the sysfs-bus-counter documentation file.
However, I'd like to wait until we get another counter driver for a
device with a prescaler before we make that move. From there, we'll have
a better vantage point to determine a fitting standard prescaler
attribute behavior.

So for now, we'll keep these attributes documented here in the
sysfs-bus-counter-ftm-quaddec file, until the time comes to broach the
discussion again.

William Breathitt Gray