[PATCH] staging: comedi: ke_counter: expose clock source options to user space

Ian Abbott abbotti at mev.co.uk
Mon Jun 23 09:57:41 UTC 2014


On 2014-06-20 18:39, H Hartley Sweeten wrote:
> As pointed out by Ian Abbott, the INSN_CONFIG_SET_CLOCK_SRC options should
> be exposed in the user header comedi.h.
>
> Signed-off-by: H Hartley Sweeten <hsweeten at visionengravers.com>
> Cc: Ian Abbott <abbotti at mev.co.uk>
> Cc: Greg Kroah-Hartman <gregkh at linuxfoundation.org>
> ---
>   drivers/staging/comedi/comedi.h             | 11 ++++++++++
>   drivers/staging/comedi/drivers/ke_counter.c | 33 ++++++++++++++++++-----------
>   2 files changed, 32 insertions(+), 12 deletions(-)
>
> diff --git a/drivers/staging/comedi/comedi.h b/drivers/staging/comedi/comedi.h
> index 6bbbe5b..dbaeba7 100644
> --- a/drivers/staging/comedi/comedi.h
> +++ b/drivers/staging/comedi/comedi.h
> @@ -976,4 +976,15 @@ enum amplc_dio_gate_source {
>   	AMPLC_DIO_GAT_NPAT_GONE	/* negated "pattern gone away" */
>   };
>
> +/*
> + * Values for setting a clock source with INSN_CONFIG_SET_CLOCK_SRC for
> + * the counter subdevice on the Kolter Electronic PCI-Counter board
> + * (ke_counter driver).
> + */
> +enum ke_counter_clock_source {
> +	KE_CLK_20MHZ,	/* internal 20MHz (default) */
> +	KE_CLK_4MHZ,	/* internal 4MHz (option) */
> +	KE_CLK_EXT	/* external clock on pin 21 of D-Sub */
> +};
> +

Could have kept the existing hardware values, but never mind as it's 
new(-ish) functionality.

Reviewed-by: Ian Abbott <abbotti at mev.co.uk>

-- 
-=( Ian Abbott @ MEV Ltd.    E-mail: <abbotti at mev.co.uk>        )=-
-=( Tel: +44 (0)161 477 1898   FAX: +44 (0)161 718 3587         )=-


More information about the devel mailing list