On Sat, Dec 10, 2022 at 10:21:08AM +0000, Biju Das wrote:
> +What: /sys/bus/counter/devices/counterX/external_input_phase_clock_select
> +KernelVersion: 6.3
> +Contact: linux-iio@xxxxxxxxxxxxxxx
> +Description:
> + This attribute selects the external clock pin for phase
> + counting mode of counter X.
Hi Biju,
Remove the "This attribute" from the description, and capitalize the
word "counter": "Selects the external clock pin for phase counting mode
of Counter X."
> +What: /sys/bus/counter/devices/counterX/external_input_phase_clock_select_available
At some point in the future I should combine the *_available blocks, but
right now they're separated between Count and Signal configurations.
This external_input_phase_clock_select_available is a device-level
configuration so it'll need its own block as well, such as the
following.
What: /sys/bus/counter/devices/counterX/external_input_phase_clock_select_available
KernelVersion: 6.3
Contact: linux-iio@xxxxxxxxxxxxxxx
Description:
Discrete set of available values for the respective device
configuration are listed in this file.
> +What: /sys/bus/counter/devices/counterX/cascade_counts_enable
> +KernelVersion: 6.3
> +Contact: linux-iio@xxxxxxxxxxxxxxx
> +Description:
> + This attribute indicates the cascading of counts on
> + counter X.
Similar to before, remove the "This attribute" from the description, and
capitalize "counts" and "counter": "Indicates the cascading of Counts on
Counter X."
William Breathitt Gray
Attachment:
signature.asc
Description: PGP signature
